Re: [PATCH] [media] tvp5150: remove signal generator as input from the DT binding

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

 



Hello Laurent,

On 02/23/2016 01:16 PM, Laurent Pinchart wrote:
Hi Javier,

On Tuesday 23 February 2016 13:09:58 Javier Martinez Canillas wrote:
The chip internal signal generator was modelled as an input connector
and represented as a media entity but isn't really a connector so the
driver was changed to use the V4L2_CID_TEST_PATTERN control instead.

Remove the signal generator input from the list of connectors in the
tvp5150 DT binding document as well since isn't a connector anymore.

Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>

---
Hello,

I think is OK to change this DT binding because is only in the media tree
for now and not in mainline yet and also is expected to change more since
there are still discussions about how input connectors will be supported
by the Media Controller framework in the media subsystem.

I think that's fine, yes

Acked-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>


Thanks.
I haven't noticed the patch that introduced this early enough I'm afraid, and
I think we still have issues with those bindings.


Yes, I posted those patches and got merged before we had the discussion about
input connectors over IRC so I didn't know what was the correct way to do it.

The tvp5150 node should *not* contain connector subnodes, the connectors nodes
should use the bindings defined in
Documentation/devicetree/bindings/display/connector/ and be linked to the
tvp5150 node using the OF graph bindings (ports and endpoints).


Agreed.

Do you think you could fix that ?


Yes I will, I'm waiting for the input connectors discussions to settle so I
can post a final version of the DT bindings following what is agreed by all.

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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