Hello! On Tue, Dec 13, 2022 at 08:39:46PM -0800, Simon Glass wrote: > Hi Alexey, > > On Tue, 13 Dec 2022 at 10:32, Alexey Romanov <[email protected]> wrote: > > > > This function can be used by other modules. > > > > Signed-off-by: Alexey Romanov <[email protected]> > > --- > > arch/arm/include/asm/arch-meson/sm.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/arch/arm/include/asm/arch-meson/sm.h > > b/arch/arm/include/asm/arch-meson/sm.h > > index f3ae46a6d6b..b68edf842e4 100644 > > --- a/arch/arm/include/asm/arch-meson/sm.h > > +++ b/arch/arm/include/asm/arch-meson/sm.h > > @@ -8,6 +8,8 @@ > > > > ssize_t meson_sm_read_efuse(uintptr_t offset, void *buffer, size_t size); > > > > +ssize_t meson_sm_write_efuse(uintptr_t offset, void *buffer, size_t size); > > + > > #define SM_SERIAL_SIZE 12 > > > > int meson_sm_get_serial(void *buffer, size_t size); > > -- > > 2.25.1 > > > > Should add function comments. > > Also, you might consider adding a uclass for this. It seems like an > important use case that is not handled right now. > > Regards, > Simon
I'm afraid we can't generalize sm API for each arch types. It's very close to Secure Monitor vendor implementation and can provide various APIs for different purposes. -- Thank you, Alexey
