Received-SPF: pass (google.com: domain of linux-kernel+bounces-96625-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Avri Altman <Avri.Altman@wdc.com>
To: Daniel Golle <daniel@makrotopia.org>, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley
	<conor+dt@kernel.org>, Ulf Hansson <ulf.hansson@linaro.org>, Jens Axboe
	<axboe@kernel.dk>, Dave Chinner <dchinner@redhat.com>, Jan Kara
	<jack@suse.cz>, =?iso-8859-1?Q?Thomas_Wei=DFschuh?= <linux@weissschuh.net>,
	Christian Brauner <brauner@kernel.org>, Li Lingfeng <lilingfeng3@huawei.com>,
	Damien Le Moal <dlemoal@kernel.org>, Min Li <min15.li@samsung.com>, Adrian
 Hunter <adrian.hunter@intel.com>, Hannes Reinecke <hare@suse.de>, Christian
 Loehle <CLoehle@hyperstone.com>, Bean Huo <beanhuo@micron.com>, Yeqi Fu
	<asuk4.q@gmail.com>, Victor Shih <victor.shih@genesyslogic.com.tw>,
	Christophe JAILLET <christophe.jaillet@wanadoo.fr>, "Ricardo B. Marliere"
	<ricardo@marliere.net>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
	"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>
CC: Diping Zhang <diping.zhang@gl-inet.com>, Jianhui Zhao
	<zhaojh329@gmail.com>, Jieying Zeng <jieying.zeng@gl-inet.com>, Chad Monroe
	<chad.monroe@adtran.com>, Adam Fox <adam.fox@adtran.com>, John Crispin
	<john@phrozen.org>
Subject: RE: [RFC PATCH v2 6/8] mmc: core: set card fwnode_handle
Thread-Topic: [RFC PATCH v2 6/8] mmc: core: set card fwnode_handle
Thread-Index: AQHabzsoqnu1cWr8NEek9PCEYV4EaLEtf6cA
Date: Fri, 8 Mar 2024 08:04:54 +0000
Message-ID:
 <DM6PR04MB6575C0FB6376681697C97DE8FC272@DM6PR04MB6575.namprd04.prod.outlook.com>
References: <cover.1709667858.git.daniel@makrotopia.org>
 <055787bb6085c32907ee1772522a6bfa49d5d2ef.1709667858.git.daniel@makrotopia.org>
In-Reply-To:
 <055787bb6085c32907ee1772522a6bfa49d5d2ef.1709667858.git.daniel@makrotopia.org>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?iso-8859-1?Q?JNk1zdkdEEy2DabasCPQlewVFdZFgUtVNcA59F97DJ8ejNscnAKeS/nkdX?=
 =?iso-8859-1?Q?ae9Y3q9Tt8bew2Zw9dTMguZe4TZquA+eYCtn64Wa8y6A6mc9BuBhuKabJH?=
 =?iso-8859-1?Q?rA5edW+y62sx/b5JTl3QpjDyI4PYNIWqzUW4tMKppF4BG6Kgr7aoZ5xPbI?=
 =?iso-8859-1?Q?XqdjbugJOQDpkseb3iFSaAhf7l4AzvjNZs4H0wEI/2j+QS+BW6wqMAz3wO?=
 =?iso-8859-1?Q?832QzFVKsu1Oa4n3iRK4eJ7Lp+F7Q15/SgRxydxVVa7eV8ZkE4e05QrDF7?=
 =?iso-8859-1?Q?eV+6FYQfb2XKUsPX86szK5SkkCBBGeWTD9p1q5DLh4Sqy/j46Ft+C7w2/7?=
 =?iso-8859-1?Q?COpd0Hrrcnh+T7sqBWM6J9bTP/m4fx0vG+krioub3YqYZWvbU1ZisuB/A5?=
 =?iso-8859-1?Q?T1NQlFfbPJtDHaCgH4hSumoksXFiOdasEyBOgT//icmfZG156J5J1WlUzk?=
 =?iso-8859-1?Q?Xv940ni45d6vGR+GeM+4jl06ElfI63ITcNm9o0mR8n0nr6zDbZjvWbhDQV?=
 =?iso-8859-1?Q?Ox63Mm6raQuK/+fAWjS2VJbrukJP3N9v+VJ8pyMflFa27bJYDvzs/KpCfr?=
 =?iso-8859-1?Q?WU2bjBtOT7MtOBfTDD+DrBZ9ZeBRyCF0YgGNeX8+C+7G5kACwe4xN0RGL2?=
 =?iso-8859-1?Q?BUy9K1w7VBg3+6nP/NtKB3ZyOVbkO7NVsTGzHzGwEanWneqj/VK5iugrh1?=
 =?iso-8859-1?Q?ZxfpRlLDwBQJGFrXZrtTtakgcI8s+Qai/ht8bzf++i9qPWm83YTMI0gm39?=
 =?iso-8859-1?Q?gumObBgyPZ6bvQugEvXlVFL70ePZoys+Kvc314Gc+uPbssRk+shk2zwFsQ?=
 =?iso-8859-1?Q?okrSF7J0zEa6NOVutVHK0BlvtmpvLNboZU7b64UYgbpoQN02b75/SVDEXb?=
 =?iso-8859-1?Q?+RLM1tLFbybumwx6etOQya33V1SvQw0rVCwDmavnMIg4m52PEdk1qvsq9X?=
 =?iso-8859-1?Q?QdvsKBF7/1EdAWNLmSaM0e1EPeu9f01IeE211r4IIS4mba9xj+9JwOZczy?=
 =?iso-8859-1?Q?nZaorUJir3/eW77NADCtM2eCZeLZWe94sMDGs8wEIrxKYpD9tOK7Ksi9vV?=
 =?iso-8859-1?Q?H7IwF5kIIRiyFxFOKsI2Eqk10UUXWcxWliiXNmHzc0m5BqVrA0muDw3tw9?=
 =?iso-8859-1?Q?xbZIelrs1q5Fn3WsPuEXb7m0QeDT7kl5mi0wk6EoE4AoomWjEws5BTA/bi?=
 =?iso-8859-1?Q?ilG+/W+tdBxYirA/OIfqTt/rDn3q6bGo9bu3I1BG1TcTI5YcSL9iMNBUYB?=
 =?iso-8859-1?Q?1Pl1hMIZuh/tAKl24j3DDptYwtjsZ/S/J+g/InYssWmRNQ8MY6bsdKnUVP?=
 =?iso-8859-1?Q?WTAQIhBDePEWI1Nh0OGZB247LGyFo9WG5NUOG1SrO3K+FeE8HQc5aJvQ2M?=
 =?iso-8859-1?Q?mEEeRw9CKfgdyzq0WCedcBVo8bG8kx571xhqoBbCD9KaLo9Y3pSeudLmg5?=
 =?iso-8859-1?Q?XWhveY9iwCVla91XnF/rmtGIeD7MkVfuWnYspE9xTuVweeP5kM0C3cXWyW?=
 =?iso-8859-1?Q?Gdzd0bEOFSjecxNGvBN8dmwO1Ya42Z+bvLeJ4kdO6s3Jd56GLyxxgWjJVd?=
 =?iso-8859-1?Q?IOnqmugcsUaNbH+KzsjS+kV3GSN6XIyYsMiK7jSPccT464f+dVOBhbHtnh?=
 =?iso-8859-1?Q?FUilyReSlFevMkbYUNYhSkcZmq2JFd0Z20?=
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0:
	JSM48ALQn5YRsG2eKQeauKM6OmH/5aLOlyrkwPQ35D6C2rHC0N3hZLourSZjfWHmEbaMbhfxxdqX3Y6sq1TsCgrBfNY4kuEocQFhUcqYPatFFjDfNbJIxvnqo3ljjEjN7nEst1CijQDk8WjB/7mncUAFuYXUJQh1ixkJbpw9CRnDBHJreBe1xE8NujZopnPht/QvOrwbT7k/4yyZryn7ydoC/mXtLWPwORW43FT0XQWnIZnjDN8Sp9sp3uwUZTP2iRnhIcWueBrPawqSRbgY2pt3XpUXMN6Siiv5yo0yBqyYDp+/A6AX5UGJCk3FyEA5GdDiDYbxOkqmSkfbgbGI8KPsLKrf+x2l4T9JqZmPiAkGpi6ZoKUQyxIuOEHADv902cuCtv06Zt/3Xt5H48JLZ2L3yh32S0bv9JRxqFezjusO0Q0r2KtmKV49Ei1EvwDKmWfvSyleWkhj+4ayR8/ai7ltuE76O4cf4wUIrOoFEc25ohFiO4bWCsOgaeUYSNozxnBvX2ruMmKFRZ0vUZ6QG2l4S5wf1URML8fmNlaID1n/5gKeZ7F/AkhG9iEO2SonqQCjK3XQrvi9y+VeEBcV4+mH0SZM5y9UwFn9NttpL5LrAwGxD9NaDibCY0ItGZ3J
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB6575.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fca20e09-979f-4286-723b-08dc3f46704d
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Mar 2024 08:04:54.4090
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: +ADaP+VnWmxsqfoq3/IImFLPBB09UGS06VnDT1wmrnuNMMAAxCa+Tzd/nI0j3NzSJAWUGRvQKOFiZUw8ZV7wDg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR04MB8861

=20
> Set fwnode in case it isn't set yet and of_node is present.
>=20
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>
> ---
>  drivers/mmc/core/bus.c | 2 ++
>  1 file changed, 2 insertions(+)
>=20
> diff --git a/drivers/mmc/core/bus.c b/drivers/mmc/core/bus.c index
> 0ddaee0eae54f..e1c5fc1b3ce4b 100644
> --- a/drivers/mmc/core/bus.c
> +++ b/drivers/mmc/core/bus.c
> @@ -364,6 +364,8 @@ int mmc_add_card(struct mmc_card *card)
>=20
>         mmc_add_card_debugfs(card);
>         card->dev.of_node =3D mmc_of_find_child_device(card->host, 0);
> +       if (card->dev.of_node && !card->dev.fwnode)
> +               card->dev.fwnode =3D &card->dev.of_node->fwnode;
Should this be restricted to eMMC only, or is it fine to be called for SD a=
s well?

Thanks,
Avri

>=20
>         device_enable_async_suspend(&card->dev);
>=20
> --
> 2.44.0