From: Leon Romanovsky <leon@kernel.org>
To: Ajit Khaparde <ajit.khaparde@broadcom.com>, kuba@kernel.org
Cc: andrew.gospodarek@broadcom.com, davem@davemloft.net,
edumazet@google.com, jgg@ziepe.ca, linux-kernel@vger.kernel.org,
linux-rdma@vger.kernel.org, michael.chan@broadcom.com,
netdev@vger.kernel.org, pabeni@redhat.com,
selvin.xavier@broadcom.com
Subject: Re: [PATCH v3 0/6] Add Auxiliary driver support
Date: Mon, 7 Nov 2022 08:37:13 +0200 [thread overview]
Message-ID: <Y2inmdbpoRm2VbuE@unreal> (raw)
In-Reply-To: <20221104162733.73345-1-ajit.khaparde@broadcom.com>
On Fri, Nov 04, 2022 at 09:27:27AM -0700, Ajit Khaparde 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.
>
> v1->v2:
> - Incorporated review comments including usage of ulp_id &
> complex function indirections.
> - Used function calls provided by the auxiliary bus interface
> instead of proprietary calls.
> - Refactor code to remove ROCE driver's access to bnxt structure.
>
> v2->v3:
> - Addressed review comments including cleanup of some unnecessary wrappers
> - Fixed warnings seen during cross compilation
>
> Please apply. Thanks.
Please send this series as standalone one and not as a reply
to previous discussion. Reply-to messes review flow, especially
for series.
Jakub, I'll review it once Ajit will send it properly.
Thanks
next prev parent reply other threads:[~2022-11-07 6:37 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-24 23:14 [PATCH 0/2] Add Auxiliary driver support Ajit Khaparde
2022-07-24 23:14 ` [PATCH 1/2] net/bnxt: Add auxiliary " Ajit Khaparde
2022-08-02 12:50 ` kernel test robot
2022-07-24 23:14 ` [PATCH 2/2] RDMA/bnxt_re: Use auxiliary driver interface Ajit Khaparde
2022-07-25 12:15 ` Leon Romanovsky
2022-07-26 4:58 ` Ajit Khaparde
2022-07-26 5:36 ` Leon Romanovsky
2022-07-26 14:31 ` Ajit Khaparde
2022-10-25 17:31 ` [PATCH v2 0/6] Add Auxiliary driver support Ajit Khaparde
2022-10-25 17:31 ` [PATCH v2 1/6] bnxt_en: Add auxiliary " Ajit Khaparde
2022-10-25 17:31 ` [PATCH v2 2/6] RDMA/bnxt_re: Use auxiliary driver interface Ajit Khaparde
2022-10-26 9:24 ` Leon Romanovsky
2022-10-26 15:58 ` Ajit Khaparde
2022-11-04 16:27 ` [PATCH v3 0/6] Add Auxiliary driver support Ajit Khaparde
2022-11-04 16:27 ` [PATCH v3 1/6] bnxt_en: Add auxiliary " Ajit Khaparde
2022-11-08 7:21 ` kernel test robot
2022-11-04 16:27 ` [PATCH v3 2/6] RDMA/bnxt_re: Use auxiliary driver interface Ajit Khaparde
2022-11-04 16:27 ` [PATCH v3 3/6] bnxt_en: Remove usage of ulp_id Ajit Khaparde
2022-11-04 16:27 ` [PATCH v3 4/6] bnxt_en: Use direct API instead of indirection Ajit Khaparde
2022-11-04 16:27 ` [PATCH v3 5/6] bnxt_en: Use auxiliary bus calls over proprietary calls Ajit Khaparde
2022-11-07 6:41 ` Leon Romanovsky
2022-11-09 18:45 ` Ajit Khaparde
2022-11-04 16:27 ` [PATCH v3 6/6] bnxt_en: Remove struct bnxt access from RoCE driver Ajit Khaparde
2022-11-07 6:37 ` Leon Romanovsky [this message]
2022-11-07 16:06 ` [PATCH v3 0/6] Add Auxiliary driver support Jakub Kicinski
2022-11-07 17:26 ` Leon Romanovsky
2022-11-07 18:21 ` Ajit Khaparde
2022-10-25 17:31 ` [PATCH v2 3/6] bnxt_en: Remove usage of ulp_id Ajit Khaparde
2022-10-26 0:57 ` kernel test robot
2022-10-25 17:31 ` [PATCH v2 4/6] bnxt_en: Use direct API instead of indirection Ajit Khaparde
2022-10-27 17:56 ` kernel test robot
2022-10-25 17:31 ` [PATCH v2 5/6] bnxt_en: Use auxiliary bus calls over proprietary calls Ajit Khaparde
2022-10-25 21:18 ` kernel test robot
2022-10-26 3:21 ` kernel test robot
2022-10-25 17:31 ` [PATCH v2 6/6] bnxt_en: Remove struct bnxt access from RoCE driver Ajit Khaparde
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y2inmdbpoRm2VbuE@unreal \
--to=leon@kernel.org \
--cc=ajit.khaparde@broadcom.com \
--cc=andrew.gospodarek@broadcom.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=jgg@ziepe.ca \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
--cc=michael.chan@broadcom.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=selvin.xavier@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).