Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX media driver
From: Rob Herring
Date: Mon Apr 03 2017 - 10:07:55 EST
On Tue, Mar 28, 2017 at 05:35:52PM -0700, Steve Longerbeam wrote:
>
>
> On 03/28/2017 05:21 PM, Rob Herring wrote:
> > On Mon, Mar 27, 2017 at 7:40 PM, Steve Longerbeam <slongerbeam@xxxxxxxxx> wrote:
> > > Add bindings documentation for the i.MX media driver.
> > >
> > > <snip>
> > > +
> > > +mipi_csi2 node
> > > +--------------
> > > +
> > > +This is the device node for the MIPI CSI-2 Receiver, required for MIPI
> > > +CSI-2 sensors.
> > > +
> > > +Required properties:
> > > +- compatible : "fsl,imx6-mipi-csi2", "snps,dw-mipi-csi2";
> >
> > As I mentioned in v5, there's a DW CSI2 binding in progress. This
> > needs to be based on that.
>
> Hi Rob, I'm not sure what you are asking me to do.
>
> I assume if there's another binding doc in progress, it means
> someone is working on another Synopsys DW CSI-2 subdevice driver.
Yes. see http://patchwork.ozlabs.org/patch/736177/
> Unfortunately I don't have the time to contribute and switch to
> this other subdevice, and do test/debug.
>From a DT perspective, I'm not asking that you share the subdevice
driver, only the binding. Simply put, there's 1 h/w block here, so there
should only be 1 binding. The binding is an ABI, so you can't just merge
it and change it later.
The driver side is a decision for the V4L2 maintainers.
> For now I would prefer if this patchset is merged as is, and
> then contribute/switch to another CSI-2 subdev later. It is
> also getting very difficult managing all these patches (39 as
> of this version), and I'd prefer not to spam the lists with
> such large patchsets for too much longer.
Then maybe you should figure out how to split up the series. I've not
looked at it to provide suggestions.
Rob