[U-Boot] [PATCH V3 1/6] power: pfuze100 correct SWBST macro definition
Przemyslaw Marczak
p.marczak at samsung.com
Wed Aug 12 12:09:47 CEST 2015
Hello,
On 08/07/2015 10:43 AM, Peng Fan wrote:
> According to datasheet, SWBST_MODE starts from bit 2 and it occupies 2 bits.
> So SWBST_MODE_MASK should be 0xC, and SWBST_MODE_xx should be ([mode] << 2).
>
> Signed-off-by: Peng Fan <Peng.Fan at freescale.com>
> Cc: Przemyslaw Marczak <p.marczak at samsung.com>
> Cc: Stefano Babic <sbabic at denx.de>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes v3:
> None
>
> Changes v2:
> None
>
> include/power/pfuze100_pmic.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/power/pfuze100_pmic.h b/include/power/pfuze100_pmic.h
> index 57b9ca9..cb10605 100644
> --- a/include/power/pfuze100_pmic.h
> +++ b/include/power/pfuze100_pmic.h
> @@ -193,11 +193,11 @@ enum {
> #define SWBST_5_15V 3
>
> #define SWBST_VOL_MASK 0x3
> -#define SWBST_MODE_MASK 0x6
> -#define SWBST_MODE_OFF (2 << 0)
> -#define SWBST_MODE_PFM (2 << 1)
> +#define SWBST_MODE_MASK 0xC
> +#define SWBST_MODE_OFF (0 << 2)
> +#define SWBST_MODE_PFM (1 << 2)
> #define SWBST_MODE_AUTO (2 << 2)
> -#define SWBST_MODE_APS (2 << 3)
> +#define SWBST_MODE_APS (3 << 2)
>
> /*
> * Regulator Mode Control
>
Applied to: u-boot-pmic.
Best regards,
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
More information about the U-Boot
mailing list