build bot (Jenkins) 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 23:
(4 comments)
https://review.coreboot.org/c/coreboot/+/32604/23/src/mainboard/apple/macboo... File src/mainboard/apple/macbookair5_2/hda_verb.c:
https://review.coreboot.org/c/coreboot/+/32604/23/src/mainboard/apple/macboo... PS23, Line 75: 0x00270500, /* Set PS to D0 on DAC1 */ please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/32604/23/src/mainboard/apple/macboo... PS23, Line 76: 0x00470500, /* Set PS to D0 on DAC3 */ please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/32604/23/src/mainboard/apple/macboo... PS23, Line 77: 0x00270650, /* Set converter stream on DAC1 */ please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/32604/23/src/mainboard/apple/macboo... PS23, Line 78: 0x00470650, /* Set converted stream on DAC3 */ please, no spaces at the start of a line