| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CADGdYn67ny-_--6fYYugE+3Nydnd=uDgyf3rUSj9TMAugpEs8w@mail.gmail.com> Date: Mon, 10 Nov 2014 10:37:04 +0530 From: amit daniel kachhap <amit.daniel@...sung.com> To: Pankaj Dubey <pankaj.dubey@...sung.com> Cc: "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, linux-samsung-soc <linux-samsung-soc@...r.kernel.org>, Kukjin Kim <kgene.kim@...sung.com>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Sylwester Nawrocki <s.nawrocki@...sung.com>, Lee Jones <lee.jones@...aro.org> Subject: Re: [PATCH v2 2/6] drivers: soc: Add support for Exynos PMU driver On Sat, Nov 8, 2014 at 10:45 PM, Pankaj Dubey <pankaj.dubey@...sung.com> wrote: > Hi Amit, > > On 8 November 2014 18:46, Amit Daniel Kachhap <amit.daniel@...sung.com> wrote: >> This patch moves Exynos PMU driver implementation from "arm/mach-exynos" >> to "drivers/soc/samsung". This driver is mainly used for setting misc bits of >> register from PMU IP of Exynos SoC which will be required to configure >> before Suspend/Resume. Currently all these settings are done in >> "arch/arm/mach-exynos/pmu.c" but moving ahead for ARM64 based SoC >> support,there is a need of DT based implementation of PMU driver. >> >> This driver uses already existing DT binding information. >> >> Signed-off-by: Pankaj Dubey <pankaj.dubey@...sung.com> >> Signed-off-by: Amit Daniel Kachhap <amit.daniel@...sung.com> >> --- >> arch/arm/mach-exynos/Makefile | 2 +- >> drivers/soc/Kconfig | 1 + >> drivers/soc/Makefile | 1 + >> drivers/soc/samsung/Kconfig | 19 +++++++++++++++++++ >> drivers/soc/samsung/Makefile | 1 + >> .../pmu.c => drivers/soc/samsung/exynos-pmu.c | 0 >> 6 files changed, 23 insertions(+), 1 deletion(-) >> create mode 100644 drivers/soc/samsung/Kconfig >> create mode 100644 drivers/soc/samsung/Makefile >> rename arch/arm/mach-exynos/pmu.c => drivers/soc/samsung/exynos-pmu.c (100%) >> >> diff --git a/arch/arm/mach-exynos/Makefile b/arch/arm/mach-exynos/Makefile >> index bcefb54..b91b382 100644 >> --- a/arch/arm/mach-exynos/Makefile >> +++ b/arch/arm/mach-exynos/Makefile >> @@ -9,7 +9,7 @@ ccflags-$(CONFIG_ARCH_MULTIPLATFORM) += -I$(srctree)/$(src)/include -I$(srctree) >> >> # Core >> >> -obj-$(CONFIG_ARCH_EXYNOS) += exynos.o pmu.o exynos-smc.o firmware.o >> +obj-$(CONFIG_ARCH_EXYNOS) += exynos.o exynos-smc.o firmware.o >> >> obj-$(CONFIG_EXYNOS_CPU_SUSPEND) += pm.o sleep.o >> obj-$(CONFIG_PM_SLEEP) += suspend.o >> diff --git a/drivers/soc/Kconfig b/drivers/soc/Kconfig >> index 76d6bd4..90f33b9 100644 >> --- a/drivers/soc/Kconfig >> +++ b/drivers/soc/Kconfig >> @@ -3,5 +3,6 @@ menu "SOC (System On Chip) specific Drivers" >> source "drivers/soc/qcom/Kconfig" >> source "drivers/soc/ti/Kconfig" >> source "drivers/soc/versatile/Kconfig" >> +source "drivers/soc/samsung/Kconfig" >> >> endmenu >> diff --git a/drivers/soc/Makefile b/drivers/soc/Makefile >> index 063113d..44d220d 100644 >> --- a/drivers/soc/Makefile >> +++ b/drivers/soc/Makefile >> @@ -6,3 +6,4 @@ obj-$(CONFIG_ARCH_QCOM) += qcom/ >> obj-$(CONFIG_ARCH_TEGRA) += tegra/ >> obj-$(CONFIG_SOC_TI) += ti/ >> obj-$(CONFIG_PLAT_VERSATILE) += versatile/ >> +obj-$(CONFIG_ARCH_EXYNOS) += samsung/ >> diff --git a/drivers/soc/samsung/Kconfig b/drivers/soc/samsung/Kconfig >> new file mode 100644 >> index 0000000..beb6dfc >> --- /dev/null >> +++ b/drivers/soc/samsung/Kconfig >> @@ -0,0 +1,19 @@ >> +# >> +# SAMSUNG SOC drivers >> +# >> +menuconfig SOC_SAMSUNG >> + bool "Samsung SOC drivers support" >> + >> +if SOC_SAMSUNG >> + >> +config MFD_EXYNOS_PMU > > MFD_EXYNOS_PMU? I think it should be only EXYNOS_PMU, > as we are moving here in drivers/soc giving reason as it does not fit > into MFD. > >> + tristate "Support Exynos Power Management Unit" > > I think this config should not be user visible config option, as I remember > same was pointed out by Tomasz some times back. I agree to some extent so set its property to default yes. But as this is now a platform driver so I let it visible. > >> + depends on ARM || ARM64 > > How about only making depends on ARCH_EXYNOS? Right makes sense. Will post v3 with this change. Regards, Amit > >> + default y >> + help >> + Exynos SoC have Power Management Unit (PMU) which controls power and >> + operation state of Exynos SoC in two different ways. This driver >> + provides implementation of PMU driver and provides basic functionality >> + required during these operation state. >> + >> +endif #SOC_SAMSUNG >> diff --git a/drivers/soc/samsung/Makefile b/drivers/soc/samsung/Makefile >> new file mode 100644 >> index 0000000..9d30f61 >> --- /dev/null >> +++ b/drivers/soc/samsung/Makefile >> @@ -0,0 +1 @@ >> +obj-$(CONFIG_MFD_EXYNOS_PMU) += exynos-pmu.o >> diff --git a/arch/arm/mach-exynos/pmu.c b/drivers/soc/samsung/exynos-pmu.c >> similarity index 100% >> rename from arch/arm/mach-exynos/pmu.c >> rename to drivers/soc/samsung/exynos-pmu.c >> -- >> 1.7.9.5 >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@...ts.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists