[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20201207163251.184B6C43463@smtp.codeaurora.org>
Date: Mon, 7 Dec 2020 16:32:51 +0000 (UTC)
From: Kalle Valo <kvalo@...eaurora.org>
To: Lee Jones <lee.jones@...aro.org>
Cc: lee.jones@...aro.org, linux-kernel@...r.kernel.org,
Yan-Hsuan Chuang <yhchuang@...ltek.com>,
"David S. Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>,
linux-wireless@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [17/17] rtw88: pci: Add prototypes for .probe,
.remove and .shutdown
Lee Jones <lee.jones@...aro.org> wrote:
> Also strip out other duplicates from driver specific headers.
>
> Ensure 'main.h' is explicitly included in 'pci.h' since the latter
> uses some defines from the former. It avoids issues like:
>
> from drivers/net/wireless/realtek/rtw88/rtw8822be.c:5:
> drivers/net/wireless/realtek/rtw88/pci.h:209:28: error: ‘RTK_MAX_TX_QUEUE_NUM’ undeclared here (not in a function); did you mean ‘RTK_MAX_RX_DESC_NUM’?
> 209 | DECLARE_BITMAP(tx_queued, RTK_MAX_TX_QUEUE_NUM);
> | ^~~~~~~~~~~~~~~~~~~~
>
> Fixes the following W=1 kernel build warning(s):
>
> drivers/net/wireless/realtek/rtw88/pci.c:1488:5: warning: no previous prototype for ‘rtw_pci_probe’ [-Wmissing-prototypes]
> 1488 | int rtw_pci_probe(struct pci_dev *pdev,
> | ^~~~~~~~~~~~~
> drivers/net/wireless/realtek/rtw88/pci.c:1568:6: warning: no previous prototype for ‘rtw_pci_remove’ [-Wmissing-prototypes]
> 1568 | void rtw_pci_remove(struct pci_dev *pdev)
> | ^~~~~~~~~~~~~~
> drivers/net/wireless/realtek/rtw88/pci.c:1590:6: warning: no previous prototype for ‘rtw_pci_shutdown’ [-Wmissing-prototypes]
> 1590 | void rtw_pci_shutdown(struct pci_dev *pdev)
> | ^~~~~~~~~~~~~~~~
>
> Cc: Yan-Hsuan Chuang <yhchuang@...ltek.com>
> Cc: Kalle Valo <kvalo@...eaurora.org>
> Cc: "David S. Miller" <davem@...emloft.net>
> Cc: Jakub Kicinski <kuba@...nel.org>
> Cc: linux-wireless@...r.kernel.org
> Cc: netdev@...r.kernel.org
> Signed-off-by: Lee Jones <lee.jones@...aro.org>
Patch applied to wireless-drivers-next.git, thanks.
2e86ef413ab3 rtw88: pci: Add prototypes for .probe, .remove and .shutdown
--
https://patchwork.kernel.org/project/linux-wireless/patch/20201126133152.3211309-18-lee.jones@linaro.org/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
Powered by blists - more mailing lists