Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings

[Krzysztof Kozlowski]

2015-10-21 17:45 GMT+09:00 Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>:
>
> Hello Krzysztof,
>
> Patch looks good to me, I've only one comment.
>
> On 10/21/2015 03:33 AM, Krzysztof Kozlowski wrote:
> > Extend the Samsung Exynos maintainer entry to match SoC documentation
> > and SoC dt-bindings directories. Without that some files, like
> > bindings/arm/samsung/pmu.txt, are not matched by existing patterns.
> >
> > This also may serve as a hint where new documentation and bindings (not
> > matching specific subsystem) should be put.
> >
> > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>
> > Cc: Kukjin Kim <kgene@xxxxxxxxxx>
> > ---
> >  MAINTAINERS | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index b556aad90930..cad7b6b628b2 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1459,6 +1459,8 @@ F:      drivers/*/*s3c2410*
> >  F:   drivers/*/*/*s3c2410*
> >  F:   drivers/spi/spi-s3c*
> >  F:   sound/soc/samsung/*
> > +F:   Documentation/arm/Samsung/
>
> That there is also a Documentation/arm/Samsung-S3C24XX/ so
> I think this should be Documentation/arm/Samsung*/ instead.

I am not sure... I think this is documentation from the Ben Dooks'
work and there is a entry for S3C241x machines:
SIMTEC EB2410ITX (BAST)

Best regards,
Krzysztof

>
> > +F:   Documentation/devicetree/bindings/arm/samsung/
> >  N:   exynos
> >
> >  ARM/SAMSUNG MOBILE MACHINE SUPPORT
> >
>
> Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
>
> Best regards,
> --
> Javier Martinez Canillas
> Open Source Group
> Samsung Research America
> --
> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/