[coreboot-gerrit] Change in coreboot[master]: src/soc/tigerlake: Add memory configuration support for Jasper Lake

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39135 )

Change subject: src/soc/tigerlake: Add memory configuration support for Jasper Lake
......................................................................

Patch Set 25:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39135/23/src/soc/intel/tigerlake/me... 
File src/soc/intel/tigerlake/meminit_jsl.c:

https://review.coreboot.org/c/coreboot/+/39135/23/src/soc/intel/tigerlake/me... 
PS23, Line 25: 		memcpy(&_mem_cfg->DqByteMapCh ## _ch,		\
             : 			&_b_cfg->dq_map[_ch],			\
             : 			sizeof(_b_cfg->dq_map[_ch]));		\
             : 		memcpy(&_mem_cfg->DqsMapCpu2DramCh ## _ch,	\
             : 			&_b_cfg->dqs_map[_ch],			\
             : 			sizeof(_b_cfg->dqs_map[_ch]));		\
> I really think this looks a lot better as just a little static function. […]

tried creating helper function bt looks like macro append won't work with static function. restoring the old method