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

[Miquel Raynal <miquel.raynal@bootlin.com>]

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