From: "Varadarajan, Charulatha" <charu@ti.com> To: "Cousson, Benoit" <b-cousson@ti.com> Cc: "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "khilman@deeprootsystems.com" <khilman@deeprootsystems.com>, "paul@pwsan.com" <paul@pwsan.com>, "tony@atomide.com" <tony@atomide.com>, "Basak, Partha" <p-basak2@ti.com>, "Nayak, Rajendra" <rnayak@ti.com> Subject: Re: [PATCH v7 07/11] OMAP3: hwmod data: Add GPIO Date: Thu, 25 Nov 2010 10:07:45 +0530 [thread overview] Message-ID: <AANLkTik4dtCv9uVbL4dJgw+mpK-WWGZykUHSsKv4zvEZ@mail.gmail.com> (raw) In-Reply-To: <4CED96EC.1040608@ti.com> Benoit, On Thu, Nov 25, 2010 at 04:21, Cousson, Benoit <b-cousson@ti.com> wrote: > On 11/23/2010 3:56 PM, Varadarajan, Charulatha wrote: >> >> Add GPIO hwmod data for OMAP3 >> >> Signed-off-by: Charulatha V<charu@ti.com> >> Signed-off-by: Rajendra Nayak<rnayak@ti.com> > > Acked-by: Benoit Cousson <b-cousson@ti.com> Thanks. > >> --- >> arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 360 >> ++++++++++++++++++++++++++++ >> 1 files changed, 360 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> b/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> index a8bed84..9ca2206 100644 >> --- a/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> +++ b/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> @@ -20,6 +20,7 @@ >> #include<plat/serial.h> >> #include<plat/l4_3xxx.h> >> #include<plat/i2c.h> >> +#include<plat/gpio.h> >> #include<plat/omap34xx.h> >> >> #include "omap_hwmod_common_data.h" >> @@ -45,6 +46,12 @@ static struct omap_hwmod omap3xxx_wd_timer2_hwmod; >> static struct omap_hwmod omap3xxx_i2c1_hwmod; >> static struct omap_hwmod omap3xxx_i2c2_hwmod; <<snip>> >> +}; >> + >> +/* gpio1 */ >> +static struct omap_hwmod_irq_info omap3xxx_gpio1_irqs[] = { >> + { .name = "gpio_mpu_irq", .irq = 29 }, /* INT_34XX_GPIO_BANK1 */ > > In this case the name is not necessarily needed since there is only one irq > line. > You can save some memory by just adding the .irq field. Yes, I too thought about it. But to be consistent with other devices, I included it here as I did not receive any comments before on this. Anyways, will remove them while re-sending patches as per Kevin's suggestions to rebase them on pm-core branch. > > Benoit > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: charu@ti.com (Varadarajan, Charulatha) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v7 07/11] OMAP3: hwmod data: Add GPIO Date: Thu, 25 Nov 2010 10:07:45 +0530 [thread overview] Message-ID: <AANLkTik4dtCv9uVbL4dJgw+mpK-WWGZykUHSsKv4zvEZ@mail.gmail.com> (raw) In-Reply-To: <4CED96EC.1040608@ti.com> Benoit, On Thu, Nov 25, 2010 at 04:21, Cousson, Benoit <b-cousson@ti.com> wrote: > On 11/23/2010 3:56 PM, Varadarajan, Charulatha wrote: >> >> Add GPIO hwmod data for OMAP3 >> >> Signed-off-by: Charulatha V<charu@ti.com> >> Signed-off-by: Rajendra Nayak<rnayak@ti.com> > > Acked-by: Benoit Cousson <b-cousson@ti.com> Thanks. > >> --- >> ?arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | ?360 >> ++++++++++++++++++++++++++++ >> ?1 files changed, 360 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> b/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> index a8bed84..9ca2206 100644 >> --- a/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> +++ b/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c >> @@ -20,6 +20,7 @@ >> ?#include<plat/serial.h> >> ?#include<plat/l4_3xxx.h> >> ?#include<plat/i2c.h> >> +#include<plat/gpio.h> >> ?#include<plat/omap34xx.h> >> >> ?#include "omap_hwmod_common_data.h" >> @@ -45,6 +46,12 @@ static struct omap_hwmod omap3xxx_wd_timer2_hwmod; >> ?static struct omap_hwmod omap3xxx_i2c1_hwmod; >> ?static struct omap_hwmod omap3xxx_i2c2_hwmod; <<snip>> >> +}; >> + >> +/* gpio1 */ >> +static struct omap_hwmod_irq_info omap3xxx_gpio1_irqs[] = { >> + ? ? ? { .name = "gpio_mpu_irq", .irq = 29 }, /* INT_34XX_GPIO_BANK1 */ > > In this case the name is not necessarily needed since there is only one irq > line. > You can save some memory by just adding the .irq field. Yes, I too thought about it. But to be consistent with other devices, I included it here as I did not receive any comments before on this. Anyways, will remove them while re-sending patches as per Kevin's suggestions to rebase them on pm-core branch. > > Benoit >
next prev parent reply other threads:[~2010-11-25 4:38 UTC|newest] Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top 2010-11-23 14:56 [PATCH v7 00/11] OMAP: GPIO: Implement GPIO as a platform device Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 01/11] OMAP: GPIO: prepare for platform driver Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-26 4:38 ` Olof Johansson 2010-11-26 4:38 ` Olof Johansson 2010-11-26 5:03 ` Varadarajan, Charulatha 2010-11-26 5:03 ` Varadarajan, Charulatha 2010-11-26 6:47 ` [Query]McSPI testing help on N800 Datta, Shubhrajyoti 2010-11-28 20:15 ` Paul Walmsley 2010-11-23 14:56 ` [PATCH v7 02/11] OMAP15xx: GPIO: Introduce support for GPIO init Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 03/11] OMAP16xx: " Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 04/11] OMAP7xx: " Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 05/11] OMAP2420: hwmod data: Add GPIO Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-24 22:13 ` Cousson, Benoit 2010-11-24 22:13 ` Cousson, Benoit 2010-11-25 4:36 ` Varadarajan, Charulatha 2010-11-25 4:36 ` Varadarajan, Charulatha 2010-11-25 7:55 ` Cousson, Benoit 2010-11-25 7:55 ` Cousson, Benoit 2010-11-25 8:02 ` Cousson, Benoit 2010-11-25 8:02 ` Cousson, Benoit 2010-11-25 8:31 ` Varadarajan, Charulatha 2010-11-25 8:31 ` Varadarajan, Charulatha 2010-11-25 9:33 ` Varadarajan, Charulatha 2010-11-25 9:33 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 06/11] OMAP2430: " Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-24 22:42 ` Cousson, Benoit 2010-11-24 22:42 ` Cousson, Benoit 2010-11-23 14:56 ` [PATCH v7 07/11] OMAP3: " Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-24 22:51 ` Cousson, Benoit 2010-11-24 22:51 ` Cousson, Benoit 2010-11-25 4:37 ` Varadarajan, Charulatha [this message] 2010-11-25 4:37 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 08/11] OMAP4: " Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 09/11] OMAP2+: GPIO: device registration Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-24 23:00 ` Cousson, Benoit 2010-11-24 23:00 ` Cousson, Benoit 2010-11-23 14:56 ` [PATCH v7 10/11] OMAP: GPIO: Implement GPIO as a platform device Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-23 14:56 ` [PATCH v7 11/11] OMAP: GPIO: Remove omap_gpio_init() Varadarajan, Charulatha 2010-11-23 14:56 ` Varadarajan, Charulatha 2010-11-24 23:28 ` Tony Lindgren 2010-11-24 23:28 ` Tony Lindgren 2010-11-24 20:58 ` [PATCH v7 00/11] OMAP: GPIO: Implement GPIO as a platform device Cousson, Benoit 2010-11-25 4:27 ` Varadarajan, Charulatha 2010-11-25 1:40 ` Kevin Hilman 2010-11-25 1:40 ` Kevin Hilman 2010-11-25 4:31 ` Varadarajan, Charulatha 2010-11-25 4:31 ` Varadarajan, Charulatha 2010-11-25 6:19 ` Kevin Hilman 2010-11-25 6:19 ` Kevin Hilman 2010-11-25 6:20 ` Varadarajan, Charulatha 2010-11-25 6:20 ` Varadarajan, Charulatha
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=AANLkTik4dtCv9uVbL4dJgw+mpK-WWGZykUHSsKv4zvEZ@mail.gmail.com \ --to=charu@ti.com \ --cc=b-cousson@ti.com \ --cc=khilman@deeprootsystems.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=p-basak2@ti.com \ --cc=paul@pwsan.com \ --cc=rnayak@ti.com \ --cc=tony@atomide.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.