2022-10-24 14:31:01

by Linus Walleij

[permalink] [raw]
Subject: [PATCH] bcma: Use the proper include

The <linux/bcma/bcma_driver_chipcommon.h> is including the legacy
header <linux/gpio.h> to obtain struct gpio_chip. Instead, include
<linux/gpio/driver.h> where this struct is defined.

Signed-off-by: Linus Walleij <[email protected]>
---
include/linux/bcma/bcma_driver_chipcommon.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/bcma/bcma_driver_chipcommon.h b/include/linux/bcma/bcma_driver_chipcommon.h
index 2d94c30ed439..0cb6638b55e5 100644
--- a/include/linux/bcma/bcma_driver_chipcommon.h
+++ b/include/linux/bcma/bcma_driver_chipcommon.h
@@ -4,7 +4,7 @@

#include <linux/platform_device.h>
#include <linux/platform_data/brcmnand.h>
-#include <linux/gpio.h>
+#include <linux/gpio/driver.h>

/** ChipCommon core registers. **/
#define BCMA_CC_ID 0x0000
--
2.34.1


2022-10-25 12:51:58

by Arend van Spriel

[permalink] [raw]
Subject: Re: [PATCH] bcma: Use the proper include

On 10/24/2022 2:28 PM, Linus Walleij wrote:
> The <linux/bcma/bcma_driver_chipcommon.h> is including the legacy
> header <linux/gpio.h> to obtain struct gpio_chip. Instead, include
> <linux/gpio/driver.h> where this struct is defined.

Reviewed-by: Arend van Spriel <[email protected]>
> Signed-off-by: Linus Walleij <[email protected]>
> ---
> include/linux/bcma/bcma_driver_chipcommon.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)


Attachments:
smime.p7s (4.12 kB)
S/MIME Cryptographic Signature

2022-10-26 08:03:02

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH] bcma: Use the proper include

On Mon, Oct 24, 2022 at 2:28 PM Linus Walleij <[email protected]> wrote:

> The <linux/bcma/bcma_driver_chipcommon.h> is including the legacy
> header <linux/gpio.h> to obtain struct gpio_chip. Instead, include
> <linux/gpio/driver.h> where this struct is defined.
>
> Signed-off-by: Linus Walleij <[email protected]>

I'm squashing this with the other patch and resending due to the co-dependence
found by the kernel test robot.

Yours,
Linus Walleij