Re: [PATCH] MAINTAINERS: Add EXYNOS5433 ARM architectures entry as a supporter

[Krzysztof Kozlowski]

On Fri, Jan 06, 2017 at 12:21:44PM +0900, Chanwoo Choi wrote:
> On 2017ë 01ì 06ì 01:59, Krzysztof Kozlowski wrote:
> > On Thu, Jan 05, 2017 at 10:41:04PM +0900, Andi Shyti wrote:
> >> Hi Daniel,
> >>
> >>>> +ARM/SAMSUNG EXYNOS5433 ARM ARCHITECTURES
> >>>> +M:	Chanwoo Choi <cw00.choi@xxxxxxxxxxx>
> >>>> +R:	Andi Shyti <andi.shyti@xxxxxxxxxxx>
> >>>> +L:	linux-samsung-soc@xxxxxxxxxxxxxxx (moderated for non-subscribers)
> >>>> +S:	Supported
> >>>> +F:	arch/arm64/boot/dts/exynos/exynos5433*
> >>>> +
> >>>
> >>> Does it make sense to add a MAINTAINER entry for some dts files ?
> >>
> >> I think it does considering that exynos5433* files are more then
> >> the half of the arm64/.../exynos/ directory.
> >>
> >> Besides Chanwoo has authored all of them (almost) and doesn't
> >> appear from the get_maintainers script.
> > 
> > He appears:
> > 1. Kukjin Kim <kgene@xxxxxxxxxx> (maintainer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES)
> > 
> > 2. Krzysztof Kozlowski <krzk@xxxxxxxxxx> (maintainer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES,commit_signer:26/24=100%,authored:4/24=17%)
> > 
> > 3. Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> (reviewer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES,commit_signer:6/24=25%)
> > 
> > 4. Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> (commit_signer:12/24=50%,authored:8/24=33%)
> > 
> > 5. Chanwoo Choi <cw00.choi@xxxxxxxxxxx> (commit_signer:9/24=38%,authored:3/24=12%)
> > 
> > 6. Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> (commit_signer:9/24=38%)
> > 
> > (sorry if lines are wrapped)
> 
> I have a question.
> 
> When I tested the get_maintainer script with Exynos5444 patch,
> I cannot get the same result from you.

The behavior of get_maintainers.pl changed few months ago and now it
prints recent GIT activity only when there are no maintainers. To me
personally, this is desired because otherwise a lot of people will be
spammed on every patch. If someone wants to notify not only maintainers
but also other people then he should use --git (which was used for
output above).

Best regards,
Krzysztof

> 
> I tested the following patch[1] which was already merged by Marek.
> [1] commit-id: 74c78036d5b5970266fa2075a17c9b89cabf873e
> - ("arm64: dts: exynos: TM2 - add support for MFC video codec device"
> 
> ./scripts/get_maintainer.pl 0001-arm64-dts-exynos-TM2-add-support-for-MFC-video-codec.patch 
> Rob Herring <robh+dt@xxxxxxxxxx> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> Mark Rutland <mark.rutland@xxxxxxx> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> Catalin Marinas <catalin.marinas@xxxxxxx> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE))
> Will Deacon <will.deacon@xxxxxxx> (maintainer:ARM64 PORT (AARCH64 ARCHITECTURE))
> Kukjin Kim <kgene@xxxxxxxxxx> (maintainer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES)
> Krzysztof Kozlowski <krzk@xxxxxxxxxx> (maintainer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES)
> Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> (reviewer:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES)
> devicetree@xxxxxxxxxxxxxxx (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated list:ARM64 PORT (AARCH64 ARCHITECTURE))
> linux-samsung-soc@xxxxxxxxxxxxxxx (moderated list:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES)
> linux-kernel@xxxxxxxxxxxxxxx (open list)
> 
> Also, when I tested the exynos5433-tm2e.dts patch, I got the same result.
> 
> > 
> > Best regards,
> > Krzysztof
> > 
> > 
> > 
> 
> 
> -- 
> Best Regards,
> Chanwoo Choi
> Samsung Electronics

> null