Hi Rob Thank you for your review > > @@ -922,7 +923,7 @@ static int dss_init_ports(struct platform_device *pdev) > > if (parent == NULL) > > return 0; > > of_graph_get_next_port() handles parent==NULL, so you can drop this > check. And the other cases. Yes, indeed. will update in v5 Thank you for your help !! Best regards --- Kuninori Morimoto