build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42502 )
Change subject: google/kukui: Add Hynix 4GB discrete LPDDR4X DDR support ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/42502/1/src/mainboard/google/kukui/... File src/mainboard/google/kukui/sdram_configs.c:
https://review.coreboot.org/c/coreboot/+/42502/1/src/mainboard/google/kukui/... PS1, Line 18: [10] = "sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB", code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/42502/1/src/mainboard/google/kukui/... PS1, Line 18: [10] = "sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB", please, no spaces at the start of a line