[U-Boot] [PATCH v4 06/24] spi: cadence_qspi_apb: Use BIT macro
Vikas MANOCHA
vikas.manocha at st.com
Fri Oct 23 19:11:20 CEST 2015
Thanks Jagan for this patch.
Acked-by: vikas.manocha at st.com
Rgds,
Vikas
> -----Original Message-----
> From: Jagan Teki [mailto:jteki at openedev.com]
> Sent: Thursday, October 22, 2015 1:50 PM
> To: u-boot at lists.denx.de
> Cc: Jagan Teki; Stefan Roese; Vikas MANOCHA; Marek Vasut
> Subject: [PATCH v4 06/24] spi: cadence_qspi_apb: Use BIT macro
>
> Replace numerical bit shift with BIT macro in cadence_qspi_apb
>
> :%s/(1 << nr)/BIT(nr)/g
> where nr = 0, 1, 2 .... 31
>
> Cc: Stefan Roese <sr at denx.de>
> Cc: Vikas Manocha <vikas.manocha at st.com>
> Cc: Marek Vasut <marex at denx.de>
> Signed-off-by: Jagan Teki <jteki at openedev.com>
> ---
> drivers/spi/cadence_qspi_apb.c | 28 ++++++++++++++--------------
> 1 file changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/spi/cadence_qspi_apb.c
> b/drivers/spi/cadence_qspi_apb.c index d053407..7786dd6 100644
> --- a/drivers/spi/cadence_qspi_apb.c
> +++ b/drivers/spi/cadence_qspi_apb.c
> @@ -58,10 +58,10 @@
> #define CQSPI_REG_CONFIG 0x00
> #define CQSPI_REG_CONFIG_CLK_POL_LSB 1
> #define CQSPI_REG_CONFIG_CLK_PHA_LSB 2
> -#define CQSPI_REG_CONFIG_ENABLE_MASK (1 << 0)
> -#define CQSPI_REG_CONFIG_DIRECT_MASK (1 << 7)
> -#define CQSPI_REG_CONFIG_DECODE_MASK (1 << 9)
> -#define CQSPI_REG_CONFIG_XIP_IMM_MASK (1 << 18)
> +#define CQSPI_REG_CONFIG_ENABLE_MASK BIT(0)
> +#define CQSPI_REG_CONFIG_DIRECT_MASK BIT(7)
> +#define CQSPI_REG_CONFIG_DECODE_MASK BIT(9)
> +#define CQSPI_REG_CONFIG_XIP_IMM_MASK BIT(18)
> #define CQSPI_REG_CONFIG_CHIPSELECT_LSB 10
> #define CQSPI_REG_CONFIG_BAUD_LSB 19
> #define CQSPI_REG_CONFIG_IDLE_LSB 31
> @@ -122,18 +122,18 @@
> #define CQSPI_REG_IRQMASK 0x44
>
> #define CQSPI_REG_INDIRECTRD 0x60
> -#define CQSPI_REG_INDIRECTRD_START_MASK (1 <<
> 0)
> -#define CQSPI_REG_INDIRECTRD_CANCEL_MASK (1 << 1)
> -#define CQSPI_REG_INDIRECTRD_INPROGRESS_MASK (1 << 2)
> -#define CQSPI_REG_INDIRECTRD_DONE_MASK (1 <<
> 5)
> +#define CQSPI_REG_INDIRECTRD_START_MASK BIT(0)
> +#define CQSPI_REG_INDIRECTRD_CANCEL_MASK BIT(1)
> +#define CQSPI_REG_INDIRECTRD_INPROGRESS_MASK BIT(2)
> +#define CQSPI_REG_INDIRECTRD_DONE_MASK BIT(5)
>
> #define CQSPI_REG_INDIRECTRDWATERMARK 0x64
> #define CQSPI_REG_INDIRECTRDSTARTADDR 0x68
> #define CQSPI_REG_INDIRECTRDBYTES 0x6C
>
> #define CQSPI_REG_CMDCTRL 0x90
> -#define CQSPI_REG_CMDCTRL_EXECUTE_MASK (1 <<
> 0)
> -#define CQSPI_REG_CMDCTRL_INPROGRESS_MASK (1 << 1)
> +#define CQSPI_REG_CMDCTRL_EXECUTE_MASK BIT(0)
> +#define CQSPI_REG_CMDCTRL_INPROGRESS_MASK BIT(1)
> #define CQSPI_REG_CMDCTRL_DUMMY_LSB 7
> #define CQSPI_REG_CMDCTRL_WR_BYTES_LSB 12
> #define CQSPI_REG_CMDCTRL_WR_EN_LSB 15
> @@ -149,10 +149,10 @@
> #define CQSPI_REG_CMDCTRL_OPCODE_MASK 0xFF
>
> #define CQSPI_REG_INDIRECTWR 0x70
> -#define CQSPI_REG_INDIRECTWR_START_MASK (1 <<
> 0)
> -#define CQSPI_REG_INDIRECTWR_CANCEL_MASK (1 << 1)
> -#define CQSPI_REG_INDIRECTWR_INPROGRESS_MASK (1 <<
> 2)
> -#define CQSPI_REG_INDIRECTWR_DONE_MASK (1 <<
> 5)
> +#define CQSPI_REG_INDIRECTWR_START_MASK BIT(0)
> +#define CQSPI_REG_INDIRECTWR_CANCEL_MASK BIT(1)
> +#define CQSPI_REG_INDIRECTWR_INPROGRESS_MASK BIT(2)
> +#define CQSPI_REG_INDIRECTWR_DONE_MASK BIT(5)
>
> #define CQSPI_REG_INDIRECTWRWATERMARK 0x74
> #define CQSPI_REG_INDIRECTWRSTARTADDR 0x78
> --
> 1.9.1
More information about the U-Boot
mailing list