Hi > >> --- /dev/null >> +++ b/board/bachmann/ot1200/Kconfig >> @@ -0,0 +1,23 @@ >> +if TARGET_OT1200 >> + >> +config SYS_CPU >> + string >> + default "armv7" >> + >> +config SYS_BOARD >> + string >> + default "ot1200" >> + >> +config SYS_VENDOR >> + string >> + default "bachmann" >> + >> +config SYS_SOC >> + string >> + default "mx6" >> + >> +config SYS_CONFIG_NAME >> + string >> + default "ot1200" >> + > > > The type "string" is not mandatory since > commit 461be2f96e4b87e5065208c6659a47dd0ad9e9f8. > You can save 5 lines.
Nice to know. > Since it is not a big deal, > I don't think you have to resend it just for fixing this. > (I am just pointing out a minor thing I found by chance.) > > If you like or you have a chance to send v4 for another fix, > I recommend you to drop "string". > I am not sure if the current patch set gets accepted or not. Either way I will send a v4 or a followup patch to correct this. I have some more patches in my queue but I need to wait until this patch hits u-boot-imx. greets -- Christian Gmeiner, MSc https://soundcloud.com/christian-gmeiner _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot