[coreboot-gerrit] Change in coreboot[master]: mb/google/volteer: Update flashmap descriptor for CSE Lite FW update

[Jamie Ryu (Code Review)]

Jamie Ryu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43790 )

Change subject: mb/google/volteer: Update flashmap descriptor for CSE Lite FW update
......................................................................

Patch Set 2:
> Patch Set 2:
> > Patch Set 2:
> > > Patch Set 2:
> > >
> > > This breaks the update flow, right? Any coordination needed here?
> >
> > Can you share more information why this breaks the update flow? Thanks.
>
> firmware updates overwrite one of the RW_SECTION_[AB] regions, keeping everything else intact but that requires that they keep their location and size.
>
> I'm not sure at which state this device is, and if there are any such incremential updates already. If not (and all updates are always full rewrites) there's no issue, otherwise we might have to look into it.

Thanks for comments. Can I have comments from others reviewers regarding this?
To enable CSE FW update, I think FMD update is required anyway to reserve enough space for RW_SECTION_A/B, and it is better FMD change is done sooner. Any comments would be appreciated.