[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede Add Audio support for waddledoo

[Aamir Bohra (Code Review)]

Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39285 )

Change subject: mb/google/dedede Add Audio support for waddledoo
......................................................................

Patch Set 9:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/39285/8/src/mainboard/google/dedede... 
PS8, Line 41: .scl_lcnt = 176,
            : 				.scl_hcnt = 95,
            : 				.sda_hold = 36,
> Audio is verified working, but these parameters are not electrically verified. […]

from the device spec:

CLKL SCL low time 500 ns
CLKH SCL high time 260 ns

the count high and low count I believe need to be calculated as 133(soc clk)* scl_low(or high)_time/Khz. Correct?