Re: [PATCH v3 16/23] v4l: fwnode: Initialise the V4L2 fwnode endpoints to zero

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 13, 2018 at 12:55:33PM +0300, Sakari Ailus wrote:
> 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?

Same for "pclk-sample". DT bindings do not document that; it should go to
the same patch.

-- 
Sakari Ailus
sakari.ailus@xxxxxxxxxxxxxxx



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux