[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Add P-sensor for Boten

[Alec Wang (Code Review)]

Alec Wang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43478 )

Change subject: mb/google/dedede: Add P-sensor for Boten
......................................................................

Patch Set 9:

(2 comments)

https://review.coreboot.org/c/coreboot/+/43478/8/src/mainboard/google/dedede... 
File src/mainboard/google/dedede/variants/boten/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/43478/8/src/mainboard/google/dedede... 
PS8, Line 57: ""SAR Proximity Sensor""
> Just to clarify, is this sensor added in Sub-board? […]

Yes, it is added in Sub-board.
I checked I2C5 have already enabled:
PAD_CFG_NF(GPP_D22, NONE, DEEP, NF1),
PAD_CFG_NF(GPP_D23, NONE, DEEP, NF1),

https://review.coreboot.org/c/coreboot/+/43478/8/src/mainboard/google/dedede... 
PS8, Line 63: 	end
> While adding P-Sensor, you removed the Audio amplifier codec configuration. Audio will be broken. […]

Done. I have recovered it.