[coreboot-gerrit] Change in coreboot[master]: sc7180: Add display dsi interface programming

[Vinod Polimera (Code Review)]

Attention is currently required from: Ravi kumar.
Vinod Polimera has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39614 )

Change subject: sc7180: Add display dsi interface programming
......................................................................

Patch Set 40:

(2 comments)

File src/soc/qualcomm/sc7180/display/dsi.c:

https://review.coreboot.org/c/coreboot/+/39614/comment/24abb351_2b488626 
PS40, Line 105: 	hbp = edid->mode.hbl - edid->mode.hso;
> Hi Vinod, […]

video mode active h part is active + backporch+ pusle width. 
Variable name has to be renamed.

https://review.coreboot.org/c/coreboot/+/39614/comment/2a419550_26e706c0 
PS40, Line 122: 		 hbp - 1));
> This too is a bit confusing because you'd really just need to write […]

VIDEO_V_TOTAL-->Number of lines in one frame minus one. This value is equivalent to the period of the vertical sync.
similarly for video_h_total.
we can rewrite it to (edid->mode.ha + edid->mode.hbl -1)