lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Fri, 14 Jun 2013 00:32:23 +0200
From:	Tomasz Figa <tomasz.figa@...il.com>
To:	Kukjin Kim <kgene.kim@...sung.com>
Cc:	'Doug Anderson' <dianders@...omium.org>,
	'Stephen Rothwell' <sfr@...b.auug.org.au>,
	'linux-samsung-soc' <linux-samsung-soc@...r.kernel.org>,
	'Linus Walleij' <linus.walleij@...aro.org>,
	'Olof Johansson' <olof@...om.net>,
	'Mauro Carvalho Chehab' <mchehab@...hat.com>,
	'Andrew Morton' <akpm@...ux-foundation.org>,
	'Cesar Eduardo Barros' <cesarb@...arb.net>,
	"'David S. Miller'" <davem@...emloft.net>,
	'Hans Verkuil' <hans.verkuil@...co.com>,
	linux-kernel@...r.kernel.org,
	'Thomas Abraham' <thomas.abraham@...aro.org>
Subject: Re: [PATCH] MAINTAINERS: Add Samsung pinctrl entries

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@...il.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@...aro.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@...omium.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@...ts.infradead.org
> > > >> (moderated for non-subscribers) S:    Maintained
> > > >> 
> > > >>  F:   drivers/pinctrl/pinctrl-at91.c
> > > >> 
> > > >> +PIN CONTROLLER - SAMSUNG
> > > >> +M:   Linus Walleij <linus.walleij@...aro.org>
> > > >> +M:   Kukjin Kim <kgene.kim@...sung.com>
> > > > 
> > > > So actually this would be:
> > > > M:      Thomas Abraham <thomas.abraham@...aro.org>
> > > > M:      Tomasz Figa <t.figa@...sung.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@...sung.com>
> > M:      Joonyoung Shim <jy0922.shim@...sung.com>
> > M:      Seung-Woo Kim <sw0312.kim@...sung.com>
> > M:      Kyungmin Park <kyungmin.park@...sung.com>
> > L:      dri-devel@...ts.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@...sung.com>
> > L:      linux-fbdev@...r.kernel.org
> > S:      Maintained
> > F:      drivers/video/exynos/exynos_dp*
> > F:      include/video/exynos_dp*
> > 
> > EXYNOS MIPI DISPLAY DRIVERS
> > M:      Inki Dae <inki.dae@...sung.com>
> > M:      Donghwa Lee <dh09.lee@...sung.com>
> > M:      Kyungmin Park <kyungmin.park@...sung.com>
> > L:      linux-fbdev@...r.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@...sung.com>
> > M:      Kamil Debski <k.debski@...sung.com>
> > L:      linux-arm-kernel@...ts.infradead.org
> > L:      linux-media@...r.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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists