Hi Jacopo, On Thu, Sep 13, 2018 at 11:46:14AM +0200, jacopo mondi wrote: > Hi Sakari, > > On Thu, Sep 13, 2018 at 12:29:35AM +0300, Sakari Ailus wrote: > > Initialise the V4L2 fwnode endpoints to zero in all drivers using > > v4l2_fwnode_endpoint_parse(). This prepares for setting default endpoint > > flags as well as the bus type. Setting bus type to zero will continue to > > guess the bus among the guessable set (parallel, Bt.656 and CSI-2 D-PHY). > > > > I've played around with this patch, trying to use defaults in the > renesas-ceu driver. > > This is the resulting patch, if you want I can send it as follow-up or > send it so that you can include it in your series if it's correct): > https://paste.debian.net/hidden/a7795d3e/ Looks nice; could you send it out to the list for review? The bus width default isn't specified in DT bindings; could you write a patch that defines it? Thanks. -- Sakari Ailus sakari.ailus@xxxxxxxxxxxxxxx
