From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Sugaya, Taichi" Subject: Re: [PATCH] dt-bindings: cpu: Fix JSON schema Date: Tue, 2 Apr 2019 20:42:02 +0900 Message-ID: References: <20190318100521.29109-1-maxime.ripard@bootlin.com> <20190401091050.cwph7xlpv3pmkwnq@flea> <78bb248e-56b8-b825-e763-583f75148eb3@socionext.com> <20190401113546.d6p2mg2y47abhumz@flea> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Maxime Ripard Cc: ARM-SoC Maintainers , Mark Rutland , devicetree@vger.kernel.org, Rob Herring , "linux-kernel@vger.kernel.org" List-Id: devicetree@vger.kernel.org Hi, On 2019/04/01 22:02, Sugaya, Taichi wrote: > Hi, > > On 2019/04/01 20:35, Maxime Ripard wrote: >> Hi, >> >> On Mon, Apr 01, 2019 at 07:52:06PM +0900, Sugaya, Taichi wrote: >>> On 2019/04/01 18:10, Maxime Ripard wrote: >>>> Hi Sugaya, Arnd, Olof, >>>> >>>> On Thu, Mar 28, 2019 at 02:35:54PM -0500, Rob Herring wrote: >>>>> +arm-soc >>>>> >>>>> On Mon, Mar 18, 2019 at 5:05 AM Maxime Ripard wrote: >>>>>> >>>>>> Commit fd73403a4862 ("dt-bindings: arm: Add SMP enable-method for >>>>>> Milbeaut") added support for a new cpu enable-method, but did so using >>>>>> tabulations to ident. This is however invalid in the syntax, and resulted >>>>>> in a failure when trying to use that schemas for validation. >>>>>> >>>>>> Use spaces instead of tabs to indent to fix this. >>>>>> >>>>>> Fixes: fd73403a4862 ("dt-bindings: arm: Add SMP enable-method for Milbeaut") >>>>>> Signed-off-by: Maxime Ripard >>>>>> --- >>>>>>    Documentation/devicetree/bindings/arm/cpus.yaml | 2 +- >>>>>>    1 file changed, 1 insertion(+), 1 deletion(-) >>>>> >>>>> Reviewed-by: Rob Herring >>>>> >>>>> This should be applied to arm-soc tree having the commit. >>>> >>>> Could you pick up that patch? >>> >>> I got it. >>> My mission is to resubmit your fix-patch added my singned-off tag and >>> Rob's reviewed tag...right? >> >> If you are part of arm-soc and send your pull requests to Arnd and >> Olof, simply pick it up in the same branch than the initial patch was >> in. And when you apply that patch, you indeed need to add your >> Signed-off-by (note that most commands to commit have an option to do >> that automatically: -s and that includes git am) and the Rob's >> reviewed-by. >> >> Then, send a new pull request to arm-soc. >> > > Okay, thanks for your suggestion. > I try to make pull request! > Sorry I realized that I couldn't make a private repository, so I'll send my Ack instead. Acked-by: Sugaya Taichi > Thanks, > Sugaya Taichi > >> Maxime >> >> -- >> Maxime Ripard, Bootlin >> Embedded Linux and Kernel engineering >> https://bootlin.com >>