regulator: lp8788-buck: Remove buck[1|2]_vout_addr array
PATCH 1/1 regulator: lp8788-buck: Remove buck[1|2]_vout_addr array
The vout address for buck[1|2] can be easily calculated, thus remote these arrays.
Signed-off-by Axel Lin <[email protected]>
--- drivers/regulator/lp8788-buck.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-)
drivers/regulator/lp8788-buck.c +2 -12
@@ -103,16 +103,6 @@ static const int lp8788_buck_vtbl[] = { 1950000, 2000000, }; -static const u8 buck1_vout_addr[] = { -LP8788_BUCK1_VOUT0 ,LP8788_BUCK1_VOUT1 , -LP8788_BUCK1_VOUT2 ,LP8788_BUCK1_VOUT3 , -}; - -static const u8 buck2_vout_addr[] = { -LP8788_BUCK2_VOUT0 ,LP8788_BUCK2_VOUT1 , -LP8788_BUCK2_VOUT2 ,LP8788_BUCK2_VOUT3 , -}; -static void lp8788_buck1_set_dvs (struct lp8788_buck *buck) {struct lp8788_buck1_dvs *dvs = (struct lp8788_buck1_dvs *)buck->dvs;
@@ -235,7 +225,7 @@ static u8 lp8788_select_buck_vout_addr(struct lp8788_buck *buck,lp8788_read_byte (buck->lp,LP8788_BUCK_DVS_SEL , &val); idx = (val &LP8788_BUCK1_DVS_M ) >>LP8788_BUCK1_DVS_S ; } - addr = buck1_vout_addr[idx]; + addr =LP8788_BUCK1_VOUT0 + idx;break ;case BUCK2 :if (mode ==EXTPIN ) {
@@ -258,7 +248,7 @@ static u8 lp8788_select_buck_vout_addr(struct lp8788_buck *buck,lp8788_read_byte (buck->lp,LP8788_BUCK_DVS_SEL , &val); idx = (val &LP8788_BUCK2_DVS_M ) >>LP8788_BUCK2_DVS_S ; } - addr = buck2_vout_addr[idx]; + addr =LP8788_BUCK2_VOUT0 + idx;break ;default : goto err;
Signature · 1 line
1.7.9.5
Quoted 4 lines
The vout address for buck[1|2] can be easily calculated, thus remote these arrays.
Signed-off-by Axel Lin <[email protected]>
Acked-by Milo Kim <[email protected]>
���{.n�+�������+%�����ݶ��w��{.n�+����{��G�����{ay�ʇڙ�,j��f���h��������z_�(�階�ݢj"���m������G������?����&���~��iO��z��v�^�m����������?�I�
Mark Brown
AppliedQuoted 2 lines · Axel Lin
The vout address for buck[1|2] can be easily calculated, thus remote these arrays.
Applied, thanks.