On 02/03/2023 10:19, jack.zhu wrote: > Add DT binding document for Starfive MIPI CSI2 receiver Ehh... you have entire commit msg to explain what you do here. Yet there is nothing mentioning that you actually have Cadence MIPI CSI here. Since you decided to add new bindings, you receive review matching new bindings. I don't think this is correct approach (duplicated bindings), but could work for me. However how are you going to solve all the points of my review? Best regards, Krzysztof