On 01/31/2015 07:00 AM, Russell King - ARM Linux wrote:
On Fri, Jan 30, 2015 at 06:23:51AM -0500, Yakir Yang wrote:
We found Designware hdmi driver only support audio clock config, we can not play sound through it.
To add Designware HDMI Audio support, we make those patch set:
1): modify n/cts config order, according to dw_hdmi document.
2): add Audio Sample Channel Status config interfaces to dw_hdmi driver.
3): add audio support for more display resolutions(eg. 800x600).
4): add audio support for No-CEA display resolutions.
5): fixed dw_hdmi irq bug, add irq control to suspend/resume interfaces.
6): add suspend/resume callback for dw_hdmi rockchip driver.
7): filter interlace mode in rockchip vop driver.
8): add hdmi audio config interfaces to dw_hdmi driver.
9): creat "dw_hdmi-audio" platform device in dw_hdmi driver.
10): add codec driver for hdmi audio, callback dw_hdmi audio config functions.
11): add sound driver for hdmi audio, creat hdmi audio sound card.
12): add dt-bings file and add hdmi_audio node to corresponding dt file.
I think the overall issue with this patch is working out how to support
both the iMX6 version of this IP, and the Rockchip version of the IP.
These two hardware IPs seem to be configured at synthesis time with
entirely different audio architectures, which change which registers
are available, and sometimes which bits in the registers are present,
which makes it more difficult to come up with a unified audio driver.
Also, I think that it would be a good idea to start documenting which
registers are available in which versions of the IP in dw_hdmi.h,
otherwise I can see that it's going to be very easy for someone to
assume that some register or bit which is available in one IP is
present on all.
The CONFIGx_ID register values for the iMX6 SoC are:
CONFIG0_ID 0x8f
CONFIG1_ID 0x01
CONFIG2_ID 0xf2
CONFIG3_ID 0x02
CONFIG0_ID appears to contain bits which indicate whether the IP
supports I2S and SPDIF mode. Presumably your IP has bit 4 set for I2S,
and maybe bit 5 for SPDIF?
CONFIG1_ID bit 0 indicates whether the AHB interface is present, which
is presumably zero for your IP?
CONFIG3_ID bit 0 indicates whether "generic parallel audio, GPAUD" is
present.
Could you provide (in addition to the values printed in the message I
requested in another reply) the values of the CONFIGx_ID registers
please, and whether any of the bits in there are documented as being
applicable to audio.
Thanks.
The IP version on rk3288 : dwhdmi-rockchip ff980000.hdmi: Detected HDMI
controller 0x20:0xa:0xa0:0xc1
The CONFIGx_ID register values for the rk3288 soc are:
CONFIG0_ID 0xbf
CONFIG1_ID 0x22
CONFIG2_ID 0xc2
CONFIG3_ID 0x0
After looking at iMX6 DQRM, i found only CONFIG1_ID & CONFIG3_ID are
different.
CONFIG1_ID bit 5 indicates whether the HDMI2.0 is present. Bit 1
indicates whether
configuration interface is APB interface.
CONFIG3_ID bit 3 indicates whether the AHBAUDDMA is present. Bit 1
indicates whether
Generic Parallel Audio is present.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
