Re: [PATCH v6 4/4] dt-bindings: media: Document Synopsys Designware HDMI RX

From: Sylwester Nawrocki
Date: Thu Jul 06 2017 - 16:31:15 EST

On 07/06/2017 12:24 PM, Jose Abreu wrote:
>>> +- edid-phandle: phandle to the EDID handler block.
>> Could you make this property optional and when it is missing assume that device
>> corresponding to the parent node of this node handles EDID? This way we could
>> avoid having property pointing to the parent node.
> Hmm, this is for the CEC notifier. Do you mean I should grab the
> parent device for the notifier? This property is already optional
> if cec is not enabled though.

Yes, device associated with the parent node. Something like:

- edid-phandle - phandle to the EDID handler block; if this property is
not specified it is assumed that EDID is handled by device described
by parent node of the HDMI RX node

Not sure if it is any better than always requiring edid-phandle property,
even when it is pointing to the parent node. We would need a DT maintainer's
opinion on that.