[coreboot-gerrit] Change in coreboot[master]: sc7180: Add support for sn65dsi86 bridge.

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42899 )

Change subject: sc7180: Add support for sn65dsi86 bridge.
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42899/3/src/drivers/ti/sn65dsi86bri... 
File src/drivers/ti/sn65dsi86bridge/sn65dsi86bridge.c:

https://review.coreboot.org/c/coreboot/+/42899/3/src/drivers/ti/sn65dsi86bri... 
PS3, Line 337: 360
> Datasheet says to wait for max 400 ms

Then let's wait 400ms instead of 360ms?

In general I think this is fine since this is only a timeout. I expect that in most "normal" cases it will be a lot faster. I'm more worried about those plain mdelay() that we have in some places.