[U-Boot] [PATCH 1/2] SECURE BOOT: add version info for ISBC

[Wolfgang Denk]

Dear Aneesh,

In message <caf0224e47434bdba2623faeea12bd9a at DM2PR03MB415.namprd03.prod.outlook.com> you wrote:
>
> > But this is what I had in mind.  It appears to be a simple 1:1 mapping
> > from SoC to ISBC version to me, or am I wrong here?
...
> Yes, it is a 1:1 mapping from SoC to ISBC version but we have multiple SoC's
> having the same ISBC version. So using the SoC names would result in having 
> a long list of SoC's wherever we are doing some stuff specific to the ISBC 
> version.
> Also in future, we might have new versions of ISBC with more features.
> SO having multiple SoC specific checks inside the code would lead to confusion.

So don't use code, and instead make this data-driven.  Use a
lookup-table.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"The algorithm to do that is extremely nasty. You might want  to  mug
someone with it."                   - M. Devine, Computer Science 340