Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754776AbcC1QH5 (ORCPT ); Mon, 28 Mar 2016 12:07:57 -0400 Received: from smtprelay4.synopsys.com ([198.182.47.9]:51955 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752838AbcC1QHz (ORCPT ); Mon, 28 Mar 2016 12:07:55 -0400 Subject: Re: [PATCH 2/3 v2] ASoC: dwc: Add I2S HDMI audio support To: Alexey Brodkin References: <538be366488bf0d7633d702f2d0bab16707b7a47.1459174494.git.joabreu@synopsys.com> <1459179345.4785.127.camel@synopsys.com> CC: "lars@metafoo.de" , "laurent.pinchart+renesas@ideasonboard.com" , "robh+dt@kernel.org" , "pawel.moll@arm.com" , "Carlos Palminha" , "nariman@opensource.wolfsonmicro.com" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "ville.syrjala@linux.intel.com" , "alexander.deucher@amd.com" , "tixy@linaro.org" , "yitian.bu@tangramtek.com" , "linux-snps-arc@lists.infradead.org" , "broonie@kernel.org" , "mark.rutland@arm.com" , "galak@codeaurora.org" , "ijc+devicetree@hellion.org.uk" , "tiwai@suse.com" , "buyitian@gmail.com" , "lgirdwood@gmail.com" , Vineet Gupta , "wsa+renesas@sang-engineering.com" , "airlied@linux.ie" , "Maruthi.Bayyavarapu@amd.com" , "perex@perex.cz" , "dri-devel@lists.freedesktop.org" , "alsa-devel@alsa-project.org" From: Jose Abreu Message-ID: <56F956C0.80705@synopsys.com> Date: Mon, 28 Mar 2016 17:07:28 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <1459179345.4785.127.camel@synopsys.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.107.19.57] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4413 Lines: 127 Hi Alexey, On 28-03-2016 16:35, Alexey Brodkin wrote: > Hi Jose, > > On Mon, 2016-03-28 at 15:36 +0100, Jose Abreu wrote: >> HDMI audio support was added to the AXS board using an >> I2S cpu driver and a custom platform driver. >> >> The platform driver supports two channels @ 16 bits with >> rates 32k, 44.1k and 48k. ALSA Simple audio card is used to >> glue the cpu, platform and codec driver (adv7511). >> >> Signed-off-by: Jose Abreu >> --- >> >> No changes v1 -> v2. >> >> sound/soc/dwc/Kconfig | 1 + >> sound/soc/dwc/designware_i2s.c | 385 +++++++++++++++++++++++++++++++++++++++-- >> 2 files changed, 373 insertions(+), 13 deletions(-) >> >> diff --git a/sound/soc/dwc/Kconfig b/sound/soc/dwc/Kconfig >> index d50e085..bc3fae7 100644 >> --- a/sound/soc/dwc/Kconfig >> +++ b/sound/soc/dwc/Kconfig >> @@ -2,6 +2,7 @@ config SND_DESIGNWARE_I2S >> tristate "Synopsys I2S Device Driver" >> depends on CLKDEV_LOOKUP >> select SND_SOC_GENERIC_DMAENGINE_PCM >> + select SND_SIMPLE_CARD >> help >> Say Y or M if you want to add support for I2S driver for >> Synopsys desigwnware I2S device. The device supports upto >> diff --git a/sound/soc/dwc/designware_i2s.c b/sound/soc/dwc/designware_i2s.c >> index bff258d..0f2f588 100644 >> --- a/sound/soc/dwc/designware_i2s.c >> +++ b/sound/soc/dwc/designware_i2s.c >> @@ -84,11 +84,37 @@ >> #define MAX_CHANNEL_NUM 8 >> #define MIN_CHANNEL_NUM 2 >> >> +/* FPGA Version Info */ >> +#define FPGA_VER_INFO 0xE0011230 >> +#define FPGA_VER_27M 0x000FBED9 >> + >> +/* PLL registers addresses */ >> +#define PLL_IDIV_ADDR 0xE00100A0 >> +#define PLL_FBDIV_ADDR 0xE00100A4 >> +#define PLL_ODIV0_ADDR 0xE00100A8 >> +#define PLL_ODIV1_ADDR 0xE00100AC > Well I think all is not acceptable. > See all these FPGA_VER_xxx as well as PLL_xxx > are strictly ARC SDP specific things and have nothing to do with generic driver. > > That's so pity we don't have a driver for all clocks/PLLs on ARC SDP yet. > So as of now I may only propose to use hard-coded fixed clocks as I did with > ARC PGU, see "pguclk" here: > http://lists.infradead.org/pipermail/linux-snps-arc/2016-March/000790.html > > Again I'll try to implement missing clock driver sometime soon because > more and more stuff requires it but for now let's use a work-around. Yes, this is a workaround that we are using so that the driver works in ARC SDP platforms. The driver still has the functionality to operate using a clock driver (it must be declared in device tree) but if the clock handle is not declared the driver will assume that must use the internal PLL config options. This is currently the only option to make it work in ARC SDP. I will send a v3 soon without this workaround and when the missing clock drivers are implemented I will re-test this. >> +struct dw_i2s_pll { >> + unsigned int rate; >> + unsigned int data_width; >> + unsigned int idiv; >> + unsigned int fbdiv; >> + unsigned int odiv0; >> + unsigned int odiv1; >> +}; >> + >> +static const struct dw_i2s_pll dw_i2s_pll_cfg_27m[] = { >> + /* 27Mhz */ >> + { 32000, 16, 0x104, 0x451, 0x10E38, 0x2000 }, >> + { 44100, 16, 0x104, 0x596, 0x10D35, 0x2000 }, >> + { 48000, 16, 0x208, 0xA28, 0x10B2C, 0x2000 }, >> + { 0, 0, 0, 0, 0, 0 }, >> }; >> >> +static const struct dw_i2s_pll dw_i2s_pll_cfg_28m[] = { >> + /* 28.224Mhz */ >> + { 32000, 16, 0x82, 0x105, 0x107DF, 0x2000 }, >> + { 44100, 16, 0x28A, 0x1, 0x10001, 0x2000 }, >> + { 48000, 16, 0xA28, 0x187, 0x10042, 0x2000 }, >> + { 0, 0, 0, 0, 0, 0 }, >> +}; > These 2 hunks as well should go in ARC SDP clocks. > >> +static int i2s_pll_cfg(struct i2s_clk_config_data *config) >> +{ >> + const struct dw_i2s_pll *pll_cfg; >> + u32 rate = config->sample_rate; >> + u32 data_width = config->data_width; >> + int i; >> + >> + if (readl((void *)FPGA_VER_INFO) <= FPGA_VER_27M) >> + pll_cfg = dw_i2s_pll_cfg_27m; >> + else >> + pll_cfg = dw_i2s_pll_cfg_28m; >> + >> + for (i = 0; pll_cfg[i].rate != 0; i++) { >> + if ((pll_cfg[i].rate == rate) && >> + (pll_cfg[i].data_width == data_width)) { >> + writel(pll_cfg[i].idiv, (void *)PLL_IDIV_ADDR); >> + writel(pll_cfg[i].fbdiv, (void *)PLL_FBDIV_ADDR); >> + writel(pll_cfg[i].odiv0, (void *)PLL_ODIV0_ADDR); >> + writel(pll_cfg[i].odiv1, (void *)PLL_ODIV1_ADDR); >> + return 0; >> + } >> + } >> + >> + return -EINVAL; >> +} > Ditto. > > -Alexey Best regards, Jose Miguel Abreu