[coreboot-gerrit] [XS] Change in coreboot[master]: mb/google/dedede/var/pirika: Support for Samsung K4U6E3S4AB-MGCL

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Anson Tseng, Daniel Peng, Derek Huang, Paul Menzel, Shou-Chieh Hsu.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76542?usp=email )

Change subject: mb/google/dedede/var/pirika: Support for Samsung K4U6E3S4AB-MGCL
......................................................................

Patch Set 5: Code-Review+2

(1 comment)

File src/mainboard/google/dedede/variants/pirika/memory/dram_id.generated.txt:

https://review.coreboot.org/c/coreboot/+/76542/comment/9e686a26_c2be25ea :
PS4, Line 4: /tmp/go-build352956073/b001/exe/part_id_gen
> The changed was modified automatically after run command.  […]

It seems this is an artifact of "go run" which compiles the source code, installs the executable in a /tmp path and then executes it from there. There is nothing wrong with that. This line will not change if you run "go build" followed by executing the binary explicitly.