On Jan 24, 2024 at 15:45:58 +0530, Dhruva Gole wrote: > The secure_hdr needs to be 0 init-ed however this was never being put > into the secure_buf, leading to possibility of the first 4 bytes of > secure_buf being possibly garbage. > > Fix this by initialising the secure_hdr itself to the secure_buf > location, thus when we make it 0, it automatically ensures the first 4 > bytes are 0. > > Fixes: 32cd25128bd849 ("firmware: Add basic support for TI System Control > Interface (TI SCI)") > Signed-off-by: Dhruva Gole <d-g...@ti.com> > --- > > Boot tested for sanity on AM62x SK > https://gist.github.com/DhruvaG2000/724ceba3a0db03f4b0bff47de1160074 > > Cc: Nishanth Menon <n...@ti.com> > Cc: Tom Rini <tr...@konsulko.com> > > drivers/firmware/ti_sci.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) >
Kindly ignore this patch, seems like it's from an older u-boot. I have rebased an sent new patch with updated boot logs. Please forgive the spam. -- Best regards, Dhruva Gole <d-g...@ti.com>