From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754878AbcHSBdS (ORCPT ); Thu, 18 Aug 2016 21:33:18 -0400 Received: from mailout3.samsung.com ([203.254.224.33]:55458 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754387AbcHSBdI (ORCPT ); Thu, 18 Aug 2016 21:33:08 -0400 MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 X-AuditID: cbfee68e-f79cb6d000006cfe-5e-57b661af1636 Content-transfer-encoding: 8BIT Message-id: <57B661AF.8010102@samsung.com> Date: Fri, 19 Aug 2016 10:32:31 +0900 From: Chanwoo Choi Organization: Samsung Electronics User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: Sylwester Nawrocki , lee.jones@linaro.org Cc: broonie@kernel.org, alsa-devel@alsa-project.org, robh@kernel.org, devicetree@vger.kernel.org, b.zolnierkie@samsung.com, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Inha Song , Beomho Seo Subject: Re: [PATCH v6 2/2] mfd: Add Samsung Exynos Low Power Audio Subsystem driver References: <1470840500-2428-1-git-send-email-s.nawrocki@samsung.com> <1470840500-2428-2-git-send-email-s.nawrocki@samsung.com> In-reply-to: <1470840500-2428-2-git-send-email-s.nawrocki@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrMIsWRmVeSWpSXmKPExsWyRsSkQHd94rZwg/kv9CyuXDzEZLFxxnpW i9OftrFbTH34hM1i/pFzrBa7/t5ntLj/9SijxeVdc9gsZpzfx2Txf88OdovDb9pZHbg9Nnxu YvPYtKqTzePOtT1sHn1bVjF6fN4kF8AaxWWTkpqTWZZapG+XwJVxeGcDY8Fk34ptt/8xNzDe tOti5OSQEDCRaJt5jBXCFpO4cG89WxcjF4eQwApGiZX3PzDBFK3pmsAEkZjFKLHv6yUWkASv gKDEj8n3gGwODmYBeYkjl7JBwswC6hKT5i1ihqh/wCgx+dEUZoh6LYlvpyeygdgsAqoSc9u2 gsXZgOL7X9wAi/MLKEpc/fGYEWSmqECERPeJSpCwiICrxObunWAzmQW6mCRO3F0F1issECbx 68cyVohlVxklGu/eB0twCrhJvN88GywhIdDIIdF6dx8jxGYBiW+TD4FdLSEgK7HpADPEl5IS B1fcYJnAKD4LyW+zEH6bheS3BYzMqxhFUwuSC4qT0ouM9IoTc4tL89L1kvNzNzEC4/b0v2d9 OxhvHrA+xCjAwajEw2vxfGu4EGtiWXFl7iFGU6AjJjJLiSbnA5NDXkm8obGZkYWpiamxkbml mZI4b4LUz2AhgfTEktTs1NSC1KL4otKc1OJDjEwcnFINjArhgp9CZ3GUMvjkCwp2xEvrVK7K jFedbaRxUb34xtHPVxVr7Bd1Xa9YmPbGUzr4muL6z9XZp6ZIW86Kt455LHGG4chSk/nB9/5E +YX/7//asL+oaPo/p5npxkkuQlOdzMqrkrwCCtc23ft49yT3pMw1r7dqfDy06Zpj15fgRAl7 57tq3sLHlViKMxINtZiLihMBs/LU3NYCAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprAKsWRmVeSWpSXmKPExsVy+t9jQd31idvCDZ72q1tcuXiIyWLjjPWs Fqc/bWO3mPrwCZvF/CPnWC12/b3PaHH/61FGi8u75rBZzDi/j8ni/54d7BaH37SzOnB7bPjc xOaxaVUnm8eda3vYPPq2rGL0+LxJLoA1qoHRJiM1MSW1SCE1Lzk/JTMv3VbJOzjeOd7UzMBQ 19DSwlxJIS8xN9VWycUnQNctMwfoNiWFssScUqBQQGJxsZK+HaYJoSFuuhYwjRG6viFBcD1G BmggYQ1jxuGdDYwFk30rtt3+x9zAeNOui5GTQ0LARGJN1wQmCFtM4sK99WxdjFwcQgKzGCX2 fb3EApLgFRCU+DH5HpDNwcEsIC9x5FI2SJhZQF1i0rxFzBD1DxglJj+awgxRryXx7fRENhCb RUBVYm7bVrA4G1B8/4sbYHF+AUWJqz8eM4LMFBWIkOg+UQkSFhFwldjcvRNsJrNAF5PEibur wHqFBcIkfv1Yxgqx7CqjROPd+2AJTgE3ifebZ7NOYBScheTWWQi3zkJy6wJG5lWMEqkFyQXF Sem5hnmp5XrFibnFpXnpesn5uZsYwanhmdQOxoO73A8xCnAwKvHw7ji4NVyINbGsuDL3EKME B7OSCO/O+G3hQrwpiZVVqUX58UWlOanFhxhNgZ6dyCwlmpwPTFt5JfGGxiZmRpZG5oYWRsbm SuK8j/+vCxMSSE8sSc1OTS1ILYLpY+LglGpgXJpVPYmL59SW9KkVieuvZLSUfat//ik/+/ev 28umGQZMWX/JOLP0PtOsTOP388v12NlK7ETZG8/Nqvn4aeqd3ZyJRU1R1aEM3MoOHutYjI99 KNk+NbLim8TXf9VTfP5tdTO4wfaVMfrVRs5E/2lV4jLB83b+MAo9ov+0985Ja/P5+1liP210 VmIpzkg01GIuKk4EAJerBVcjAwAA DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear all, I tested this patch with TM2 dt patches[1] based on v4.8-rc2. To test the playback, I adjust the dt node according to LPASS documentation. [1] https://lkml.org/lkml/2016/8/16/61 [PATCH 0/7] arm64: dts: Add the dts file for Exynos5433 and TM/TM2E board Tested-by: Chanwoo Choi Regards, Chanwoo Choi On 2016년 08월 10일 23:48, Sylwester Nawrocki wrote: > This patch adds common driver for the Top block of the Samsung Exynos > SoC Low Power Audio Subsystem. This is a minimal driver which prepares > resources for IP blocks like I2S, audio DMA and UART and exposes > a regmap for the Top block registers. Also system power ops are added > to ensure the Audio Subsystem is operational after system suspend/resume > cycle. > > Signed-off-by: Inha Song > Signed-off-by: Beomho Seo > Signed-off-by: Sylwester Nawrocki > --- > > Changes since v5: > - BIT() used for the register bit macro definitions, > - removed unneeded dev->of_node test, > - exynos_lpass_{suspend,resume} functions compiled-in conditionally, > - added comments for struct exynos_lpass fields. > > Changes since v4: > - none. > > Changes since v3: > - moved from sound/soc/samsung and rewritten as a MFD driver, > - PMU register definitions moved to include/linux/mfd/syscon/exynos5-pmu.h, > - added regmap for LPASS Top SFR region, > - cleaned up register bit field defintions. > > Changes since v2: > - move misplaced SND_SAMSUNG_AUDSS Kconfig symbol addition > to this patch. > --- > drivers/mfd/Kconfig | 8 ++ > drivers/mfd/Makefile | 1 + > drivers/mfd/exynos-lpass.c | 187 +++++++++++++++++++++++++++++++++ > include/linux/mfd/syscon/exynos5-pmu.h | 4 +- > 4 files changed, 199 insertions(+), 1 deletion(-) > create mode 100644 drivers/mfd/exynos-lpass.c > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 2d1fb64..a7ec890 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -281,6 +281,14 @@ config MFD_DLN2 > etc. must be enabled in order to use the functionality of > the device. > > +config MFD_EXYNOS_LPASS > + tristate "Samsung Exynos SoC Low Power Audio Subsystem" > + select MFD_CORE > + select REGMAP_MMIO > + help > + Select this option to enable support for Samsung Exynos Low Power > + Audio Subsystem. > + > config MFD_MC13XXX > tristate > depends on (SPI_MASTER || I2C) > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 2ba3ba3..41510bb 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -13,6 +13,7 @@ obj-$(CONFIG_MFD_BCM590XX) += bcm590xx.o > obj-$(CONFIG_MFD_CROS_EC) += cros_ec.o > obj-$(CONFIG_MFD_CROS_EC_I2C) += cros_ec_i2c.o > obj-$(CONFIG_MFD_CROS_EC_SPI) += cros_ec_spi.o > +obj-$(CONFIG_MFD_EXYNOS_LPASS) += exynos-lpass.o > > rtsx_pci-objs := rtsx_pcr.o rts5209.o rts5229.o rtl8411.o rts5227.o rts5249.o > obj-$(CONFIG_MFD_RTSX_PCI) += rtsx_pci.o > diff --git a/drivers/mfd/exynos-lpass.c b/drivers/mfd/exynos-lpass.c > new file mode 100644 > index 0000000..578ac7b > --- /dev/null > +++ b/drivers/mfd/exynos-lpass.c > @@ -0,0 +1,187 @@ > +/* > + * Copyright (C) 2015 - 2016 Samsung Electronics Co., Ltd. > + * > + * Authors: Inha Song > + * Sylwester Nawrocki > + * > + * Samsung Exynos SoC series Low Power Audio Subsystem driver. > + * > + * This module provides regmap for the Top SFR region and instantiates > + * devices for IP blocks like DMAC, I2S, UART. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 and > + * only version 2 as published by the Free Software Foundation. > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +/* LPASS Top register definitions */ > +#define SFR_LPASS_CORE_SW_RESET 0x08 > +#define LPASS_SB_SW_RESET BIT(11) > +#define LPASS_UART_SW_RESET BIT(10) > +#define LPASS_PCM_SW_RESET BIT(9) > +#define LPASS_I2S_SW_RESET BIT(8) > +#define LPASS_WDT1_SW_RESET BIT(4) > +#define LPASS_WDT0_SW_RESET BIT(3) > +#define LPASS_TIMER_SW_RESET BIT(2) > +#define LPASS_MEM_SW_RESET BIT(1) > +#define LPASS_DMA_SW_RESET BIT(0) > + > +#define SFR_LPASS_INTR_CA5_MASK 0x48 > +#define SFR_LPASS_INTR_CPU_MASK 0x58 > +#define LPASS_INTR_APM BIT(9) > +#define LPASS_INTR_MIF BIT(8) > +#define LPASS_INTR_TIMER BIT(7) > +#define LPASS_INTR_DMA BIT(6) > +#define LPASS_INTR_GPIO BIT(5) > +#define LPASS_INTR_I2S BIT(4) > +#define LPASS_INTR_PCM BIT(3) > +#define LPASS_INTR_SLIMBUS BIT(2) > +#define LPASS_INTR_UART BIT(1) > +#define LPASS_INTR_SFR BIT(0) > + > +struct exynos_lpass { > + /* pointer to the Power Management Unit regmap */ > + struct regmap *pmu; > + /* pointer to the LPASS TOP regmap */ > + struct regmap *top; > +}; > + > +static void exynos_lpass_core_sw_reset(struct exynos_lpass *lpass, int mask) > +{ > + unsigned int val = 0; > + > + regmap_read(lpass->top, SFR_LPASS_CORE_SW_RESET, &val); > + > + val &= ~mask; > + regmap_write(lpass->top, SFR_LPASS_CORE_SW_RESET, val); > + > + usleep_range(100, 150); > + > + val |= mask; > + regmap_write(lpass->top, SFR_LPASS_CORE_SW_RESET, val); > +} > + > +static void exynos_lpass_enable(struct exynos_lpass *lpass) > +{ > + /* Unmask SFR, DMA and I2S interrupt */ > + regmap_write(lpass->top, SFR_LPASS_INTR_CA5_MASK, > + LPASS_INTR_SFR | LPASS_INTR_DMA | LPASS_INTR_I2S); > + > + regmap_write(lpass->top, SFR_LPASS_INTR_CPU_MASK, > + LPASS_INTR_SFR | LPASS_INTR_DMA | LPASS_INTR_I2S); > + > + /* Activate related PADs from retention state */ > + regmap_write(lpass->pmu, EXYNOS5433_PAD_RETENTION_AUD_OPTION, > + EXYNOS5433_PAD_INITIATE_WAKEUP_FROM_LOWPWR); > + > + exynos_lpass_core_sw_reset(lpass, LPASS_I2S_SW_RESET); > + exynos_lpass_core_sw_reset(lpass, LPASS_DMA_SW_RESET); > + exynos_lpass_core_sw_reset(lpass, LPASS_MEM_SW_RESET); > +} > + > +static void exynos_lpass_disable(struct exynos_lpass *lpass) > +{ > + /* Mask any unmasked IP interrupt sources */ > + regmap_write(lpass->top, SFR_LPASS_INTR_CPU_MASK, 0); > + regmap_write(lpass->top, SFR_LPASS_INTR_CA5_MASK, 0); > + > + /* Deactivate related PADs from retention state */ > + regmap_write(lpass->pmu, EXYNOS5433_PAD_RETENTION_AUD_OPTION, 0); > +} > + > +static const struct regmap_config exynos_lpass_reg_conf = { > + .reg_bits = 32, > + .reg_stride = 4, > + .val_bits = 32, > + .max_register = 0xfc, > + .fast_io = true, > +}; > + > +static int exynos_lpass_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct exynos_lpass *lpass; > + void __iomem *base_top; > + struct resource *res; > + > + lpass = devm_kzalloc(dev, sizeof(*lpass), GFP_KERNEL); > + if (!lpass) > + return -ENOMEM; > + > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > + base_top = devm_ioremap_resource(dev, res); > + if (IS_ERR(base_top)) > + return PTR_ERR(base_top); > + > + lpass->top = regmap_init_mmio(dev, base_top, > + &exynos_lpass_reg_conf); > + if (IS_ERR(lpass->top)) { > + dev_err(dev, "LPASS top regmap initialization failed\n"); > + return PTR_ERR(lpass->top); > + } > + > + lpass->pmu = syscon_regmap_lookup_by_phandle(dev->of_node, > + "samsung,pmu-syscon"); > + if (IS_ERR(lpass->pmu)) { > + dev_err(dev, "Failed to lookup PMU regmap\n"); > + return PTR_ERR(lpass->pmu); > + } > + > + platform_set_drvdata(pdev, lpass); > + exynos_lpass_enable(lpass); > + > + return of_platform_populate(dev->of_node, NULL, NULL, dev); > +} > + > +#ifdef CONFIG_PM_SLEEP > +static int exynos_lpass_suspend(struct device *dev) > +{ > + struct exynos_lpass *lpass = dev_get_drvdata(dev); > + > + exynos_lpass_disable(lpass); > + > + return 0; > +} > + > +static int exynos_lpass_resume(struct device *dev) > +{ > + struct exynos_lpass *lpass = dev_get_drvdata(dev); > + > + exynos_lpass_enable(lpass); > + > + return 0; > +} > +#endif > + > +static SIMPLE_DEV_PM_OPS(lpass_pm_ops, exynos_lpass_suspend, > + exynos_lpass_resume); > + > +static const struct of_device_id exynos_lpass_of_match[] = { > + { .compatible = "samsung,exynos5433-lpass" }, > + { }, > +}; > +MODULE_DEVICE_TABLE(of, exynos_lpass_of_match); > + > +static struct platform_driver exynos_lpass_driver = { > + .driver = { > + .name = "exynos-lpass", > + .pm = &lpass_pm_ops, > + .of_match_table = exynos_lpass_of_match, > + }, > + .probe = exynos_lpass_probe, > +}; > +module_platform_driver(exynos_lpass_driver); > + > +MODULE_DESCRIPTION("Samsung Low Power Audio Subsystem driver"); > +MODULE_LICENSE("GPL v2"); > diff --git a/include/linux/mfd/syscon/exynos5-pmu.h b/include/linux/mfd/syscon/exynos5-pmu.h > index 76f30f9..c28ff21 100644 > --- a/include/linux/mfd/syscon/exynos5-pmu.h > +++ b/include/linux/mfd/syscon/exynos5-pmu.h > @@ -43,8 +43,10 @@ > #define EXYNOS5433_MIPI_PHY2_CONTROL (0x718) > > #define EXYNOS5_PHY_ENABLE BIT(0) > - > #define EXYNOS5_MIPI_PHY_S_RESETN BIT(1) > #define EXYNOS5_MIPI_PHY_M_RESETN BIT(2) > > +#define EXYNOS5433_PAD_RETENTION_AUD_OPTION (0x3028) > +#define EXYNOS5433_PAD_INITIATE_WAKEUP_FROM_LOWPWR BIT(28) > + > #endif /* _LINUX_MFD_SYSCON_PMU_EXYNOS5_H_ */ >