[coreboot-gerrit] [S] Change in coreboot[master]: util/amdfwtool/amdfwread: Fix AMDFW_OPT* bit mask

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Robert Zieba, Raul Rangel, Rob Barnes, Felix Held.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/66573 )

Change subject: util/amdfwtool/amdfwread: Fix AMDFW_OPT* bit mask
......................................................................

Patch Set 6:

(2 comments)

File util/amdfwtool/amdfwread.c:

https://review.coreboot.org/c/coreboot/+/66573/comment/52dcd472_60628972 
PS5, Line 191: }
> We should probably have a `default` case. […]

Done. Made some updates to option handling to include default case as well.

https://review.coreboot.org/c/coreboot/+/66573/comment/b381065d_60c9d1fc 
PS5, Line 198: selected_functions |= opt;
> I would be inclined to just remove the BIT(31) in this CL since it causes problems in https://review […]

Done