[dpdk-dev] [PATCH 0/3] patchset for bnxt PMD

[Ajit Khaparde]

On Tue, Oct 12, 2021 at 1:30 PM Ajit Khaparde
<ajit.khaparde at broadcom.com> wrote:
>
> On Tue, Oct 12, 2021 at 12:49 PM Ferruh Yigit <ferruh.yigit at intel.com> wrote:
> >
> > On 10/12/2021 8:03 PM, Ajit Khaparde wrote:
> > > On Tue, Oct 12, 2021 at 11:16 AM Ajit Khaparde
> > > <ajit.khaparde at broadcom.com> wrote:
> > >>
> > >> On Tue, Oct 12, 2021 at 11:14 AM Ferruh Yigit <ferruh.yigit at intel.com> wrote:
> > >>>
> > >>> On 10/7/2021 5:54 PM, Ajit Khaparde wrote:
> > >>>> On Wed, Oct 6, 2021 at 8:23 PM Ajit Khaparde <ajit.khaparde at broadcom.com>
> > >>>> wrote:
> > >>>>
> > >>>>> Apart from fixing an issue with Rx queue state at startup,
> > >>>>> this patchset enhances support for RSS action and
> > >>>>> optimizes the mbuf requirement at startup by creating the
> > >>>>> aggregation rings only when needed (rx_scatter is enabled).
> > >>>>>
> > >>>>> Please apply!
> > >>>>>
> > >>>> Patchset applied to dpdk-next-net-brcm.
> > >>>>
> > >>>>
> > >>>
> > >>> Hi Ajit,
> > >>>
> > >>> dpdk-next-net-brcm conflicts with next-net because of some patches
> > >>> updating bnxt in next-net.
> > >>> Can you please rebase 'dpdk-next-net-brcm' on top of latest next-net?
> > >> Sure. I will send an email once it's ready.
> > > Done. I have rebased the repo to the latest next-net/main
> > > and merged the patches. Please try now.
> > >
> >
> > Hi Ajit,
> >
> > I am getting lots of checkpatch warnings on the patches in your tree,
> > but I don't see them in the patchwork. Warnings can be because of the
> > updates during rebase.
> >
> > If there is code change during rebase, to not have a diverge between
> > the patch in the patchwork and what we are merging, can you please send
> > and updated version of the effected patches first?
> There is no code change made during the rebase.
> My checkpatch run did not point out anything.
> But let me try again. Thanks Ferruh.
Ok, got it.
I had done the merge on a new system which did not have the checkpatch
setup correctly.
I have fixed the issues and updated the tree. Apologies for the trouble.

Thanks
Ajit

>
> Thanks
> Ajit
>
>
>
> > This lets us see the CI results and later they can be merge in your tree
> > again.
> >