[U-Boot] [PATCH V2 05/12] board:samsung:common: remove unused max77686 init function
Rajeshwari Birje
rajeshwari.birje at gmail.com
Fri Feb 14 06:32:09 CET 2014
Hi Piotr,
On Thu, Feb 13, 2014 at 7:40 PM, Piotr Wilczek <p.wilczek at samsung.com> wrote:
> This patch removes currently unused max77686_init function.
> Despite being not used, it's implementation is board specific.
>
MAX77686 is required for 5250, but missed it somehow when adding 5420
support and making a common config file for both. It is my mistake
will correct the same
You can refer:
"[U-Boot] [PATCH V5 0/6] SMDK5420: Add S2MPS11 pmic support to
SMDK5420" by Leela Krishna Amudala
It adds a generic way for PMIC support.
http://lists.denx.de/pipermail/u-boot/2014-January/171113.html
Regards,
Rajeshwari
> Signed-off-by: Piotr Wilczek <p.wilczek at samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> Cc: Rajeshwari S Shinde <rajeshwari.s at samsung.com>
> Cc: Minkyu Kang <mk7.kang at samsung.com>
> ---
> Changes for v2:
> - new patch
>
> board/samsung/common/board.c | 120 ------------------------------------------
> 1 file changed, 120 deletions(-)
>
> diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c
> index cd873bc..3ac8005 100644
> --- a/board/samsung/common/board.c
> +++ b/board/samsung/common/board.c
> @@ -22,7 +22,6 @@
> #include <asm/arch/power.h>
> #include <power/pmic.h>
> #include <asm/arch/sromc.h>
> -#include <power/max77686_pmic.h>
>
> DECLARE_GLOBAL_DATA_PTR;
>
> @@ -160,133 +159,14 @@ static int board_init_cros_ec_devices(const void *blob)
> }
> #endif
>
> -#if defined(CONFIG_POWER)
> -#ifdef CONFIG_POWER_MAX77686
> -static int pmic_reg_update(struct pmic *p, int reg, uint regval)
> -{
> - u32 val;
> - int ret = 0;
> -
> - ret = pmic_reg_read(p, reg, &val);
> - if (ret) {
> - debug("%s: PMIC %d register read failed\n", __func__, reg);
> - return -1;
> - }
> - val |= regval;
> - ret = pmic_reg_write(p, reg, val);
> - if (ret) {
> - debug("%s: PMIC %d register write failed\n", __func__, reg);
> - return -1;
> - }
> - return 0;
> -}
> -
> -static int max77686_init(void)
> -{
> - struct pmic *p;
> -
> - if (pmic_init(I2C_PMIC))
> - return -1;
> -
> - p = pmic_get("MAX77686_PMIC");
> - if (!p)
> - return -ENODEV;
> -
> - if (pmic_probe(p))
> - return -1;
> -
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_32KHZ, MAX77686_32KHCP_EN))
> - return -1;
> -
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_BBAT,
> - MAX77686_BBCHOSTEN | MAX77686_BBCVS_3_5V))
> - return -1;
> -
> - /* VDD_MIF */
> - if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK1OUT,
> - MAX77686_BUCK1OUT_1V)) {
> - debug("%s: PMIC %d register write failed\n", __func__,
> - MAX77686_REG_PMIC_BUCK1OUT);
> - return -1;
> - }
> -
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK1CRTL,
> - MAX77686_BUCK1CTRL_EN))
> - return -1;
> -
> - /* VDD_ARM */
> - if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK2DVS1,
> - MAX77686_BUCK2DVS1_1_3V)) {
> - debug("%s: PMIC %d register write failed\n", __func__,
> - MAX77686_REG_PMIC_BUCK2DVS1);
> - return -1;
> - }
> -
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK2CTRL1,
> - MAX77686_BUCK2CTRL_ON))
> - return -1;
> -
> - /* VDD_INT */
> - if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK3DVS1,
> - MAX77686_BUCK3DVS1_1_0125V)) {
> - debug("%s: PMIC %d register write failed\n", __func__,
> - MAX77686_REG_PMIC_BUCK3DVS1);
> - return -1;
> - }
> -
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK3CTRL,
> - MAX77686_BUCK3CTRL_ON))
> - return -1;
> -
> - /* VDD_G3D */
> - if (pmic_reg_write(p, MAX77686_REG_PMIC_BUCK4DVS1,
> - MAX77686_BUCK4DVS1_1_2V)) {
> - debug("%s: PMIC %d register write failed\n", __func__,
> - MAX77686_REG_PMIC_BUCK4DVS1);
> - return -1;
> - }
> -
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_BUCK4CTRL1,
> - MAX77686_BUCK3CTRL_ON))
> - return -1;
> -
> - /* VDD_LDO2 */
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO2CTRL1,
> - MAX77686_LD02CTRL1_1_5V | EN_LDO))
> - return -1;
> -
> - /* VDD_LDO3 */
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO3CTRL1,
> - MAX77686_LD03CTRL1_1_8V | EN_LDO))
> - return -1;
> -
> - /* VDD_LDO5 */
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO5CTRL1,
> - MAX77686_LD05CTRL1_1_8V | EN_LDO))
> - return -1;
> -
> - /* VDD_LDO10 */
> - if (pmic_reg_update(p, MAX77686_REG_PMIC_LDO10CTRL1,
> - MAX77686_LD10CTRL1_1_8V | EN_LDO))
> - return -1;
> -
> - return 0;
> -}
> -#endif
> -
> int power_init_board(void)
> {
> int ret = 0;
>
> set_ps_hold_ctrl();
>
> -#ifdef CONFIG_POWER_MAX77686
> - ret = max77686_init();
> -#endif
> -
> return ret;
> }
> -#endif
>
> #ifdef CONFIG_OF_CONTROL
> static int decode_sromc(const void *blob, struct fdt_sromc *config)
> --
> 1.7.9.5
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
--
Regards,
Rajeshwari Shinde
More information about the U-Boot
mailing list