mirror of
https://source.denx.de/u-boot/u-boot.git
synced 2026-06-02 09:46:37 +03:00
usb: ulpi: Incorrect operator used
Combining two bits into a mask to be used so that the same write code can be used to set or reset bits in a register clearly needs to use the binary 'or' operator, not the binary 'and'. Fix it. This issue was found by Smatch. Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
This commit is contained in:
committed by
Marek Vasut
parent
08f4c52c97
commit
dd01b0541b
@@ -127,7 +127,7 @@ int ulpi_set_vbus_indicator(struct ulpi_viewport *ulpi_vp, int external,
|
||||
if (val == ULPI_ERROR)
|
||||
return val;
|
||||
|
||||
val = val & ~(ULPI_IFACE_PASSTHRU & ULPI_IFACE_EXTVBUS_COMPLEMENT);
|
||||
val = val & ~(ULPI_IFACE_PASSTHRU | ULPI_IFACE_EXTVBUS_COMPLEMENT);
|
||||
val |= flags;
|
||||
val = ulpi_write(ulpi_vp, &ulpi->iface_ctrl, val);
|
||||
if (val)
|
||||
|
||||
Reference in New Issue
Block a user