Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1252409 > unrolled thread
| Started by | Krzysztof Kozlowski <k.kozlowski@samsung.com> |
|---|---|
| First post | 2015-10-21 03:40 +0200 |
| Last post | 2015-10-22 03:20 +0200 |
| Articles | 5 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-21 03:40 +0200
Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-21 10:50 +0200
Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-21 14:50 +0200
Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-21 15:10 +0200
Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-22 03:20 +0200
| From | Krzysztof Kozlowski <k.kozlowski@samsung.com> |
|---|---|
| Date | 2015-10-21 03:40 +0200 |
| Subject | [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings |
| Message-ID | <qlQxI-4GQ-7@gated-at.bofh.it> |
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@samsung.com> Cc: Kukjin Kim <kgene@kernel.org> --- 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/ +F: Documentation/devicetree/bindings/arm/samsung/ N: exynos ARM/SAMSUNG MOBILE MACHINE SUPPORT -- 1.9.1 -- 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/
[toc] | [next] | [standalone]
| From | Javier Martinez Canillas <javier@osg.samsung.com> |
|---|---|
| Date | 2015-10-21 10:50 +0200 |
| Subject | Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings |
| Message-ID | <qlXfR-6bq-29@gated-at.bofh.it> |
| In reply to | #1252409 |
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@samsung.com> > Cc: Kukjin Kim <kgene@kernel.org> > --- > 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. > +F: Documentation/devicetree/bindings/arm/samsung/ > N: exynos > > ARM/SAMSUNG MOBILE MACHINE SUPPORT > Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com> Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- 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/
[toc] | [prev] | [next] | [standalone]
| From | Krzysztof Kozlowski <k.kozlowski@samsung.com> |
|---|---|
| Date | 2015-10-21 14:50 +0200 |
| Message-ID | <qm106-3fX-19@gated-at.bofh.it> |
| In reply to | #1252576 |
2015-10-21 17:45 GMT+09:00 Javier Martinez Canillas <javier@osg.samsung.com>: > > 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@samsung.com> > > Cc: Kukjin Kim <kgene@kernel.org> > > --- > > 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@osg.samsung.com> > > 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@vger.kernel.org > 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Javier Martinez Canillas <javier@osg.samsung.com> |
|---|---|
| Date | 2015-10-21 15:10 +0200 |
| Subject | Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings |
| Message-ID | <qm1js-3S5-21@gated-at.bofh.it> |
| In reply to | #1252786 |
Hello Krzysztof, On 10/21/2015 02:43 PM, Krzysztof Kozlowski wrote: > 2015-10-21 17:45 GMT+09:00 Javier Martinez Canillas <javier@osg.samsung.com>: >> >> 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@samsung.com> >>> Cc: Kukjin Kim <kgene@kernel.org> >>> --- >>> 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 ) > I'm not familiar with S3C241x but the EB2410ITX entry only covers a couple of files under arch/arm/mach-s3c24xx, basically all that are related to the BAST machine (board file and platform data). But arch/arm/mach-s3c24* and drivers/*/*s3c2410* are covered by the ARM/SAMSUNG entry so that's why I thought the Samsung-S3C24XX docs should be there too since are SoC and not board specific. It's your call but I would put it in ARM/SAMSUNG and only the file Documentation/arm/Samsung-S3C24XX/EB2410ITX.txt in the BAST entry. > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- 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/
[toc] | [prev] | [next] | [standalone]
| From | Krzysztof Kozlowski <k.kozlowski@samsung.com> |
|---|---|
| Date | 2015-10-22 03:20 +0200 |
| Message-ID | <qmcHU-3Ao-17@gated-at.bofh.it> |
| In reply to | #1252798 |
On 21.10.2015 22:08, Javier Martinez Canillas wrote: > Hello Krzysztof, > > On 10/21/2015 02:43 PM, Krzysztof Kozlowski wrote: >> 2015-10-21 17:45 GMT+09:00 Javier Martinez Canillas <javier@osg.samsung.com>: >>> >>> 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@samsung.com> >>>> Cc: Kukjin Kim <kgene@kernel.org> >>>> --- >>>> 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 ) >> > > I'm not familiar with S3C241x but the EB2410ITX entry only covers > a couple of files under arch/arm/mach-s3c24xx, basically all that > are related to the BAST machine (board file and platform data). > > But arch/arm/mach-s3c24* and drivers/*/*s3c2410* are covered by > the ARM/SAMSUNG entry so that's why I thought the Samsung-S3C24XX > docs should be there too since are SoC and not board specific. Right, but in the same time the documentation clearly is related to the boards from that time (pre-Exynos). > It's your call but I would put it in ARM/SAMSUNG and only the file > Documentation/arm/Samsung-S3C24XX/EB2410ITX.txt in the BAST entry. I would prefer to leave it as is... Anyway thanks for comments and if anyone feels like doing re-organization and cleanup on this old documents - please go ahead. Best regards, Krzysztof -- 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/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web