Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1615225

Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX media driver

Show all headers | View raw

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@gmail.com> 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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX media driver Steve Longerbeam <slongerbeam@gmail.com> - 2017-03-28 02:50 +0200
  Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX media driver Rob Herring <robh+dt@kernel.org> - 2017-03-29 02:30 +0200
    Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX media  driver Steve Longerbeam <steve_longerbeam@mentor.com> - 2017-03-29 02:40 +0200
      Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX  media driver Rob Herring <robh@kernel.org> - 2017-04-03 16:10 +0200
        Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX  media driver Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-04-03 17:20 +0200
    Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX  media driver Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-03-29 10:50 +0200
      Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX  media driver Rob Herring <robh@kernel.org> - 2017-04-03 16:20 +0200
        Re: [PATCH v6 02/39] [media] dt-bindings: Add bindings for i.MX  media driver Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-04-03 17:10 +0200

csiph-web