[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede/var/beadrix: Add a Proximity Sensor SX9324 for SAR

[Teddy Shih (Code Review)]

Attention is currently required from: Henry Sun, Super Ni, Teddy Shih, Paul Menzel, Simon Yang, Aseda Aboagye, Ivan Chen, Karthik Ramasubramanian, Felix Held.
Teddy Shih has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63652 )

Change subject: mb/google/dedede/var/beadrix: Add a Proximity Sensor SX9324 for SAR
......................................................................

Patch Set 8: Code-Review+1

(2 comments)

Patchset:

PS4:
> Hi Teddy, […]

Hi Simon,

For I2C clock frequency, I will remove the value of parameter speed_config until ECS HW's comment to ensure it.

I have checked the JSL I2C bus 0-5 according mainboard schematic [1]. As I2C bus 1 and 3 are unused, therefore I will set them to PchSerialIoDisabled.

Reference:
[1] https://drive.google.com/file/d/1ZO_Ba0UOBihc2teO0XPZnALM-xkhy9t2/view?usp=s...
[2] EVT schematic (V B 220207)
https://drive.google.com/file/d/1Tb6sPUu9BQmiu77HEhp_q43iWLMpstEf/view?usp=s...

Patchset:

PS8: 
Dear All,

Thank you for all reviewers' comments. I have replied and modified CL regarding reviewers' comments. Please kindly review my CL.

Best Regards,

Teddy