Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <af9a8506-7c8e-d76c-c14f-1bf1c913a18c@kernel.org>
Date:   Thu, 21 Apr 2022 09:37:35 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.7.0
Subject: Re: [PATCH 2/2] arm: dts: Add device tree for bosch acc board
Content-Language: en-US
To:     Philip Oberfichtner <pro@denx.de>, devicetree@vger.kernel.org
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzk+dt@kernel.org>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Winker Matthias <Matthias.Winker@de.bosch.com>
References: <20220412101905.514310-1-pro@denx.de>
 <20220412101905.514310-3-pro@denx.de>
 <f2b1359d-a003-ae38-e8af-ab6aa1d0fe72@kernel.org>
 <cd7546a341d475eca99a446b6c8a5141056b3bc8.camel@denx.de>
From:   Krzysztof Kozlowski <krzk@kernel.org>
In-Reply-To: <cd7546a341d475eca99a446b6c8a5141056b3bc8.camel@denx.de>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 21/04/2022 09:06, Philip Oberfichtner wrote:
> 
> The "operating-points" and "fsl,soc-operating-points" properties are
> defined in imx6q.dtsi. We are just overwriting them here.     

OK, thanks for explanation.

(...)

>> Please run `make dtbs_check` (see Docs for this) and fix the
>> warnings.
>> Please fix automated check warnings before using reviewers time.
> 
> I built with W=1 and used checkpatch.pl. But I'm having trouble using
> make dtbs_check. Seems like I get all warnings for all possible dts. Is
> there a way to get warnings for a single dt only?

I think no, it is not possible at the moment. I understand it's a pain
and already someone proposed to have a ability to check only subset of
DTS files.

Workaround is to build imx defconfig, run dtbs_check, touch your files
and run dtbs_check again looking for results.

Best regards,
Krzysztof