RE: [PATCH] MAINTAINERS: Add Samsung pinctrl entries

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux