Hi Cédric, Thanks for the review.
Best Regards, Kane > -----Original Message----- > From: Cédric Le Goater <[email protected]> > Sent: Wednesday, September 17, 2025 2:23 PM > To: Kane Chen <[email protected]>; Peter Maydell > <[email protected]>; Steven Lee <[email protected]>; Troy > Lee <[email protected]>; Jamin Lin <[email protected]>; Andrew > Jeffery <[email protected]>; Joel Stanley <[email protected]>; > open list:ASPEED BMCs <[email protected]>; open list:All patches CC > here <[email protected]> > Cc: Troy Lee <[email protected]>; [email protected] > Subject: Re: [SPAM] [PATCH v2 0/3] tests/functional/arm: Add OTP functional > test > > On 9/17/25 05:59, Kane Chen wrote: > > From: Kane-Chen-AS <[email protected]> > > > > Hi all, > > > > During early boot, SoC firmware reads from the OTP region to obtain > > the chip ID and default configuration. This series adds functional > > tests that boot with a generated OTP image and verify that the > > firmware reads these contents correctly during initialization. > > > > Any feedback or suggestions are appreciated. > > > > Best Regards, > > Kane > > --- > > Change log: > > v2: Fine-tune test scripts > > > > v1: Initial version > > --- > > Kane-Chen-AS (3): > > tests/functional/arm: Add helper to generate OTP images > > tests/functional/arm: Add AST1030 boot test with generated OTP image > > tests/functional/arm: Add AST2600 boot test with generated OTP > > image > > > > tests/functional/arm/test_aspeed_ast1030.py | 24 > +++++++++++++++++---- > > tests/functional/arm/test_aspeed_ast2600.py | 14 ++++++++++++ > > tests/functional/aspeed.py | 8 +++++++ > > 3 files changed, 42 insertions(+), 4 deletions(-) > > > > For the series, > > Reviewed-by: Cédric Le Goater <[email protected]> > > Thanks, > > C. >
