Hi Patrick On 1/6/23 13:20, Patrick Delaunay wrote: > Remove unnecessary return in stm32mp_bsec_write_lock and replace tab > by space for plat_auto opts. > > Signed-off-by: Patrick Delaunay <patrick.delau...@foss.st.com> > --- > > arch/arm/mach-stm32mp/bsec.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/arch/arm/mach-stm32mp/bsec.c b/arch/arm/mach-stm32mp/bsec.c > index c00130b08b36..51ccff9aa560 100644 > --- a/arch/arm/mach-stm32mp/bsec.c > +++ b/arch/arm/mach-stm32mp/bsec.c > @@ -468,8 +468,6 @@ static int stm32mp_bsec_write_lock(struct udevice *dev, > u32 val, u32 otp) > plat = dev_get_plat(dev); > > return bsec_permanent_lock_otp(dev, plat->base, otp); > - > - return -EINVAL; > } > > static int stm32mp_bsec_read(struct udevice *dev, int offset, > @@ -608,7 +606,7 @@ U_BOOT_DRIVER(stm32mp_bsec) = { > .id = UCLASS_MISC, > .of_match = stm32mp_bsec_ids, > .of_to_plat = stm32mp_bsec_of_to_plat, > - .plat_auto = sizeof(struct stm32mp_bsec_plat), > + .plat_auto = sizeof(struct stm32mp_bsec_plat), > .ops = &stm32mp_bsec_ops, > .probe = stm32mp_bsec_probe, > };
Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com> Thanks Patrice