Message ID | 1446728090-5969-5-git-send-email-thomas@wytron.com.tw |
---|---|
State | Accepted, archived |
Delegated to: | Thomas Chou |
Headers | show |
On Thu, Nov 5, 2015 at 6:54 AM, Thomas Chou <thomas@wytron.com.tw> wrote: > Replace numerical bit shift with BIT macro > in altera_tse > > :%s/(1 << nr)/BIT(nr)/g > where nr = 0, 1, 2 .... 31 > > Signed-off-by: Thomas Chou <thomas@wytron.com.tw> > Reviewed-by: Marek Vasut <marex@denx.de> > Reviewed-by: Chin Liang See <clsee@altera.com> > Reviewed-by: Jagan Teki <jteki@openedev.com> Acked-by: Joe Hershberger <joe.hershberger@ni.com>
On 2015年11月05日 20:54, Thomas Chou wrote: > Replace numerical bit shift with BIT macro > in altera_tse > > :%s/(1 << nr)/BIT(nr)/g > where nr = 0, 1, 2 .... 31 > > Signed-off-by: Thomas Chou <thomas@wytron.com.tw> > Reviewed-by: Marek Vasut <marex@denx.de> > Reviewed-by: Chin Liang See <clsee@altera.com> > Reviewed-by: Jagan Teki <jteki@openedev.com> > --- > drivers/net/altera_tse.h | 30 +++++++++++++++--------------- > 1 file changed, 15 insertions(+), 15 deletions(-) > Applied to u-boot-nios. > diff --git a/drivers/net/altera_tse.h b/drivers/net/altera_tse.h > index 0981308..471a880 100644 > --- a/drivers/net/altera_tse.h > +++ b/drivers/net/altera_tse.h > @@ -14,11 +14,11 @@ > #define __packed_1_ __packed __aligned(1) > > /* SGDMA Stuff */ > -#define ALT_SGDMA_STATUS_BUSY_MSK 0x00000010 > +#define ALT_SGDMA_STATUS_BUSY_MSK BIT(4) > > -#define ALT_SGDMA_CONTROL_RUN_MSK 0x00000020 > -#define ALT_SGDMA_CONTROL_STOP_DMA_ER_MSK 0x00000040 > -#define ALT_SGDMA_CONTROL_SOFTWARERESET_MSK 0x00010000 > +#define ALT_SGDMA_CONTROL_RUN_MSK BIT(5) > +#define ALT_SGDMA_CONTROL_STOP_DMA_ER_MSK BIT(6) > +#define ALT_SGDMA_CONTROL_SOFTWARERESET_MSK BIT(16) > > /* > * Descriptor control bit masks & offsets > @@ -27,10 +27,10 @@ > * The following bit-offsets are expressed relative to the LSB of > * the control register bitfield. > */ > -#define ALT_SGDMA_DESCRIPTOR_CONTROL_GENERATE_EOP_MSK 0x00000001 > -#define ALT_SGDMA_DESCRIPTOR_CONTROL_READ_FIXED_ADDRESS_MSK 0x00000002 > -#define ALT_SGDMA_DESCRIPTOR_CONTROL_WRITE_FIXED_ADDRESS_MSK 0x00000004 > -#define ALT_SGDMA_DESCRIPTOR_CONTROL_OWNED_BY_HW_MSK 0x00000080 > +#define ALT_SGDMA_DESCRIPTOR_CONTROL_GENERATE_EOP_MSK BIT(0) > +#define ALT_SGDMA_DESCRIPTOR_CONTROL_READ_FIXED_ADDRESS_MSK BIT(1) > +#define ALT_SGDMA_DESCRIPTOR_CONTROL_WRITE_FIXED_ADDRESS_MSK BIT(2) > +#define ALT_SGDMA_DESCRIPTOR_CONTROL_OWNED_BY_HW_MSK BIT(7) > > /* > * Descriptor status bit masks & offsets > @@ -39,7 +39,7 @@ > * The following bit-offsets are expressed relative to the LSB of > * the status register bitfield. > */ > -#define ALT_SGDMA_DESCRIPTOR_STATUS_TERMINATED_BY_EOP_MSK 0x00000080 > +#define ALT_SGDMA_DESCRIPTOR_STATUS_TERMINATED_BY_EOP_MSK BIT(7) > > /* > * The SGDMA controller buffer descriptor allocates > @@ -85,12 +85,12 @@ struct alt_sgdma_registers { > }; > > /* TSE Stuff */ > -#define ALTERA_TSE_CMD_TX_ENA_MSK 0x00000001 > -#define ALTERA_TSE_CMD_RX_ENA_MSK 0x00000002 > -#define ALTERA_TSE_CMD_ETH_SPEED_MSK 0x00000008 > -#define ALTERA_TSE_CMD_HD_ENA_MSK 0x00000400 > -#define ALTERA_TSE_CMD_SW_RESET_MSK 0x00002000 > -#define ALTERA_TSE_CMD_ENA_10_MSK 0x02000000 > +#define ALTERA_TSE_CMD_TX_ENA_MSK BIT(0) > +#define ALTERA_TSE_CMD_RX_ENA_MSK BIT(1) > +#define ALTERA_TSE_CMD_ETH_SPEED_MSK BIT(3) > +#define ALTERA_TSE_CMD_HD_ENA_MSK BIT(10) > +#define ALTERA_TSE_CMD_SW_RESET_MSK BIT(13) > +#define ALTERA_TSE_CMD_ENA_10_MSK BIT(25) > > #define ALT_TSE_SW_RESET_TIMEOUT (3 * CONFIG_SYS_HZ) > #define ALT_TSE_SGDMA_BUSY_TIMEOUT (3 * CONFIG_SYS_HZ) >
diff --git a/drivers/net/altera_tse.h b/drivers/net/altera_tse.h index 0981308..471a880 100644 --- a/drivers/net/altera_tse.h +++ b/drivers/net/altera_tse.h @@ -14,11 +14,11 @@ #define __packed_1_ __packed __aligned(1) /* SGDMA Stuff */ -#define ALT_SGDMA_STATUS_BUSY_MSK 0x00000010 +#define ALT_SGDMA_STATUS_BUSY_MSK BIT(4) -#define ALT_SGDMA_CONTROL_RUN_MSK 0x00000020 -#define ALT_SGDMA_CONTROL_STOP_DMA_ER_MSK 0x00000040 -#define ALT_SGDMA_CONTROL_SOFTWARERESET_MSK 0x00010000 +#define ALT_SGDMA_CONTROL_RUN_MSK BIT(5) +#define ALT_SGDMA_CONTROL_STOP_DMA_ER_MSK BIT(6) +#define ALT_SGDMA_CONTROL_SOFTWARERESET_MSK BIT(16) /* * Descriptor control bit masks & offsets @@ -27,10 +27,10 @@ * The following bit-offsets are expressed relative to the LSB of * the control register bitfield. */ -#define ALT_SGDMA_DESCRIPTOR_CONTROL_GENERATE_EOP_MSK 0x00000001 -#define ALT_SGDMA_DESCRIPTOR_CONTROL_READ_FIXED_ADDRESS_MSK 0x00000002 -#define ALT_SGDMA_DESCRIPTOR_CONTROL_WRITE_FIXED_ADDRESS_MSK 0x00000004 -#define ALT_SGDMA_DESCRIPTOR_CONTROL_OWNED_BY_HW_MSK 0x00000080 +#define ALT_SGDMA_DESCRIPTOR_CONTROL_GENERATE_EOP_MSK BIT(0) +#define ALT_SGDMA_DESCRIPTOR_CONTROL_READ_FIXED_ADDRESS_MSK BIT(1) +#define ALT_SGDMA_DESCRIPTOR_CONTROL_WRITE_FIXED_ADDRESS_MSK BIT(2) +#define ALT_SGDMA_DESCRIPTOR_CONTROL_OWNED_BY_HW_MSK BIT(7) /* * Descriptor status bit masks & offsets @@ -39,7 +39,7 @@ * The following bit-offsets are expressed relative to the LSB of * the status register bitfield. */ -#define ALT_SGDMA_DESCRIPTOR_STATUS_TERMINATED_BY_EOP_MSK 0x00000080 +#define ALT_SGDMA_DESCRIPTOR_STATUS_TERMINATED_BY_EOP_MSK BIT(7) /* * The SGDMA controller buffer descriptor allocates @@ -85,12 +85,12 @@ struct alt_sgdma_registers { }; /* TSE Stuff */ -#define ALTERA_TSE_CMD_TX_ENA_MSK 0x00000001 -#define ALTERA_TSE_CMD_RX_ENA_MSK 0x00000002 -#define ALTERA_TSE_CMD_ETH_SPEED_MSK 0x00000008 -#define ALTERA_TSE_CMD_HD_ENA_MSK 0x00000400 -#define ALTERA_TSE_CMD_SW_RESET_MSK 0x00002000 -#define ALTERA_TSE_CMD_ENA_10_MSK 0x02000000 +#define ALTERA_TSE_CMD_TX_ENA_MSK BIT(0) +#define ALTERA_TSE_CMD_RX_ENA_MSK BIT(1) +#define ALTERA_TSE_CMD_ETH_SPEED_MSK BIT(3) +#define ALTERA_TSE_CMD_HD_ENA_MSK BIT(10) +#define ALTERA_TSE_CMD_SW_RESET_MSK BIT(13) +#define ALTERA_TSE_CMD_ENA_10_MSK BIT(25) #define ALT_TSE_SW_RESET_TIMEOUT (3 * CONFIG_SYS_HZ) #define ALT_TSE_SGDMA_BUSY_TIMEOUT (3 * CONFIG_SYS_HZ)