IronPort-SDR: WQJqQ7/OWABuyJj9f0h+RFQU3v1Ce1atzZHeYFwnBGU3FnUGH6VnHdu2bD0YRfCpxrcWWgBC03
 QOrjVW/7CXzfx6L0BfVSvNkPadaqdMT8IjdzgqKgjaKi50HSA7XzTY4NuiqGA6KhT8MSqugfnB
 g/Aj1VD8cvvW8uiizUi3vcKkCGlK6v7W8hUQRm6FA4Q0N3ILjhyWbg18VHqvnSBggQSO1l9EQM
 llKC7D3r74DmCAnFjYQ1l74NrEGLA9CMq3NI3jOjCm5HUVOGYFSKJyGrjh2WLe6QXCui2nlFtR
 dcMxkVDqH3R+QkLMyVhsCErt
From:   <Eugen.Hristev@microchip.com>
To:     <Nicolas.Ferre@microchip.com>, <devicetree@vger.kernel.org>
CC:     <alexandre.belloni@bootlin.com>, <Ludovic.Desroches@microchip.com>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>, <Mihai.Sain@microchip.com>
Subject: Re: [PATCH] ARM: dts: at91: sama5d2_xplained: remove
 PA11__SDMMC0_VDDSEL from pinctrl
Thread-Topic: [PATCH] ARM: dts: at91: sama5d2_xplained: remove
 PA11__SDMMC0_VDDSEL from pinctrl
Thread-Index: AQHXymxb5xC4iJlwgE+oBKrTcDSgSqwotsuAgAABlAA=
Date:   Wed, 8 Dec 2021 11:27:44 +0000
Message-ID: <f943539b-bc95-35c9-a634-ad75f9df54b0@microchip.com>
References: <20211026132034.678655-1-eugen.hristev@microchip.com>
 <28f0898c-0d6c-a77e-4068-b49c6dd0aa64@microchip.com>
