Message ID | 20190416135728.27163-1-david.wu@rock-chips.com |
---|---|
State | Accepted |
Delegated to: | Kever Yang |
Headers | show |
Series | pinctrl: Split the common mux/drive/pull/schmitt func into per Soc | expand |
On 04/16/2019 09:57 PM, David Wu wrote: > RK3288 pmu_gpio0 pull setting have no higher 16 writing corresponding > bits, need to read before write the register. > > Signed-off-by: David Wu <david.wu@rock-chips.com> Reviewed-by: Kever Yang <kever.yang@rock-chips.com> Thanks, - Kever > --- > > Change in v3: > - None > > drivers/pinctrl/rockchip/pinctrl-rk3288.c | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/drivers/pinctrl/rockchip/pinctrl-rk3288.c b/drivers/pinctrl/rockchip/pinctrl-rk3288.c > index 9192aa3949..24af3597ec 100644 > --- a/drivers/pinctrl/rockchip/pinctrl-rk3288.c > +++ b/drivers/pinctrl/rockchip/pinctrl-rk3288.c > @@ -116,8 +116,15 @@ static int rk3288_set_pull(struct rockchip_pin_bank *bank, > return ret; > } > > - /* enable the write to the equivalent lower bits */ > - data = ((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << (bit + 16); > + /* bank0 is special, there are no higher 16 bit writing bits */ > + if (bank->bank_num == 0) { > + regmap_read(regmap, reg, &data); > + data &= ~(((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << bit); > + } else { > + /* enable the write to the equivalent lower bits */ > + data = ((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << (bit + 16); > + } > + > data |= (ret << bit); > ret = regmap_write(regmap, reg, data); >
On 05/07/2019 11:44 AM, Kever Yang wrote: > > On 04/16/2019 09:57 PM, David Wu wrote: >> RK3288 pmu_gpio0 pull setting have no higher 16 writing corresponding >> bits, need to read before write the register. >> >> Signed-off-by: David Wu <david.wu@rock-chips.com> > Reviewed-by: Kever Yang <kever.yang@rock-chips.com> Applied to u-boot-rockchip, thanks! > > Thanks, > - Kever >> --- >> >> Change in v3: >> - None >> >> drivers/pinctrl/rockchip/pinctrl-rk3288.c | 11 +++++++++-- >> 1 file changed, 9 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/pinctrl/rockchip/pinctrl-rk3288.c b/drivers/pinctrl/rockchip/pinctrl-rk3288.c >> index 9192aa3949..24af3597ec 100644 >> --- a/drivers/pinctrl/rockchip/pinctrl-rk3288.c >> +++ b/drivers/pinctrl/rockchip/pinctrl-rk3288.c >> @@ -116,8 +116,15 @@ static int rk3288_set_pull(struct rockchip_pin_bank *bank, >> return ret; >> } >> >> - /* enable the write to the equivalent lower bits */ >> - data = ((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << (bit + 16); >> + /* bank0 is special, there are no higher 16 bit writing bits */ >> + if (bank->bank_num == 0) { >> + regmap_read(regmap, reg, &data); >> + data &= ~(((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << bit); >> + } else { >> + /* enable the write to the equivalent lower bits */ >> + data = ((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << (bit + 16); >> + } >> + >> data |= (ret << bit); >> ret = regmap_write(regmap, reg, data); >> > > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot
diff --git a/drivers/pinctrl/rockchip/pinctrl-rk3288.c b/drivers/pinctrl/rockchip/pinctrl-rk3288.c index 9192aa3949..24af3597ec 100644 --- a/drivers/pinctrl/rockchip/pinctrl-rk3288.c +++ b/drivers/pinctrl/rockchip/pinctrl-rk3288.c @@ -116,8 +116,15 @@ static int rk3288_set_pull(struct rockchip_pin_bank *bank, return ret; } - /* enable the write to the equivalent lower bits */ - data = ((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << (bit + 16); + /* bank0 is special, there are no higher 16 bit writing bits */ + if (bank->bank_num == 0) { + regmap_read(regmap, reg, &data); + data &= ~(((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << bit); + } else { + /* enable the write to the equivalent lower bits */ + data = ((1 << ROCKCHIP_PULL_BITS_PER_PIN) - 1) << (bit + 16); + } + data |= (ret << bit); ret = regmap_write(regmap, reg, data);
RK3288 pmu_gpio0 pull setting have no higher 16 writing corresponding bits, need to read before write the register. Signed-off-by: David Wu <david.wu@rock-chips.com> --- Change in v3: - None drivers/pinctrl/rockchip/pinctrl-rk3288.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-)