From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 6/7] mfd: menelaus: make vcore and regulator controls internal Date: Tue, 20 Jan 2015 13:58:40 +0000 Message-ID: <20150120135840.GO13701@x1> References: <1419702779-16001-1-git-send-email-aaro.koskinen@iki.fi> <1419702779-16001-7-git-send-email-aaro.koskinen@iki.fi> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <1419702779-16001-7-git-send-email-aaro.koskinen@iki.fi> Sender: linux-omap-owner@vger.kernel.org To: Aaro Koskinen Cc: Tony Lindgren , Samuel Ortiz , devicetree@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Sat, 27 Dec 2014, Aaro Koskinen wrote: > Make vcore and regulator control functions internal to the driver. >=20 > Signed-off-by: Aaro Koskinen > --- > drivers/mfd/menelaus.c | 13 +++++++++++-- > include/linux/mfd/menelaus.h | 12 ------------ > 2 files changed, 11 insertions(+), 14 deletions(-) >=20 > diff --git a/drivers/mfd/menelaus.c b/drivers/mfd/menelaus.c > index 4a900cc..8660adf 100644 > --- a/drivers/mfd/menelaus.c > +++ b/drivers/mfd/menelaus.c > @@ -163,6 +163,15 @@ > #define MCT_PIN_ST_S1_CD_ST (1 << 0) > #define MCT_PIN_ST_S2_CD_ST (1 << 1) > =20 > +#define EN_VPLL_SLEEP (1 << 7) > +#define EN_VMMC_SLEEP (1 << 6) > +#define EN_VAUX_SLEEP (1 << 5) > +#define EN_VIO_SLEEP (1 << 4) > +#define EN_VMEM_SLEEP (1 << 3) > +#define EN_DC3_SLEEP (1 << 2) > +#define EN_DC2_SLEEP (1 << 1) > +#define EN_VC_SLEEP (1 << 0) Better to use the BIT() macro for this kind of stuff. Please take this opportunity to use it. --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html