[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/nissa/var/craask: remove SAR UNUSED fw_config

[Martin L Roth (Code Review)]

Attention is currently required from: Tarun Tuli, Kangheui Won, Eric Lai.

Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71690 )

Change subject: mb/google/nissa/var/craask: remove SAR UNUSED fw_config
......................................................................

Patch Set 2:

(1 comment)

File src/mainboard/google/brya/variants/craask/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/71690/comment/c59a8a94_e25b3b01 
PS2, Line 23: 	field WIFI_SAR_ID 13 15
            : 		option ID_0			0
            : 	end
So get rid of the option and just hardcode it to option ID_0?

The stuff that's going on with these fields makes no sense to me.  Yeah, value 7 wasn't used, because you can't get a 7 with just 2 bits.  Why was that even added?