[coreboot-gerrit] Change in coreboot[master]: mb/google/volteer: Update I2C5 bus freq and devicetree.

[Stanley Wu (Code Review)]

Stanley Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48150 )

Change subject: mb/google/volteer: Update I2C5 bus freq and devicetree.
......................................................................

Patch Set 5:

(2 comments)

https://review.coreboot.org/c/coreboot/+/48150/4/src/mainboard/google/voltee... 
File src/mainboard/google/volteer/variants/lindar/gpio.c:

https://review.coreboot.org/c/coreboot/+/48150/4/src/mainboard/google/voltee... 
PS4, Line 137: 	/* E15 : ISH_GP6 ==> NC */
             : 	PAD_NC(GPP_E15, NONE),
             :
> May i know the reason behind of setting this?

Because TRACKPAD_INT_ODL pin no response on Synaptics trackpad device after power on. And it can be fixed after config GPP_E15 to early GPIO table.

https://review.coreboot.org/c/coreboot/+/48150/4/src/mainboard/google/voltee... 
File src/mainboard/google/volteer/variants/lindar/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/48150/4/src/mainboard/google/voltee... 
PS4, Line 39: 			.rise_time_ns = 0,
            : 			.fall_time_ns = 0,
            : 			.data_hold_time_ns = 0,
> Is this needed? I think the default value should be zero, isn't it?

You`re right, i remove useless code change.