* [PATCH 2.6.33] mfd: Correct WM835x ISINK ramp time defines
@ 2010-01-04 18:05 Mark Brown
2010-01-05 19:37 ` Samuel Ortiz
0 siblings, 1 reply; 2+ messages in thread
From: Mark Brown @ 2010-01-04 18:05 UTC (permalink / raw)
To: Samuel Ortiz; +Cc: linux-kernel, Mark Brown, stable
The constants used to specify ISINK ramp times for WM835x had the
wrong shifts so that the on times applied to the off ramp and vice
versa. The masks for the bitfields are correct.
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: stable@kernel.org
---
include/linux/mfd/wm8350/pmic.h | 28 ++++++++++++++--------------
1 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/include/linux/mfd/wm8350/pmic.h b/include/linux/mfd/wm8350/pmic.h
index be3264e..e786fe9 100644
--- a/include/linux/mfd/wm8350/pmic.h
+++ b/include/linux/mfd/wm8350/pmic.h
@@ -666,20 +666,20 @@
#define WM8350_ISINK_FLASH_DUR_64MS (1 << 8)
#define WM8350_ISINK_FLASH_DUR_96MS (2 << 8)
#define WM8350_ISINK_FLASH_DUR_1024MS (3 << 8)
-#define WM8350_ISINK_FLASH_ON_INSTANT (0 << 4)
-#define WM8350_ISINK_FLASH_ON_0_25S (1 << 4)
-#define WM8350_ISINK_FLASH_ON_0_50S (2 << 4)
-#define WM8350_ISINK_FLASH_ON_1_00S (3 << 4)
-#define WM8350_ISINK_FLASH_ON_1_95S (1 << 4)
-#define WM8350_ISINK_FLASH_ON_3_91S (2 << 4)
-#define WM8350_ISINK_FLASH_ON_7_80S (3 << 4)
-#define WM8350_ISINK_FLASH_OFF_INSTANT (0 << 0)
-#define WM8350_ISINK_FLASH_OFF_0_25S (1 << 0)
-#define WM8350_ISINK_FLASH_OFF_0_50S (2 << 0)
-#define WM8350_ISINK_FLASH_OFF_1_00S (3 << 0)
-#define WM8350_ISINK_FLASH_OFF_1_95S (1 << 0)
-#define WM8350_ISINK_FLASH_OFF_3_91S (2 << 0)
-#define WM8350_ISINK_FLASH_OFF_7_80S (3 << 0)
+#define WM8350_ISINK_FLASH_ON_INSTANT (0 << 0)
+#define WM8350_ISINK_FLASH_ON_0_25S (1 << 0)
+#define WM8350_ISINK_FLASH_ON_0_50S (2 << 0)
+#define WM8350_ISINK_FLASH_ON_1_00S (3 << 0)
+#define WM8350_ISINK_FLASH_ON_1_95S (1 << 0)
+#define WM8350_ISINK_FLASH_ON_3_91S (2 << 0)
+#define WM8350_ISINK_FLASH_ON_7_80S (3 << 0)
+#define WM8350_ISINK_FLASH_OFF_INSTANT (0 << 4)
+#define WM8350_ISINK_FLASH_OFF_0_25S (1 << 4)
+#define WM8350_ISINK_FLASH_OFF_0_50S (2 << 4)
+#define WM8350_ISINK_FLASH_OFF_1_00S (3 << 4)
+#define WM8350_ISINK_FLASH_OFF_1_95S (1 << 4)
+#define WM8350_ISINK_FLASH_OFF_3_91S (2 << 4)
+#define WM8350_ISINK_FLASH_OFF_7_80S (3 << 4)
/*
* Regulator Interrupts.
--
1.6.5.7
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 2.6.33] mfd: Correct WM835x ISINK ramp time defines
2010-01-04 18:05 [PATCH 2.6.33] mfd: Correct WM835x ISINK ramp time defines Mark Brown
@ 2010-01-05 19:37 ` Samuel Ortiz
0 siblings, 0 replies; 2+ messages in thread
From: Samuel Ortiz @ 2010-01-05 19:37 UTC (permalink / raw)
To: Mark Brown; +Cc: linux-kernel, stable
Hi Mark,
On Mon, Jan 04, 2010 at 06:05:00PM +0000, Mark Brown wrote:
> The constants used to specify ISINK ramp times for WM835x had the
> wrong shifts so that the on times applied to the off ramp and vice
> versa. The masks for the bitfields are correct.
Patch applied to my for-linus branch, thanks.
Cheers,
Samuel.
> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
> Cc: stable@kernel.org
> ---
> include/linux/mfd/wm8350/pmic.h | 28 ++++++++++++++--------------
> 1 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/include/linux/mfd/wm8350/pmic.h b/include/linux/mfd/wm8350/pmic.h
> index be3264e..e786fe9 100644
> --- a/include/linux/mfd/wm8350/pmic.h
> +++ b/include/linux/mfd/wm8350/pmic.h
> @@ -666,20 +666,20 @@
> #define WM8350_ISINK_FLASH_DUR_64MS (1 << 8)
> #define WM8350_ISINK_FLASH_DUR_96MS (2 << 8)
> #define WM8350_ISINK_FLASH_DUR_1024MS (3 << 8)
> -#define WM8350_ISINK_FLASH_ON_INSTANT (0 << 4)
> -#define WM8350_ISINK_FLASH_ON_0_25S (1 << 4)
> -#define WM8350_ISINK_FLASH_ON_0_50S (2 << 4)
> -#define WM8350_ISINK_FLASH_ON_1_00S (3 << 4)
> -#define WM8350_ISINK_FLASH_ON_1_95S (1 << 4)
> -#define WM8350_ISINK_FLASH_ON_3_91S (2 << 4)
> -#define WM8350_ISINK_FLASH_ON_7_80S (3 << 4)
> -#define WM8350_ISINK_FLASH_OFF_INSTANT (0 << 0)
> -#define WM8350_ISINK_FLASH_OFF_0_25S (1 << 0)
> -#define WM8350_ISINK_FLASH_OFF_0_50S (2 << 0)
> -#define WM8350_ISINK_FLASH_OFF_1_00S (3 << 0)
> -#define WM8350_ISINK_FLASH_OFF_1_95S (1 << 0)
> -#define WM8350_ISINK_FLASH_OFF_3_91S (2 << 0)
> -#define WM8350_ISINK_FLASH_OFF_7_80S (3 << 0)
> +#define WM8350_ISINK_FLASH_ON_INSTANT (0 << 0)
> +#define WM8350_ISINK_FLASH_ON_0_25S (1 << 0)
> +#define WM8350_ISINK_FLASH_ON_0_50S (2 << 0)
> +#define WM8350_ISINK_FLASH_ON_1_00S (3 << 0)
> +#define WM8350_ISINK_FLASH_ON_1_95S (1 << 0)
> +#define WM8350_ISINK_FLASH_ON_3_91S (2 << 0)
> +#define WM8350_ISINK_FLASH_ON_7_80S (3 << 0)
> +#define WM8350_ISINK_FLASH_OFF_INSTANT (0 << 4)
> +#define WM8350_ISINK_FLASH_OFF_0_25S (1 << 4)
> +#define WM8350_ISINK_FLASH_OFF_0_50S (2 << 4)
> +#define WM8350_ISINK_FLASH_OFF_1_00S (3 << 4)
> +#define WM8350_ISINK_FLASH_OFF_1_95S (1 << 4)
> +#define WM8350_ISINK_FLASH_OFF_3_91S (2 << 4)
> +#define WM8350_ISINK_FLASH_OFF_7_80S (3 << 4)
>
> /*
> * Regulator Interrupts.
> --
> 1.6.5.7
>
--
Intel Open Source Technology Centre
http://oss.intel.com/
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-01-05 19:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-01-04 18:05 [PATCH 2.6.33] mfd: Correct WM835x ISINK ramp time defines Mark Brown
2010-01-05 19:37 ` Samuel Ortiz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome