[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede: Add ACPI support for camera in waddledee

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42205 )

Change subject: mb/google/dedede: Add ACPI support for camera in waddledee
......................................................................

Patch Set 9:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42205/9/src/mainboard/google/dedede... 
File src/mainboard/google/dedede/variants/waddledee/include/variant/acpi/cam0.asl:

https://review.coreboot.org/c/coreboot/+/42205/9/src/mainboard/google/dedede... 
PS9, Line 36: 			Sleep(5)	/* 5 us */
So all of the comments in here for the Sleeps say `us` implying microseconds, but the ASL Sleep op sleeps for *milliseconds*. So are the comments wrong, or do you actually only need to sleep for 5 us? Because the finest granularity is the 1ms of Sleep, so a Sleep(1) would be the best you could do.