[PATCH v2 01/15] Revert "dm: SMEM (Shared memory) uclass"
Casey Connolly
casey.connolly at linaro.org
Thu May 7 22:05:46 CEST 2026
On 07/05/2026 17:31, Simon Glass wrote:
> Hi Casey,
>
> On Mon, 4 May 2026 at 12:57, Casey Connolly <casey.connolly at linaro.org> wrote:
>>
>> SMEM is a highly Qualcomm specific interface, while having a dedicated
>> UCLASS for it offers a nice abstraction, for things like memory layout
>> parsing we need to use it before the driver model is available.
>
> Do you mean before or after relocation?
Before, we actually use it and read the memory map in
board_fdt_blob_setup() since we support the usecase where you chainload
U-Boot from ABL but with its own DTB baked in. This allows you to leave
the original Android DTB and read the memory map from it while using a
proper DTB for U-Boot without having to hardcode the memory map.
Basically it exists because some Android phones have awful bootloaders
that will crash or break stuff if you don't have very specific arbitrary
things in your devicetree.
>
>>
>> Therefore, it doesn't make sense to fit SMEM into the driver model.
>> Instead let's adopt a model closer to Linux, and parse SMEM really early
>> during boot (as soon as we have the FDT).
>>
>> This reverts commit 7b384eccc785b596f68448b155cbda26df57fb23.
>>
>> Signed-off-by: Casey Connolly <casey.connolly at linaro.org>
>> ---
>> arch/arm/Kconfig | 1 -
>> drivers/Kconfig | 2 --
>> drivers/Makefile | 1 -
>> drivers/smem/Kconfig | 25 -------------
>> drivers/smem/Makefile | 7 ----
>> drivers/smem/smem-uclass.c | 46 ------------------------
>> include/dm/uclass-id.h | 1 -
>> include/smem.h | 90 ----------------------------------------------
>> 8 files changed, 173 deletions(-)
>
> [..]
>
> Regards,
> Simon
--
// Casey (she/her)
More information about the U-Boot
mailing list