Re: [PATCH v7 0/4] drm: add support for Cadence MHDP DPI/DP bridge.

On 27/03/2019 16:58, Damian Kos wrote:

We would be very happy to have a separate driver for mhdp8546 instead of
mixing it with Rockchip's driver. Unfortunately we need to have one driver
for both IP's, so I'll just change mhdp8546.ko to cdns-mhdp.ko. (Unless,
maintainers give us a green light for the dedicated driver?)

Ok. I don't know about the differences, so I can't say much here. But I
do think that you shouldn't combine drivers for two different IPs unless
they are very similar. And if two different IPs have a block that's the
same/similar, the code for that block can be extracted to a "library"
used by both drivers.

quoted

quoted

I see some audio functions in the code, but it's not mentioned in the DT
bindings. I'm not an audio guy, but the display bridges with audio
support I have seen have had DT bindings for the audio source too. Is
audio supported in the current driver?

As far as I know, audio is working, but I can't double check it right now
due to heavy load on my back right now. I'll try to find time for that in the
next week. But, like I've mentioned earlier I don't see a reason why audio
code (that exists in the mainline) wouldn't work.
I'm not an audio guy myself. When updating dt bindings I took the dt bindings
for other bridges as an example, but I did not see any bindings for audio in
them. I didn't check all of them. Can you please let me know which bridges
are you were referring to? In our test environment we didn't use any audio
driver as audio generator is so simple that it is part of the register space
of IP and we didn't even need to include this device in .dts file. Audio (like
the video) is configured and enabled by code that is not included in the patch
(for obvious reasons).
So, in short, yes, driver supports audio and we will include description for
audio input port in the dt bindings.

Ok. Jyri Sarha [off-list ref] recently sent "drm/bridge: sii902x:
HDMI-audio support and some fixes" series, and I think tda998x_drv.c
also has some audio support. But I'm no audio guy either =). I was just
curious about the status of the audio.

 Tomi

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel