diff options
| author | Andre Przywara <andre.przywara@arm.com> | 2016-11-16 00:50:10 +0000 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2016-12-04 13:55:02 -0500 |
| commit | 429033659d574bec966718de19eb732b99f3a9af (patch) | |
| tree | b518c2929f861e855ddc0dc91bc24feacd8f7fd6 /drivers/phy/marvell/comphy_a3700.h | |
| parent | b8d4fad3bcf606b286962cc0c2251ec6be7d7909 (diff) | |
marvell: comphy_a3700: fix bitmask
Obviously the mask for the rx and tx select field cannot be right,
as it would overlap in one and exceed the 32-bit register in the other
case. From looking at the neighbouring bits it looks like the mask
should be really 4 bits wide instead of 8.
Pointed out by a GCC 6.2 (default) warning.
Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Stefan Roese <sr@denx.de>
Reviewed-by: Stefan Roese <sr@denx.de>
Diffstat (limited to 'drivers/phy/marvell/comphy_a3700.h')
| -rw-r--r-- | drivers/phy/marvell/comphy_a3700.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/phy/marvell/comphy_a3700.h b/drivers/phy/marvell/comphy_a3700.h index eb2ed7b317..dd60b882dd 100644 --- a/drivers/phy/marvell/comphy_a3700.h +++ b/drivers/phy/marvell/comphy_a3700.h @@ -33,9 +33,9 @@ #define rb_pin_pu_tx BIT(18) #define rb_pin_tx_idle BIT(19) #define rf_gen_rx_sel_shift 22 -#define rf_gen_rx_select (0xFF << rf_gen_rx_sel_shift) +#define rf_gen_rx_select (0x0F << rf_gen_rx_sel_shift) #define rf_gen_tx_sel_shift 26 -#define rf_gen_tx_select (0xFF << rf_gen_tx_sel_shift) +#define rf_gen_tx_select (0x0F << rf_gen_tx_sel_shift) #define rb_phy_rx_init BIT(30) #define COMPHY_PHY_STAT1_ADDR(lane) MVEBU_REG(0x018318 + (lane) * 0x28) |