[coreboot-gerrit] [L] Change in coreboot[master]: mb/apple: Add MacBook Air 5,2 (A1466) support

[Nicholas Chin (Code Review)]

Attention is currently required from: Evgeny Zinoviev, Felix Held.

Nicholas Chin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32604 )

Change subject: mb/apple: Add MacBook Air 5,2 (A1466) support
......................................................................

Patch Set 41:

(1 comment)

File src/mainboard/apple/macbookair5_2/early_init.c:

https://review.coreboot.org/c/coreboot/+/32604/comment/c6cf632a_059f8f71 
PS41, Line 62: 	case 2:
I was helping a user on the libreboot IRC get their 8 GiB Hynix working, and they reported that the generated SPD from inteltool for it is exactly the same as the 4 GiB Samsung one. Adding `case 4:` as a fallthrough here did work for them. It's probably just a matter of the chips having similar timings and the 8 GiB config using dual vs single channel for the 4 GiB config.