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: <02a9b4ca-baf8-9f06-12be-57853893fb50@kontron.de>
Date:   Wed, 13 Jul 2022 10:15:58 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH 4/6] arm64: dts: imx8mm-kontron: Use upper case PMIC
 regulator names
Content-Language: en-US
To:     Heiko Thiery <heiko.thiery@gmail.com>,
        Frieder Schrempf <frieder@fris.de>
Cc:     devicetree@vger.kernel.org,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Rob Herring <robh+dt@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Shawn Guo <shawnguo@kernel.org>,
        Fabio Estevam <festevam@gmail.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        NXP Linux Team <linux-imx@nxp.com>,
        Pengutronix Kernel Team <kernel@pengutronix.de>
References: <20220713074118.14733-1-frieder@fris.de>
 <20220713074118.14733-5-frieder@fris.de>
 <CAEyMn7a7S6UZOTsubM3pY95diY0oR0WNvhQpTUSmedjG59Qn9g@mail.gmail.com>
From:   Frieder Schrempf <frieder.schrempf@kontron.de>
In-Reply-To: <CAEyMn7a7S6UZOTsubM3pY95diY0oR0WNvhQpTUSmedjG59Qn9g@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dVhRMHgrd09ZTmMySGdVWkhNVVNDd1ZjTG5Pc2xxejJLMkVwS3M1TEpJbmE3?=
 =?utf-8?B?SjZJV3VBQTNCT0lRVzk4V29GYWZrYlBsUUVQZmdmazRVeFZWd0w1c1dLT1R0?=
 =?utf-8?B?ZnBoWUkzSDZHemcrOUJNV2JJSnNKbk9VWHIzeGtxM0FUQm5GcHJlNktUU2F3?=
 =?utf-8?B?ZWZtejdxSWV1ZWtoQUtHOHdQams3Q2hmTFMwbzRxRGtUbFpVcmpxT20rc2du?=
 =?utf-8?B?QmxnckR4VFkxYlJJdXRIems1djFndXhWTWhKSWg4MVV2Um9wdUxzMG12c1pE?=
 =?utf-8?B?K1FHUUt0VldUU1JSSUU0VW9sZkVudEUvMHg5Ly8rUjVkMUgrbzdVQnkrRVdv?=
 =?utf-8?B?ZWdLWDhEYm5mSE96Q3hBcndKVUVnK0d4c1hsOXZyaFZwVnFjMjRHTmpKVmY4?=
 =?utf-8?B?VVloTXNIbUxxelMxR2xHZXBwZXE3NmJ1T3lQNHBBNFlQSmdOLzhUSHFSdUIv?=
 =?utf-8?B?V3hiY2xjOVY2TE5Pd1FMaUtvOTFBMjNWT05XWjFtblRzaXJ6bkhZcHl3eDVG?=
 =?utf-8?B?QzVGbmJSeGpYVU02a21vTlltd2F2RXpwRitMMWZUMjByWDBPY0lzSDg0Ynps?=
 =?utf-8?B?N0IzWEdVdDIzeFB0ajZ3QTRsdFZKZVZCT3ZyZU84OEdXWnJqcnB2L3V4aGlh?=
 =?utf-8?B?MzVHOGpNMWEvR3FtMlV2b1UyYzN5RmpEazZpb3BkZGxvb0o0UGVsd0dZbFV5?=
 =?utf-8?B?a0JEbzJHNjExMm0vVkhTV1NzUm55ZXhaK2g5MWZZUFpMaE9idDhiMzBOcTRs?=
 =?utf-8?B?eENmWUtqdnBXckgwVFQra3JhSlVuNUdtUmx4VHB0azBjT0FlT1NkTElhaHpZ?=
 =?utf-8?B?VXo3amJMYk9PbldmQUt3bDBZNFEwZjNYaHJpN3hzNHg4QVRoMWZHMThSZW5u?=
 =?utf-8?B?K3VIWW03REpsSjNkUFRmNW1rM1ZkR2ZnQnNLdmRWdy8xMXFZYWhIVTE1YVFo?=
 =?utf-8?B?UXFpK0FRbnRoSVljZDJ3dytMOFZITXdsakhWUmF4RVllZW40eUE4SC9kUFJU?=
 =?utf-8?B?OXBHVjlKUVdqYWpGeDA1dnViU1BFOG5sZkVBN3hCVmZXN3BsaE53ZnFsS0Ni?=
 =?utf-8?B?NmxzUXZxNVJDdjRMdWhYVkJPSTc0NkpCOEZvQkZHcUp5Y1REUlZubk5vc1BT?=
 =?utf-8?B?K3VhNDhMendYQWcyTVFoRkdXblZON1c5Y1Q3NVpXaFBsUDUrOHQwUUhiVHZj?=
 =?utf-8?B?WHRwbzAvaWRLeEhLazFXWGNwcXJ3OUVxQmFhbktJYWRrcFBPbGVHbHBReW9n?=
 =?utf-8?B?Z1FOdVpqcTB3d1Vlc2ZvK2tmUjZKbURRVllwMERYdzdOQlU5ZTdQOGdhZ3lU?=
 =?utf-8?B?RkJDa0tVa08wQkFDMkpJOXZYTlBFOFZYWkpKSzQwWk9MT3dPWnZzS2xxNkRn?=
 =?utf-8?B?RXhVaEV2ZDVoR0hMeFVSc0hmNjExYm5JYkJETEloaCtCRHg1WHlvQXBCMUsv?=
 =?utf-8?B?eUlOeWRZWVNXSXh3R0k4aXE3R0ZOeERlazl2UFRheExwQ0cyQ0t2VXRPOFF0?=
 =?utf-8?B?MmIzWmVKQmhQUExReG42cU4xNWlKWkU2eFlRalVHOUc3b0tYc0R2QkJIWVNU?=
 =?utf-8?B?TG4vOGIrZmpoR1lpSmpFbEhXL1EvMWJtbWtCNzZQbnQ2ZmxvbDFjeGJNbllI?=
 =?utf-8?B?Y2NLQ092b1ZreGVaTHNXQW4zc01VK1ZQMFNpbTJiV3dJVWtnNEs3NHRXVXc4?=
 =?utf-8?B?MTNwKzhzTjZSL0xRZmN3SjlVbGhvNWRuTitaMzlvYklKZzlrdFhhc1BLNENx?=
 =?utf-8?B?bXlxcXNkaHlaUXlDRDRQcllzeVBsY1V4TkVzOGMzYmFMN3RMbmhUUWlaR1U0?=
 =?utf-8?B?UDBYKzJQU3JKQjVtK2cvTExXN1E4b1Y1NkxJVHgxcWhTQlNTclZrR0hNei9K?=
 =?utf-8?B?cWtVRmFKT256dFBta3ZrdnVTa1pWVVpaaVczVFEvbTVibGJyYlhOZ2VwN0Y2?=
 =?utf-8?B?TGo0TFJ6cHozaUpwTkVYdVpVeEZVZ093MkFXNDJpM3RtNldhSkcyczR3cFdo?=
 =?utf-8?B?MVhweEszQmdjU2FzdVZraHhQQ3dKRWFLZEt2Y21ZR3hzbjFONU9rbUlWdFFU?=
 =?utf-8?B?VnZSVXppc3pRWFFDdjJ3NStNTDU1dG0yaXYwOERZSTZGekUwVDF4aHVvaEVa?=
 =?utf-8?B?dDNMUHBLUm5lUGdLSnd5Z1NFdEJDbTRtdWhRYjNnMmVGQXRvNjY2YzA5ZGlI?=
 =?utf-8?Q?qLtVN3EaGkdBFqjonTcNBuESAhZu/0iUANJANHyBrnK9?=
