From 02a78261315045011a5a987761d7a76e51b04480 Mon Sep 17 00:00:00 2001 From: Max Krummenacher Date: Wed, 26 Sep 2018 13:42:55 +0200 Subject: usb/misc/usb3503: use regmap write to set nrd and pds Using regmap update bits the way it was used is wrong as only additional bits gets set but no bits would be cleared. The whole register or NRD and PDS are reserved for those two function. There are no bits which needs to be preserved in the registers so just writting the new value is all that is needed. Signed-off-by: Max Krummenacher --- drivers/usb/misc/usb3503.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/usb/misc/usb3503.c b/drivers/usb/misc/usb3503.c index 81f74654e5a3..77127466f32e 100644 --- a/drivers/usb/misc/usb3503.c +++ b/drivers/usb/misc/usb3503.c @@ -105,8 +105,7 @@ static int usb3503_connect(struct usb3503 *hub) /* PDS : Set the ports which are disabled in self-powered mode. */ if (hub->port_off_mask) { - err = regmap_update_bits(hub->regmap, USB3503_PDS, - hub->port_off_mask, + err = regmap_write(hub->regmap, USB3503_PDS, hub->port_off_mask); if (err < 0) { dev_err(dev, "PDS failed (%d)\n", err); @@ -125,8 +124,7 @@ static int usb3503_connect(struct usb3503 *hub) /* NRD : Set non removable ports. */ if (hub->port_nrd) { - err = regmap_update_bits(hub->regmap, USB3503_NRD, - hub->port_nrd, + err = regmap_write(hub->regmap, USB3503_NRD, hub->port_nrd); if (err < 0) { dev_err(dev, "NRD failed (%d)\n", err); -- cgit v1.2.3