From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932173AbbJUIp0 (ORCPT ); Wed, 21 Oct 2015 04:45:26 -0400 Received: from lists.s-osg.org ([54.187.51.154]:42060 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753231AbbJUIpY (ORCPT ); Wed, 21 Oct 2015 04:45:24 -0400 Subject: Re: [PATCH] MAINTAINERS: ARM: EXYNOS: Add documentation and dt-bindings To: Krzysztof Kozlowski , Kukjin Kim , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org References: <1445391203-3819-1-git-send-email-k.kozlowski@samsung.com> Cc: Jonathan Corbet , arm@kernel.org, Arnd Bergmann , Olof Johansson , Kevin Hilman From: Javier Martinez Canillas X-Enigmail-Draft-Status: N1110 Message-ID: <5627509E.6090503@osg.samsung.com> Date: Wed, 21 Oct 2015 10:45:18 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <1445391203-3819-1-git-send-email-k.kozlowski@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > Cc: Kukjin Kim > --- > 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 Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America