X-OriginatorOrg: kontron.de
X-MS-Exchange-CrossTenant-Network-Message-Id: 12602f7b-3aa2-4a9b-ef93-08da64a7ebe2
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5681.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Jul 2022 08:16:01.3691
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 8c9d3c97-3fd9-41c8-a2b1-646f3942daf1
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: QlKzSKgFHE6kXSBjdZTLcSMLA+gRRo9M2/ITXHM9tb70HEYVZ0qliVULc317yiID0OgbSIIjK8c5k9fxpa5WxJcnV4DXdQ6JRWoOj/kA0Ss=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR10MB3255
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,
        SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Am 13.07.22 um 09:52 schrieb Heiko Thiery:
> Hi Frieder,
> 
> Am Mi., 13. Juli 2022 um 09:41 Uhr schrieb Frieder Schrempf <frieder@fris.de>:
>>
>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>
>> Use upper case PMIC regulator names to comply with the bindings.
>>
>> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
>> ---
>>  .../freescale/imx8mm-kontron-n801x-som.dtsi   | 22 +++++++++----------
>>  1 file changed, 11 insertions(+), 11 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi
>> index 2d0661cce89b..2e3d51bbf92e 100644
>> --- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi
>> +++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi
>> @@ -85,7 +85,7 @@ pca9450: pmic@25 {
>>
>>                 regulators {
>>                         reg_vdd_soc: BUCK1 {
>> -                               regulator-name = "buck1";
>> +                               regulator-name = "BUCK1";
> 
> Wouldn't it be better to use the real signal or voltage rail names
> used in the schematics?

Sounds like a good idea, thanks! I will try to include in the next
version of this patchset.