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]

 



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



[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