[coreboot-gerrit] [M] Change in coreboot[master]: mb/google/nissa/var/pujjo: Tune SX9324 register for pujjoteen5

[Leo Chou (Code Review)]

Attention is currently required from: Derek Huang, Paul Menzel, Shou-Chieh Hsu, Tarun Tuli.

Leo Chou has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75900?usp=email )

Change subject: mb/google/nissa/var/pujjo: Tune SX9324 register for pujjoteen5
......................................................................

Patch Set 7:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/75900/comment/35bc5227_60f8c5ad :
PS4, Line 7: Tuning
> Imperative mood: Tune

Done

https://review.coreboot.org/c/coreboot/+/75900/comment/274eb91a_cb8b1487 :
PS4, Line 10: P-sensor vendor
> Who is that?

The vendor is SEMTECH

https://review.coreboot.org/c/coreboot/+/75900/comment/74f94028_de0b5552 :
PS4, Line 13: TEST=i2cdump -y -f 13 0x28 on Pujjoteen5
> What is the output?

Check i2c dump register settings on Pujjoteen5

https://review.coreboot.org/c/coreboot/+/75900/comment/861353ac_2c8c8c54 :
PS4, Line 14:
> What user visible change is visible?

The P sensor function can work