On Fri, Feb 26, 2016 at 12:14:44PM +0200, Jyri Sarha wrote:

On 02/26/16 02:43, Russell King - ARM Linux wrote:

quoted

On Thu, Feb 25, 2016 at 03:42:50PM +0200, Jyri Sarha wrote:

quoted

On 02/18/16 16:35, Rob Herring wrote:

quoted

This should be implied from the port unit address. In other words,
port@0 is defined to be the rgb port. Now, if this is one of several
modes for the video port, then that is a different story.

Do you suggest that also the audio i2s and s/p-dif port-types should be
coded in the port unit addresses? Something like: port@0 is always rgb,
port@1 is i2s, and port@2 is spdif?

For the audio inputs, the port address corresponds to the input pin.
TDA998x devices can have multiple streams routed to the pins, and can
select between them.

For example, there may be four I2S data pins and one I2S clock pin.
When using stereo, you can select which of the four I2S data pins
carries the audio data.

Sure, but I do not think that would be the usual setup. The only "normal"
situation I can think for having a need to have two alternative audio setups
would one for i2s and another for s/p-dif. But then again it is possible to
come up with a design with multiple alternative audio wirings and it
relatively simple handle that in DT binding, so why not.

There's another reason: if you want to support 8 channel audio using I2S
rather than SPDIF, then you need to use four I2S data inputs.  Each I2S
data input can support only two channels.

quoted

When using SPDIF, there may be two SPDIF inputs, and you can select
which SPDIF input is used.

So, "reg" may not be an address in terms of a CPU visible address, but
it's an address as far as selecting the appropriate input - and it
fits in with the requirements of ePAPR, which are that if you have
a unit-address (which is required to distinguish different port nodes)
then you must have a matching "reg" property.

Still I do not see why it is desirable to reuse reg property, when we can
introduce new property for describing the audio wiring.

Different people have different opinions.  Your opinion is just another
example of someone holding a different view.

You _have_ to have a unit address, and therefore you _have_ to have a
reg property.  If you want to use some other property to describe the
audio input pin, then you will need to make up a totally ficticious
unit-address and reg property for each audio input pin.

That's adding complexity, arguably unnecessary complexity, and making
the binding unnecessarily more complex for no good reason.

quoted

I don't particularly like the video node using the RGB routing register
value either for the reg property, but I've kept quiet because I have
nothing to offer there: again, this comes down to ePAPR requirements
and the need to specify multiple "port { }" nodes.  You can't have two
"port { }" nodes without using a unit-address, and we'd need to chose
a unit-address for it which doesn't conflict with the audio ports...
so there's a kind of logic to using the RGB routing value, which will
never conflict.

If we after all decide to go with using reg property for audio wiring (and
essentially writing the value directly to AP_ENA register), then we could
also agree that video port's unit address is always 0 as it corresponds to
audio disabled in AP_ENA register and would not collide with any audio
"address". Then we could keep the old video-ports property to configure the
video wiring. How does this sound?

Sub-standard :)

This has actually been discussed before.  See the thread:

"[PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio"

from January 2015.

-- 
RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel