* [PATCH] MAINTAINERS: Add Samsung pinctrl entries @ 2013-06-13 17:05 Doug Anderson 2013-06-13 17:13 ` Tomasz Figa 2013-06-16 22:17 ` [PATCH v2 1/2] " Doug Anderson 0 siblings, 2 replies; 25+ messages in thread From: Doug Anderson @ 2013-06-13 17:05 UTC (permalink / raw) To: Stephen Rothwell Cc: linux-samsung-soc, Linus Walleij, Tomasz Figa, Kukjin Kim, Olof Johansson, Doug Anderson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel It's convenient if get_maintainer suggests sending samsung/exynos pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry for that. Signed-off-by: Doug Anderson <dianders@chromium.org> --- MAINTAINERS | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 8d97b3e..d026e38 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -6284,6 +6284,16 @@ L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) S: Maintained F: drivers/pinctrl/pinctrl-at91.c +PIN CONTROLLER - SAMSUNG +M: Linus Walleij <linus.walleij@linaro.org> +M: Kukjin Kim <kgene.kim@samsung.com> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) +L: linux-samsung-soc@vger.kernel.org (moderated for non-subscribers) +S: Maintained +F: drivers/pinctrl/pinctrl-exynos.* +F: drivers/pinctrl/pinctrl-s3c* +F: drivers/pinctrl/pinctrl-samsung.* + PIN CONTROLLER - ST SPEAR M: Viresh Kumar <viresh.linux@gmail.com> L: spear-devel@list.st.com -- 1.8.3 ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:05 [PATCH] MAINTAINERS: Add Samsung pinctrl entries Doug Anderson @ 2013-06-13 17:13 ` Tomasz Figa 2013-06-13 17:25 ` Doug Anderson 2013-06-13 22:47 ` Kukjin Kim 2013-06-16 22:17 ` [PATCH v2 1/2] " Doug Anderson 1 sibling, 2 replies; 25+ messages in thread From: Tomasz Figa @ 2013-06-13 17:13 UTC (permalink / raw) To: Doug Anderson Cc: Stephen Rothwell, linux-samsung-soc, Linus Walleij, Kukjin Kim, Olof Johansson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel, thomas.abraham On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > It's convenient if get_maintainer suggests sending samsung/exynos > pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry for > that. Well, if we are already at it, we could list the actual maintainers of the driver. The original maintainer was Thomas Abraham <thomas.abraham@linaro.org> (added to CC) and after joining to development of this driver last year I have become a de facto maintainer as well, so... > Signed-off-by: Doug Anderson <dianders@chromium.org> > --- > MAINTAINERS | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 8d97b3e..d026e38 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -6284,6 +6284,16 @@ L: linux-arm-kernel@lists.infradead.org > (moderated for non-subscribers) S: Maintained > F: drivers/pinctrl/pinctrl-at91.c > > +PIN CONTROLLER - SAMSUNG > +M: Linus Walleij <linus.walleij@linaro.org> > +M: Kukjin Kim <kgene.kim@samsung.com> So actually this would be: M: Thomas Abraham <thomas.abraham@linaro.org> M: Tomasz Figa <t.figa@samsung.com> Thomas, do you agree on this? Best regards, Tomasz > +L: linux-arm-kernel@lists.infradead.org (moderated for non- subscribers) > +L: linux-samsung-soc@vger.kernel.org (moderated for non-subscribers) > +S: Maintained > +F: drivers/pinctrl/pinctrl-exynos.* > +F: drivers/pinctrl/pinctrl-s3c* > +F: drivers/pinctrl/pinctrl-samsung.* > + > PIN CONTROLLER - ST SPEAR > M: Viresh Kumar <viresh.linux@gmail.com> > L: spear-devel@list.st.com ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:13 ` Tomasz Figa @ 2013-06-13 17:25 ` Doug Anderson 2013-06-13 17:32 ` Tomasz Figa 2013-06-13 18:25 ` Linus Walleij 2013-06-13 22:47 ` Kukjin Kim 1 sibling, 2 replies; 25+ messages in thread From: Doug Anderson @ 2013-06-13 17:25 UTC (permalink / raw) To: Tomasz Figa Cc: Stephen Rothwell, linux-samsung-soc, Linus Walleij, Kukjin Kim, Olof Johansson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org, Thomas Abraham Tomasz, On Thu, Jun 13, 2013 at 10:13 AM, Tomasz Figa <tomasz.figa@gmail.com> wrote: > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: >> It's convenient if get_maintainer suggests sending samsung/exynos >> pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry for >> that. > > Well, if we are already at it, we could list the actual maintainers of the > driver. > > The original maintainer was Thomas Abraham <thomas.abraham@linaro.org> > (added to CC) and after joining to development of this driver last year I > have become a de facto maintainer as well, so... > >> Signed-off-by: Doug Anderson <dianders@chromium.org> >> --- >> MAINTAINERS | 10 ++++++++++ >> 1 file changed, 10 insertions(+) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index 8d97b3e..d026e38 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -6284,6 +6284,16 @@ L: linux-arm-kernel@lists.infradead.org >> (moderated for non-subscribers) S: Maintained >> F: drivers/pinctrl/pinctrl-at91.c >> >> +PIN CONTROLLER - SAMSUNG >> +M: Linus Walleij <linus.walleij@linaro.org> >> +M: Kukjin Kim <kgene.kim@samsung.com> > > So actually this would be: > M: Thomas Abraham <thomas.abraham@linaro.org> > M: Tomasz Figa <t.figa@samsung.com> > > Thomas, do you agree on this? Ah. Wasn't sure where the official maintainership should lie. ;) If Thomas is happy with this then I'll spin and re-send. Shall I remove Kukjin and Linus, then? It's convenient to have Linus on the list since he tends to be the one applying the patches... -Doug ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:25 ` Doug Anderson @ 2013-06-13 17:32 ` Tomasz Figa 2013-06-13 17:46 ` Joe Perches 2013-06-13 22:22 ` Kukjin Kim 2013-06-13 18:25 ` Linus Walleij 1 sibling, 2 replies; 25+ messages in thread From: Tomasz Figa @ 2013-06-13 17:32 UTC (permalink / raw) To: Doug Anderson Cc: Stephen Rothwell, linux-samsung-soc, Linus Walleij, Kukjin Kim, Olof Johansson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org, Thomas Abraham On Thursday 13 of June 2013 10:25:02 Doug Anderson wrote: > Tomasz, > > On Thu, Jun 13, 2013 at 10:13 AM, Tomasz Figa <tomasz.figa@gmail.com> wrote: > > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > >> It's convenient if get_maintainer suggests sending samsung/exynos > >> pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry for > >> that. > > > > Well, if we are already at it, we could list the actual maintainers of > > the driver. > > > > The original maintainer was Thomas Abraham <thomas.abraham@linaro.org> > > (added to CC) and after joining to development of this driver last > > year I have become a de facto maintainer as well, so... > > > >> Signed-off-by: Doug Anderson <dianders@chromium.org> > >> --- > >> > >> MAINTAINERS | 10 ++++++++++ > >> 1 file changed, 10 insertions(+) > >> > >> diff --git a/MAINTAINERS b/MAINTAINERS > >> index 8d97b3e..d026e38 100644 > >> --- a/MAINTAINERS > >> +++ b/MAINTAINERS > >> @@ -6284,6 +6284,16 @@ L: linux-arm-kernel@lists.infradead.org > >> (moderated for non-subscribers) S: Maintained > >> > >> F: drivers/pinctrl/pinctrl-at91.c > >> > >> +PIN CONTROLLER - SAMSUNG > >> +M: Linus Walleij <linus.walleij@linaro.org> > >> +M: Kukjin Kim <kgene.kim@samsung.com> > > > > So actually this would be: > > M: Thomas Abraham <thomas.abraham@linaro.org> > > M: Tomasz Figa <t.figa@samsung.com> > > > > Thomas, do you agree on this? > > Ah. Wasn't sure where the official maintainership should lie. ;) If > Thomas is happy with this then I'll spin and re-send. Shall I remove > Kukjin and Linus, then? It's convenient to have Linus on the list > since he tends to be the one applying the patches... >From my understanding how it works, there is no need to list Linus in particular drivers, because he's already listed as a maintainer of the whole pinctrl subsystem and get-maintainer accounts for this. Similarly for Kukjin, he's listed as the main Samsung maintainer already, but AFAIK in this case the script can't infer this based on directory structure. Let's see how it's done for other Samsung drivers: DRM DRIVERS FOR EXYNOS M: Inki Dae <inki.dae@samsung.com> M: Joonyoung Shim <jy0922.shim@samsung.com> M: Seung-Woo Kim <sw0312.kim@samsung.com> M: Kyungmin Park <kyungmin.park@samsung.com> L: dri-devel@lists.freedesktop.org T: git git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm- exynos.git S: Supported F: drivers/gpu/drm/exynos F: include/drm/exynos* F: include/uapi/drm/exynos* EXYNOS DP DRIVER M: Jingoo Han <jg1.han@samsung.com> L: linux-fbdev@vger.kernel.org S: Maintained F: drivers/video/exynos/exynos_dp* F: include/video/exynos_dp* EXYNOS MIPI DISPLAY DRIVERS M: Inki Dae <inki.dae@samsung.com> M: Donghwa Lee <dh09.lee@samsung.com> M: Kyungmin Park <kyungmin.park@samsung.com> L: linux-fbdev@vger.kernel.org S: Maintained F: drivers/video/exynos/exynos_mipi* F: include/video/exynos_mipi* ARM/SAMSUNG S5P SERIES 2D GRAPHICS ACCELERATION (G2D) SUPPORT M: Kyungmin Park <kyungmin.park@samsung.com> M: Kamil Debski <k.debski@samsung.com> L: linux-arm-kernel@lists.infradead.org L: linux-media@vger.kernel.org S: Maintained F: drivers/media/platform/s5p-g2d/ and so on. So I think, based on this precedent, we can do the same for pinctrl. Let's wait to see Thomas' view on this. Best regards, Tomasz ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:32 ` Tomasz Figa @ 2013-06-13 17:46 ` Joe Perches 2013-06-13 22:22 ` Kukjin Kim 1 sibling, 0 replies; 25+ messages in thread From: Joe Perches @ 2013-06-13 17:46 UTC (permalink / raw) To: Tomasz Figa Cc: Doug Anderson, Stephen Rothwell, linux-samsung-soc, Linus Walleij, Kukjin Kim, Olof Johansson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org, Thomas Abraham On Thu, 2013-06-13 at 19:32 +0200, Tomasz Figa wrote: [] > Similarly for Kukjin, he's listed as the main Samsung maintainer already, > but AFAIK in this case the script can't infer this based on directory > structure. Let's see how it's done for other Samsung drivers: A pattern could be added that matches any file with "exynos" in it. N: Files and directories with regex patterns. N: [^a-z]tegra all files whose path contains the word tegra One pattern per line. Multiple N: lines acceptable. like: N: exynos or N: drivers/pinctrl/.*exynos ^ permalink raw reply [flat|nested] 25+ messages in thread
* RE: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:32 ` Tomasz Figa 2013-06-13 17:46 ` Joe Perches @ 2013-06-13 22:22 ` Kukjin Kim 2013-06-13 22:32 ` Tomasz Figa 2013-06-13 22:49 ` Tomasz Figa 1 sibling, 2 replies; 25+ messages in thread From: Kukjin Kim @ 2013-06-13 22:22 UTC (permalink / raw) To: 'Tomasz Figa', 'Doug Anderson' Cc: 'Stephen Rothwell', 'linux-samsung-soc', 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, 'Thomas Abraham' Tomasz Figa wrote: > > On Thursday 13 of June 2013 10:25:02 Doug Anderson wrote: > > Tomasz, > > > > On Thu, Jun 13, 2013 at 10:13 AM, Tomasz Figa <tomasz.figa@gmail.com> > wrote: > > > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > > >> It's convenient if get_maintainer suggests sending samsung/exynos > > >> pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry for > > >> that. > > > > > > Well, if we are already at it, we could list the actual maintainers of > > > the driver. > > > > > > The original maintainer was Thomas Abraham <thomas.abraham@linaro.org> > > > (added to CC) and after joining to development of this driver last > > > year I have become a de facto maintainer as well, so... > > > > > >> Signed-off-by: Doug Anderson <dianders@chromium.org> > > >> --- > > >> > > >> MAINTAINERS | 10 ++++++++++ > > >> 1 file changed, 10 insertions(+) > > >> > > >> diff --git a/MAINTAINERS b/MAINTAINERS > > >> index 8d97b3e..d026e38 100644 > > >> --- a/MAINTAINERS > > >> +++ b/MAINTAINERS > > >> @@ -6284,6 +6284,16 @@ L: linux-arm-kernel@lists.infradead.org > > >> (moderated for non-subscribers) S: Maintained > > >> > > >> F: drivers/pinctrl/pinctrl-at91.c > > >> > > >> +PIN CONTROLLER - SAMSUNG > > >> +M: Linus Walleij <linus.walleij@linaro.org> > > >> +M: Kukjin Kim <kgene.kim@samsung.com> > > > > > > So actually this would be: > > > M: Thomas Abraham <thomas.abraham@linaro.org> > > > M: Tomasz Figa <t.figa@samsung.com> > > > > > > Thomas, do you agree on this? > > > > Ah. Wasn't sure where the official maintainership should lie. ;) If > > Thomas is happy with this then I'll spin and re-send. Shall I remove > > Kukjin and Linus, then? It's convenient to have Linus on the list > > since he tends to be the one applying the patches... > > From my understanding how it works, there is no need to list Linus in > particular drivers, because he's already listed as a maintainer of the > whole pinctrl subsystem and get-maintainer accounts for this. > > Similarly for Kukjin, he's listed as the main Samsung maintainer already, If so, the directory like pinctrl should be added under Samsung ARM Architectures like spi and sound for Samsung stuff... > but AFAIK in this case the script can't infer this based on directory > structure. Let's see how it's done for other Samsung drivers: > Well, I don't think so, because pinctrl has the close relationship with exynos arch part and it's little bit different with following. Just in my understanding. > DRM DRIVERS FOR EXYNOS > M: Inki Dae <inki.dae@samsung.com> > M: Joonyoung Shim <jy0922.shim@samsung.com> > M: Seung-Woo Kim <sw0312.kim@samsung.com> > M: Kyungmin Park <kyungmin.park@samsung.com> > L: dri-devel@lists.freedesktop.org > T: git git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm- > exynos.git > S: Supported > F: drivers/gpu/drm/exynos > F: include/drm/exynos* > F: include/uapi/drm/exynos* > > EXYNOS DP DRIVER > M: Jingoo Han <jg1.han@samsung.com> > L: linux-fbdev@vger.kernel.org > S: Maintained > F: drivers/video/exynos/exynos_dp* > F: include/video/exynos_dp* > > EXYNOS MIPI DISPLAY DRIVERS > M: Inki Dae <inki.dae@samsung.com> > M: Donghwa Lee <dh09.lee@samsung.com> > M: Kyungmin Park <kyungmin.park@samsung.com> > L: linux-fbdev@vger.kernel.org > S: Maintained > F: drivers/video/exynos/exynos_mipi* > F: include/video/exynos_mipi* > > ARM/SAMSUNG S5P SERIES 2D GRAPHICS ACCELERATION (G2D) SUPPORT > M: Kyungmin Park <kyungmin.park@samsung.com> > M: Kamil Debski <k.debski@samsung.com> > L: linux-arm-kernel@lists.infradead.org > L: linux-media@vger.kernel.org > S: Maintained > F: drivers/media/platform/s5p-g2d/ > And I'm asking above all guys _really_ do review and maintaining the file or directories? I don't think so. > and so on. > > So I think, based on this precedent, we can do the same for pinctrl. Let's > wait to see Thomas' view on this. > ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 22:22 ` Kukjin Kim @ 2013-06-13 22:32 ` Tomasz Figa 2013-06-13 22:59 ` Kukjin Kim 2013-06-13 23:02 ` Kukjin Kim 2013-06-13 22:49 ` Tomasz Figa 1 sibling, 2 replies; 25+ messages in thread From: Tomasz Figa @ 2013-06-13 22:32 UTC (permalink / raw) To: Kukjin Kim Cc: 'Doug Anderson', 'Stephen Rothwell', 'linux-samsung-soc', 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, 'Thomas Abraham' On Friday 14 of June 2013 07:22:53 Kukjin Kim wrote: > Tomasz Figa wrote: > > On Thursday 13 of June 2013 10:25:02 Doug Anderson wrote: > > > Tomasz, > > > > > > On Thu, Jun 13, 2013 at 10:13 AM, Tomasz Figa > > > <tomasz.figa@gmail.com> > > > > wrote: > > > > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > > > >> It's convenient if get_maintainer suggests sending samsung/exynos > > > >> pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry > > > >> for > > > >> that. > > > > > > > > Well, if we are already at it, we could list the actual > > > > maintainers of > > > > the driver. > > > > > > > > The original maintainer was Thomas Abraham > > > > <thomas.abraham@linaro.org> > > > > (added to CC) and after joining to development of this driver last > > > > year I have become a de facto maintainer as well, so... > > > > > > > >> Signed-off-by: Doug Anderson <dianders@chromium.org> > > > >> --- > > > >> > > > >> MAINTAINERS | 10 ++++++++++ > > > >> 1 file changed, 10 insertions(+) > > > >> > > > >> diff --git a/MAINTAINERS b/MAINTAINERS > > > >> index 8d97b3e..d026e38 100644 > > > >> --- a/MAINTAINERS > > > >> +++ b/MAINTAINERS > > > >> @@ -6284,6 +6284,16 @@ L: > > > >> linux-arm-kernel@lists.infradead.org > > > >> (moderated for non-subscribers) S: Maintained > > > >> > > > >> F: drivers/pinctrl/pinctrl-at91.c > > > >> > > > >> +PIN CONTROLLER - SAMSUNG > > > >> +M: Linus Walleij <linus.walleij@linaro.org> > > > >> +M: Kukjin Kim <kgene.kim@samsung.com> > > > > > > > > So actually this would be: > > > > M: Thomas Abraham <thomas.abraham@linaro.org> > > > > M: Tomasz Figa <t.figa@samsung.com> > > > > > > > > Thomas, do you agree on this? > > > > > > Ah. Wasn't sure where the official maintainership should lie. ;) > > > If > > > Thomas is happy with this then I'll spin and re-send. Shall I > > > remove > > > Kukjin and Linus, then? It's convenient to have Linus on the list > > > since he tends to be the one applying the patches... > > > > From my understanding how it works, there is no need to list Linus in > > particular drivers, because he's already listed as a maintainer of the > > whole pinctrl subsystem and get-maintainer accounts for this. > > > > Similarly for Kukjin, he's listed as the main Samsung maintainer > > already, > If so, the directory like pinctrl should be added under Samsung ARM > Architectures like spi and sound for Samsung stuff... I don't think so, because it's a part of the pin control subsystem. > > but AFAIK in this case the script can't infer this based on directory > > > structure. Let's see how it's done for other Samsung drivers: > Well, I don't think so, because pinctrl has the close relationship with > exynos arch part and it's little bit different with following. Just in > my understanding. I'm not really sure what you mean. Pin control is a separate kernel subsystem, which shouldn't have any dependencies on arch code. > > DRM DRIVERS FOR EXYNOS > > M: Inki Dae <inki.dae@samsung.com> > > M: Joonyoung Shim <jy0922.shim@samsung.com> > > M: Seung-Woo Kim <sw0312.kim@samsung.com> > > M: Kyungmin Park <kyungmin.park@samsung.com> > > L: dri-devel@lists.freedesktop.org > > T: git git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm- > > exynos.git > > S: Supported > > F: drivers/gpu/drm/exynos > > F: include/drm/exynos* > > F: include/uapi/drm/exynos* > > > > EXYNOS DP DRIVER > > M: Jingoo Han <jg1.han@samsung.com> > > L: linux-fbdev@vger.kernel.org > > S: Maintained > > F: drivers/video/exynos/exynos_dp* > > F: include/video/exynos_dp* > > > > EXYNOS MIPI DISPLAY DRIVERS > > M: Inki Dae <inki.dae@samsung.com> > > M: Donghwa Lee <dh09.lee@samsung.com> > > M: Kyungmin Park <kyungmin.park@samsung.com> > > L: linux-fbdev@vger.kernel.org > > S: Maintained > > F: drivers/video/exynos/exynos_mipi* > > F: include/video/exynos_mipi* > > > > ARM/SAMSUNG S5P SERIES 2D GRAPHICS ACCELERATION (G2D) SUPPORT > > M: Kyungmin Park <kyungmin.park@samsung.com> > > M: Kamil Debski <k.debski@samsung.com> > > L: linux-arm-kernel@lists.infradead.org > > L: linux-media@vger.kernel.org > > S: Maintained > > F: drivers/media/platform/s5p-g2d/ > > And I'm asking above all guys _really_ do review and maintaining the > file or directories? I don't think so. I'm not really sure what makes you think they don't. I can see all of them involved in works on mentioned drivers. Best regards, Tomasz > > and so on. > > > > So I think, based on this precedent, we can do the same for pinctrl. > > Let's wait to see Thomas' view on this. ^ permalink raw reply [flat|nested] 25+ messages in thread
* RE: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 22:32 ` Tomasz Figa @ 2013-06-13 22:59 ` Kukjin Kim 2013-06-13 23:02 ` Kukjin Kim 1 sibling, 0 replies; 25+ messages in thread From: Kukjin Kim @ 2013-06-13 22:59 UTC (permalink / raw) To: 'Tomasz Figa' Cc: 'Doug Anderson', 'Stephen Rothwell', 'linux-samsung-soc', 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, 'Thomas Abraham' Tomasz Figa wrote: > [...] > > > From my understanding how it works, there is no need to list Linus in > > > particular drivers, because he's already listed as a maintainer of the > > > whole pinctrl subsystem and get-maintainer accounts for this. > > > > > > Similarly for Kukjin, he's listed as the main Samsung maintainer > > > already, > > If so, the directory like pinctrl should be added under Samsung ARM > > Architectures like spi and sound for Samsung stuff... > > I don't think so, because it's a part of the pin control subsystem. > Yes right. I mean, pinctrl is clos > > > but AFAIK in this case the script can't infer this based on directory > > > > > structure. Let's see how it's done for other Samsung drivers: > > Well, I don't think so, because pinctrl has the close relationship with > > exynos arch part and it's little bit different with following. Just in > > my understanding. > > I'm not really sure what you mean. Pin control is a separate kernel > subsystem, which shouldn't have any dependencies on arch code. > Yes, right. I have no objection about it is a subsystem. I meant just relationship and it helps people find the overall architecture. [...] > > And I'm asking above all guys _really_ do review and maintaining the > > file or directories? I don't think so. > > I'm not really sure what makes you think they don't. I can see all of them > involved in works on mentioned drivers. > What's the role of maintainer? I mean, as you said, above guys are really do it... - Kukjin ^ permalink raw reply [flat|nested] 25+ messages in thread
* RE: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 22:32 ` Tomasz Figa 2013-06-13 22:59 ` Kukjin Kim @ 2013-06-13 23:02 ` Kukjin Kim 1 sibling, 0 replies; 25+ messages in thread From: Kukjin Kim @ 2013-06-13 23:02 UTC (permalink / raw) To: 'Kukjin Kim', 'Tomasz Figa' Cc: 'Doug Anderson', 'Stephen Rothwell', 'linux-samsung-soc', 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, 'Thomas Abraham' Kukjin Kim wrote: > > Tomasz Figa wrote: > > [...] > > > > And I'm asking above all guys _really_ do review and maintaining the > > > file or directories? I don't think so. > > > > I'm not really sure what makes you think they don't. I can see all of > them > > involved in works on mentioned drivers. > > > What's the role of maintainer? I mean, as you said, above guys are really > do it... > One more, I don't blame and I'm not saying it was wrong :) - Kukjin ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 22:22 ` Kukjin Kim 2013-06-13 22:32 ` Tomasz Figa @ 2013-06-13 22:49 ` Tomasz Figa 2013-06-13 23:09 ` Tomasz Figa 1 sibling, 1 reply; 25+ messages in thread From: Tomasz Figa @ 2013-06-13 22:49 UTC (permalink / raw) To: Kukjin Kim Cc: 'Doug Anderson', 'Stephen Rothwell', 'linux-samsung-soc', 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, 'Thomas Abraham', arnd, olof On Friday 14 of June 2013 07:22:53 Kukjin Kim wrote: > Tomasz Figa wrote: > > On Thursday 13 of June 2013 10:25:02 Doug Anderson wrote: > > > Tomasz, > > > > > > On Thu, Jun 13, 2013 at 10:13 AM, Tomasz Figa > > > <tomasz.figa@gmail.com> > > > > wrote: > > > > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > > > >> It's convenient if get_maintainer suggests sending samsung/exynos > > > >> pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry > > > >> for > > > >> that. > > > > > > > > Well, if we are already at it, we could list the actual > > > > maintainers of > > > > the driver. > > > > > > > > The original maintainer was Thomas Abraham > > > > <thomas.abraham@linaro.org> > > > > (added to CC) and after joining to development of this driver last > > > > year I have become a de facto maintainer as well, so... > > > > > > > >> Signed-off-by: Doug Anderson <dianders@chromium.org> > > > >> --- > > > >> > > > >> MAINTAINERS | 10 ++++++++++ > > > >> 1 file changed, 10 insertions(+) > > > >> > > > >> diff --git a/MAINTAINERS b/MAINTAINERS > > > >> index 8d97b3e..d026e38 100644 > > > >> --- a/MAINTAINERS > > > >> +++ b/MAINTAINERS > > > >> @@ -6284,6 +6284,16 @@ L: > > > >> linux-arm-kernel@lists.infradead.org > > > >> (moderated for non-subscribers) S: Maintained > > > >> > > > >> F: drivers/pinctrl/pinctrl-at91.c > > > >> > > > >> +PIN CONTROLLER - SAMSUNG > > > >> +M: Linus Walleij <linus.walleij@linaro.org> > > > >> +M: Kukjin Kim <kgene.kim@samsung.com> > > > > > > > > So actually this would be: > > > > M: Thomas Abraham <thomas.abraham@linaro.org> > > > > M: Tomasz Figa <t.figa@samsung.com> > > > > > > > > Thomas, do you agree on this? > > > > > > Ah. Wasn't sure where the official maintainership should lie. ;) > > > If > > > Thomas is happy with this then I'll spin and re-send. Shall I > > > remove > > > Kukjin and Linus, then? It's convenient to have Linus on the list > > > since he tends to be the one applying the patches... > > > > From my understanding how it works, there is no need to list Linus in > > particular drivers, because he's already listed as a maintainer of the > > whole pinctrl subsystem and get-maintainer accounts for this. > > > > Similarly for Kukjin, he's listed as the main Samsung maintainer > > already, > If so, the directory like pinctrl should be added under Samsung ARM > Architectures like spi and sound for Samsung stuff... One more thing. As Joe Perches suggested, we might as well add N: exynos to ARM/S5P EXYNOS ARM ARCHITECTURE entry, but I'm not really sure if it's the right thing to do. Best regards, Tomasz > > but AFAIK in this case the script can't infer this based on directory > > > structure. Let's see how it's done for other Samsung drivers: > Well, I don't think so, because pinctrl has the close relationship with > exynos arch part and it's little bit different with following. Just in > my understanding. > > > DRM DRIVERS FOR EXYNOS > > M: Inki Dae <inki.dae@samsung.com> > > M: Joonyoung Shim <jy0922.shim@samsung.com> > > M: Seung-Woo Kim <sw0312.kim@samsung.com> > > M: Kyungmin Park <kyungmin.park@samsung.com> > > L: dri-devel@lists.freedesktop.org > > T: git git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm- > > exynos.git > > S: Supported > > F: drivers/gpu/drm/exynos > > F: include/drm/exynos* > > F: include/uapi/drm/exynos* > > > > EXYNOS DP DRIVER > > M: Jingoo Han <jg1.han@samsung.com> > > L: linux-fbdev@vger.kernel.org > > S: Maintained > > F: drivers/video/exynos/exynos_dp* > > F: include/video/exynos_dp* > > > > EXYNOS MIPI DISPLAY DRIVERS > > M: Inki Dae <inki.dae@samsung.com> > > M: Donghwa Lee <dh09.lee@samsung.com> > > M: Kyungmin Park <kyungmin.park@samsung.com> > > L: linux-fbdev@vger.kernel.org > > S: Maintained > > F: drivers/video/exynos/exynos_mipi* > > F: include/video/exynos_mipi* > > > > ARM/SAMSUNG S5P SERIES 2D GRAPHICS ACCELERATION (G2D) SUPPORT > > M: Kyungmin Park <kyungmin.park@samsung.com> > > M: Kamil Debski <k.debski@samsung.com> > > L: linux-arm-kernel@lists.infradead.org > > L: linux-media@vger.kernel.org > > S: Maintained > > F: drivers/media/platform/s5p-g2d/ > > And I'm asking above all guys _really_ do review and maintaining the > file or directories? I don't think so. > > > and so on. > > > > So I think, based on this precedent, we can do the same for pinctrl. > > Let's wait to see Thomas' view on this. ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 22:49 ` Tomasz Figa @ 2013-06-13 23:09 ` Tomasz Figa 0 siblings, 0 replies; 25+ messages in thread From: Tomasz Figa @ 2013-06-13 23:09 UTC (permalink / raw) To: Kukjin Kim Cc: 'Doug Anderson', 'Stephen Rothwell', 'linux-samsung-soc', 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, 'Thomas Abraham', arnd On Friday 14 of June 2013 00:49:08 Tomasz Figa wrote: > On Friday 14 of June 2013 07:22:53 Kukjin Kim wrote: > > Tomasz Figa wrote: > > > On Thursday 13 of June 2013 10:25:02 Doug Anderson wrote: > > > > Tomasz, > > > > > > > > On Thu, Jun 13, 2013 at 10:13 AM, Tomasz Figa > > > > <tomasz.figa@gmail.com> > > > > > > wrote: > > > > > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > > > > >> It's convenient if get_maintainer suggests sending > > > > >> samsung/exynos > > > > >> pinctrl changes to linux-samsung-soc and to Kukjin. Add an > > > > >> entry > > > > >> for > > > > >> that. > > > > > > > > > > Well, if we are already at it, we could list the actual > > > > > maintainers of > > > > > the driver. > > > > > > > > > > The original maintainer was Thomas Abraham > > > > > <thomas.abraham@linaro.org> > > > > > (added to CC) and after joining to development of this driver > > > > > last > > > > > year I have become a de facto maintainer as well, so... > > > > > > > > > >> Signed-off-by: Doug Anderson <dianders@chromium.org> > > > > >> --- > > > > >> > > > > >> MAINTAINERS | 10 ++++++++++ > > > > >> 1 file changed, 10 insertions(+) > > > > >> > > > > >> diff --git a/MAINTAINERS b/MAINTAINERS > > > > >> index 8d97b3e..d026e38 100644 > > > > >> --- a/MAINTAINERS > > > > >> +++ b/MAINTAINERS > > > > >> @@ -6284,6 +6284,16 @@ L: > > > > >> linux-arm-kernel@lists.infradead.org > > > > >> (moderated for non-subscribers) S: Maintained > > > > >> > > > > >> F: drivers/pinctrl/pinctrl-at91.c > > > > >> > > > > >> +PIN CONTROLLER - SAMSUNG > > > > >> +M: Linus Walleij <linus.walleij@linaro.org> > > > > >> +M: Kukjin Kim <kgene.kim@samsung.com> > > > > > > > > > > So actually this would be: > > > > > M: Thomas Abraham <thomas.abraham@linaro.org> > > > > > M: Tomasz Figa <t.figa@samsung.com> > > > > > > > > > > Thomas, do you agree on this? > > > > > > > > Ah. Wasn't sure where the official maintainership should lie. ;) > > > > If > > > > Thomas is happy with this then I'll spin and re-send. Shall I > > > > remove > > > > Kukjin and Linus, then? It's convenient to have Linus on the list > > > > since he tends to be the one applying the patches... > > > > > > From my understanding how it works, there is no need to list Linus > > > in > > > particular drivers, because he's already listed as a maintainer of > > > the > > > whole pinctrl subsystem and get-maintainer accounts for this. > > > > > > Similarly for Kukjin, he's listed as the main Samsung maintainer > > > already, > > > > If so, the directory like pinctrl should be added under Samsung ARM > > Architectures like spi and sound for Samsung stuff... > > One more thing. > > As Joe Perches suggested, we might as well add > > N: exynos > > to ARM/S5P EXYNOS ARM ARCHITECTURE entry, but I'm not really sure if > it's the right thing to do. To clarify things a bit, I'm not against doing it. Just I don't think I'm the one to decide about it. Best regards, Tomasz > > > but AFAIK in this case the script can't infer this based on > > > directory > > > > > structure. Let's see how it's done for other Samsung drivers: > > Well, I don't think so, because pinctrl has the close relationship > > with > > exynos arch part and it's little bit different with following. Just in > > my understanding. > > > > > DRM DRIVERS FOR EXYNOS > > > M: Inki Dae <inki.dae@samsung.com> > > > M: Joonyoung Shim <jy0922.shim@samsung.com> > > > M: Seung-Woo Kim <sw0312.kim@samsung.com> > > > M: Kyungmin Park <kyungmin.park@samsung.com> > > > L: dri-devel@lists.freedesktop.org > > > T: git > > > git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm- > > > exynos.git > > > S: Supported > > > F: drivers/gpu/drm/exynos > > > F: include/drm/exynos* > > > F: include/uapi/drm/exynos* > > > > > > EXYNOS DP DRIVER > > > M: Jingoo Han <jg1.han@samsung.com> > > > L: linux-fbdev@vger.kernel.org > > > S: Maintained > > > F: drivers/video/exynos/exynos_dp* > > > F: include/video/exynos_dp* > > > > > > EXYNOS MIPI DISPLAY DRIVERS > > > M: Inki Dae <inki.dae@samsung.com> > > > M: Donghwa Lee <dh09.lee@samsung.com> > > > M: Kyungmin Park <kyungmin.park@samsung.com> > > > L: linux-fbdev@vger.kernel.org > > > S: Maintained > > > F: drivers/video/exynos/exynos_mipi* > > > F: include/video/exynos_mipi* > > > > > > ARM/SAMSUNG S5P SERIES 2D GRAPHICS ACCELERATION (G2D) SUPPORT > > > M: Kyungmin Park <kyungmin.park@samsung.com> > > > M: Kamil Debski <k.debski@samsung.com> > > > L: linux-arm-kernel@lists.infradead.org > > > L: linux-media@vger.kernel.org > > > S: Maintained > > > F: drivers/media/platform/s5p-g2d/ > > > > And I'm asking above all guys _really_ do review and maintaining the > > file or directories? I don't think so. > > > > > and so on. > > > > > > So I think, based on this precedent, we can do the same for pinctrl. > > > Let's wait to see Thomas' view on this. ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:25 ` Doug Anderson 2013-06-13 17:32 ` Tomasz Figa @ 2013-06-13 18:25 ` Linus Walleij 1 sibling, 0 replies; 25+ messages in thread From: Linus Walleij @ 2013-06-13 18:25 UTC (permalink / raw) To: Doug Anderson Cc: Tomasz Figa, Stephen Rothwell, linux-samsung-soc, Kukjin Kim, Olof Johansson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org, Thomas Abraham On Thu, Jun 13, 2013 at 7:25 PM, Doug Anderson <dianders@chromium.org> wrote: >>> +PIN CONTROLLER - SAMSUNG >>> +M: Linus Walleij <linus.walleij@linaro.org> >>> +M: Kukjin Kim <kgene.kim@samsung.com> >> >> So actually this would be: >> M: Thomas Abraham <thomas.abraham@linaro.org> >> M: Tomasz Figa <t.figa@samsung.com> >> >> Thomas, do you agree on this? > > Ah. Wasn't sure where the official maintainership should lie. ;) If > Thomas is happy with this then I'll spin and re-send. Shall I remove > Kukjin and Linus, then? It's convenient to have Linus on the list > since he tends to be the one applying the patches... Check what output you get from scripts/get-maintainer before and after this patch. I don't know how it works but I think I will end up on this list anyway due to the way the bigger regex for the subsystem works. I.e. this: PIN CONTROL SUBSYSTEM M: Linus Walleij <linus.walleij@linaro.org> S: Maintained F: drivers/pinctrl/ F: include/linux/pinctrl/ Yours, Linus Walleij ^ permalink raw reply [flat|nested] 25+ messages in thread
* RE: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:13 ` Tomasz Figa 2013-06-13 17:25 ` Doug Anderson @ 2013-06-13 22:47 ` Kukjin Kim 2013-06-13 23:03 ` Tomasz Figa 1 sibling, 1 reply; 25+ messages in thread From: Kukjin Kim @ 2013-06-13 22:47 UTC (permalink / raw) To: 'Tomasz Figa', 'Doug Anderson' Cc: 'Stephen Rothwell', linux-samsung-soc, 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, thomas.abraham Tomasz Figa wrote: > > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > > It's convenient if get_maintainer suggests sending samsung/exynos > > pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry for > > that. > > Well, if we are already at it, we could list the actual maintainers of the > driver. > > The original maintainer was Thomas Abraham <thomas.abraham@linaro.org> Should be original author?! I think, original maintainer was/is Linus Walleij. Let's see, probably original author should have some knowledge for it but the meaning of the maintainer is different. Of course, would be helpful if original author could review and maintain... - Kukjin ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 22:47 ` Kukjin Kim @ 2013-06-13 23:03 ` Tomasz Figa 2013-06-16 12:47 ` Linus Walleij 0 siblings, 1 reply; 25+ messages in thread From: Tomasz Figa @ 2013-06-13 23:03 UTC (permalink / raw) To: Kukjin Kim Cc: 'Doug Anderson', 'Stephen Rothwell', linux-samsung-soc, 'Linus Walleij', 'Olof Johansson', 'Mauro Carvalho Chehab', 'Andrew Morton', 'Cesar Eduardo Barros', 'David S. Miller', 'Hans Verkuil', linux-kernel, thomas.abraham On Friday 14 of June 2013 07:47:15 Kukjin Kim wrote: > Tomasz Figa wrote: > > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: > > > It's convenient if get_maintainer suggests sending samsung/exynos > > > pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry > > > for > > > that. > > > > Well, if we are already at it, we could list the actual maintainers of > > the driver. > > > > The original maintainer was Thomas Abraham <thomas.abraham@linaro.org> > > Should be original author?! I think, original maintainer was/is Linus > Walleij. Linus is the maintainer of the pin control subsystem. In addition drivers can have their own maintainers, but of course they are lower in the hierarchy than subsystem maintainers. > Let's see, probably original author should have some knowledge for it > but the meaning of the maintainer is different. Of course, would be > helpful if original author could review and maintain... Maintainer is the person who takes care of the code, including testing, fixes, review of patches, but also development as well to keep the driver up to date with core code, improve things, add support for new hardware, etc. Originally Thomas Abraham was the person doing it for this driver. At least I remember, he was the person who reviewed and acked my patches when I started my work on improving this driver. Later on I took over most of those tasks and I've been doing so ever since that time. Best regards, Tomasz ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 23:03 ` Tomasz Figa @ 2013-06-16 12:47 ` Linus Walleij 0 siblings, 0 replies; 25+ messages in thread From: Linus Walleij @ 2013-06-16 12:47 UTC (permalink / raw) To: Tomasz Figa Cc: Kukjin Kim, Doug Anderson, Stephen Rothwell, linux-samsung-soc, Olof Johansson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org, Thomas Abraham On Fri, Jun 14, 2013 at 1:03 AM, Tomasz Figa <tomasz.figa@gmail.com> wrote: > On Friday 14 of June 2013 07:47:15 Kukjin Kim wrote: >> Tomasz Figa wrote: >> > On Thursday 13 of June 2013 10:05:44 Doug Anderson wrote: >> > > It's convenient if get_maintainer suggests sending samsung/exynos >> > > pinctrl changes to linux-samsung-soc and to Kukjin. Add an entry >> > > for >> > > that. >> > >> > Well, if we are already at it, we could list the actual maintainers of >> > the driver. >> > >> > The original maintainer was Thomas Abraham <thomas.abraham@linaro.org> >> >> Should be original author?! I think, original maintainer was/is Linus >> Walleij. > > Linus is the maintainer of the pin control subsystem. In addition drivers > can have their own maintainers, but of course they are lower in the > hierarchy than subsystem maintainers. Well I don't really like hiearchies, as I'm basically anarchist :-P Anyway, as long as the get-mainater script output both Thomas and my name I don't really care, the only practical intent with this file is to be able to mail the right people. Yours, Linus Walleij ^ permalink raw reply [flat|nested] 25+ messages in thread
* [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries 2013-06-13 17:05 [PATCH] MAINTAINERS: Add Samsung pinctrl entries Doug Anderson 2013-06-13 17:13 ` Tomasz Figa @ 2013-06-16 22:17 ` Doug Anderson 2013-06-16 22:17 ` [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES Doug Anderson ` (2 more replies) 1 sibling, 3 replies; 25+ messages in thread From: Doug Anderson @ 2013-06-16 22:17 UTC (permalink / raw) To: Linus Walleij Cc: linux-samsung-soc, Tomasz Figa, Kukjin Kim, Olof Johansson, Thomas Abraham, Joe Perches, Stephen Rothwell, Doug Anderson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel It's convenient if get_maintainer suggests sending samsung/exynos pinctrl changes to linux-samsung-soc and to Tomasz and Thomas. Signed-off-by: Doug Anderson <dianders@chromium.org> --- Changes in v2: - Updated with Thomas and Tomasz; removed Linus since he's already there as part of the general pinctrl match. MAINTAINERS | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 8d97b3e..f55e3c7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -6284,6 +6284,16 @@ L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) S: Maintained F: drivers/pinctrl/pinctrl-at91.c +PIN CONTROLLER - SAMSUNG +M: Tomasz Figa <t.figa@samsung.com> +M: Thomas Abraham <thomas.abraham@linaro.org> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) +L: linux-samsung-soc@vger.kernel.org (moderated for non-subscribers) +S: Maintained +F: drivers/pinctrl/pinctrl-exynos.* +F: drivers/pinctrl/pinctrl-s3c* +F: drivers/pinctrl/pinctrl-samsung.* + PIN CONTROLLER - ST SPEAR M: Viresh Kumar <viresh.linux@gmail.com> L: spear-devel@list.st.com -- 1.8.3 ^ permalink raw reply related [flat|nested] 25+ messages in thread
* [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES 2013-06-16 22:17 ` [PATCH v2 1/2] " Doug Anderson @ 2013-06-16 22:17 ` Doug Anderson 2013-06-17 15:37 ` Kukjin Kim 2013-06-17 3:35 ` [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries Kyungmin Park 2013-06-17 16:35 ` Linus Walleij 2 siblings, 1 reply; 25+ messages in thread From: Doug Anderson @ 2013-06-16 22:17 UTC (permalink / raw) To: Linus Walleij Cc: linux-samsung-soc, Tomasz Figa, Kukjin Kim, Olof Johansson, Thomas Abraham, Joe Perches, Stephen Rothwell, Doug Anderson, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel This will help direct patches related to exynos to the linux-samsung-soc mailing list for discussion. Suggested by Joe Perches. Signed-off-by: Doug Anderson <dianders@chromium.org> --- Changes in v2: - New for this version. MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index f55e3c7..2e95fd8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1140,6 +1140,7 @@ L: linux-samsung-soc@vger.kernel.org (moderated for non-subscribers) S: Maintained F: arch/arm/mach-s5p*/ F: arch/arm/mach-exynos*/ +N: exynos ARM/SAMSUNG MOBILE MACHINE SUPPORT M: Kyungmin Park <kyungmin.park@samsung.com> -- 1.8.3 ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES 2013-06-16 22:17 ` [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES Doug Anderson @ 2013-06-17 15:37 ` Kukjin Kim 2013-06-17 15:48 ` Doug Anderson 0 siblings, 1 reply; 25+ messages in thread From: Kukjin Kim @ 2013-06-17 15:37 UTC (permalink / raw) To: Doug Anderson Cc: Linus Walleij, linux-samsung-soc, Tomasz Figa, Kukjin Kim, Olof Johansson, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel On 06/17/13 07:17, Doug Anderson wrote: > This will help direct patches related to exynos to the > linux-samsung-soc mailing list for discussion. > > Suggested by Joe Perches. > > Signed-off-by: Doug Anderson<dianders@chromium.org> Acked-by: Kukjin Kim <kgene.kim@samsung.com> Thanks, - Kukjin > --- > Changes in v2: > - New for this version. > > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index f55e3c7..2e95fd8 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1140,6 +1140,7 @@ L: linux-samsung-soc@vger.kernel.org (moderated for non-subscribers) > S: Maintained > F: arch/arm/mach-s5p*/ > F: arch/arm/mach-exynos*/ > +N: exynos > > ARM/SAMSUNG MOBILE MACHINE SUPPORT > M: Kyungmin Park<kyungmin.park@samsung.com> ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES 2013-06-17 15:37 ` Kukjin Kim @ 2013-06-17 15:48 ` Doug Anderson 2013-06-17 15:55 ` Kukjin Kim 0 siblings, 1 reply; 25+ messages in thread From: Doug Anderson @ 2013-06-17 15:48 UTC (permalink / raw) To: Kukjin Kim Cc: Linus Walleij, linux-samsung-soc, Tomasz Figa, Olof Johansson, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org Kukjin, On Mon, Jun 17, 2013 at 8:37 AM, Kukjin Kim <kgene.kim@samsung.com> wrote: > On 06/17/13 07:17, Doug Anderson wrote: >> >> This will help direct patches related to exynos to the >> linux-samsung-soc mailing list for discussion. >> >> Suggested by Joe Perches. >> >> Signed-off-by: Doug Anderson<dianders@chromium.org> > > > Acked-by: Kukjin Kim <kgene.kim@samsung.com> I _think_ you can even apply this to your tree. Looking at commits to MAINTAINERS it looks individual maintainers just include this in their trees, though I could be wrong. -Doug ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES 2013-06-17 15:48 ` Doug Anderson @ 2013-06-17 15:55 ` Kukjin Kim 0 siblings, 0 replies; 25+ messages in thread From: Kukjin Kim @ 2013-06-17 15:55 UTC (permalink / raw) To: Doug Anderson Cc: Kukjin Kim, Linus Walleij, linux-samsung-soc, Tomasz Figa, Olof Johansson, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org On 06/18/13 00:48, Doug Anderson wrote: > Kukjin, > > On Mon, Jun 17, 2013 at 8:37 AM, Kukjin Kim<kgene.kim@samsung.com> wrote: >> On 06/17/13 07:17, Doug Anderson wrote: >>> >>> This will help direct patches related to exynos to the >>> linux-samsung-soc mailing list for discussion. >>> >>> Suggested by Joe Perches. >>> >>> Signed-off-by: Doug Anderson<dianders@chromium.org> >> >> >> Acked-by: Kukjin Kim<kgene.kim@samsung.com> > > I _think_ you can even apply this to your tree. Looking at commits to > MAINTAINERS it looks individual maintainers just include this in their > trees, though I could be wrong. > OK, let me take this into samsung tree. Thanks, - Kukjin ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries 2013-06-16 22:17 ` [PATCH v2 1/2] " Doug Anderson 2013-06-16 22:17 ` [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES Doug Anderson @ 2013-06-17 3:35 ` Kyungmin Park 2013-06-17 3:43 ` Olof Johansson 2013-06-17 16:35 ` Linus Walleij 2 siblings, 1 reply; 25+ messages in thread From: Kyungmin Park @ 2013-06-17 3:35 UTC (permalink / raw) To: Doug Anderson Cc: Linus Walleij, linux-samsung-soc, Tomasz Figa, Kukjin Kim, Olof Johansson, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel On Mon, Jun 17, 2013 at 7:17 AM, Doug Anderson <dianders@chromium.org> wrote: > It's convenient if get_maintainer suggests sending samsung/exynos > pinctrl changes to linux-samsung-soc and to Tomasz and Thomas. > > Signed-off-by: Doug Anderson <dianders@chromium.org> Acked-by: Kyungmin Park <kyungmin.park@samsung.com> > --- > Changes in v2: > - Updated with Thomas and Tomasz; removed Linus since he's already > there as part of the general pinctrl match. > > MAINTAINERS | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 8d97b3e..f55e3c7 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -6284,6 +6284,16 @@ L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) > S: Maintained > F: drivers/pinctrl/pinctrl-at91.c > > +PIN CONTROLLER - SAMSUNG > +M: Tomasz Figa <t.figa@samsung.com> > +M: Thomas Abraham <thomas.abraham@linaro.org> > +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) > +L: linux-samsung-soc@vger.kernel.org (moderated for non-subscribers) > +S: Maintained > +F: drivers/pinctrl/pinctrl-exynos.* > +F: drivers/pinctrl/pinctrl-s3c* > +F: drivers/pinctrl/pinctrl-samsung.* > + > PIN CONTROLLER - ST SPEAR > M: Viresh Kumar <viresh.linux@gmail.com> > L: spear-devel@list.st.com > -- > 1.8.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries 2013-06-17 3:35 ` [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries Kyungmin Park @ 2013-06-17 3:43 ` Olof Johansson 2013-06-17 15:37 ` Kukjin Kim 0 siblings, 1 reply; 25+ messages in thread From: Olof Johansson @ 2013-06-17 3:43 UTC (permalink / raw) To: Kyungmin Park, Doug Anderson Cc: Linus Walleij, linux-samsung-soc, Tomasz Figa, Kukjin Kim, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel Kyungmin Park <kmpark@infradead.org> skrev: >On Mon, Jun 17, 2013 at 7:17 AM, Doug Anderson <dianders@chromium.org> >wrote: >> It's convenient if get_maintainer suggests sending samsung/exynos >> pinctrl changes to linux-samsung-soc and to Tomasz and Thomas. >> >> Signed-off-by: Doug Anderson <dianders@chromium.org> >Acked-by: Kyungmin Park <kyungmin.park@samsung.com> Acked-by: Olof Johansson <olof@lixom.net> -- Skickat från min Android-telefon med K-9 E-post. Ursäkta min fåordighet. ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries 2013-06-17 3:43 ` Olof Johansson @ 2013-06-17 15:37 ` Kukjin Kim 2013-06-17 15:50 ` Doug Anderson 0 siblings, 1 reply; 25+ messages in thread From: Kukjin Kim @ 2013-06-17 15:37 UTC (permalink / raw) To: Olof Johansson Cc: Kyungmin Park, Doug Anderson, Linus Walleij, linux-samsung-soc, Tomasz Figa, Kukjin Kim, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel On 06/17/13 12:43, Olof Johansson wrote: > > > Kyungmin Park<kmpark@infradead.org> skrev: > >> On Mon, Jun 17, 2013 at 7:17 AM, Doug Anderson<dianders@chromium.org> >> wrote: >>> It's convenient if get_maintainer suggests sending samsung/exynos >>> pinctrl changes to linux-samsung-soc and to Tomasz and Thomas. >>> >>> Signed-off-by: Doug Anderson<dianders@chromium.org> >> Acked-by: Kyungmin Park<kyungmin.park@samsung.com> > > Acked-by: Olof Johansson<olof@lixom.net> > Acked-by: Kukjin Kim <kgene.kim@samsung.com> Thanks, - Kukjin ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries 2013-06-17 15:37 ` Kukjin Kim @ 2013-06-17 15:50 ` Doug Anderson 0 siblings, 0 replies; 25+ messages in thread From: Doug Anderson @ 2013-06-17 15:50 UTC (permalink / raw) To: Linus Walleij Cc: Olof Johansson, Kyungmin Park, linux-samsung-soc, Tomasz Figa, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org, Kukjin Kim Linus, On Mon, Jun 17, 2013 at 8:37 AM, Kukjin Kim <kgene.kim@samsung.com> wrote: > On 06/17/13 12:43, Olof Johansson wrote: >> >> Kyungmin Park<kmpark@infradead.org> skrev: >> >>> On Mon, Jun 17, 2013 at 7:17 AM, Doug Anderson<dianders@chromium.org> >>> wrote: >>>> >>>> It's convenient if get_maintainer suggests sending samsung/exynos >>>> pinctrl changes to linux-samsung-soc and to Tomasz and Thomas. >>>> >>>> Signed-off-by: Doug Anderson<dianders@chromium.org> >>> >>> Acked-by: Kyungmin Park<kyungmin.park@samsung.com> >> >> >> Acked-by: Olof Johansson<olof@lixom.net> >> > Acked-by: Kukjin Kim <kgene.kim@samsung.com> I'm assuming that you'll take this and apply? If I'm supposed to find someone else to apply it, please yell. :) -Doug ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries 2013-06-16 22:17 ` [PATCH v2 1/2] " Doug Anderson 2013-06-16 22:17 ` [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES Doug Anderson 2013-06-17 3:35 ` [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries Kyungmin Park @ 2013-06-17 16:35 ` Linus Walleij 2 siblings, 0 replies; 25+ messages in thread From: Linus Walleij @ 2013-06-17 16:35 UTC (permalink / raw) To: Doug Anderson Cc: linux-samsung-soc, Tomasz Figa, Kukjin Kim, Olof Johansson, Thomas Abraham, Joe Perches, Stephen Rothwell, Mauro Carvalho Chehab, Andrew Morton, Cesar Eduardo Barros, David S. Miller, Hans Verkuil, linux-kernel@vger.kernel.org On Mon, Jun 17, 2013 at 12:17 AM, Doug Anderson <dianders@chromium.org> wrote: > It's convenient if get_maintainer suggests sending samsung/exynos > pinctrl changes to linux-samsung-soc and to Tomasz and Thomas. > > Signed-off-by: Doug Anderson <dianders@chromium.org> > --- > Changes in v2: > - Updated with Thomas and Tomasz; removed Linus since he's already > there as part of the general pinctrl match. Patch applied with all the ACKs. Yours, Linus Walleij ^ permalink raw reply [flat|nested] 25+ messages in thread
end of thread, other threads:[~2013-06-17 16:35 UTC | newest] Thread overview: 25+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-06-13 17:05 [PATCH] MAINTAINERS: Add Samsung pinctrl entries Doug Anderson 2013-06-13 17:13 ` Tomasz Figa 2013-06-13 17:25 ` Doug Anderson 2013-06-13 17:32 ` Tomasz Figa 2013-06-13 17:46 ` Joe Perches 2013-06-13 22:22 ` Kukjin Kim 2013-06-13 22:32 ` Tomasz Figa 2013-06-13 22:59 ` Kukjin Kim 2013-06-13 23:02 ` Kukjin Kim 2013-06-13 22:49 ` Tomasz Figa 2013-06-13 23:09 ` Tomasz Figa 2013-06-13 18:25 ` Linus Walleij 2013-06-13 22:47 ` Kukjin Kim 2013-06-13 23:03 ` Tomasz Figa 2013-06-16 12:47 ` Linus Walleij 2013-06-16 22:17 ` [PATCH v2 1/2] " Doug Anderson 2013-06-16 22:17 ` [PATCH v2 2/2] MAINTAINERS: Add exynos filename match to ARM/S5P EXYNOS ARM ARCHITECTURES Doug Anderson 2013-06-17 15:37 ` Kukjin Kim 2013-06-17 15:48 ` Doug Anderson 2013-06-17 15:55 ` Kukjin Kim 2013-06-17 3:35 ` [PATCH v2 1/2] MAINTAINERS: Add Samsung pinctrl entries Kyungmin Park 2013-06-17 3:43 ` Olof Johansson 2013-06-17 15:37 ` Kukjin Kim 2013-06-17 15:50 ` Doug Anderson 2013-06-17 16:35 ` Linus Walleij
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox