| Message ID | 20210510114107.43006-1-andriy.shevchenko@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2,1/1] ARM: Drop ARCH_NR_GPIOS definition | expand |
On Mon, May 10, 2021 at 1:40 PM Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > The conditional by the generic header is the same, > hence drop unnecessary duplication. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> I think this should go into Russell's patch tracker for convenience, if you're not familiar with it I can sign it off and put it in there, just tell me. Yours, Linus Walleij
On Thu, May 20, 2021 at 2:26 AM Linus Walleij <linus.walleij@linaro.org> wrote: > On Mon, May 10, 2021 at 1:40 PM Andy Shevchenko > <andriy.shevchenko@linux.intel.com> wrote: > > > The conditional by the generic header is the same, > > hence drop unnecessary duplication. > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > Reviewed-by: Linus Walleij <linus.walleij@linaro.org> > > I think this should go into Russell's patch tracker for convenience, > if you're not familiar with it I can sign it off and put it in there, > just tell me. Nope, I'm not. Please do, thanks! Since he kept silent I suppose it means an agreement on the change. It's quite straightforward and can be tested easily.
On Thu, May 20, 2021 at 11:18:11AM +0300, Andy Shevchenko wrote: > On Thu, May 20, 2021 at 2:26 AM Linus Walleij <linus.walleij@linaro.org> wrote: > > On Mon, May 10, 2021 at 1:40 PM Andy Shevchenko > > <andriy.shevchenko@linux.intel.com> wrote: > > > > > The conditional by the generic header is the same, > > > hence drop unnecessary duplication. > > > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > > > Reviewed-by: Linus Walleij <linus.walleij@linaro.org> > > > > I think this should go into Russell's patch tracker for convenience, > > if you're not familiar with it I can sign it off and put it in there, > > just tell me. > > Nope, I'm not. Please do, thanks! > > Since he kept silent I suppose it means an agreement on the change. > It's quite straightforward and can be tested easily. Any new on this, please? It's kinda not good when the (simplest) patch is stuck like this :-(
On Mon, Jul 12, 2021 at 3:37 PM Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > Any new on this, please? > > It's kinda not good when the (simplest) patch is stuck like this :-( Ooops, my fault. Now it is in Russell's patch tracker: https://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=9103/1 Yours, Linus Walleij
On Fri, Jul 30, 2021 at 11:44:05AM +0200, Linus Walleij wrote: > On Mon, Jul 12, 2021 at 3:37 PM Andy Shevchenko > <andriy.shevchenko@linux.intel.com> wrote: > > > Any new on this, please? > > > > It's kinda not good when the (simplest) patch is stuck like this :-( > > Ooops, my fault. Now it is in Russell's patch tracker: > https://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=9103/1 Thanks!
diff --git a/arch/arm/include/asm/gpio.h b/arch/arm/include/asm/gpio.h index c50e383358c4..f3bb8a2bf788 100644 --- a/arch/arm/include/asm/gpio.h +++ b/arch/arm/include/asm/gpio.h @@ -2,10 +2,6 @@ #ifndef _ARCH_ARM_GPIO_H #define _ARCH_ARM_GPIO_H -#if CONFIG_ARCH_NR_GPIO > 0 -#define ARCH_NR_GPIOS CONFIG_ARCH_NR_GPIO -#endif - /* Note: this may rely upon the value of ARCH_NR_GPIOS set in mach/gpio.h */ #include <asm-generic/gpio.h>
The conditional by the generic header is the same, hence drop unnecessary duplication. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- v2: rebased on top of v5.13-rc1 arch/arm/include/asm/gpio.h | 4 ---- 1 file changed, 4 deletions(-)