| Message ID | 1312582534-10871-2-git-send-email-swarren@nvidia.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, Aug 05, 2011 at 04:15:34PM -0600, Stephen Warren wrote: > irq_to_gpio is deprecated and in the process of being removed. Make that > happen now for ARM Tegra. > > Signed-off-by: Stephen Warren <swarren@nvidia.com> Applied both. Thanks. g. > --- > I assume this will go into one of arm-soc.git's branches, but it depends > on the previous patch in this series to compile. Arnd, I see you already > have Grant's gpio/next tree in your repo as depends/gpio, so I assume > you'll just pick up and merge the latest version of Grant's tree before > applying this patch? > > Both patches compiled with a number of Tegra boards enable and tested on > Harmony. All without Device Tree support. > > arch/arm/mach-tegra/include/mach/gpio.h | 8 -------- > 1 files changed, 0 insertions(+), 8 deletions(-) > > diff --git a/arch/arm/mach-tegra/include/mach/gpio.h b/arch/arm/mach-tegra/include/mach/gpio.h > index 196f114..b357808 100644 > --- a/arch/arm/mach-tegra/include/mach/gpio.h > +++ b/arch/arm/mach-tegra/include/mach/gpio.h > @@ -32,7 +32,6 @@ > #define gpio_cansleep __gpio_cansleep > > #define TEGRA_GPIO_TO_IRQ(gpio) (INT_GPIO_BASE + (gpio)) > -#define TEGRA_IRQ_TO_GPIO(irq) ((irq) - INT_GPIO_BASE) > > static inline int gpio_to_irq(unsigned int gpio) > { > @@ -41,13 +40,6 @@ static inline int gpio_to_irq(unsigned int gpio) > return -EINVAL; > } > > -static inline int irq_to_gpio(unsigned int irq) > -{ > - if ((irq >= INT_GPIO_BASE) && (irq < INT_GPIO_BASE + INT_GPIO_NR)) > - return irq - INT_GPIO_BASE; > - return -EINVAL; > -} > - > struct tegra_gpio_table { > int gpio; /* GPIO number */ > bool enable; /* Enable for GPIO at init? */ > -- > 1.7.0.4 >
Grant Likely wrote at Saturday, August 06, 2011 11:45 AM: > On Fri, Aug 05, 2011 at 04:15:34PM -0600, Stephen Warren wrote: > > irq_to_gpio is deprecated and in the process of being removed. Make that > > happen now for ARM Tegra. > > > > Signed-off-by: Stephen Warren <swarren@nvidia.com> > > Applied both. Thanks. Didn't you want an Ack by a Tegra maintainer first? I assume they were both applied to gpio-next (although a fresh fetch of that doesn't show the commits) > > I assume this will go into one of arm-soc.git's branches, but it depends > > on the previous patch in this series to compile. Arnd, I see you already > > have Grant's gpio/next tree in your repo as depends/gpio, so I assume > > you'll just pick up and merge the latest version of Grant's tree before > > applying this patch? > > > > Both patches compiled with a number of Tegra boards enable and tested on > > Harmony. All without Device Tree support. > > > > arch/arm/mach-tegra/include/mach/gpio.h | 8 -------- > > 1 files changed, 0 insertions(+), 8 deletions(-)
On Mon, Aug 8, 2011 at 7:26 AM, Stephen Warren <swarren@nvidia.com> wrote: > Grant Likely wrote at Saturday, August 06, 2011 11:45 AM: >> On Fri, Aug 05, 2011 at 04:15:34PM -0600, Stephen Warren wrote: >> > irq_to_gpio is deprecated and in the process of being removed. Make that >> > happen now for ARM Tegra. >> > >> > Signed-off-by: Stephen Warren <swarren@nvidia.com> >> >> Applied both. Thanks. > > Didn't you want an Ack by a Tegra maintainer first? Acked-by: Olof Johansson <olof@lixom.net> But either way is OK. No need to rebase to add an acked-by, as far as I am concerned. -Olof
On Aug 8, 2011 8:26 AM, "Stephen Warren" <swarren@nvidia.com> wrote: > > Grant Likely wrote at Saturday, August 06, 2011 11:45 AM: > > On Fri, Aug 05, 2011 at 04:15:34PM -0600, Stephen Warren wrote: > > > irq_to_gpio is deprecated and in the process of being removed. Make that > > > happen now for ARM Tegra. > > > > > > Signed-off-by: Stephen Warren <swarren@nvidia.com> > > > > Applied both. Thanks. > > Didn't you want an Ack by a Tegra maintainer first? It looks like a pretty low risk change. I didn't see much risk in picking it up and getting it off my plate. I can always pull things back out of my tree if someone complains. I'll add Olof's ack before it gets into linux-next. > > I assume they were both applied to gpio-next (although a fresh fetch of that > doesn't show the commits) I can't push it out until after 3.1-rc1 is released because it is targeted at 3.2 now. g.
On Mon, Aug 08, 2011 at 11:50:47AM -0600, Grant Likely wrote: > On Aug 8, 2011 8:26 AM, "Stephen Warren" <swarren@nvidia.com> wrote: > > > > Grant Likely wrote at Saturday, August 06, 2011 11:45 AM: > > > On Fri, Aug 05, 2011 at 04:15:34PM -0600, Stephen Warren wrote: > > > > irq_to_gpio is deprecated and in the process of being removed. Make that > > > > happen now for ARM Tegra. > > > > > > > > Signed-off-by: Stephen Warren <swarren@nvidia.com> > > > > > > Applied both. Thanks. > > > > Didn't you want an Ack by a Tegra maintainer first? > > It looks like a pretty low risk change. I didn't see much risk in > picking it up and getting it off my plate. I can always pull things > back out of my tree if someone complains. I'll add Olof's ack before > it gets into linux-next. > > > > > I assume they were both applied to gpio-next (although a fresh fetch of that > > doesn't show the commits) > > I can't push it out until after 3.1-rc1 is released because it is > targeted at 3.2 now. It'd probably make sense for it to be merged into my tree along with the rest of my gpio changes, otherwise we're going to head for merge conflicts. I'll be publishing those gpio.h changes in the next few days.
On Mon, Aug 8, 2011 at 12:05 PM, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote: > On Mon, Aug 08, 2011 at 11:50:47AM -0600, Grant Likely wrote: >> On Aug 8, 2011 8:26 AM, "Stephen Warren" <swarren@nvidia.com> wrote: >> > >> > Grant Likely wrote at Saturday, August 06, 2011 11:45 AM: >> > > On Fri, Aug 05, 2011 at 04:15:34PM -0600, Stephen Warren wrote: >> > > > irq_to_gpio is deprecated and in the process of being removed. Make that >> > > > happen now for ARM Tegra. >> > > > >> > > > Signed-off-by: Stephen Warren <swarren@nvidia.com> >> > > >> > > Applied both. Thanks. >> > >> > Didn't you want an Ack by a Tegra maintainer first? >> >> It looks like a pretty low risk change. I didn't see much risk in >> picking it up and getting it off my plate. I can always pull things >> back out of my tree if someone complains. I'll add Olof's ack before >> it gets into linux-next. >> >> > >> > I assume they were both applied to gpio-next (although a fresh fetch of that >> > doesn't show the commits) >> >> I can't push it out until after 3.1-rc1 is released because it is >> targeted at 3.2 now. > > It'd probably make sense for it to be merged into my tree along with > the rest of my gpio changes, otherwise we're going to head for merge > conflicts. I'll be publishing those gpio.h changes in the next few > days. Okay, I'll drop it from my tree and you can add my: Acked-by: Grant Likely <grant.likely@secretlab.ca>
diff --git a/arch/arm/mach-tegra/include/mach/gpio.h b/arch/arm/mach-tegra/include/mach/gpio.h index 196f114..b357808 100644 --- a/arch/arm/mach-tegra/include/mach/gpio.h +++ b/arch/arm/mach-tegra/include/mach/gpio.h @@ -32,7 +32,6 @@ #define gpio_cansleep __gpio_cansleep #define TEGRA_GPIO_TO_IRQ(gpio) (INT_GPIO_BASE + (gpio)) -#define TEGRA_IRQ_TO_GPIO(irq) ((irq) - INT_GPIO_BASE) static inline int gpio_to_irq(unsigned int gpio) { @@ -41,13 +40,6 @@ static inline int gpio_to_irq(unsigned int gpio) return -EINVAL; } -static inline int irq_to_gpio(unsigned int irq) -{ - if ((irq >= INT_GPIO_BASE) && (irq < INT_GPIO_BASE + INT_GPIO_NR)) - return irq - INT_GPIO_BASE; - return -EINVAL; -} - struct tegra_gpio_table { int gpio; /* GPIO number */ bool enable; /* Enable for GPIO at init? */
irq_to_gpio is deprecated and in the process of being removed. Make that happen now for ARM Tegra. Signed-off-by: Stephen Warren <swarren@nvidia.com> --- I assume this will go into one of arm-soc.git's branches, but it depends on the previous patch in this series to compile. Arnd, I see you already have Grant's gpio/next tree in your repo as depends/gpio, so I assume you'll just pick up and merge the latest version of Grant's tree before applying this patch? Both patches compiled with a number of Tegra boards enable and tested on Harmony. All without Device Tree support. arch/arm/mach-tegra/include/mach/gpio.h | 8 -------- 1 files changed, 0 insertions(+), 8 deletions(-)