Attention is currently required from: Vinod Polimera, Shelley Chen, Hung-Te Lin. Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56966 )
Change subject: qualcomm/sc7180: Switch to common MIPI panel library ......................................................................
Patch Set 1:
(1 comment)
File src/soc/qualcomm/sc7180/display/dsi.c:
https://review.coreboot.org/c/coreboot/+/56966/comment/ac4d9e1d_1e9f0762 PS1, Line 226: size = ALIGN_UP(cm->size, DSI_PAYLOAD_SIZE_ALIGN); This looks like it has always been wrong to me. If the payload size is not a multiple of 4 (and at least for some of the new Wormdingler panels it won't be) the memcpy() below would read over the end of the array. I changed it to just zero-fill to the next alignment boundary instead, I hope that will work. Let me know if something else needs to be done here, Vinod.