2023-02-02 03:49:40

by Jakub Kicinski

[permalink] [raw]
Subject: Re: [PATCH net-next v11 0/8] Add Auxiliary driver support

On Wed, 1 Feb 2023 19:38:01 -0800 Ajit Khaparde wrote:
> v10->v11:
> - Addressed unused variable warning in patch 1 reported by kernel test
> robot.

:/ Why are you reposting this so early, read the rules please:

https://www.kernel.org/doc/html/next/process/maintainer-netdev.html

We should bump that to 48h for those who can't be bothered to compile
their code before reposting, if you ask me.

I'm discarding your series, come back next week.

2023-02-06 16:59:03

by Ajit Khaparde

[permalink] [raw]
Subject: Re: [PATCH net-next v11 0/8] Add Auxiliary driver support

On Wed, Feb 1, 2023 at 7:49 PM Jakub Kicinski <[email protected]> wrote:
>
> On Wed, 1 Feb 2023 19:38:01 -0800 Ajit Khaparde wrote:
> > v10->v11:
> > - Addressed unused variable warning in patch 1 reported by kernel test
> > robot.
>
> :/ Why are you reposting this so early, read the rules please:
>
> https://www.kernel.org/doc/html/next/process/maintainer-netdev.html
>
> We should bump that to 48h for those who can't be bothered to compile
> their code before reposting, if you ask me.
>
> I'm discarding your series, come back next week.
Please let me know if I should rebase and send a new version.
FYI - I rebased the v11 patchset against the latest and it applied fine.

Thanks
Ajit


Attachments:
smime.p7s (4.12 kB)
S/MIME Cryptographic Signature

2023-02-07 06:40:26

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH net-next v11 0/8] Add Auxiliary driver support

Hello:

This series was applied to netdev/net-next.git (master)
by Ajit Khaparde <[email protected]>:

On Wed, 1 Feb 2023 19:38:01 -0800 you wrote:
> Add auxiliary device driver for Broadcom devices.
> The bnxt_en driver will register and initialize an aux device
> if RDMA is enabled in the underlying device.
> The bnxt_re driver will then probe and initialize the
> RoCE interfaces with the infiniband stack.
>
> We got rid of the bnxt_en_ops which the bnxt_re driver used to
> communicate with bnxt_en.
> Similarly We have tried to clean up most of the bnxt_ulp_ops.
> In most of the cases we used the functions and entry points provided
> by the auxiliary bus driver framework.
> And now these are the minimal functions needed to support the functionality.
>
> [...]

Here is the summary with links:
- [net-next,v11,1/8] bnxt_en: Add auxiliary driver support
https://git.kernel.org/netdev/net-next/c/d80d88b0dfff
- [net-next,v11,2/8] RDMA/bnxt_re: Use auxiliary driver interface
https://git.kernel.org/netdev/net-next/c/6d758147c7b8
- [net-next,v11,3/8] bnxt_en: Remove usage of ulp_id
https://git.kernel.org/netdev/net-next/c/dafcdf5e2bd0
- [net-next,v11,4/8] bnxt_en: Use direct API instead of indirection
https://git.kernel.org/netdev/net-next/c/63669ab384ea
- [net-next,v11,5/8] bnxt_en: Use auxiliary bus calls over proprietary calls
https://git.kernel.org/netdev/net-next/c/3b65e9456c29
- [net-next,v11,6/8] bnxt_en: Remove struct bnxt access from RoCE driver
https://git.kernel.org/netdev/net-next/c/848dc857c8de
- [net-next,v11,7/8] RDMA/bnxt_re: Remove the sriov config callback
https://git.kernel.org/netdev/net-next/c/a43c26fa2e6c
- [net-next,v11,8/8] bnxt_en: Remove runtime interrupt vector allocation
https://git.kernel.org/netdev/net-next/c/303432211324

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html