On 6/30/25 07:17, Kane Chen wrote:
> From: Kane-Chen-AS <kane_chen@aspeedtech.com>
>
> The has_otpmem attribute is enabled in the SBC subclasses for AST2600
> to control the presence of OTP support per SoC type.
>
> Signed-off-by: Kane-Chen-AS <kane_chen@aspeedtech.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Thanks,
C.
> ---
> hw/arm/aspeed_ast2600.c | 2 +-
> hw/misc/aspeed_sbc.c | 2 ++
> 2 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/hw/arm/aspeed_ast2600.c b/hw/arm/aspeed_ast2600.c
> index d12707f0ab..59ffd41a4a 100644
> --- a/hw/arm/aspeed_ast2600.c
> +++ b/hw/arm/aspeed_ast2600.c
> @@ -261,7 +261,7 @@ static void aspeed_soc_ast2600_init(Object *obj)
>
> object_initialize_child(obj, "i3c", &s->i3c, TYPE_ASPEED_I3C);
>
> - object_initialize_child(obj, "sbc", &s->sbc, TYPE_ASPEED_SBC);
> + object_initialize_child(obj, "sbc", &s->sbc, TYPE_ASPEED_AST2600_SBC);
>
> object_initialize_child(obj, "iomem", &s->iomem, TYPE_UNIMPLEMENTED_DEVICE);
> object_initialize_child(obj, "video", &s->video, TYPE_UNIMPLEMENTED_DEVICE);
> diff --git a/hw/misc/aspeed_sbc.c b/hw/misc/aspeed_sbc.c
> index 46a038337c..b56a8b7678 100644
> --- a/hw/misc/aspeed_sbc.c
> +++ b/hw/misc/aspeed_sbc.c
> @@ -273,8 +273,10 @@ static const TypeInfo aspeed_sbc_info = {
> static void aspeed_ast2600_sbc_class_init(ObjectClass *klass, const void *data)
> {
> DeviceClass *dc = DEVICE_CLASS(klass);
> + AspeedSBCClass *sc = ASPEED_SBC_CLASS(klass);
>
> dc->desc = "AST2600 Secure Boot Controller";
> + sc->has_otp = true;
> }
>
> static const TypeInfo aspeed_ast2600_sbc_info = {