From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@linaro.org (Kevin Hilman) Date: Fri, 30 Aug 2013 09:18:34 -0700 Subject: [PATCH v2 4/4] ARM: keystone: add PM bus support for clock management In-Reply-To: <1377819404-9671-5-git-send-email-santosh.shilimkar@ti.com> (Santosh Shilimkar's message of "Thu, 29 Aug 2013 19:36:44 -0400") References: <1377819404-9671-1-git-send-email-santosh.shilimkar@ti.com> <1377819404-9671-5-git-send-email-santosh.shilimkar@ti.com> Message-ID: <87k3j3p2xx.fsf@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Santosh Shilimkar writes: > Add runtime PM core support to Keystone SOCs by using the pm_clk > infrastructure of the PM core. Patch is based on Kevin's pm_domain > work on DaVinci SOCs. > > Keystone SOC doesn't have depedency to enable clocks in early > in the boot and hence the clock and PM bus initialisation is done > at subsys_init() level. > > Cc: Kevin Hilman > > Signed-off-by: Santosh Shilimkar > --- > drivers/bus/Makefile | 2 ++ > drivers/bus/keystone_pm_bus.c | 70 +++++++++++++++++++++++++++++++++++++++++ Maybe I missed some earlier discussion on this, but why drivers/bus? I called the davinci stuff 'bus' initially because it piggy-backed the platform_bus, but now that we have pm_domains, it's using that, and is unrelated the bus. Therefore, as with davinci, I suspect this belongs in mach-keystone. [...] > diff --git a/drivers/bus/keystone_pm_bus.c b/drivers/bus/keystone_pm_bus.c > new file mode 100644 > index 0000000..6cc56f1 > --- /dev/null > +++ b/drivers/bus/keystone_pm_bus.c > @@ -0,0 +1,70 @@ > +/* > + * PM Bus driver for Keystone2 devices > + * > + * Copyright 2013 Texas Instruments, Inc. > + * Santosh Shilimkar > + * > + * Based on Kevins work on DAVINCI SOCs > + * Kevin Hilman Dead email address (as you know) ;) You can just leave off the email, or use khilman at kernel.org as one that shouldn't change. Kevin