From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4ED2CC4360F for ; Wed, 3 Apr 2019 01:29:33 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 1A93D2146E for ; Wed, 3 Apr 2019 01:29:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726582AbfDCB3b (ORCPT ); Tue, 2 Apr 2019 21:29:31 -0400 Received: from mx.socionext.com ([202.248.49.38]:49084 "EHLO mx.socionext.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726071AbfDCB3b (ORCPT ); Tue, 2 Apr 2019 21:29:31 -0400 Received: from unknown (HELO kinkan-ex.css.socionext.com) ([172.31.9.52]) by mx.socionext.com with ESMTP; 03 Apr 2019 10:29:29 +0900 Received: from mail.mfilter.local (m-filter-2 [10.213.24.62]) by kinkan-ex.css.socionext.com (Postfix) with ESMTP id 43057180B60; Wed, 3 Apr 2019 10:29:29 +0900 (JST) Received: from 172.31.9.51 (172.31.9.51) by m-FILTER with ESMTP; Wed, 3 Apr 2019 10:29:29 +0900 Received: from yuzu.css.socionext.com (yuzu [172.31.8.45]) by kinkan.css.socionext.com (Postfix) with ESMTP id AF0171A15B7; Wed, 3 Apr 2019 10:29:28 +0900 (JST) Received: from [127.0.0.1] (unknown [10.213.119.83]) by yuzu.css.socionext.com (Postfix) with ESMTP id A19D0120499; Wed, 3 Apr 2019 10:29:28 +0900 (JST) Subject: Re: [PATCH] dt-bindings: cpu: Fix JSON schema From: "Sugaya, Taichi" To: Maxime Ripard Cc: ARM-SoC Maintainers , Mark Rutland , devicetree@vger.kernel.org, Rob Herring , "linux-kernel@vger.kernel.org" References: <20190318100521.29109-1-maxime.ripard@bootlin.com> <20190401091050.cwph7xlpv3pmkwnq@flea> <78bb248e-56b8-b825-e763-583f75148eb3@socionext.com> <20190401113546.d6p2mg2y47abhumz@flea> Message-ID: Date: Wed, 3 Apr 2019 10:29:27 +0900 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2019/04/02 20:42, Sugaya, Taichi wrote: > 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. One correction: "a public repository" > > Acked-by: Sugaya Taichi > >> Thanks, >> Sugaya Taichi >> >>> Maxime >>> >>> -- >>> Maxime Ripard, Bootlin >>> Embedded Linux and Kernel engineering >>> https://bootlin.com >>>