| Message ID | 20240907160926.39556-1-riyandhiman14@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [next,V2] PCI: brmstb: Fix type mismatch for num_inbound_wins in brcm_pcie_get_inbound_wins | expand |
Hello, > Change num_inbound_wins from u8 to int to correctly handle > potential negative error codes returned by brcm_pcie_get_inbound_wins(). > The u8 type was inappropriate for capturing the function's return value, > which can include error codes. I squashed with the current code on the controller/brcmstb branch, see: - https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git/commit/?h=controller/brcmstb&id=ae6476c6de187bea90c729e3e0188143300fa671 And credited you via the Co-developed-by tag such that you get credit for fixing this issue. Thank you, by the way. There is no Fixes: tag as this code is not yet merged into the mainline. Krzysztof
Hello, > > Change num_inbound_wins from u8 to int to correctly handle > > potential negative error codes returned by brcm_pcie_get_inbound_wins(). > > The u8 type was inappropriate for capturing the function's return value, > > which can include error codes. > > I squashed with the current code on the controller/brcmstb branch, see: > > - https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git/commit/?h=controller/brcmstb&id=ae6476c6de187bea90c729e3e0188143300fa671 > > And credited you via the Co-developed-by tag such that you get credit for > fixing this issue. Thank you, by the way. > > There is no Fixes: tag as this code is not yet merged into the mainline. For the record, I would prefer if we went with Florian's first recommendation per the following message: - https://lore.kernel.org/linux-pci/159c5fcf-709d-42ba-8d45-a70b109fe261@broadcom.com To clearly separate errors returned from the value being updated. Albeit, this is fine too, especially as Jim expressed no opinion either way regarding his preference. Krzysztof
diff --git a/drivers/pci/controller/pcie-brcmstb.c b/drivers/pci/controller/pcie-brcmstb.c index e8332fe5396e..b2859c4fd931 100644 --- a/drivers/pci/controller/pcie-brcmstb.c +++ b/drivers/pci/controller/pcie-brcmstb.c @@ -1030,7 +1030,8 @@ static int brcm_pcie_setup(struct brcm_pcie *pcie) struct pci_host_bridge *bridge; struct resource_entry *entry; u32 tmp, burst, aspm_support; - u8 num_out_wins = 0, num_inbound_wins = 0; + u8 num_out_wins = 0; + int num_inbound_wins = 0; int memc, ret; /* Reset the bridge */
Change num_inbound_wins from u8 to int to correctly handle potential negative error codes returned by brcm_pcie_get_inbound_wins(). The u8 type was inappropriate for capturing the function's return value, which can include error codes. Signed-off-by: Riyan Dhiman <riyandhiman14@gmail.com> --- V2: Added missing semicolon in variable declaration drivers/pci/controller/pcie-brcmstb.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)