On 2023/4/4 13:07, Laurent Pinchart wrote: > Hi Jack, > > Thank you for the patch. > > On Fri, Mar 31, 2023 at 08:18:25PM +0800, Jack Zhu wrote: >> Add support for Starfive JH7110 SoC which >> has the cadence csi2 receiver. > > This fits on one line: > > Add support for Starfive JH7110 SoC which has the cadence csi2 receiver. > OK, I will modify it in my next commit msg. >> Signed-off-by: Jack Zhu <jack.zhu@xxxxxxxxxxxxxxxx> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > >> --- >> drivers/media/platform/cadence/cdns-csi2rx.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/media/platform/cadence/cdns-csi2rx.c b/drivers/media/platform/cadence/cdns-csi2rx.c >> index add982edce64..5e36b994b8d5 100644 >> --- a/drivers/media/platform/cadence/cdns-csi2rx.c >> +++ b/drivers/media/platform/cadence/cdns-csi2rx.c >> @@ -560,6 +560,7 @@ static int csi2rx_remove(struct platform_device *pdev) >> } >> >> static const struct of_device_id csi2rx_of_table[] = { >> + { .compatible = "starfive,jh7110-csi2rx" }, >> { .compatible = "cdns,csi2rx" }, >> { }, >> }; >