Re: [PATCH v2 0/2] mtd: devices: Qualcomm SCM storage support

From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>,
	 Junhao Xie <bigfoot@radxa.com>,
	 Konrad Dybcio <konradybcio@kernel.org>,
	 Richard Weinberger <richard@nod.at>,
	 Vignesh Raghavendra <vigneshr@ti.com>,
	linux-arm-msm@vger.kernel.org,  linux-kernel@vger.kernel.org,
	linux-mtd@lists.infradead.org,  Xilin Wu <sophon@radxa.com>
Subject: Re: [PATCH v2 0/2] mtd: devices: Qualcomm SCM storage support
Date: Thu, 29 Jan 2026 19:38:44 +0100	[thread overview]
Message-ID: <87fr7ol0wb.fsf@bootlin.com> (raw)
In-Reply-To: <njrzcyhdkohsmnlisdzu25b6obqotjjrzlrs42jhq5bhlz6qco@mdk5blghdqbc> (Bjorn Andersson's message of "Wed, 28 Jan 2026 14:31:33 -0600")

Hello Bjorn,

>> If I may, I even skipped the mail entirely because it was not about mtd
>> in the end. There is a single big patch labelled "firmware: qcom:". I
>> believe the series would better be split in smaller chunks. Typically
>> one for the storage support and another one for the firmware part, at
>> least.
>> 
>
> The firmware patch adds wrappers for the secure world interface, which
> without the MTD driver will lack consumers of the API - i.e. I'd prefer
> not to pick that part, or parts thereof, alone.

Yes, I understand.

> So I think it does make sense to introduce the two parts together in one
> series (if you have any requests for splitting the MTD patch up in any
> way I have no objections).
>
> This leaves us with the practical problem of getting it merged, which I
> believe would best be handled by me picking the firmware patch and
> sharing this in an immutable branch with you, once we're happy with the
> set (which won't be for 6.20).

Works for me!

I'll send a minor review with a couple of comments, but we can apply
this close after next -rc1, if you are good with the fw part.

Thanks,
Miquèl