In-Reply-To: <28f0898c-0d6c-a77e-4068-b49c6dd0aa64@microchip.com>
Accept-Language: en-US, ro-RO
Content-Language: en-US
user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.8.0
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?Windows-1252?Q?2c4KFmcCIkFZClSxIPPdP//8u3OHAikDMr6s7RAYth+Yd/OhtX1Sf5Qb?=
 =?Windows-1252?Q?JMZ5qL0dFiZ+bSrvMeookzm9SsygMD9ZiBm2xzzlFi8tViOab/rxWyco?=
 =?Windows-1252?Q?7KWwD49gKnmSrDjN/WjfFWoX9pVT7HRY8vxAPnjPDisZ3qbrkH1/J5ql?=
 =?Windows-1252?Q?GjEE+YgKNlbIxMApIS8TdD3bIX9AfRShAfP5wUcSV/0uPIwQNWHLsEfx?=
 =?Windows-1252?Q?P1swesSj8uCi689Uc+LK4uLAxPK3Nw0py/AEiWqb35OIlxv5OdP7MT78?=
 =?Windows-1252?Q?A59i/TthQE6Q/5JFXroDusS1FmeHGvxBhYohS3mOLVyZw6RqXiyhbXXx?=
 =?Windows-1252?Q?mwt+hx56gfPvCPayPBYdpnzagsjUIMbdvPhZgisEw07e8AuAw2+nH+yt?=
 =?Windows-1252?Q?32W75wyp9wh9tssiQA5Ex/DeGpF5lyzWiP305M58WKGk7MGgCR6xj2Tr?=
 =?Windows-1252?Q?k7QPq5bPHka2M8dgVdkWCFp3DmJQGN2XdN+N0RJnXSqsQBBSSKGbHVlP?=
 =?Windows-1252?Q?it6d6fOBssqJ52cKQsepnU2vhLCr3UY4F8iFUoqcTtNQ8IMVHza5EqCa?=
 =?Windows-1252?Q?gwSA0XOpIH6cYS56XuTt+nSEv1uwhsdRN3szQCU/F5sZBSyTXWV0gk28?=
 =?Windows-1252?Q?oByk8xOrZH6SKuqWllowk0I+yj5/trmiLx5BlOFhZfVfwVDJt2fZXlAe?=
 =?Windows-1252?Q?hKxy2yFGYO1tvFk0TAk3N+dP+1sC9bnT1SCjSnfHoTyWsRKlEGdM/jUQ?=
 =?Windows-1252?Q?pUur7MMGV5DFfRSyd+Cf65E+Zfb/L7+FMEiQsZhqOqqkW28thhTmLBGd?=
 =?Windows-1252?Q?dlXvEW7gBwD41Je0ls7gZV+kUacw54dD6X0rCI1nBMZqkyHbzDn/sSGY?=
 =?Windows-1252?Q?TEUQsB7+u3v0DvudDysXoFt4FTN+Ux6TSvqGNmLbYdN36Hgm4480psFv?=
 =?Windows-1252?Q?9u5/0QcSIOIjPE4X1dc+ACICRW/tIA5b9IFaFdeEu046FKJY4l0VGWRz?=
 =?Windows-1252?Q?E/UXm0aPKJ/0ux//S5CtKr6BhBRbDD/Qj3HB0D0lTXCd0qm5sCV5sb+P?=
 =?Windows-1252?Q?wiavpZNeBxT8vr2mwKRO9ravTw9pVCWs/TSYmgrSYUc4NNllnvU8j8EK?=
 =?Windows-1252?Q?Cw4ctVy6S21rAWXus+SKZ8lDFT5NGN8qBGjaaCEj5Zqxs9bN1elLfM/Y?=
 =?Windows-1252?Q?+a30AzPGP6Y6WbkjAYRvvQ5HUf6vKpD40MacUU60bUXqSBx8O+YggTvz?=
 =?Windows-1252?Q?IGjezXxjoMZCP5INYhwyJNVUFo7UKnbcuQwgRmjwfMEnUtn1z+OM1ew9?=
 =?Windows-1252?Q?EcjddU1Y+wYhLxajSXLokbK35qZVoMhaZlR8KhExY4d4FZxSvkVlOkXo?=
 =?Windows-1252?Q?poxpU6jQ6g9ab2GBc/GUrQFCUjDb+pldVF+Cqz4zmLX50J3Qi8PgIIHf?=
 =?Windows-1252?Q?HIprR71xFN4Fg9lsm3nFs3Q8qpF8t95DhbtPr+mXBXldRokjS27q35iy?=
 =?Windows-1252?Q?eavzHzh60qjcGH39NvrEBqznZs/Jxs7hsEZ8JDO7CKO/g6KNIe2rL269?=
 =?Windows-1252?Q?rA41vj2gzHiGktBJa7WkAq2QlejB6Vif448ZrKdEYfgnDIGDKnZ7uXnP?=
 =?Windows-1252?Q?544nSJsbYv55wTclOzg3V0hzJwcARFhJIcEWo3hIZWq8geV+/+bfPJHA?=
 =?Windows-1252?Q?anpbEINVlhhMBWsz5jrMBTVYzOgUZMmaIf+mJ/2DhSYXiygEDZ00pOR9?=
 =?Windows-1252?Q?WMPpS6g+1dqa9MP08chWdCeorgTFgBDjl6Eo51K7zzw0Nq5jRbYoAv6t?=
 =?Windows-1252?Q?NmnH6Q=3D=3D?=
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <D052519973D32B489BF382D0EBB88915@namprd11.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB4920.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 85c2dc74-08f7-40b7-afec-08d9ba3dc149
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Dec 2021 11:27:44.7104
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 8JwiT1JqU/84Di4eCjzQnWBMd1jzDzdWppYrBOutlxhnt/QImwd4IV7DBKG3PYSYP2181YJ+KGucKa5GzG7gP4EyIMVLNIwLGHVM5ffp2ds=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5904
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 12/8/21 1:22 PM, Nicolas Ferre wrote:
> On 26/10/2021 at 15:20, Eugen Hristev wrote:
>> From: Mihai Sain <mihai.sain@microchip.com>
>>
>> I/O voltage for eMMC is always 3.3V because PA11__SDMMC0_VDDSEL is
>> tied with 10K resistor to GND. U13 switch S1 is always selected as
>> voltage rail of 3.3V for VCCQ power pin from MPU controller and eMMC=20
>> flash.
>> Removing PA11 from pinctrl because it remains unused.
>>
>> Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
>=20
> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>

You can also add my
Reviewed-by: Eugen Hristev <eugen.hristev@microchip.com>

and also
Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
because I sent the patch.


>=20
> Thanks, best regards,
>  =A0 Nicolas
>=20
>> ---
>> =A0 arch/arm/boot/dts/at91-sama5d2_xplained.dts | 5 ++---
>> =A0 1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts=20
>> b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
>> index b1e854f658de..9bf2ec0ba3e2 100644
>> --- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts
>> +++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
>> @@ -66,7 +66,7 @@ sdmmc0: sdio-host@a0000000 {
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 pinctrl-names =3D "default";
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 pinctrl-0 =3D <&pinctrl_sdmmc0_d=
efault>;
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 non-removable;
>> -=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 mmc-ddr-1_8v;
>> +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 mmc-ddr-3_3v;
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 status =3D "okay";
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0 };
>> @@ -619,10 +619,9 @@ cmd_data {
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=
=A0 bias-disable;
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 };
>> -=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 ck_cd_rstn_vd=
dsel {
>> +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 ck_cd_rstn {
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=
=A0 pinmux =3D <PIN_PA0__SDMMC0_CK>,
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=
=A0=A0=A0=A0=A0=A0 <PIN_PA10__SDMMC0_RSTN>,
>> -=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=
=A0=A0=A0=A0=A0 <PIN_PA11__SDMMC0_VDDSEL>,
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=
=A0=A0=A0=A0=A0=A0 <PIN_PA13__SDMMC0_CD>;
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=
=A0 bias-disable;
>> =A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 };
>>
>=20
>=20