Skip to content

fw/drivers/sf32lb52: f32lb52 Solve the boot abnormality caused by the rng driver#121

Closed
w200024212 wants to merge 1 commit intocoredevices:mainfrom
w200024212:rng_drv
Closed

fw/drivers/sf32lb52: f32lb52 Solve the boot abnormality caused by the rng driver#121
w200024212 wants to merge 1 commit intocoredevices:mainfrom
w200024212:rng_drv

Conversation

@w200024212
Copy link
Copy Markdown
Contributor

rng driver parameter is_seed need set to 1;

Copy link
Copy Markdown
Contributor

@jwise jwise left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, though it would be nice to have a short // comment in there saying why this always needs to be called in is_seed mode (it already tripped us up once... probably if we continue to use this API it might trip us up again :) )

@gmarull
Copy link
Copy Markdown
Member

gmarull commented Jul 17, 2025

@w200024212 please add comment and we’ll then merge

@w200024212
Copy link
Copy Markdown
Contributor Author

@w200024212 please add comment and we’ll then merge

ok!

… rng driver

rng driver  parameter is_seed need set to 1;

Signed-off-by: Jerrywang <w200024212@163.com>
@w200024212
Copy link
Copy Markdown
Contributor Author

Before generating random numbers, it is necessary to ensure that the seed is valid.So, I made some adjustments to the code.

@gmarull
Copy link
Copy Markdown
Member

gmarull commented Jul 23, 2025

merged here 44572c6

also refactored code as I was having difficulties in following the logic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants