[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <877c1ivcym.fsf@bootlin.com>
Date: Wed, 11 Jun 2025 11:34:25 +0200
From: Miquel Raynal <miquel.raynal@...tlin.com>
To: Guenter Roeck <linux@...ck-us.net>
Cc: Richard Weinberger <richard@....at>, Alexander Usyskin
<alexander.usyskin@...el.com>, Vignesh Raghavendra <vigneshr@...com>,
Lucas De Marchi <lucas.demarchi@...el.com>, Thomas Hellström
<thomas.hellstrom@...ux.intel.com>, Rodrigo Vivi
<rodrigo.vivi@...el.com>, Maarten Lankhorst
<maarten.lankhorst@...ux.intel.com>, Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>, David Airlie
<airlied@...il.com>, Simona Vetter <simona@...ll.ch>, Jani Nikula
<jani.nikula@...ux.intel.com>, Joonas Lahtinen
<joonas.lahtinen@...ux.intel.com>, Tvrtko Ursulin <tursulin@...ulin.net>,
Karthik Poosa <karthik.poosa@...el.com>, Reuven Abliyev
<reuven.abliyev@...el.com>, Oren Weil <oren.jer.weil@...el.com>,
linux-mtd <linux-mtd@...ts.infradead.org>, DRI mailing list
<dri-devel@...ts.freedesktop.org>, intel-gfx
<intel-gfx@...ts.freedesktop.org>, linux-kernel
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v6 01/11] mtd: core: always create master device
On 10/06/2025 at 09:15:25 -07, Guenter Roeck <linux@...ck-us.net> wrote:
> On 6/10/25 05:54, Richard Weinberger wrote:
>> ----- Ursprüngliche Mail -----
>>> Von: "Alexander Usyskin" <alexander.usyskin@...el.com>
>>> Richard, I've reproduced your setup (modulo that I must load mtdram manually)
>>> and patch provided in this thread helps to fix the issue.
>>> Can you apply and confirm?
>> Yes, it fixes the issue here! :-)
>>
>
> It doesn't seem to fix the issue if the partition data is in
> devicetree.
I had a look at the patch again. The whole mtd core makes assumptions on
parenting, which is totally changed with this patch. There are so many
creative ways this can break, I don't believe we are going to continue
this route. I propose to revert the patch entirely for now. We need to
find another approach, I'm sorry.
Alexander, can you please remind me what was your initial problem? I
believe you needed to anchor runtime PM on the master device. Can you
please elaborate again? Why taking the controller as source (the
default, before your change) did not work? Also why was selecting
MTD_PARTITIONED_MASTER not an option for you? I'm trying to get to the
root of this change again, so we can find a solution fixing "the world"
(fast) and in a second time a way to address your problem.
Thanks,
Miquèl
Powered by blists - more mailing lists