Hi Marek,
On Friday 11 Aug 2017 08:39:31 Marek Szyprowski wrote:
> On 2017-08-10 15:39, Laurent Pinchart wrote:
> > On Thursday 10 Aug 2017 15:25:29 Maciej Purski wrote:
> >> The driver should be switched on if an external connector is plugged and
> >> switched off if it is unplugged. Extcon is optional. If it is not found,
> >> the driver stays in "always-on" mode.
> >>
> >> Signed-off-by: Maciej Purski <m.purski@xxxxxxxxxxx>
> >> ---
> >>
> >> .../bindings/display/bridge/sil-sii8620.txt | 4 ++
> >> drivers/gpu/drm/bridge/sil-sii8620.c | 83 ++++++++++++++-
> >> 2 files changed, 85 insertions(+), 2 deletions(-)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/display/bridge/sil-sii8620.txt
> >> b/Documentation/devicetree/bindings/display/bridge/sil-sii8620.txt index
> >> 9409d9c..1f230bf 100644
> >> --- a/Documentation/devicetree/bindings/display/bridge/sil-sii8620.txt
> >> +++ b/Documentation/devicetree/bindings/display/bridge/sil-sii8620.txt
> >> @@ -10,6 +10,9 @@ Required properties:
> >> - clocks, clock-names: specification and name of "xtal" clock
> >> - video interfaces: Device node can contain video interface port
> >> node for HDMI encoder according to [1].
> >>
> >> +Optional properties:
> >> + - extcon: phandle to external connector for MHL cable changes
> >> + detection
> >
> > The sii8620 DT node should model its connection to the MHL connector using
> > OF graph, connecting a port to the MHL connector DT node through
> > endpoints. I believe the extcon property should then be added to the MHL
> > connector DT node, not the bridge.
>
> I don't get this. Generic extcon bindings doesn't define connector DT
> node, so I don't see how could we use endpoint DT approach here. Could you
> give the example?
Sure, here you are.
&hsi2c_7 {
...
sii8620@39 {
reg = <0x39>;
compatible = "sil,sii8620";
cvcc10-supply = <&ldo36_reg>;
iovcc18-supply = <&ldo34_reg>;
interrupt-parent = <&gpf0>;
interrupts = <2 IRQ_TYPE_LEVEL_HIGH>;
reset-gpios = <&gpv7 0 GPIO_ACTIVE_LOW>;
clocks = <&pmu_system_controller 0>;
clock-names = "xtal";
ports {
#address-cells = <1>;
#size-cells = <0>;
port@0 {
reg = <0>;
mhl_to_hdmi: endpoint {
remote-endpoint = <&hdmi_to_mhl>;
};
};
port@1 {
reg = <1>;
mhl_bridge_out: endpoint {
remote-endpoint = <&mhl_connector_in>;
};
};
};
};
};
/ {
mhl-connector {
compatible = "mhl-connector";
extcon = <&muic>;
port {
mhl_connector_in: endpoint {
remote-endpoint = <&mhl_bridge_out>;
};
};
};
};
Note that there's currently no DT binding for MHL connectors. One needs to be
developed, the HDMI connector DT binding can be used as a source of
inspiration.
> >> [1]: Documentation/devicetree/bindings/media/video-interfaces.txt
--
Regards,
Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
