Re: [PATCH net-next v2 04/11] bnxt_en: introduce new firmware message API based on DMA pools
From: Michael Chan <michael.chan@broadcom.com>
Date: 2021-09-01 17:35:19
On Wed, Sep 1, 2021 at 9:57 AM Guenter Roeck [off-list ref] wrote:
On Sun, Aug 29, 2021 at 03:34:59AM -0400, Michael Chan wrote:quoted
From: Edwin Peer <redacted> This change constitutes a major step towards supporting multiple firmware commands in flight by maintaining a separate response buffer for the duration of each request. These firmware commands are also known as Hardware Resource Manager (HWRM) commands. Using separate response buffers requires an API change in order for callers to be able to free the buffer when done.parisc:allmodconfig, and probably others: drivers/net/ethernet/broadcom/bnxt/bnxt.c: In function 'bnxt_init_board': drivers/net/ethernet/broadcom/bnxt/bnxt.c:12280:27: error: 'struct bnxt' has no member named 'db_lock' There is a difference between "#ifdef writeq" and "#if BITS_PER_LONG == 32".
Yes. It is already fixed in this commit: c6132f6f2e682c958f7022ecfd8bec35723a1a9d bnxt_en: Fix 64-bit doorbell operation on 32-bit kernels Thanks.