[v1] ARM: Drop ARCH_NR_GPIOS definition

Message ID 20201012154709.68521-1-andriy.shevchenko@linux.intel.com
State New
Headers show
Series
  • [v1] ARM: Drop ARCH_NR_GPIOS definition
Related show

Commit Message

Andy Shevchenko Oct. 12, 2020, 3:47 p.m.
The conditional by the generic header is the same,
hence drop unnecessary duplication.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 arch/arm/include/asm/gpio.h | 4 ----
 1 file changed, 4 deletions(-)

Comments

Andy Shevchenko Nov. 9, 2020, 12:13 p.m. | #1
On Mon, Oct 12, 2020 at 06:47:09PM +0300, Andy Shevchenko wrote:
> The conditional by the generic header is the same,

> hence drop unnecessary duplication.


Any comment on this?

> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> ---

>  arch/arm/include/asm/gpio.h | 4 ----

>  1 file changed, 4 deletions(-)

> 

> 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>

>  

> -- 

> 2.28.0

> 


-- 
With Best Regards,
Andy Shevchenko
Andy Shevchenko Jan. 21, 2021, 5:13 p.m. | #2
On Mon, Oct 12, 2020 at 06:47:09PM +0300, Andy Shevchenko wrote:
> The conditional by the generic header is the same,

> hence drop unnecessary duplication.


Can we get this proceed somehow?

> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> ---

>  arch/arm/include/asm/gpio.h | 4 ----

>  1 file changed, 4 deletions(-)

> 

> 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>

>  

> -- 

> 2.28.0

> 


-- 
With Best Regards,
Andy Shevchenko
Andy Shevchenko Feb. 24, 2021, 1:14 p.m. | #3
On Mon, Oct 12, 2020 at 06:47:09PM +0300, Andy Shevchenko wrote:
> The conditional by the generic header is the same,

> hence drop unnecessary duplication.


Any comment on this anybody?

> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> ---

>  arch/arm/include/asm/gpio.h | 4 ----

>  1 file changed, 4 deletions(-)

> 

> 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>

>  

> -- 

> 2.28.0

> 


-- 
With Best Regards,
Andy Shevchenko
Andy Shevchenko March 1, 2021, 5:10 p.m. | #4
On Wed, Feb 24, 2021 at 4:21 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>

> On Mon, Oct 12, 2020 at 06:47:09PM +0300, Andy Shevchenko wrote:

> > The conditional by the generic header is the same,

> > hence drop unnecessary duplication.

>

> Any comment on this anybody?


+Cc: Linus and Arnd, maybe you can help to advance this to somewhere?

> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> > ---

> >  arch/arm/include/asm/gpio.h | 4 ----

> >  1 file changed, 4 deletions(-)

> >

> > 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>

> >

> > --

> > 2.28.0

> >

>

> --

> With Best Regards,

> Andy Shevchenko

>

>



-- 
With Best Regards,
Andy Shevchenko
Linus Walleij March 2, 2021, 3:51 p.m. | #5
Hi Andy,

On Mon, Oct 12, 2020 at 5:47 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>

sorry for extremely slow reply.

I think we could merge this into the GPIO tree if Russell is OK with it,
alternatively put it in Russell's patch tracker. It looks absolutely correct.

Yours,
Linus Walleij
Andy Shevchenko May 10, 2021, 11:41 a.m. | #6
On Tue, Mar 02, 2021 at 04:51:37PM +0100, Linus Walleij wrote:
> Hi Andy,

> 

> On Mon, Oct 12, 2020 at 5:47 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>

> 

> sorry for extremely slow reply.

> 

> I think we could merge this into the GPIO tree if Russell is OK with it,

> alternatively put it in Russell's patch tracker. It looks absolutely correct.


I have sent a v2 (same patch, just in case somebody lost this thread).

-- 
With Best Regards,
Andy Shevchenko

Patch

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>