Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [net-next][bisected da41788] modules fails to build
 drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c:12199
To:     Abdul Haleem <abdhalee@linux.vnet.ibm.com>, davem@davemloft.net,
        netdev@vger.kernel.org
Cc:     linux-kernel <linux-kernel@vger.kernel.org>,
        linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
        Brian King <brking@linux.vnet.ibm.com>, kuba@kernel.org
References: <16a5465e-2357-3277-e2eb-dc0668e6cb01@linux.vnet.ibm.com>
From:   Heiner Kallweit <hkallweit1@gmail.com>
Message-ID: <6c87b191-13c3-c52d-34ad-79e23b5d71a6@gmail.com>
Date:   Mon, 23 Aug 2021 08:38:58 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.13.0
MIME-Version: 1.0
In-Reply-To: <16a5465e-2357-3277-e2eb-dc0668e6cb01@linux.vnet.ibm.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 23.08.2021 08:00, Abdul Haleem wrote:
> Greeting's
> 
> Todays net-next branch fails to build on my powerpc box with below error
> 
>   CC [M]  drivers/gpu/drm/virtio/virtgpu_trace_points.o
>   CC [M]  drivers/net/ethernet/atheros/atl1e/atl1e_ethtool.o
> drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c: In function ‘bnx2x_read_fwinfo’:
> drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c:12199:13: error: implicit declaration of function ‘pci_vpd_alloc’; did you mean ‘pci_pool_alloc’? [-Werror=implicit-function-declaration]
>   vpd_data = pci_vpd_alloc(bp->pdev, &vpd_len);
>              ^~~~~~~~~~~~~
>              pci_pool_alloc
> drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c:12199:11: warning: assignment to ‘u8 *’ {aka ‘unsigned char *’} from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
>   vpd_data = pci_vpd_alloc(bp->pdev, &vpd_len);
>            ^
>   CC [M]  drivers/scsi/qla2xxx/qla_nx.o
>   CC [M]  net/netfilter/xt_MASQUERADE.o
>   CC [M]  net/bluetooth/ecdh_helper.o
>   CC [M]  drivers/gpu/drm/amd/amdgpu/uvd_v5_0.o
> drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c:12203:9: error: implicit declaration of function ‘pci_vpd_find_ro_info_keyword’; did you mean ‘pci_vpd_find_info_keyword’? [-Werror=implicit-function-declaration]
>   rodi = pci_vpd_find_ro_info_keyword(vpd_data, vpd_len,
>          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
>          pci_vpd_find_info_keyword
>   CC [M]  net/netfilter/xt_SECMARK.o
>   CC [M]  net/netfilter/xt_TPROXY.o
>   CC [M]  net/mac80211/driver-ops.o
> 
> 
> The recent changes to the code causing build fail is
> 
> commit da417885a99d36036cc7d2778f94b846e6582434
> Author: Heiner Kallweit <hkallweit1@gmail.com>
> Date:   Sun Aug 22 15:54:23 2021 +0200
> 
>     bnx2x: Search VPD with pci_vpd_find_ro_info_keyword()
>         Use pci_vpd_find_ro_info_keyword() to search for keywords in VPD to
>     simplify the code.
>         str_id_reg and str_id_cap hold the same string and are used in the same
>     comparison. This doesn't make sense, use one string str_id instead.
>         Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
>     Signed-off-by: David S. Miller <davem@davemloft.net>
> 
> @Heiner, Could you please have a look, I am attaching the kernel config
> 
This series was supposed to go via the PCI tree because it builds on patches
that are in the PCI tree but not in linux-next and net-next yet.