[S] Change in coreboot[main]: mb/google/brox: Fix CPU crashlog device MMIO memory access

[Angel Pons (Code Review)]

Attention is currently required from: Benjamin Doron, Dinesh Gehlot, Kane Chen, Kapil Porwal, Karthik Ramasubramanian, Nick Vaccaro, Paul Menzel, Sowmya Aralguppe, Subrata Banik.

Angel Pons has posted comments on this change by Sowmya Aralguppe. ( https://review.coreboot.org/c/coreboot/+/82136?usp=email )

Change subject: mb/google/brox: Fix CPU crashlog device MMIO memory access
......................................................................

Patch Set 10:

(2 comments)

File src/soc/intel/alderlake/crashlog.c:

https://review.coreboot.org/c/coreboot/+/82136/comment/f23e4f0f_12e26589?usp... :
PS10, Line 205: &
> ACK , using logical operator is the correct thing  […]

What if the address is below 4G (the high dword is zero, but the low dword is non-zero)? Is this a valid value?

https://review.coreboot.org/c/coreboot/+/82136/comment/ce0c7050_315df61f?usp... :
PS10, Line 210: 	cpu_cl_disc_tab.header.data = (((u64)dw0) + ((u64)dw1 << 32));
Since this combines two 32-bit values into a 64-bit value, I would recommend using bitwise-OR `|` instead:

```suggestion
    cpu_cl_disc_tab.header.data = (((u64)dw0) | ((u64)dw1 << 32));
```