[coreboot-gerrit] Change in coreboot[master]: mb/google/dedede/var/drawcia: Add MIPI camera support

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44939 )

Change subject: mb/google/dedede/var/drawcia: Add MIPI camera support
......................................................................

Patch Set 8:

(3 comments)

https://review.coreboot.org/c/coreboot/+/44939/8//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/44939/8//COMMIT_MSG@10 
PS8, Line 10: 2. add IPU/VCM/NVM/CAM1 in devicetree
Can you please mention it is WFC. I interpret it so, but is not clear.

https://review.coreboot.org/c/coreboot/+/44939/8//COMMIT_MSG@13 
PS8, Line 13: grap
Nit: grap -> grab

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

https://review.coreboot.org/c/coreboot/+/44939/8/src/mainboard/google/dedede... 
PS8, Line 233: 				register "on_seq.ops[1]" = "SEQ_OPS_GPIO_ENABLE(0, 5)"
             : 				register "on_seq.ops[2]" = "SEQ_OPS_GPIO_ENABLE(1, 5)
In the initial static ACPI ASL file, there is no delay between turning on Power Enable GPIOs. Do we still need the delay?