From: "Cousson, Benoit" <b-cousson@ti.com> To: "Varadarajan, Charulatha" <charu@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> Subject: Re: [PATCH v7 05/11] OMAP2420: hwmod data: Add GPIO Date: Thu, 25 Nov 2010 08:55:35 +0100 [thread overview] Message-ID: <4CEE1677.8030109@ti.com> (raw) In-Reply-To: <AANLkTimRK=eKBG3pqGN8Ota5EO8EdmEibQN5C3Lwfheq@mail.gmail.com> On 11/25/2010 5:36 AM, Varadarajan, Charulatha wrote: > Benoit, > > On Thu, Nov 25, 2010 at 03:43, Cousson, Benoit<b-cousson@ti.com> wrote: >> On 11/23/2010 3:56 PM, Varadarajan, Charulatha wrote: >>> >>> Add GPIO hwmod data for OMAP2420 >>> >>> Signed-off-by: Charulatha V<charu@ti.com> [...] >>> diff --git a/arch/arm/plat-omap/include/plat/gpio.h >>> b/arch/arm/plat-omap/include/plat/gpio.h >> >> That change should not necessarily be there, it is not directly related to >> the subject. >> >> Maybe that should be in an extra patch just before that one. > > I merged these two patches because of a comment to introduce > new variables/structures only in the patch where they are used. > Since "omap_gpio_dev_attr" is being used by hwmod DB I merged > these two patches. Do you think that hwmod DB patches should not > be merged with other patches? Not necessarily, maybe you can just update the changelog and the subject to indicate that you add a new structure as well in this patch. Benoit
WARNING: multiple messages have this Message-ID (diff)
From: b-cousson@ti.com (Cousson, Benoit) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v7 05/11] OMAP2420: hwmod data: Add GPIO Date: Thu, 25 Nov 2010 08:55:35 +0100 [thread overview] Message-ID: <4CEE1677.8030109@ti.com> (raw) In-Reply-To: <AANLkTimRK=eKBG3pqGN8Ota5EO8EdmEibQN5C3Lwfheq@mail.gmail.com> On 11/25/2010 5:36 AM, Varadarajan, Charulatha wrote: > Benoit, > > On Thu, Nov 25, 2010 at 03:43, Cousson, Benoit<b-cousson@ti.com> wrote: >> On 11/23/2010 3:56 PM, Varadarajan, Charulatha wrote: >>> >>> Add GPIO hwmod data for OMAP2420 >>> >>> Signed-off-by: Charulatha V<charu@ti.com> [...] >>> diff --git a/arch/arm/plat-omap/include/plat/gpio.h >>> b/arch/arm/plat-omap/include/plat/gpio.h >> >> That change should not necessarily be there, it is not directly related to >> the subject. >> >> Maybe that should be in an extra patch just before that one. > > I merged these two patches because of a comment to introduce > new variables/structures only in the patch where they are used. > Since "omap_gpio_dev_attr" is being used by hwmod DB I merged > these two patches. Do you think that hwmod DB patches should not > be merged with other patches? Not necessarily, maybe you can just update the changelog and the subject to indicate that you add a new structure as well in this patch. Benoit
next prev parent reply other threads:[~2010-11-25 7:55 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 [this message] 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 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=4CEE1677.8030109@ti.com \ --to=b-cousson@ti.com \ --cc=charu@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=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.