[PATCH 2/2] dt-bindings: soc: imx-blk-ctrl: Drop leading label in blk-ctrl in examples

Marek Vasut marex at denx.de
Sun Mar 19 10:25:33 PDT 2023


On 3/19/23 12:21, Krzysztof Kozlowski wrote:
> On 17/03/2023 10:58, Marek Vasut wrote:
>> Drop the leading label in block controller examples, it is unused.
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> ---
>> Cc: Fabio Estevam <festevam at gmail.com>
>> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt at linaro.org>
>> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>> Cc: Lucas Stach <l.stach at pengutronix.de>
>> Cc: Marek Vasut <marex at denx.de>
>> Cc: NXP Linux Team <linux-imx at nxp.com>
>> Cc: Paul Elder <paul.elder at ideasonboard.com>
>> Cc: Peng Fan <peng.fan at nxp.com>
>> Cc: Pengutronix Kernel Team <kernel at pengutronix.de>
>> Cc: Richard Cochran <richardcochran at gmail.com>
>> Cc: Rob Herring <robh+dt at kernel.org>
>> Cc: Sascha Hauer <s.hauer at pengutronix.de>
>> Cc: Shawn Guo <shawnguo at kernel.org>
>> Cc: devicetree at vger.kernel.org
>> Cc: linux-arm-kernel at lists.infradead.org
>> ---
>> Note: This was suggested by Krzysztof as part of requested changes to commit
>>        0d2c843ce5adb ("dt-bindings: soc: imx8mp-media-blk-ctrl: Align block controller example name")
>>        Expand the change onto all the block controllers.
>> ---
>>   .../devicetree/bindings/soc/imx/fsl,imx8mm-disp-blk-ctrl.yaml   | 2 +-
>>   .../devicetree/bindings/soc/imx/fsl,imx8mm-vpu-blk-ctrl.yaml    | 2 +-
>>   .../devicetree/bindings/soc/imx/fsl,imx8mn-disp-blk-ctrl.yaml   | 2 +-
>>   .../devicetree/bindings/soc/imx/fsl,imx8mp-hsio-blk-ctrl.yaml   | 2 +-
>>   .../devicetree/bindings/soc/imx/fsl,imx8mq-vpu-blk-ctrl.yaml    | 2 +-
>>   .../devicetree/bindings/soc/imx/fsl,imx93-media-blk-ctrl.yaml   | 2 +-
>>   6 files changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/soc/imx/fsl,imx8mm-disp-blk-ctrl.yaml b/Documentation/devicetree/bindings/soc/imx/fsl,imx8mm-disp-blk-ctrl.yaml
>> index e8da8f413966b..a02a09d574a2c 100644
>> --- a/Documentation/devicetree/bindings/soc/imx/fsl,imx8mm-disp-blk-ctrl.yaml
>> +++ b/Documentation/devicetree/bindings/soc/imx/fsl,imx8mm-disp-blk-ctrl.yaml
>> @@ -70,7 +70,7 @@ examples:
>>       #include <dt-bindings/clock/imx8mm-clock.h>
>>       #include <dt-bindings/power/imx8mm-power.h>
>>   
>> -    disp_blk_ctl: blk-ctrl at 32e28000 {
>> +    blk-ctrl at 32e28000 {
> 
> This piece should be squashed with previous. You are changing the same
> line twice with two cleanups. It's still a cleanup.

I hope its OK to keep one type of change (fix node name vs. drop labels) 
per patch . The two patches also modify different files , so I don't 
want to fix them up.



More information about the linux-arm-kernel mailing list