[Skiboot] [PATCH 2/5] Change user-defined _MASK/_LSH to just mask

Dan Streetman ddstreet at ieee.org
Thu Feb 19 03:24:00 AEDT 2015


One thing I forgot to point out with this patch, the
SHPC_LOGICAL_SLOT_STATE_MASK value appeared to be set wrong; I
corrected it, as shown below...

On Tue, Feb 17, 2015 at 3:38 PM, Dan Streetman <ddstreet at ieee.org> wrote:
> The last patch changed the SETFIELD() and GETFIELD() macros to automatically
> calculate the shift of a given mask, so manually specifying the shift is no
> longer needed.  Additionally, any masks should have the _MASK suffix removed
> since the GETFIELD() and SETFIELD() operations expected to be passed the
> mask name without the _MASK suffix (and so either the mask name or the
> get/setfield call needs to have its mask name changed).
>
> Change all _MASK masks to remove the _MASK suffix, except for any places
> that leaving _MASK makes sense (e.g. already an existing define without
> _MASK suffix).
>
> Remove all _LSH defines, as they are no longer needed.
>
> Signed-off-by: Dan Streetman <ddstreet at ieee.org>
> ---
>  hw/chiptod.c          |   3 +-
>  hw/lpc.c              |  27 ++++-----
>  hw/p8-i2c.c           |  45 +++++----------
>  hw/xscom.c            |  10 ++--
>  include/gx.h          |  18 ++----
>  include/p5ioc2-regs.h |  50 ++++++----------
>  include/p7ioc-regs.h  | 134 +++++++++++++++----------------------------
>  include/pci-cfg.h     | 116 +++++++++++++------------------------
>  include/phb3-regs.h   | 156 +++++++++++++++++---------------------------------
>  include/processor.h   |  30 ++++------
>  include/psi.h         |   6 +-
>  11 files changed, 200 insertions(+), 395 deletions(-)
>
> diff --git a/hw/chiptod.c b/hw/chiptod.c
> index 1e3448b..fa4fc0b 100644
> --- a/hw/chiptod.c
> +++ b/hw/chiptod.c
> @@ -42,8 +42,7 @@
>  /* -- TOD PIB Master reg -- */
>  #define TOD_PIB_MASTER                 0x00040027
>  #define   TOD_PIBM_ADDR_CFG_MCAST      PPC_BIT(25)
> -#define   TOD_PIBM_ADDR_CFG_SLADDR_MASK        PPC_BITMASK(26,31)
> -#define   TOD_PIBM_ADDR_CFG_SLADDR_LSH PPC_BITLSHIFT(31)
> +#define   TOD_PIBM_ADDR_CFG_SLADDR     PPC_BITMASK(26,31)
>
>  /* -- TOD Error interrupt register -- */
>  #define TOD_ERROR                      0x00040030
> diff --git a/hw/lpc.c b/hw/lpc.c
> index b6d9a63..0db674f 100644
> --- a/hw/lpc.c
> +++ b/hw/lpc.c
> @@ -37,29 +37,22 @@ DEFINE_LOG_ENTRY(OPAL_RC_LPC_WRITE, OPAL_PLATFORM_ERR_EVT, OPAL_LPC,
>  #define ECCB_DATA      3 /* b0023 -> b00218 */
>
>  #define ECCB_CTL_MAGIC         0xd000000000000000ul
> -#define ECCB_CTL_DATASZ_MASK   PPC_BITMASK(4,7)
> -#define ECCB_CTL_DATASZ_LSH    PPC_BITLSHIFT(7)
> +#define ECCB_CTL_DATASZ                PPC_BITMASK(4,7)
>  #define ECCB_CTL_READ          PPC_BIT(15)
> -#define ECCB_CTL_ADDRLEN_MASK  PPC_BITMASK(23,25)
> -#define ECCB_CTL_ADDRLEN_LSH   PPC_BITLSHIFT(25)
> +#define ECCB_CTL_ADDRLEN       PPC_BITMASK(23,25)
>  #define        ECCB_ADDRLEN_4B 0x4
> -#define ECCB_CTL_ADDR_MASK     PPC_BITMASK(32,63)
> -#define ECCB_CTL_ADDR_LSH      0
> +#define ECCB_CTL_ADDR          PPC_BITMASK(32,63)
>
> -#define ECCB_STAT_PIB_ERR_MASK PPC_BITMASK(0,5)
> -#define ECCB_STAT_PIB_ERR_LSH  PPC_BITLSHIFT(5)
> -#define ECCB_STAT_RD_DATA_MASK PPC_BITMASK(6,37)
> -#define ECCB_STAT_RD_DATA_LSH  PPC_BITLSHIFT(37)
> +#define ECCB_STAT_PIB_ERR      PPC_BITMASK(0,5)
> +#define ECCB_STAT_RD_DATA      PPC_BITMASK(6,37)
>  #define ECCB_STAT_BUSY         PPC_BIT(44)
> -#define ECCB_STAT_ERRORS1_MASK PPC_BITMASK(45,51)
> -#define ECCB_STAT_ERRORS1_LSH  PPC_BITLSHIFT(51)
> +#define ECCB_STAT_ERRORS1      PPC_BITMASK(45,51)
>  #define ECCB_STAT_OP_DONE      PPC_BIT(52)
> -#define ECCB_STAT_ERRORS2_MASK PPC_BITMASK(53,55)
> -#define ECCB_STAT_ERRORS3_LSH  PPC_BITLSHIFT(55)
> +#define ECCB_STAT_ERRORS2      PPC_BITMASK(53,55)
>
> -#define ECCB_STAT_ERR_MASK     (ECCB_STAT_PIB_ERR_MASK | \
> -                                ECCB_STAT_ERRORS1_MASK | \
> -                                ECCB_STAT_ERRORS2_MASK)
> +#define ECCB_STAT_ERR_MASK     (ECCB_STAT_PIB_ERR | \
> +                                ECCB_STAT_ERRORS1 | \
> +                                ECCB_STAT_ERRORS2)
>
>  #define ECCB_TIMEOUT   1000000
>
> diff --git a/hw/p8-i2c.c b/hw/p8-i2c.c
> index 4068024..7ae0099 100644
> --- a/hw/p8-i2c.c
> +++ b/hw/p8-i2c.c
> @@ -66,8 +66,7 @@ DEFINE_LOG_ENTRY(OPAL_RC_I2C_RESET, OPAL_INPUT_OUTPUT_ERR_EVT, OPAL_I2C,
>
>  /* I2C FIFO register */
>  #define I2C_FIFO_REG                   0x4
> -#define I2C_FIFO_MASK                  PPC_BITMASK(0, 7)
> -#define I2C_FIFO_LSH                   PPC_BITLSHIFT(7)
> +#define I2C_FIFO                       PPC_BITMASK(0, 7)
>
>  /* I2C command register */
>  #define I2C_CMD_REG                    0x5
> @@ -75,19 +74,15 @@ DEFINE_LOG_ENTRY(OPAL_RC_I2C_RESET, OPAL_INPUT_OUTPUT_ERR_EVT, OPAL_I2C,
>  #define I2C_CMD_WITH_ADDR              PPC_BIT(1)
>  #define I2C_CMD_READ_CONT              PPC_BIT(2)
>  #define I2C_CMD_WITH_STOP              PPC_BIT(3)
> -#define I2C_CMD_DEV_ADDR_MASK          PPC_BITMASK(8, 14)
> -#define I2C_CMD_DEV_ADDR_LSH           PPC_BITLSHIFT(14)
> +#define I2C_CMD_DEV_ADDR               PPC_BITMASK(8, 14)
>  #define I2C_CMD_READ_NOT_WRITE         PPC_BIT(15)
> -#define I2C_CMD_LEN_BYTES_MASK         PPC_BITMASK(16, 31)
> -#define I2C_CMD_LEN_BYTES_LSH          PPC_BITLSHIFT(31)
> +#define I2C_CMD_LEN_BYTES              PPC_BITMASK(16, 31)
>  #define I2C_MAX_TFR_LEN                        0xfff0ull
>
>  /* I2C mode register */
>  #define I2C_MODE_REG                   0x6
> -#define I2C_MODE_BIT_RATE_DIV_MASK     PPC_BITMASK(0, 15)
> -#define I2C_MODE_BIT_RATE_DIV_LSH      PPC_BITLSHIFT(15)
> -#define I2C_MODE_PORT_NUM_MASK         PPC_BITMASK(16, 21)
> -#define I2C_MODE_PORT_NUM_LSH          PPC_BITLSHIFT(21)
> +#define I2C_MODE_BIT_RATE_DIV          PPC_BITMASK(0, 15)
> +#define I2C_MODE_PORT_NUM              PPC_BITMASK(16, 21)
>  #define I2C_MODE_ENHANCED              PPC_BIT(28)
>  #define I2C_MODE_DIAGNOSTIC            PPC_BIT(29)
>  #define I2C_MODE_PACING_ALLOW          PPC_BIT(30)
> @@ -95,17 +90,14 @@ DEFINE_LOG_ENTRY(OPAL_RC_I2C_RESET, OPAL_INPUT_OUTPUT_ERR_EVT, OPAL_I2C,
>
>  /* I2C watermark register */
>  #define I2C_WATERMARK_REG              0x7
> -#define I2C_WATERMARK_HIGH_MASK                PPC_BITMASK(16, 19)
> -#define I2C_WATERMARK_HIGH_LSH         PPC_BITLSHIFT(19)
> -#define I2C_WATERMARK_LOW_MASK         PPC_BITMASK(24, 27)
> -#define I2C_WATERMARK_LOW_LSH          PPC_BITLSHIFT(27)
> +#define I2C_WATERMARK_HIGH             PPC_BITMASK(16, 19)
> +#define I2C_WATERMARK_LOW              PPC_BITMASK(24, 27)
>
>  /* I2C interrupt mask, condition and interrupt registers */
>  #define I2C_INTR_MASK_REG              0x8
>  #define I2C_INTR_COND_REG              0x9
>  #define I2C_INTR_REG                   0xa
> -#define I2C_INTR_ALL_MASK              PPC_BITMASK(16, 31)
> -#define I2C_INTR_ALL_LSH               PPC_BITLSHIFT(31)
> +#define I2C_INTR_ALL                   PPC_BITMASK(16, 31)
>  #define I2C_INTR_INVALID_CMD           PPC_BIT(16)
>  #define I2C_INTR_LBUS_PARITY_ERR       PPC_BIT(17)
>  #define I2C_INTR_BKEND_OVERRUN_ERR     PPC_BIT(18)
> @@ -135,16 +127,14 @@ DEFINE_LOG_ENTRY(OPAL_RC_I2C_RESET, OPAL_INPUT_OUTPUT_ERR_EVT, OPAL_I2C,
>  #define I2C_STAT_DATA_REQ              PPC_BIT(6)
>  #define I2C_STAT_CMD_COMP              PPC_BIT(7)
>  #define I2C_STAT_STOP_ERR              PPC_BIT(8)
> -#define I2C_STAT_UPPER_THRS_MASK       PPC_BITMASK(9, 15)
> -#define I2C_STAT_UPPER_THRS_LSH                PPC_BITLSHIFT(15)
> +#define I2C_STAT_UPPER_THRS            PPC_BITMASK(9, 15)
>  #define I2C_STAT_ANY_I2C_INTR          PPC_BIT(16)
>  #define I2C_STAT_PORT_HISTORY_BUSY     PPC_BIT(19)
>  #define I2C_STAT_SCL_INPUT_LEVEL       PPC_BIT(20)
>  #define I2C_STAT_SDA_INPUT_LEVEL       PPC_BIT(21)
>  #define I2C_STAT_PORT_BUSY             PPC_BIT(22)
>  #define I2C_STAT_INTERFACE_BUSY         PPC_BIT(23)
> -#define I2C_STAT_FIFO_ENTRY_COUNT_MASK  PPC_BITMASK(24, 31)
> -#define I2C_STAT_FIFO_ENTRY_COUNT_LSH  PPC_BITLSHIFT(31)
> +#define I2C_STAT_FIFO_ENTRY_COUNT        PPC_BITMASK(24, 31)
>
>  #define I2C_STAT_ANY_ERR (I2C_STAT_INVALID_CMD | I2C_STAT_LBUS_PARITY_ERR | \
>                           I2C_STAT_BKEND_OVERRUN_ERR | \
> @@ -153,10 +143,8 @@ DEFINE_LOG_ENTRY(OPAL_RC_I2C_RESET, OPAL_INPUT_OUTPUT_ERR_EVT, OPAL_I2C,
>
>  /* I2C extended status register */
>  #define I2C_EXTD_STAT_REG              0xc
> -#define I2C_EXTD_STAT_FIFO_SIZE_MASK   PPC_BITMASK(0, 7)
> -#define I2C_EXTD_STAT_FIFO_SIZE_LSH    PPC_BITLSHIFT(7)
> -#define I2C_EXTD_STAT_MSM_CURSTATE_MASK PPC_BITMASK(11, 15)
> -#define I2C_EXTD_STAT_MSM_CURSTATE_LSH PPC_BITLSHIFT(15)
> +#define I2C_EXTD_STAT_FIFO_SIZE                PPC_BITMASK(0, 7)
> +#define I2C_EXTD_STAT_MSM_CURSTATE     PPC_BITMASK(11, 15)
>  #define I2C_EXTD_STAT_SCL_IN_SYNC      PPC_BIT(16)
>  #define I2C_EXTD_STAT_SDA_IN_SYNC      PPC_BIT(17)
>  #define I2C_EXTD_STAT_S_SCL            PPC_BIT(18)
> @@ -167,15 +155,12 @@ DEFINE_LOG_ENTRY(OPAL_RC_I2C_RESET, OPAL_INPUT_OUTPUT_ERR_EVT, OPAL_I2C,
>  #define I2C_EXTD_STAT_LOW_WATER                PPC_BIT(23)
>  #define I2C_EXTD_STAT_I2C_BUSY         PPC_BIT(24)
>  #define I2C_EXTD_STAT_SELF_BUSY                PPC_BIT(25)
> -#define I2C_EXTD_STAT_I2C_VERSION_MASK PPC_BITMASK(27, 31)
> -#define I2C_EXTD_STAT_I2C_VERSION_LSH  PPC_BITLSHIFT(31)
> +#define I2C_EXTD_STAT_I2C_VERSION      PPC_BITMASK(27, 31)
>
>  /* I2C residual front end/back end length */
>  #define I2C_RESIDUAL_LEN_REG           0xd
> -#define I2C_RESIDUAL_FRONT_END_MASK    PPC_BITMASK(0, 15)
> -#define I2C_RESIDUAL_FRONT_END_LSH     PPC_BITLSHIFT(15)
> -#define I2C_RESIDUAL_BACK_END_MASK     PPC_BITMASK(16, 31)
> -#define I2C_RESIDUAL_BACK_END_LSH      PPC_BITLSHIFT(31)
> +#define I2C_RESIDUAL_FRONT_END         PPC_BITMASK(0, 15)
> +#define I2C_RESIDUAL_BACK_END          PPC_BITMASK(16, 31)
>
>  /* Port busy register */
>  #define I2C_PORT_BUYS_REG              0xe
> diff --git a/hw/xscom.c b/hw/xscom.c
> index 33c04b1..471d057 100644
> --- a/hw/xscom.c
> +++ b/hw/xscom.c
> @@ -30,15 +30,13 @@
>                            SPR_HMER_XSCOM_STATUS_MASK))
>
>  #define XSCOM_ADDR_IND_FLAG            PPC_BIT(0)
> -#define XSCOM_ADDR_IND_ADDR_MASK       PPC_BITMASK(12,31)
> -#define XSCOM_ADDR_IND_ADDR_LSH                PPC_BITLSHIFT(31)
> -#define XSCOM_ADDR_IND_DATA_MSK                PPC_BITMASK(48,63)
> +#define XSCOM_ADDR_IND_ADDR            PPC_BITMASK(12,31)
> +#define XSCOM_ADDR_IND_DATA            PPC_BITMASK(48,63)
>
>  #define XSCOM_DATA_IND_READ            PPC_BIT(0)
>  #define XSCOM_DATA_IND_COMPLETE                PPC_BIT(32)
> -#define XSCOM_DATA_IND_ERR_MASK                PPC_BITMASK(33,35)
> -#define XSCOM_DATA_IND_ERR_LSH         PPC_BITLSHIFT(35)
> -#define XSCOM_DATA_IND_DATA_MSK                PPC_BITMASK(48,63)
> +#define XSCOM_DATA_IND_ERR             PPC_BITMASK(33,35)
> +#define XSCOM_DATA_IND_DATA            PPC_BITMASK(48,63)
>
>  /* HB folks say: try 10 time for now */
>  #define XSCOM_IND_MAX_RETRIES          10
> diff --git a/include/gx.h b/include/gx.h
> index f633892..4ba1661 100644
> --- a/include/gx.h
> +++ b/include/gx.h
> @@ -24,30 +24,24 @@
>
>  /* P7 GX Mode 1 register (contains PSI BUID) */
>  #define GX_P7_MODE1_REG                0x0201180A
> -#define GX_P7_MODE1_PSI_BUID_MASK      PPC_BITMASK(18,26)
> -#define GX_P7_MODE1_PSI_BUID_LSH       PPC_BITLSHIFT(26)
> +#define GX_P7_MODE1_PSI_BUID           PPC_BITMASK(18,26)
>  #define GX_P7_MODE1_PSI_BUID_DISABLE   PPC_BIT(27)
>
>  /* P7+ GX Mode 4 register (PSI and NX BUIDs ) */
>  #define GX_P7P_MODE4_REG       0x02011811
>  #define GX_P7P_MODE4_ENABLE_NX_BUID    PPC_BIT(0)
> -#define GX_P7P_MODE4_NX_BUID_BASE_MASK PPC_BITMASK(1,9)
> -#define GX_P7P_MODE4_NX_BUID_BASE_LSH  PPC_BITLSHIFT(9)
> -#define GX_P7P_MODE4_NX_BUID_MASK_MASK PPC_BITMASK(10,18)
> -#define GX_P7P_MODE4_NX_BUID_MASK_LSH  PPC_BITLSHIFT(18)
> -#define GX_P7P_MODE4_PSI_BUID_MASK     PPC_BITMASK(19,27)
> -#define GX_P7P_MODE4_PSI_BUID_LSH      PPC_BITLSHIFT(27)
> +#define GX_P7P_MODE4_NX_BUID_BASE      PPC_BITMASK(1,9)
> +#define GX_P7P_MODE4_NX_BUID_MASK      PPC_BITMASK(10,18)
> +#define GX_P7P_MODE4_PSI_BUID          PPC_BITMASK(19,27)
>  #define GX_P7P_MODE4_PSI_BUID_DISABLE  PPC_BIT(28)
>
>  /* P7 GX TCE BAR and mask */
>  #define GX_P7_GX0_TCE_BAR      0x02011845
> -#define GX_P7_TCE_BAR_ADDR_MASK                PPC_BITMASK(0,25)
> -#define GX_P7_TCE_BAR_ADDR_LSH         PPC_BITLSHIFT(25)
> +#define GX_P7_TCE_BAR_ADDR             PPC_BITMASK(0,25)
>  #define GX_P7_TCE_BAR_ADDR_SHIFT       PPC_BITLSHIFT(43)
>  #define GX_P7_TCE_BAR_ENABLE           PPC_BIT(26)
>  #define GX_P7_GX0_TCE_MASK     0x0201184B
> -#define GX_P7_TCE_MASK_MASK            PPC_BITMASK(0,25)
> -#define GX_P7_TCE_MASK_LSH             PPC_BITLSHIFT(25)
> +#define GX_P7_TCE_MASK                 PPC_BITMASK(0,25)
>  #define GX_P7_GX1_TCE_BAR      0x02011885
>  #define GX_P7_GX1_TCE_MASK     0x0201188B
>
> diff --git a/include/p5ioc2-regs.h b/include/p5ioc2-regs.h
> index 4cfd2e7..c4a63c6 100644
> --- a/include/p5ioc2-regs.h
> +++ b/include/p5ioc2-regs.h
> @@ -65,10 +65,8 @@
>  #define P5IOC2_IBASE           0x0048  /* Interrupt base address */
>  #define P5IOC2_IRBM            0x00d8  /* Interrupt re-issue broadcast mask */
>  #define P5IOC2_SID             0x0038  /* P5IOC2 ID register */
> -#define   P5IOC2_SID_BUID_BASE_MASK    PPC_BITMASK(14,22)
> -#define   P5IOC2_SID_BUID_BASE_LSH     PPC_BITLSHIFT(22)
> -#define   P5IOC2_SID_BUID_MASK_MASK    PPC_BITMASK(27,30)
> -#define   P5IOC2_SID_BUID_MASK_LSH     PPC_BITLSHIFT(30)
> +#define   P5IOC2_SID_BUID_BASE PPC_BITMASK(14,22)
> +#define   P5IOC2_SID_BUID_MASK PPC_BITMASK(27,30)
>  #define P5IOC2_SBUID           0x00f8  /* P5IOC2 HUB BUID */
>
>  /* XIPM area */
> @@ -90,17 +88,12 @@
>  #define CA_CCR                         0x108
>  #define CA_DEVBUID                     0x118
>  #define   CA_DEVBUID_MASK              PPC_BITMASK32(7,15)
> -#define   CA_DEVBUID_LSH               PPC_BITLSHIFT32(15)
>  #define CA_TAR0                                0x580
> -#define   CA_TAR_HUBID_MASK            PPC_BITMASK(0,5)
> -#define   CA_TAR_HUBID_LSH             PPC_BITLSHIFT(5)
> -#define   CA_TAR_ALTHUBID_MASK         PPC_BITMASK(6,11)
> -#define   CA_TAR_ALTHUBID_LSH          PPC_BITLSHIFT(11)
> -#define   CA_TAR_TCE_ADDR_MASK         PPC_BITMASK(16,48)
> -#define   CA_TAR_TCE_ADDR_LSH          PPC_BITLSHIFT(48)
> +#define   CA_TAR_HUBID                 PPC_BITMASK(0,5)
> +#define   CA_TAR_ALTHUBID              PPC_BITMASK(6,11)
> +#define   CA_TAR_TCE_ADDR              PPC_BITMASK(16,48)
>  #define   CA_TAR_VALID                 PPC_BIT(60)
> -#define   CA_TAR_NUM_TCE_MASK          PPC_BITMASK(61,63)
> -#define   CA_TAR_NUM_TCE_LSH           PPC_BITLSHIFT(63)
> +#define   CA_TAR_NUM_TCE               PPC_BITMASK(61,63)
>  #define CA_TAR1                                0x588
>  #define CA_TAR2                                0x590
>  #define CA_TAR3                                0x598
> @@ -109,8 +102,7 @@
>  #define CA_PHBID0                      0x650
>  #define   CA_PHBID_PHB_ENABLE          PPC_BIT32(0)
>  #define   CA_PHBID_ADDRSPACE_ENABLE    PPC_BIT32(1)
> -#define   CA_PHBID_PHB_TYPE_MASK       PPC_BITMASK32(4,7)
> -#define   CA_PHBID_PHB_TYPE_LSH                PPC_BITLSHIFT32(7)
> +#define   CA_PHBID_PHB_TYPE            PPC_BITMASK32(4,7)
>  #define     CA_PHBTYPE_PCIX1_0 0
>  #define     CA_PHBTYPE_PCIX2_0 1
>  #define     CA_PHBTYPE_PCIE_G1 4
> @@ -120,8 +112,7 @@
>  #define   CA_PHBID_XBUS_64BIT          PPC_BIT32(9)
>  #define   CA_PHBID_XBUS_266MHZ         PPC_BIT32(10)
>  /* PCI-E bits */
> -#define   CA_PHBID_EWIDTH_MASK         PPC_BITMASK32(8,10)
> -#define   CA_PHBID_EWIDTH_LSH          PPC_BITLSHIFT32(10)
> +#define   CA_PHBID_EWIDTH              PPC_BITMASK32(8,10)
>  #define     CA_PHB_EWIDTH_X4   0
>  #define     CA_PHB_EWIDTH_X8   1
>  #define     CA_PHB_EWIDTH_X16  2
> @@ -138,16 +129,13 @@
>   */
>  #define CAP_BUID                       0x100
>  #define   CAP_BUID_MASK                        PPC_BITMASK32(7,15)
> -#define   CAP_BUID_LSH                 PPC_BITLSHIFT32(15)
>  #define CAP_MSIBASE                    0x108 /* Undocumented ! */
>  #define CAP_DMACSR                     0x110
>  #define CAP_PLSSR                      0x120
>  #define CAP_PCADR                      0x140
>  #define   CAP_PCADR_ENABLE             PPC_BIT32(0)
> -#define   CAP_PCADR_FUNC_MASK          PPC_BITMASK32(21,23)
> -#define   CAP_PCADR_FUNC_LSH           PPC_BITLSHIFT32(23)
> -#define   CAP_PCADR_EXTOFF_MASK                PPC_BITLSHIFT32(4,7)
> -#define   CAP_PCADR_EXTOFF_LSH         PPC_BITLSHIFT32(7)
> +#define   CAP_PCADR_FUNC               PPC_BITMASK32(21,23)
> +#define   CAP_PCADR_EXTOFF             PPC_BITMASK32(4,7)
>  #define CAP_PCDAT                      0x130
>  #define CAP_PCFGRW                     0x160
>  #define   CAP_PCFGRW_ERR_RECOV_EN      PPC_BIT32(1)
> @@ -177,17 +165,13 @@
>  #define   CAP_CRR_RESET1               PPC_BIT32(0)
>  #define   CAP_CRR_RESET2               PPC_BIT32(1)
>  #define CAP_XIVR0                      0x400
> -#define   CAP_XIVR_PRIO_MASK           0x000000ff
> -#define   CAP_XIVR_PRIO_LSH            0
> -#define   CAP_XIVR_SERVER_MASK         0x0000ff00
> -#define   CAP_XIVR_SERVER_LSH          8
> +#define   CAP_XIVR_PRIO                        0x000000ff
> +#define   CAP_XIVR_SERVER              0x0000ff00
>  #define CAP_XIVRn(n)                   (0x400 + ((n) << 4))
>  #define CAP_MVE0                       0x500
>  #define   CAP_MVE_VALID                        PPC_BIT32(0)
> -#define   CAP_MVE_TBL_OFF_MASK         PPC_BITMASK32(13,15)
> -#define   CAP_MVE_TBL_OFF_LSH          PPC_BITLSHIFT32(15)
> -#define   CAP_MVE_NUM_INT_MASK         PPC_BITMASK32(18,19)
> -#define   CAP_MVE_NUM_INT_LSH          PPC_BITLSHIFT32(19)
> +#define   CAP_MVE_TBL_OFF              PPC_BITMASK32(13,15)
> +#define   CAP_MVE_NUM_INT              PPC_BITMASK32(18,19)
>  #define CAP_MVE1                       0x510
>  #define CAP_MODE0                      0x880
>  #define CAP_MODE1                      0x890
> @@ -198,13 +182,11 @@
>   * SHPC Registers
>   */
>  #define SHPC_LOGICAL_SLOT              0xb40
> -#define SHPC_LOGICAL_SLOT_STATE_MASK   0x00000002
> -#define SHPC_LOGICAL_SLOT_STATE_LSH    0
> +#define SHPC_LOGICAL_SLOT_STATE                0x00000003

The mask and lsh don't agree; a mask of 2 should have a lsh of 1.
Since the actual values below are 1-3, I'm assuming the lsh was
correct and the mask should be 3, so i corrected it above.  In any
case, a git grep shows this define isn't actually used anywhere in the
code, so this shouldn't currently make any difference.

>  #define   SHPC_SLOT_STATE_POWER_ONLY   1
>  #define   SHPC_SLOT_STATE_ENABLED      2
>  #define   SHPC_SLOT_STATE_DISABLED     3
> -#define SHPC_LOGICAL_SLOT_PRSNT_MASK   0x000000c00
> -#define SHPC_LOGICAL_SLOT_PRSNT_LSH    10
> +#define SHPC_LOGICAL_SLOT_PRSNT                0x000000c00
>  #define   SHPC_SLOT_PRSTN_7_5W         0
>  #define   SHPC_SLOT_PRSTN_25W          1
>  #define   SHPC_SLOT_STATE_15W          2
> diff --git a/include/p7ioc-regs.h b/include/p7ioc-regs.h
> index 2cd5b75..e868188 100644
> --- a/include/p7ioc-regs.h
> +++ b/include/p7ioc-regs.h
> @@ -114,24 +114,18 @@
>
>  /* PHB Fundamental register set A */
>  #define PHB_BUID                       0x100
> -#define   PHB_BUID_LSI_MASK            PPC_BITMASK(7,15)
> -#define   PHB_BUID_LSI_LSH             PPC_BITLSHIFT(15)
> -#define   PHB_BUID_MSI_MASK            PPC_BITMASK(23,31)
> -#define   PHB_BUID_MSI_LSH             PPC_BITLSHIFT(31)
> +#define   PHB_BUID_LSI                 PPC_BITMASK(7,15)
> +#define   PHB_BUID_MSI                 PPC_BITMASK(23,31)
>  #define PHB_DMA_CHAN_STATUS            0x110
>  #define PHB_CPU_LOADSTORE_STATUS       0x120
>  #define PHB_CONFIG_DATA                        0x130
>  #define PHB_LOCK0                      0x138
>  #define PHB_CONFIG_ADDRESS             0x140
>  #define   PHB_CA_ENABLE                        PPC_BIT(0)
> -#define          PHB_CA_BUS_MASK               PPC_BITMASK(4,11)
> -#define   PHB_CA_BUS_LSH               PPC_BITLSHIFT(11)
> -#define   PHB_CA_DEV_MASK              PPC_BITMASK(12,16)
> -#define   PHB_CA_DEV_LSH               PPC_BITLSHIFT(16)
> -#define   PHB_CA_FUNC_MASK             PPC_BITMASK(17,19)
> -#define   PHB_CA_FUNC_LSH              PPC_BITLSHIFT(19)
> -#define   PHB_CA_REG_MASK              PPC_BITMASK(20,31)
> -#define   PHB_CA_REG_LSH               PPC_BITLSHIFT(31)
> +#define          PHB_CA_BUS                    PPC_BITMASK(4,11)
> +#define   PHB_CA_DEV                   PPC_BITMASK(12,16)
> +#define   PHB_CA_FUNC                  PPC_BITMASK(17,19)
> +#define   PHB_CA_REG                   PPC_BITMASK(20,31)
>  #define PHB_LOCK1                      0x148
>  #define PHB_PHB2_CONFIG                        0x160
>  #define   PHB_PHB2C_64B_TCE_EN         PPC_BIT(2)
> @@ -148,14 +142,12 @@
>  #define PHB_M64_UPPER_BITS             0x1f0
>  #define PHB_TCE_KILL                   0x210
>  #define   PHB_TCEKILL_PAIR             PPC_BIT(0)
> -#define   PHB_TCEKILL_ADDR_MASK                PPC_BITMASK(16,59)
> +#define   PHB_TCEKILL_ADDR             PPC_BITMASK(16,59)
>  #define PHB_TCE_PREFETCH               0x218
>  #define PHB_IODA_ADDR                  0x220
>  #define   PHB_IODA_AD_AUTOINC          PPC_BIT(0)
> -#define          PHB_IODA_AD_TSEL_MASK         PPC_BITMASK(11,15)
> -#define          PHB_IODA_AD_TSEL_LSH          PPC_BITLSHIFT(15)
> -#define          PHB_IODA_AD_TADR_MASK         PPC_BITMASK(48,63)
> -#define          PHB_IODA_AD_TADR_LSH          PPC_BITLSHIFT(63)
> +#define          PHB_IODA_AD_TSEL              PPC_BITMASK(11,15)
> +#define          PHB_IODA_AD_TADR              PPC_BITMASK(48,63)
>  #define PHB_IODA_DATA0                 0x228
>  #define PHB_IODA_DATA1                 0x230
>  #define PHB_LOCK2                      0x240
> @@ -174,14 +166,10 @@
>  #define   PHB_PAPR_ERR_INJ_CTL_FREEZE  PPC_BIT(6)
>  #define PHB_PAPR_ERR_INJ_ADDR          0x2b8
>  #define PHB_PAPR_ERR_INJ_MASK          0x2c0
> -#define   PHB_PAPR_ERR_INJ_MASK_CFG_MASK       PPC_BITMASK(4,11)
> -#define   PHB_PAPR_ERR_INJ_MASK_CFG_LSH                PPC_BITLSHIFT(11)
> -#define   PHB_PAPR_ERR_INJ_MASK_MMIO_MASK      PPC_BITMASK(16,39)      /* 16M aligned */
> -#define   PHB_PAPR_ERR_INJ_MASK_MMIO_LSH       PPC_BITLSHIFT(39)
> -#define   PHB_PAPR_ERR_INJ_MASK_IO_MASK                PPC_BITMASK(16,47)      /* 64K aligned */
> -#define   PHB_PAPR_ERR_INJ_MASK_IO_LSH         PPC_BITLSHIFT(47)
> -#define   PHB_PAPR_ERR_INJ_MASK_DMA_MASK       PPC_BITMASK(60,63)      /* 16 window */
> -#define   PHB_PAPR_ERR_INJ_MASK_DMA_LSH                PPC_BITLSHIFT(63)
> +#define   PHB_PAPR_ERR_INJ_MASK_CFG            PPC_BITMASK(4,11)
> +#define   PHB_PAPR_ERR_INJ_MASK_MMIO           PPC_BITMASK(16,39)      /* 16M aligned */
> +#define   PHB_PAPR_ERR_INJ_MASK_IO             PPC_BITMASK(16,47)      /* 64K aligned */
> +#define   PHB_PAPR_ERR_INJ_MASK_DMA            PPC_BITMASK(60,63)      /* 16 window */
>  #define PHB_ETU_ERR_SUMMARY            0x2c8
>
>  /*  UTL registers */
> @@ -235,10 +223,8 @@
>  #define   PHB_PCIE_SLOTCTL2_PWR_EN_STAT        PPC_BIT(17)
>  #define   PHB_PCIE_SLOTCTL2_RCK_EN_STAT        PPC_BIT(18)
>  #define   PHB_PCIE_SLOTCTL2_PERST_STAT PPC_BIT(19)
> -#define   PHB_PCIE_SLOTCTL2_PLED_S_MASK        PPC_BITMASK(20,21)
> -#define   PHB_PCIE_SLOTCTL2_PLED_S_LSH PPC_BITLSHIFT(21) /* use PCIE_INDIC_* */
> -#define   PHB_PCIE_SLOTCTL2_ALED_S_MASK        PPC_BITMASK(22,23)
> -#define   PHB_PCIE_SLOTCTL2_ALED_S_LSH PPC_BITLSHIFT(23)
> +#define   PHB_PCIE_SLOTCTL2_PLED_S     PPC_BITMASK(20,21) /* use PCIE_INDIC_* */
> +#define   PHB_PCIE_SLOTCTL2_ALED_S     PPC_BITMASK(22,23)
>  #define   PHB_PCIE_SLOTCTL2_PRSTN_STAT PPC_BIT(24)
>  #define   PHB_PCIE_SLOTCTL2_PWRFLT_STAT        PPC_BIT(25)
>  #define PHB_PCIE_UTL_CONFIG            0x670
> @@ -366,35 +352,24 @@
>  #define IODA_TBL_PEEV          20
>
>  /* L/M XIVT */
> -#define IODA_XIVT_SERVER_MASK          PPC_BITMASK(8,23)
> -#define IODA_XIVT_SERVER_LSH           PPC_BITLSHIFT(23)
> -#define IODA_XIVT_PRIORITY_MASK                PPC_BITMASK(24,31)
> -#define IODA_XIVT_PRIORITY_LSH         PPC_BITLSHIFT(31)
> -#define IODA_XIVT_PENUM_MASK           PPC_BITMASK(41,47)
> -#define IODA_XIVT_PENUM_LSH            PPC_BITLSHIFT(47)
> -#define IODA_XIVT_HUBNUM_MASK          PPC_BITMASK(58,59)
> -#define IODA_XIVT_HUBNUM_LSH           PPC_BITLSHIFT(59)
> +#define IODA_XIVT_SERVER               PPC_BITMASK(8,23)
> +#define IODA_XIVT_PRIORITY             PPC_BITMASK(24,31)
> +#define IODA_XIVT_PENUM                        PPC_BITMASK(41,47)
> +#define IODA_XIVT_HUBNUM               PPC_BITMASK(58,59)
>
>  /* M64BT */
>  #define IODA_M64BT_ENABLE              PPC_BIT(0)
> -#define IODA_M64BT_BASE_MASK           PPC_BITMASK(8,31)
> -#define IODA_M64BT_BASE_LSH            PPC_BITLSHIFT(31)
> -#define IODA_M64BT_MASK_MASK           PPC_BITMASK(40,63)
> -#define IODA_M64BT_MASK_LSH            PPC_BITLSHIFT(63)
> +#define IODA_M64BT_BASE                        PPC_BITMASK(8,31)
> +#define IODA_M64BT_MASK                        PPC_BITMASK(40,63)
>
>  /* IODT/M32DT/M64DX */
> -#define IODA_XXDT_PE_MASK              PPC_BITMASK(0,6)
> -#define IODA_XXDT_PE_LSH               PPC_BITLSHIFT(6)
> +#define IODA_XXDT_PE                   PPC_BITMASK(0,6)
>
>  /* PELTM */
> -#define IODA_PELTM_BUS_MASK            PPC_BITMASK(0,7)
> -#define IODA_PELTM_BUS_LSH             PPC_BITLSHIFT(7)
> -#define IODA_PELTM_DEV_MASK            PPC_BITMASK(8,12)
> -#define IODA_PELTM_DEV_LSH             PPC_BITLSHIFT(12)
> -#define IODA_PELTM_FUNC_MASK           PPC_BITMASK(13,15)
> -#define IODA_PELTM_FUNC_LSH            PPC_BITLSHIFT(15)
> -#define IODA_PELTM_BUS_VALID_MASK      PPC_BITMASK(16,18)
> -#define IODA_PELTM_BUS_VALID_LSH       PPC_BITLSHIFT(18)
> +#define IODA_PELTM_BUS                 PPC_BITMASK(0,7)
> +#define IODA_PELTM_DEV                 PPC_BITMASK(8,12)
> +#define IODA_PELTM_FUNC                        PPC_BITMASK(13,15)
> +#define IODA_PELTM_BUS_VALID           PPC_BITMASK(16,18)
>  #define  IODA_BUS_VALID_ANY            0
>  #define  IODA_BUS_VALID_3_BITS         2
>  #define  IODA_BUS_VALID_4_BITS         3
> @@ -406,49 +381,34 @@
>  #define IODA_PELTM_FUNC_VALID          PPC_BIT(20)
>
>  /* TVT */
> -#define IODA_TVT0_TABLE_ADDR_MASK      PPC_BITMASK(0,47)
> -#define IODA_TVT0_TABLE_ADDR_LSH       PPC_BITLSHIFT(47)
> -#define IODA_TVT0_BUS_VALID_MASK       PPC_BITMASK(48,50)
> -#define IODA_TVT0_BUS_VALID_LSH                PPC_BITLSHIFT(50)
> -#define IODA_TVT0_TCE_TABLE_SIZE_MASK  PPC_BITMASK(51,55)
> -#define IODA_TVT0_TCE_TABLE_SIZE_LSH   PPC_BITLSHIFT(55)
> -#define IODA_TVT0_BUS_NUM_MASK         PPC_BITMASK(56,63)
> -#define IODA_TVT0_BUS_NUM_LSH          PPC_BITLSHIFT(63)
> +#define IODA_TVT0_TABLE_ADDR           PPC_BITMASK(0,47)
> +#define IODA_TVT0_BUS_VALID            PPC_BITMASK(48,50)
> +#define IODA_TVT0_TCE_TABLE_SIZE       PPC_BITMASK(51,55)
> +#define IODA_TVT0_BUS_NUM              PPC_BITMASK(56,63)
>  #define IODA_TVT1_DEV_VALID            PPC_BIT(2)
> -#define IODA_TVT1_DEV_NUM_MASK         PPC_BITMASK(3,7)
> -#define IODA_TVT1_DEV_NUM_LSH          PPC_BITLSHIFT(7)
> -#define IODA_TVT1_HUB_NUM_MASK         PPC_BITMASK(10,11)
> -#define IODA_TVT1_HUB_NUM_LSH          PPC_BITLSHIFT(11)
> +#define IODA_TVT1_DEV_NUM              PPC_BITMASK(3,7)
> +#define IODA_TVT1_HUB_NUM              PPC_BITMASK(10,11)
>  #define IODA_TVT1_FUNC_VALID           PPC_BIT(12)
> -#define IODA_TVT1_FUNC_NUM_MASK                PPC_BITMASK(13,15)
> -#define IODA_TVT1_FUNC_NUM_LSH         PPC_BITLSHIFT(15)
> -#define IODA_TVT1_IO_PSIZE_MASK                PPC_BITMASK(19,23)
> -#define IODA_TVT1_IO_PSIZE_LSH         PPC_BITLSHIFT(23)
> -#define IODA_TVT1_PE_NUM_MASK          PPC_BITMASK(57,63)
> -#define IODA_TVT1_PE_NUM_LSH           PPC_BITLSHIFT(63)
> +#define IODA_TVT1_FUNC_NUM             PPC_BITMASK(13,15)
> +#define IODA_TVT1_IO_PSIZE             PPC_BITMASK(19,23)
> +#define IODA_TVT1_PE_NUM               PPC_BITMASK(57,63)
>
>  /* MVT */
>  #define IODA_MVT_VALID                 PPC_BIT(0)
> -#define IODA_MVT_BUS_VALID_MASK                PPC_BITMASK(21,23)
> -#define IODA_MVT_BUS_VALID_LSH         PPC_BITLSHIFT(23)
> -#define IODA_MVT_BUS_NUM_MASK          PPC_BITMASK(24,31)
> -#define IODA_MVT_BUS_NUM_LSH           PPC_BITLSHIFT(31)
> -#define IODA_MVT_PE_NUM_MASK           PPC_BITMASK(41,47)
> -#define IODA_MVT_PE_NUM_LSH            PPC_BITLSHIFT(47)
> +#define IODA_MVT_BUS_VALID             PPC_BITMASK(21,23)
> +#define IODA_MVT_BUS_NUM               PPC_BITMASK(24,31)
> +#define IODA_MVT_PE_NUM                        PPC_BITMASK(41,47)
>  #define IODA_MVT_DEV_VALID             PPC_BIT(50)
> -#define IODA_MVT_DEV_NUM_MASK          PPC_BITMASK(51,55)
> -#define IODA_MVT_DEV_NUM_LSH           PPC_BITLSHIFT(55)
> +#define IODA_MVT_DEV_NUM               PPC_BITMASK(51,55)
>  #define IODA_MVT_FUNC_VALID            PPC_BIT(60)
> -#define IODA_MVT_FUNC_NUM_MASK         PPC_BITMASK(61,63)
> -#define IODA_MVT_FUNC_NUM_LSH          PPC_BITLSHIFT(63)
> +#define IODA_MVT_FUNC_NUM              PPC_BITMASK(61,63)
>
>  /* PESTA */
>  #define IODA_PESTA_MMIO_FROZEN         PPC_BIT(0)
>  #define IODA_PESTA_MMIO_CAUSE          PPC_BIT(2)
>  #define IODA_PESTA_CFG_READ            PPC_BIT(3)
>  #define IODA_PESTA_CFG_WRITE           PPC_BIT(4)
> -#define IODA_PESTA_TTYPE_MASK          PPC_BITMASK(5,7)
> -#define IODA_PESTA_TTYPE_LSH           PPC_BITLSHIFT(7)
> +#define IODA_PESTA_TTYPE               PPC_BITMASK(5,7)
>  #define   PESTA_TTYPE_DMA_WRITE                0
>  #define   PESTA_TTYPE_MSI              1
>  #define   PESTA_TTYPE_DMA_READ         2
> @@ -472,16 +432,12 @@
>  #define IODA_PESTA_TCE_ACCESS_FAULT    PPC_BIT(19)
>  #define IODA_PESTA_DMA_RESP_TIMEOUT    PPC_BIT(20)
>  #define IODA_PESTA_AIB_SIZE_INVALID    PPC_BIT(21)
> -#define IODA_PESTA_LEM_BIT_MASK                PPC_BITMASK(26,31)
> -#define IODA_PESTA_LEM_BIT_LSH         PPC_BITLSHIFT(31)
> -#define IODA_PESTA_RID_MASK            PPC_BITMASK(32,47)
> -#define IODA_PESTA_RID_LSH             PPC_BITLSHIFT(47)
> -#define IODA_PESTA_MSI_DATA_MASK       PPC_BITMASK(48,63)
> -#define IODA_PESTA_MSI_DATA_LSH                PPC_BITLSHIFT(63)
> +#define IODA_PESTA_LEM_BIT             PPC_BITMASK(26,31)
> +#define IODA_PESTA_RID                 PPC_BITMASK(32,47)
> +#define IODA_PESTA_MSI_DATA            PPC_BITMASK(48,63)
>
>  /* PESTB */
>  #define IODA_PESTB_DMA_STOPPED         PPC_BIT(0)
> -#define IODA_PESTB_FAIL_ADDR_MASK      PPC_BITMASK(3,63)
> -#define IODA_PESTB_FAIL_ADDR_LSH       PPC_BITLSHIFT(63)
> +#define IODA_PESTB_FAIL_ADDR           PPC_BITMASK(3,63)
>
>  #endif /* __P7IOC_REGS_H */
> diff --git a/include/pci-cfg.h b/include/pci-cfg.h
> index 7c98f3f..c705d25 100644
> --- a/include/pci-cfg.h
> +++ b/include/pci-cfg.h
> @@ -46,7 +46,7 @@
>  #define PCI_CFG_HDR_TYPE               0x000e
>  #define PCI_CFG_BIST                   0x000f
>  #define PCI_CFG_BAR0                   0x0010
> -#define   PCI_CFG_BAR_TYPE_MASK                  0x00000001
> +#define   PCI_CFG_BAR_TYPE               0x00000001
>  #define   PCI_CFG_BAR_TYPE_MEM           0x00000000
>  #define   PCI_CFG_BAR_TYPE_IO            0x00000001
>  #define   PCI_CFG_BAR_MEM64              0x00000004
> @@ -117,10 +117,8 @@
>  #define PCI_CFG_CAP_ID_EXP             0x10
>  /* PCI Express capability fields */
>  #define PCICAP_EXP_CAPABILITY_REG      0x02
> -#define   PCICAP_EXP_CAP_VERSION_MASK  0x000f
> -#define   PCICAP_EXP_CAP_VERSION_LSH   0
> -#define   PCICAP_EXP_CAP_TYPE_MASK     0x00f0
> -#define   PCICAP_EXP_CAP_TYPE_LSH      4
> +#define   PCICAP_EXP_CAP_VERSION       0x000f
> +#define   PCICAP_EXP_CAP_TYPE          0x00f0
>  #define     PCIE_TYPE_ENDPOINT         0x0
>  #define     PCIE_TYPE_LEGACY           0x1
>  #define     PCIE_TYPE_ROOT_PORT                0x4
> @@ -131,27 +129,23 @@
>  #define     PCIE_TYPE_RC_INTEGRATED    0x9
>  #define     PCIE_TYPE_RC_EVT_COLL      0xa
>  #define   PCICAP_EXP_CAP_SLOT          0x0100
> -#define   PCICAP_EXP_CAP_MSI_NUM_MASK  0x3e00
> -#define   PCICAP_EXP_CAP_MSI_NUM_LSH   9
> +#define   PCICAP_EXP_CAP_MSI_NUM       0x3e00
>  #define   PCICAP_EXP_CAP_TCS_ROUTING   0x4000
>  #define PCICAP_EXP_DEVCAP              0x04
> -#define   PCICAP_EXP_DEVCAP_MPSS_MASK  0x00000007
> -#define   PCICAP_EXP_DEVCAP_MPSS_LSH   0
> +#define   PCICAP_EXP_DEVCAP_MPSS       0x00000007
>  #define      PCIE_MPSS_128             0
>  #define      PCIE_MPSS_256             1
>  #define      PCIE_MPSS_512             2
>  #define      PCIE_MPSS_1024            3
>  #define      PCIE_MPSS_2048            4
>  #define      PCIE_MPSS_4096            5
> -#define   PCICAP_EXP_DEVCAP_PHANT_MASK 0x00000018
> -#define   PCICAP_EXP_DEVCAP_PHANT_LSH  3
> +#define   PCICAP_EXP_DEVCAP_PHANT      0x00000018
>  #define     PCIE_PHANTOM_NONE          0
>  #define     PCIE_PHANTOM_1MSB          1
>  #define     PCIE_PHANTOM_2MSB          2
>  #define     PCIE_PHANTOM_3MSB          3
>  #define   PCICAP_EXP_DEVCAP_EXTTAG     0x00000020
> -#define   PCICAP_EXP_DEVCAP_L0SL_MASK  0x000001c0
> -#define   PCICAP_EXP_DEVCAP_L0SL_LSH   6
> +#define   PCICAP_EXP_DEVCAP_L0SL       0x000001c0
>  #define     PCIE_L0SL_MAX_64NS         0
>  #define     PCIE_L0SL_MAX_128NS                1
>  #define     PCIE_L0SL_MAX_256NS                2
> @@ -160,8 +154,7 @@
>  #define     PCIE_L0SL_MAX_2US          5
>  #define     PCIE_L0SL_MAX_4US          6
>  #define     PCIE_L0SL_MAX_NO_LIMIT     7
> -#define   PCICAP_EXP_DEVCAP_L1L_MASK   0x00000e00
> -#define   PCICAP_EXP_DEVCAP_L1L_LSH    9
> +#define   PCICAP_EXP_DEVCAP_L1L                0x00000e00
>  #define     PCIE_L1L_MAX_1US           0
>  #define     PCIE_L1L_MAX_2US           1
>  #define     PCIE_L1L_MAX_4US           2
> @@ -171,10 +164,8 @@
>  #define     PCIE_L1L_MAX_64US          6
>  #define     PCIE_L1L_MAX_NO_LIMIT      7
>  #define   PCICAP_EXP_ROLE_BASED_ERR    0x00008000
> -#define   PCICAP_EXP_DEVCAP_PWRVAL_MASK        0x03fc0000
> -#define   PCICAP_EXP_DEVCAP_PWRVAL_LSH 18
> -#define   PCICAP_EXP_DEVCAP_PWRSCA_MASK        0x0c000000
> -#define   PCICAP_EXP_DEVCAP_PWRSCA_LSH 26
> +#define   PCICAP_EXP_DEVCAP_PWRVAL     0x03fc0000
> +#define   PCICAP_EXP_DEVCAP_PWRSCA     0x0c000000
>  #define     PCIE_SLOT_PWR_SCALE_1x     0
>  #define     PCIE_SLOT_PWR_SCALE_0d1x   1
>  #define     PCIE_SLOT_PWR_SCALE_0d01x  2
> @@ -186,8 +177,7 @@
>  #define   PCICAP_EXP_DEVCTL_FE_REPORT  0x0004
>  #define   PCICAP_EXP_DEVCTL_UR_REPORT  0x0008
>  #define   PCICAP_EXP_DEVCTL_RELAX_ORD  0x0010
> -#define   PCICAP_EXP_DEVCTL_MPS_MASK   0x00e0
> -#define   PCICAP_EXP_DEVCTL_MPS_LSH    5
> +#define   PCICAP_EXP_DEVCTL_MPS                0x00e0
>  #define     PCIE_MPS_128B              0
>  #define     PCIE_MPS_256B              1
>  #define     PCIE_MPS_512B              2
> @@ -198,8 +188,7 @@
>  #define          PCICAP_EXP_DEVCTL_PHANTOM     0x0200
>  #define          PCICAP_EXP_DEVCTL_AUX_POW_PM  0x0400
>  #define          PCICAP_EXP_DEVCTL_NO_SNOOP    0x0800
> -#define   PCICAP_EXP_DEVCTL_MRRS_MASK  0x7000
> -#define   PCICAP_EXP_DEVCTL_MRRS_LSH   12
> +#define   PCICAP_EXP_DEVCTL_MRRS       0x7000
>  #define     PCIE_MRSS_128B             0
>  #define     PCIE_MRSS_256B             1
>  #define     PCIE_MRSS_512B             2
> @@ -216,8 +205,7 @@
>  #define   PCICAP_EXP_DEVSTAT_AUX_POW   0x0010
>  #define   PCICAP_EXP_DEVSTAT_TPEND     0x0020
>  #define PCICAP_EXP_LCAP                        0x0c
> -#define   PCICAP_EXP_LCAP_MAXSPD_MASK  0x0000000f
> -#define   PCICAP_EXP_LCAP_MAXSPD_LSH   0
> +#define   PCICAP_EXP_LCAP_MAXSPD       0x0000000f
>  #define     PCIE_LSPEED_VECBIT_0       0x1
>  #define     PCIE_LSPEED_VECBIT_1       0x2
>  #define     PCIE_LSPEED_VECBIT_2       0x3
> @@ -225,8 +213,7 @@
>  #define     PCIE_LSPEED_VECBIT_4       0x5
>  #define     PCIE_LSPEED_VECBIT_5       0x6
>  #define     PCIE_LSPEED_VECBIT_6       0x7
> -#define   PCICAP_EXP_LCAP_MAXWDTH_MASK 0x000003f0
> -#define   PCICAP_EXP_LCAP_MAXWDTH_LSH  4
> +#define   PCICAP_EXP_LCAP_MAXWDTH      0x000003f0
>  #define     PCIE_LWIDTH_1X             1
>  #define     PCIE_LWIDTH_2X             2
>  #define     PCIE_LWIDTH_4X             4
> @@ -236,8 +223,7 @@
>  #define     PCIE_LWIDTH_32X            32
>  #define   PCICAP_EXP_LCAP_ASPM_L0S     0x00000400
>  #define   PCICAP_EXP_LCAP_ASPM_L1      0x00000800
> -#define   PCICAP_EXP_LCAP_L0S_EXLT_MASK        0x00007000
> -#define   PCICAP_EXP_LCAP_L0S_EXLT_LSH 12
> +#define   PCICAP_EXP_LCAP_L0S_EXLT     0x00007000
>  #define     PCIE_L0S_EXLT_LESS_64NS    0
>  #define     PCIE_L0S_EXLT_64NS_128NS   1
>  #define     PCIE_L0S_EXLT_128NS_256NS  2
> @@ -246,8 +232,7 @@
>  #define     PCIE_L0S_EXLT_1US_2US      5
>  #define     PCIE_L0S_EXLT_2US_4US      6
>  #define     PCIE_L0S_EXLT_MORE_4US     7
> -#define   PCICAP_EXP_LCAP_L1_EXLT_MASK 0x00038000
> -#define   PCICAP_EXP_LCAP_L1_EXLT_LSH  15
> +#define   PCICAP_EXP_LCAP_L1_EXLT      0x00038000
>  #define     PCIE_L1_EXLT_LESS_1US      0
>  #define     PCIE_L1_EXLT_1US_2US       1
>  #define     PCIE_L1_EXLT_2US_4US       2
> @@ -261,8 +246,7 @@
>  #define   PCICAP_EXP_LCAP_DL_ACT_REP   0x00100000
>  #define   PCICAP_EXP_LCAP_LNKBWDTH_NOTF        0x00200000
>  #define   PCICAP_EXP_LCAP_ASPM_OPT_CMPL        0x00400000
> -#define   PCICAP_EXP_LCAP_PORTNUM_MASK 0xff000000
> -#define   PCICAP_EXP_LCAP_PORTNUM_LSH  24
> +#define   PCICAP_EXP_LCAP_PORTNUM      0xff000000
>  #define PCICAP_EXP_LCTL                        0x10
>  #define   PCICAP_EXP_LCTL_ASPM_L0S     0x0001
>  #define   PCICAP_EXP_LCTL_ASPM_L1      0x0002
> @@ -276,10 +260,8 @@
>  #define   PCICAP_EXP_LCTL_LBWM_INT_EN  0x0400
>  #define   PCICAP_EXP_LCTL_LAWD_INT_EN  0x0800
>  #define PCICAP_EXP_LSTAT               0x12
> -#define   PCICAP_EXP_LSTAT_SPEED_MASK  0x000f
> -#define   PCICAP_EXP_LSTAT_SPEED_LSH   0      /* use PCIE_LSPEED_* consts */
> -#define   PCICAP_EXP_LSTAT_WIDTH_MASK  0x03f0
> -#define   PCICAP_EXP_LSTAT_WIDTH_LSH   4      /* use PCIE_LWIDTH_* consts */
> +#define   PCICAP_EXP_LSTAT_SPEED       0x000f /* use PCIE_LSPEED_* consts */
> +#define   PCICAP_EXP_LSTAT_WIDTH       0x03f0 /* use PCIE_LWIDTH_* consts */
>  #define   PCICAP_EXP_LSTAT_TRAINING    0x0800
>  #define   PCICAP_EXP_LSTAT_SLOTCLKCFG  0x1000
>  #define   PCICAP_EXP_LSTAT_DLLL_ACT    0x2000
> @@ -293,14 +275,11 @@
>  #define   PCICAP_EXP_SLOTCAP_PWRI      0x00000010
>  #define   PCICAP_EXP_SLOTCAP_HPLUG_SURP        0x00000020
>  #define   PCICAP_EXP_SLOTCAP_HPLUG_CAP 0x00000040
> -#define   PCICAP_EXP_SLOTCAP_SPLVA_MASK        0x00007f80
> -#define   PCICAP_EXP_SLOTCAP_SPLLVA_LSH        7
> -#define   PCICAP_EXP_SLOTCAP_SPLSC_MASK        0x00018000
> -#define   PCICAP_EXP_SLOTCAP_SPLSC_LSH 15
> +#define   PCICAP_EXP_SLOTCAP_SPLVA     0x00007f80
> +#define   PCICAP_EXP_SLOTCAP_SPLSC     0x00018000
>  #define   PCICAP_EXP_SLOTCAP_EIP       0x00020000
>  #define   PCICAP_EXP_SLOTCAP_NO_CMDCOMP        0x00040000
> -#define   PCICAP_EXP_SLOTCAP_PSLOT_MASK        0xfff80000
> -#define   PCICAP_EXP_SLOTCAP_PSLOT_LSH 19
> +#define   PCICAP_EXP_SLOTCAP_PSLOT     0xfff80000
>  #define PCICAP_EXP_SLOTCTL             0x18
>  #define   PCICAP_EXP_SLOTCTL_ATTNB     0x0001
>  #define   PCICAP_EXP_SLOTCTL_PFLT      0x0002
> @@ -308,13 +287,11 @@
>  #define   PCICAP_EXP_SLOTCTL_PDETECT   0x0008
>  #define   PCICAP_EXP_SLOTCTL_CMDCOMPINT        0x0010
>  #define   PCICAP_EXP_SLOTCTL_HPINT     0x0020
> -#define   PCICAP_EXP_SLOTCTL_ATTNI_MASK        0x00c0
> -#define   PCICAP_EXP_SLOTCTL_ATTNI_LSH 6
> +#define   PCICAP_EXP_SLOTCTL_ATTNI     0x00c0
>  #define     PCIE_INDIC_ON              1
>  #define     PCIE_INDIC_BLINK           2
>  #define     PCIE_INDIC_OFF             3
> -#define   PCICAP_EXP_SLOTCTL_PWRI_MASK 0x0300
> -#define   PCICAP_EXP_SLOTCTL_PWRI_LSH  8      /* Use PCIE_INDIC_* consts */
> +#define   PCICAP_EXP_SLOTCTL_PWRI      0x0300 /* Use PCIE_INDIC_* consts */
>  #define   PCICAP_EXP_SLOTCTL_PWRCTLR   0x0400
>  #define   PCICAP_EXP_SLOTCTL_EIC       0x0800
>  #define   PCICAP_EXP_SLOTCTL_DLLSTCHG  0x1000
> @@ -337,13 +314,11 @@
>  #define PCICAP_EXP_RCAP                        0x1e
>  #define   PCICAP_EXP_RCAP_CRS_VISIBLE  0x0001
>  #define PCICAP_EXP_RSTAT               0x20
> -#define   PCICAP_EXP_RSTAT_PME_RID_MASK        0x0000ffff
> -#define   PCICAP_EXP_RSTAT_PME_RID_LSH 0
> +#define   PCICAP_EXP_RSTAT_PME_RID     0x0000ffff
>  #define   PCICAP_EXP_RSTAT_PME_STATUS  0x00010000
>  #define   PCICAP_EXP_RSTAT_PME_PENDING 0x00020000
>  #define PCIECAP_EXP_DCAP2              0x24
> -#define   PCICAP_EXP_DCAP2_CMPTOUT_MASK        0x0000000f
> -#define   PCICAP_EXP_DCAP2_CMPTOUT_LSH 0
> +#define   PCICAP_EXP_DCAP2_CMPTOUT     0x0000000f
>  #define   PCICAP_EXP_DCAP2_CMPTOUT_DIS 0x00000010
>  #define   PCICAP_EXP_DCAP2_ARI_FWD     0x00000020
>  #define   PCICAP_EXP_DCAP2_ATOMIC_RTE  0x00000040
> @@ -358,15 +333,13 @@
>  #define   PCICAP_EXP_DCAP2_OBFF_WAKE   0x00080000
>  #define   PCICAP_EXP_DCAP2_EXTFMT      0x00100000
>  #define   PCICAP_EXP_DCAP2_EETLP_PFX   0x00200000
> -#define   PCICAP_EXP_DCAP2_MAXEETP_MASK        0x00c00000
> -#define   PCICAP_EXP_DCAP2_MAXEETP_LSH 22
> +#define   PCICAP_EXP_DCAP2_MAXEETP     0x00c00000
>  #define     PCIE_EETLPP_1              1
>  #define     PCIE_EETLPP_2              2
>  #define     PCIE_EETLPP_3              3
>  #define     PCIE_EETLPP_4              0
>  #define PCICAP_EXP_DCTL2               0x28
> -#define   PCICAP_EXP_DCTL2_CMPTOUT_MASK        0x000f
> -#define   PCICAP_EXP_DCTL2_CMPTOUT_LSH 0
> +#define   PCICAP_EXP_DCTL2_CMPTOUT     0x000f
>  #define   PCICAP_EXP_DCTL2_CMPTOUT_DIS 0x0010
>  #define   PCICAP_EXP_DCTL2_ARI_FWD     0x0020
>  #define   PCICAP_EXP_DCTL2_ATOMIC_REQ  0x0040
> @@ -374,8 +347,7 @@
>  #define   PCICAP_EXP_DCTL2_IDO_REQ     0x0100
>  #define   PCICAP_EXP_DCTL2_IDO_COMPL   0x0200
>  #define   PCICAP_EXP_DCTL2_LTR         0x0400
> -#define   PCICAP_EXP_DCTL2_OBFF_MASK   0x6000
> -#define   PCICAP_EXP_DCTL2_OBFF_LSH    13
> +#define   PCICAP_EXP_DCTL2_OBFF                0x6000
>  #define     PCIE_OBFF_MODE_DISABLED    0
>  #define     PCIE_OBFF_MODE_MSG_A       1
>  #define     PCIE_OBFF_MODE_MSG_B       2
> @@ -388,17 +360,14 @@
>  #define   PCICAP_EXP_LCAP2_SP_8d0GTs   0x00000008
>  #define   PCICAP_EXP_LCAP2_XLINK       0x00000100
>  #define PCICAP_EXP_LCTL2               0x30
> -#define   PCICAP_EXP_LCTL2_TLSPD_MASK  0x000f
> -#define   PCICAP_EXP_LCTL2_TLSPD_LSH   0     /* use PCIE_LSPEED_ consts */
> +#define   PCICAP_EXP_LCTL2_TLSPD       0x000f /* use PCIE_LSPEED_ consts */
>  #define   PCICAP_EXP_LCTL2_ENTER_COMPL 0x0010
>  #define   PCICAP_EXP_LCTL2_HWAUTSPDIS  0x0020
>  #define   PCICAP_EXP_LCTL2_SEL_DEEMPH  0x0040
> -#define   PCICAP_EXP_LCTL2_XMTMARG_MASK        0x0380
> -#define   PCICAP_EXP_LCTL2_XMTMARG_LSH 7
> +#define   PCICAP_EXP_LCTL2_XMTMARG     0x0380
>  #define   PCICAP_EXP_LCTL2_ENTER_MCOMPL        0x0400
>  #define   PCICAP_EXP_LCTL2_COMPL_SOS   0x0800
> -#define   PCICAP_EXP_LCTL2_CMPPDEM_MASK        0xf000
> -#define   PCICAP_EXP_LCTL2_CMPPDEM_LSH 12
> +#define   PCICAP_EXP_LCTL2_CMPPDEM     0xf000
>  #define PCICAP_EXP_LSTA2               0x32
>  #define   PCICAP_EXP_LSTA2_DEMPH_LVL   0x0001
>  #define   PCICAP_EXP_LSTA2_EQ_COMPLETE 0x0002
> @@ -414,12 +383,9 @@
>   * PCI-E Extended capabilties
>   */
>  #define PCI_CFG_ECAP_START             0x100
> -#define PCI_CFG_ECAP_ID_MASK           0x0000ffff
> -#define PCI_CFG_ECAP_ID_LSH            0
> -#define PCI_CFG_ECAP_VERS_MASK         0x000f0000
> -#define PCI_CFG_ECAP_VERS_LSH          16
> -#define PCI_CFG_ECAP_NEXT_MASK         0xfff00000
> -#define PCI_CFG_ECAP_NEXT_LSH          20
> +#define PCI_CFG_ECAP_ID                        0x0000ffff
> +#define PCI_CFG_ECAP_VERS              0x000f0000
> +#define PCI_CFG_ECAP_NEXT              0xfff00000
>
>  /* AER Ext. Capability */
>  #define PCIECAP_ID_AER                 0x0001
> @@ -484,8 +450,7 @@
>  #define   PCIECAP_AER_CE_MASK_CORTD_INTERNAL   0x00004000
>  #define   PCIECAP_AER_CE_MASK_HDR_LOG_OVFL     0x00008000
>  #define PCIECAP_AER_CAPCTL             0x18
> -#define   PCIECAP_AER_CAPCTL_FPTR_MASK 0x0000001f
> -#define   PCIECAP_AER_CAPCTL_FPTR_LSH  0
> +#define   PCIECAP_AER_CAPCTL_FPTR      0x0000001f
>  #define   PCIECAP_AER_CAPCTL_ECRCG_CAP 0x00000020
>  #define   PCIECAP_AER_CAPCTL_ECRCG_EN  0x00000040
>  #define   PCIECAP_AER_CAPCTL_ECRCC_CAP 0x00000080
> @@ -509,13 +474,10 @@
>  #define   PCIECAP_AER_RERR_F_UFATAL    0x00000010
>  #define   PCIECAP_AER_RERR_NFE         0x00000020
>  #define   PCIECAP_AER_RERR_FE          0x00000040
> -#define   PCIECAP_AER_RERR_MSINO_MASK  0xf8000000
> -#define   PCIECAP_AER_RERR_MSINO_LSH   27
> +#define   PCIECAP_AER_RERR_MSINO       0xf8000000
>  #define PCIECAP_AER_SRCID              0x34
> -#define   PCIECAP_AER_SRCID_CORR_MASK  0x0000ffff
> -#define   PCIECAP_AER_SRCID_CORR_LSH   0
> -#define   PCIECAP_AER_SRCID_FNF_MASK   0xffff0000
> -#define   PCIECAP_AER_SRCID_FNF_LSH    16
> +#define   PCIECAP_AER_SRCID_CORR       0x0000ffff
> +#define   PCIECAP_AER_SRCID_FNF                0xffff0000
>  #define PCIECAP_AER_TLP_PFX_LOG0       0x38
>  #define PCIECAP_AER_TLP_PFX_LOG1       0x3c
>  #define PCIECAP_AER_TLP_PFX_LOG2       0x40
> diff --git a/include/phb3-regs.h b/include/phb3-regs.h
> index 686a113..144c344 100644
> --- a/include/phb3-regs.h
> +++ b/include/phb3-regs.h
> @@ -24,8 +24,7 @@
>
>  /* PHB Fundamental register set A */
>  #define PHB_LSI_SOURCE_ID              0x100
> -#define   PHB_LSI_SRC_ID_MASK          PPC_BITMASK(5,12)
> -#define   PHB_LSI_SRC_ID_LSH           PPC_BITLSHIFT(12)
> +#define   PHB_LSI_SRC_ID               PPC_BITMASK(5,12)
>  #define PHB_DMA_CHAN_STATUS            0x110
>  #define   PHB_DMA_CHAN_ANY_ERR         PPC_BIT(27)
>  #define   PHB_DMA_CHAN_ANY_ERR1                PPC_BIT(28)
> @@ -36,33 +35,24 @@
>  #define   PHB_CPU_LS_ANY_FREEZE                PPC_BIT(29)
>  #define PHB_DMA_MSI_NODE_ID            0x128
>  #define   PHB_DMAMSI_NID_FIXED         PPC_BIT(0)
> -#define   PHB_DMAMSI_NID_MASK          PPC_BITMASK(24,31)
> -#define   PHB_DMAMSI_NID_LSH           PPC_BITLSHIFT(31)
> +#define   PHB_DMAMSI_NID               PPC_BITMASK(24,31)
>  #define PHB_CONFIG_DATA                        0x130
>  #define PHB_LOCK0                      0x138
>  #define PHB_CONFIG_ADDRESS             0x140
>  #define   PHB_CA_ENABLE                        PPC_BIT(0)
> -#define          PHB_CA_BUS_MASK               PPC_BITMASK(4,11)
> -#define   PHB_CA_BUS_LSH               PPC_BITLSHIFT(11)
> -#define   PHB_CA_DEV_MASK              PPC_BITMASK(12,16)
> -#define   PHB_CA_DEV_LSH               PPC_BITLSHIFT(16)
> -#define   PHB_CA_FUNC_MASK             PPC_BITMASK(17,19)
> -#define   PHB_CA_FUNC_LSH              PPC_BITLSHIFT(19)
> -#define   PHB_CA_REG_MASK              PPC_BITMASK(20,31)
> -#define   PHB_CA_REG_LSH               PPC_BITLSHIFT(31)
> -#define   PHB_CA_PE_MASK               PPC_BITMASK(40,47)
> -#define   PHB_CA_PE_LSH                        PPC_BITLSHIFT(47)
> +#define          PHB_CA_BUS                    PPC_BITMASK(4,11)
> +#define   PHB_CA_DEV                   PPC_BITMASK(12,16)
> +#define   PHB_CA_FUNC                  PPC_BITMASK(17,19)
> +#define   PHB_CA_REG                   PPC_BITMASK(20,31)
> +#define   PHB_CA_PE                    PPC_BITMASK(40,47)
>  #define PHB_LOCK1                      0x148
>  #define PHB_IVT_BAR                    0x150
>  #define   PHB_IVT_BAR_ENABLE           PPC_BIT(0)
> -#define   PHB_IVT_BASE_ADDRESS_MASK    PPC_BITMASK(14,48)
> -#define   PHB_IVT_BASE_ADDRESS_LSH     PPC_BITLSHIFT(48)
> -#define   PHB_IVT_LENGTH_MASK          PPC_BITMASK(52,63)
> -#define   PHB_IVT_LENGTH_ADDRESS_LSH   PPC_BITLSHIFT(63)
> +#define   PHB_IVT_BASE_ADDRESS         PPC_BITMASK(14,48)
> +#define   PHB_IVT_LENGTH               PPC_BITMASK(52,63)
>  #define PHB_RBA_BAR                    0x158
>  #define   PHB_RBA_BAR_ENABLE           PPC_BIT(0)
> -#define   PHB_RBA_BASE_ADDRESS_MASK    PPC_BITMASK(14,55)
> -#define   PHB_RBA_BASE_ADDRESS_LSH     PPC_BITLSHIFT(55)
> +#define   PHB_RBA_BASE_ADDRESS         PPC_BITMASK(14,55)
>  #define PHB_PHB3_CONFIG                        0x160
>  #define   PHB_PHB3C_64B_TCE_EN         PPC_BIT(2)
>  #define   PHB_PHB3C_32BIT_MSI_EN       PPC_BIT(8)
> @@ -70,40 +60,33 @@
>  #define   PHB_PHB3C_M32_EN             PPC_BIT(16)
>  #define PHB_RTT_BAR                    0x168
>  #define   PHB_RTT_BAR_ENABLE           PPC_BIT(0)
> -#define   PHB_RTT_BASE_ADDRESS_MASK    PPC_BITMASK(14,46)
> -#define   PHB_RTT_BASE_ADDRESS_LSH     PPC_BITLSHIFT(46)
> +#define   PHB_RTT_BASE_ADDRESS         PPC_BITMASK(14,46)
>  #define PHB_PELTV_BAR                  0x188
>  #define   PHB_PELTV_BAR_ENABLE         PPC_BIT(0)
> -#define   PHB_PELTV_BASE_ADDRESS_MASK  PPC_BITMASK(14,50)
> -#define   PHB_PELTV_BASE_ADDRESS_LSH   PPC_BITLSHIFT(50)
> +#define   PHB_PELTV_BASE_ADDRESS       PPC_BITMASK(14,50)
>  #define PHB_M32_BASE_ADDR              0x190
>  #define PHB_M32_BASE_MASK              0x198
>  #define PHB_M32_START_ADDR             0x1a0
>  #define PHB_PEST_BAR                   0x1a8
>  #define   PHB_PEST_BAR_ENABLE          PPC_BIT(0)
> -#define   PHB_PEST_BASE_ADDRESS_MASK   PPC_BITMASK(14,51)
> -#define   PHB_PEST_BASE_ADDRESS_LSH    PPC_BITLSHIFT(51)
> +#define   PHB_PEST_BASE_ADDRESS                PPC_BITMASK(14,51)
>  #define PHB_M64_UPPER_BITS             0x1f0
>  #define PHB_INTREP_TIMER               0x1f8
>  #define PHB_DMARD_SYNC                 0x200
>  #define PHB_RTC_INVALIDATE             0x208
>  #define   PHB_RTC_INVALIDATE_ALL       PPC_BIT(0)
> -#define   PHB_RTC_INVALIDATE_RID_MASK  PPC_BITMASK(16,31)
> -#define   PHB_RTC_INVALIDATE_RID_LSH   PPC_BITLSHIFT(31)
> +#define   PHB_RTC_INVALIDATE_RID       PPC_BITMASK(16,31)
>  #define PHB_TCE_KILL                   0x210
>  #define   PHB_TCE_KILL_ALL             PPC_BIT(0)
>  #define PHB_TCE_SPEC_CTL               0x218
>  #define PHB_IODA_ADDR                  0x220
>  #define   PHB_IODA_AD_AUTOINC          PPC_BIT(0)
> -#define          PHB_IODA_AD_TSEL_MASK         PPC_BITMASK(11,15)
> -#define          PHB_IODA_AD_TSEL_LSH          PPC_BITLSHIFT(15)
> -#define          PHB_IODA_AD_TADR_MASK         PPC_BITMASK(55,63)
> -#define          PHB_IODA_AD_TADR_LSH          PPC_BITLSHIFT(63)
> +#define          PHB_IODA_AD_TSEL              PPC_BITMASK(11,15)
> +#define          PHB_IODA_AD_TADR              PPC_BITMASK(55,63)
>  #define PHB_IODA_DATA0                 0x228
>  #define PHB_FFI_REQUEST                        0x238
>  #define   PHB_FFI_LOCK_CLEAR           PPC_BIT(3)
> -#define   PHB_FFI_REQUEST_ISN_MASK     PPC_BITMASK(49,59)
> -#define   PHB_FFI_REQUEST_ISN_LSH      PPC_BITLSHIFT(59)
> +#define   PHB_FFI_REQUEST_ISN          PPC_BITMASK(49,59)
>  #define PHB_FFI_LOCK                   0x240
>  #define PHB_XIVE_UPDATE                        0x248 /* Broken in DD1 */
>  #define PHB_PHB3_GEN_CAP               0x250
> @@ -112,8 +95,7 @@
>  #define PHB_PHB3_EEH_CAP               0x268
>  #define PHB_IVC_INVALIDATE             0x2a0
>  #define   PHB_IVC_INVALIDATE_ALL       PPC_BIT(0)
> -#define   PHB_IVC_INVALIDATE_SID_MASK  PPC_BITMASK(16,31)
> -#define   PHB_IVC_INVALIDATE_SID_LSH   PPC_BITLSHIFT(31)
> +#define   PHB_IVC_INVALIDATE_SID       PPC_BITMASK(16,31)
>  #define PHB_IVC_UPDATE                 0x2a8
>  #define   PHB_IVC_UPDATE_ENABLE_P      PPC_BIT(0)
>  #define   PHB_IVC_UPDATE_ENABLE_Q      PPC_BIT(1)
> @@ -121,20 +103,13 @@
>  #define   PHB_IVC_UPDATE_ENABLE_PRI    PPC_BIT(3)
>  #define   PHB_IVC_UPDATE_ENABLE_GEN    PPC_BIT(4)
>  #define   PHB_IVC_UPDATE_ENABLE_CON    PPC_BIT(5)
> -#define   PHB_IVC_UPDATE_GEN_MATCH_MASK        PPC_BITMASK(6, 7)
> -#define   PHB_IVC_UPDATE_GEN_MATCH_LSH PPC_BITLSHIFT(7)
> -#define   PHB_IVC_UPDATE_SERVER_MASK   PPC_BITMASK(8, 23)
> -#define   PHB_IVC_UPDATE_SERVER_LSH    PPC_BITLSHIFT(23)
> -#define   PHB_IVC_UPDATE_PRI_MASK      PPC_BITMASK(24, 31)
> -#define   PHB_IVC_UPDATE_PRI_LSH       PPC_BITLSHIFT(31)
> -#define   PHB_IVC_UPDATE_GEN_MASK      PPC_BITMASK(32,33)
> -#define   PHB_IVC_UPDATE_GEN_LSH       PPC_BITLSHIFT(33)
> -#define   PHB_IVC_UPDATE_P_MASK                PPC_BITMASK(34,34)
> -#define   PHB_IVC_UPDATE_P_LSH         PPC_BITLSHIFT(34)
> -#define   PHB_IVC_UPDATE_Q_MASK                PPC_BITMASK(35,35)
> -#define   PHB_IVC_UPDATE_Q_LSH         PPC_BITLSHIFT(35)
> -#define   PHB_IVC_UPDATE_SID_MASK      PPC_BITMASK(48,63)
> -#define   PHB_IVC_UPDATE_SID_LSH       PPC_BITLSHIFT(63)
> +#define   PHB_IVC_UPDATE_GEN_MATCH     PPC_BITMASK(6, 7)
> +#define   PHB_IVC_UPDATE_SERVER                PPC_BITMASK(8, 23)
> +#define   PHB_IVC_UPDATE_PRI           PPC_BITMASK(24, 31)
> +#define   PHB_IVC_UPDATE_GEN           PPC_BITMASK(32,33)
> +#define   PHB_IVC_UPDATE_P             PPC_BITMASK(34,34)
> +#define   PHB_IVC_UPDATE_Q             PPC_BITMASK(35,35)
> +#define   PHB_IVC_UPDATE_SID           PPC_BITMASK(48,63)
>  #define PHB_PAPR_ERR_INJ_CTL           0x2b0
>  #define   PHB_PAPR_ERR_INJ_CTL_INB     PPC_BIT(0)
>  #define   PHB_PAPR_ERR_INJ_CTL_OUTB    PPC_BIT(1)
> @@ -144,13 +119,10 @@
>  #define   PHB_PAPR_ERR_INJ_CTL_WR      PPC_BIT(5)
>  #define   PHB_PAPR_ERR_INJ_CTL_FREEZE  PPC_BIT(6)
>  #define PHB_PAPR_ERR_INJ_ADDR          0x2b8
> -#define   PHB_PAPR_ERR_INJ_ADDR_MMIO_MASK      PPC_BITMASK(16,63)
> -#define   PHB_PAPR_ERR_INJ_ADDR_MMIO_LSH       PPC_BITLSHIFT(63)
> +#define   PHB_PAPR_ERR_INJ_ADDR_MMIO           PPC_BITMASK(16,63)
>  #define PHB_PAPR_ERR_INJ_MASK          0x2c0
> -#define   PHB_PAPR_ERR_INJ_MASK_CFG_MASK       PPC_BITMASK(4,11)
> -#define   PHB_PAPR_ERR_INJ_MASK_CFG_LSH                PPC_BITLSHIFT(11)
> -#define   PHB_PAPR_ERR_INJ_MASK_MMIO_MASK      PPC_BITMASK(16,63)
> -#define   PHB_PAPR_ERR_INJ_MASK_MMIO_LSH       PPC_BITLSHIFT(63)
> +#define   PHB_PAPR_ERR_INJ_MASK_CFG            PPC_BITMASK(4,11)
> +#define   PHB_PAPR_ERR_INJ_MASK_MMIO           PPC_BITMASK(16,63)
>  #define PHB_ETU_ERR_SUMMARY            0x2c8
>
>  /*  UTL registers */
> @@ -340,39 +312,27 @@
>  #define IODA2_TBL_PEEV         20
>
>  /* LXIVT */
> -#define IODA2_LXIVT_SERVER_MASK                PPC_BITMASK(8,23)
> -#define IODA2_LXIVT_SERVER_LSH         PPC_BITLSHIFT(23)
> -#define IODA2_LXIVT_PRIORITY_MASK      PPC_BITMASK(24,31)
> -#define IODA2_LXIVT_PRIORITY_LSH       PPC_BITLSHIFT(31)
> -#define IODA2_LXIVT_NODE_ID_MASK       PPC_BITMASK(56,63)
> -#define IODA2_LXIVT_NODE_ID_LSH                PPC_BITLSHIFT(63)
> +#define IODA2_LXIVT_SERVER             PPC_BITMASK(8,23)
> +#define IODA2_LXIVT_PRIORITY           PPC_BITMASK(24,31)
> +#define IODA2_LXIVT_NODE_ID            PPC_BITMASK(56,63)
>
>  /* IVT */
> -#define IODA2_IVT_SERVER_MASK          PPC_BITMASK(0,23)
> -#define IODA2_IVT_SERVER_LSH           PPC_BITLSHIFT(23)
> -#define IODA2_IVT_PRIORITY_MASK                PPC_BITMASK(24,31)
> -#define IODA2_IVT_PRIORITY_LSH         PPC_BITLSHIFT(31)
> -#define IODA2_IVT_P_MASK               PPC_BITMASK(39,39)
> -#define IODA2_IVT_P_LSH                        PPC_BITLSHIFT(39)
> -#define IODA2_IVT_Q_MASK               PPC_BITMASK(47,47)
> -#define IODA2_IVT_Q_LSH                        PPC_BITLSHIFT(47)
> -#define IODA2_IVT_PE_MASK              PPC_BITMASK(48,63)
> -#define IODA2_IVT_PE_LSH               PPC_BITLSHIFT(63)
> +#define IODA2_IVT_SERVER               PPC_BITMASK(0,23)
> +#define IODA2_IVT_PRIORITY             PPC_BITMASK(24,31)
> +#define IODA2_IVT_P                    PPC_BITMASK(39,39)
> +#define IODA2_IVT_Q                    PPC_BITMASK(47,47)
> +#define IODA2_IVT_PE                   PPC_BITMASK(48,63)
>
>  /* TVT */
> -#define IODA2_TVT_TABLE_ADDR_MASK      PPC_BITMASK(0,47)
> -#define IODA2_TVT_TABLE_ADDR_LSH       PPC_BITLSHIFT(47)
> -#define IODA2_TVT_NUM_LEVELS_MASK      PPC_BITMASK(48,50)
> -#define IODA2_TVT_NUM_LEVELS_LSH       PPC_BITLSHIFT(50)
> +#define IODA2_TVT_TABLE_ADDR           PPC_BITMASK(0,47)
> +#define IODA2_TVT_NUM_LEVELS           PPC_BITMASK(48,50)
>  #define   IODA2_TVE_1_LEVEL    0
>  #define   IODA2_TVE_2_LEVELS   1
>  #define   IODA2_TVE_3_LEVELS   2
>  #define   IODA2_TVE_4_LEVELS   3
>  #define   IODA2_TVE_5_LEVELS   4
> -#define IODA2_TVT_TCE_TABLE_SIZE_MASK  PPC_BITMASK(51,55)
> -#define IODA2_TVT_TCE_TABLE_SIZE_LSH   PPC_BITLSHIFT(55)
> -#define IODA2_TVT_IO_PSIZE_MASK                PPC_BITMASK(59,63)
> -#define IODA2_TVT_IO_PSIZE_LSH         PPC_BITLSHIFT(63)
> +#define IODA2_TVT_TCE_TABLE_SIZE       PPC_BITMASK(51,55)
> +#define IODA2_TVT_IO_PSIZE             PPC_BITMASK(59,63)
>
>  /* PESTA */
>  #define IODA2_PESTA_MMIO_FROZEN                PPC_BIT(0)
> @@ -381,24 +341,17 @@
>  #define IODA2_PESTB_DMA_STOPPED                PPC_BIT(0)
>
>  /* M32DT */
> -#define IODA2_M32DT_PE_MASK            PPC_BITMASK(8,15)
> -#define IODA2_M32DT_PE_LSH             PPC_BITLSHIFT(15)
> +#define IODA2_M32DT_PE                 PPC_BITMASK(8,15)
>
>  /* M64BT */
>  #define IODA2_M64BT_ENABLE             PPC_BIT(0)
>  #define IODA2_M64BT_SINGLE_PE          PPC_BIT(1)
> -#define IODA2_M64BT_BASE_MASK          PPC_BITMASK(2,31)
> -#define IODA2_M64BT_BASE_LSH           PPC_BITLSHIFT(31)
> -#define IODA2_M64BT_MASK_MASK          PPC_BITMASK(34,63)
> -#define IODA2_M64BT_MASK_LSH           PPC_BITLSHIFT(63)
> -#define IODA2_M64BT_SINGLE_BASE_MASK   PPC_BITMASK(2,26)
> -#define IODA2_M64BT_SINGLE_BASE_LSH    PPC_BITLSHIFT(26)
> -#define IODA2_M64BT_PE_HI_MASK         PPC_BITMASK(27,31)
> -#define IODA2_M64BT_PE_HI_LSH          PPC_BITLSHIFT(31)
> -#define IODA2_M64BT_SINGLE_MASK_MASK   PPC_BITMASK(34,58)
> -#define IODA2_M64BT_SINGLE_MASK_LSH    PPC_BITLSHIFT(58)
> -#define IODA2_M64BT_PE_LOW_MASK                PPC_BITMASK(59,63)
> -#define IODA2_M64BT_PE_LOW_LSH         PPC_BITLSHIFT(63)
> +#define IODA2_M64BT_BASE               PPC_BITMASK(2,31)
> +#define IODA2_M64BT_MASK               PPC_BITMASK(34,63)
> +#define IODA2_M64BT_SINGLE_BASE                PPC_BITMASK(2,26)
> +#define IODA2_M64BT_PE_HI              PPC_BITMASK(27,31)
> +#define IODA2_M64BT_SINGLE_MASK                PPC_BITMASK(34,58)
> +#define IODA2_M64BT_PE_LOW             PPC_BITMASK(59,63)
>
>  /*
>   * IODA2 in-memory tables
> @@ -412,8 +365,7 @@
>  #define IODA2_PEST0_MMIO_CAUSE         PPC_BIT(2)
>  #define IODA2_PEST0_CFG_READ           PPC_BIT(3)
>  #define IODA2_PEST0_CFG_WRITE          PPC_BIT(4)
> -#define IODA2_PEST0_TTYPE_MASK         PPC_BITMASK(5,7)
> -#define IODA2_PEST0_TTYPE_LSH          PPC_BITLSHIFT(7)
> +#define IODA2_PEST0_TTYPE              PPC_BITMASK(5,7)
>  #define   PEST_TTYPE_DMA_WRITE         0
>  #define   PEST_TTYPE_MSI               1
>  #define   PEST_TTYPE_DMA_READ          2
> @@ -435,15 +387,11 @@
>  #define IODA2_PEST0_TCE_ACCESS_FAULT   PPC_BIT(19)
>  #define IODA2_PEST0_DMA_RESP_TIMEOUT   PPC_BIT(20)
>  #define IODA2_PEST0_AIB_SIZE_INVALID   PPC_BIT(21)
> -#define IODA2_PEST0_LEM_BIT_MASK       PPC_BITMASK(26,31)
> -#define IODA2_PEST0_LEM_BIT_LSH                PPC_BITLSHIFT(31)
> -#define IODA2_PEST0_RID_MASK           PPC_BITMASK(32,47)
> -#define IODA2_PEST0_RID_LSH            PPC_BITLSHIFT(47)
> -#define IODA2_PEST0_MSI_DATA_MASK      PPC_BITMASK(48,63)
> -#define IODA2_PEST0_MSI_DATA_LSH       PPC_BITLSHIFT(63)
> +#define IODA2_PEST0_LEM_BIT            PPC_BITMASK(26,31)
> +#define IODA2_PEST0_RID                        PPC_BITMASK(32,47)
> +#define IODA2_PEST0_MSI_DATA           PPC_BITMASK(48,63)
>
> -#define IODA2_PEST1_FAIL_ADDR_MASK     PPC_BITMASK(3,63)
> -#define IODA2_PEST1_FAIL_ADDR_LSH      PPC_BITLSHIFT(63)
> +#define IODA2_PEST1_FAIL_ADDR          PPC_BITMASK(3,63)
>
>
>  #endif /* __PHB3_REGS_H */
> diff --git a/include/processor.h b/include/processor.h
> index c15e17e..31ed72d 100644
> --- a/include/processor.h
> +++ b/include/processor.h
> @@ -95,13 +95,10 @@
>
>
>  /* Bits in TFMR - control bits */
> -#define SPR_TFMR_MAX_CYC_BET_STEPS_MASK        PPC_BITMASK(0,7)
> -#define SPR_TFMR_MAX_CYC_BET_STEPS_LSH PPC_BITLSHIFT(7)
> -#define SPR_TFMR_N_CLKS_PER_STEP_MASK  PPC_BITMASK(8,9)
> -#define SPR_TFMR_N_CLKS_PER_STEP_LSH   PPC_BITLSHIFT(9)
> +#define SPR_TFMR_MAX_CYC_BET_STEPS     PPC_BITMASK(0,7)
> +#define SPR_TFMR_N_CLKS_PER_STEP       PPC_BITMASK(8,9)
>  #define SPR_TFMR_MASK_HMI              PPC_BIT(10)
> -#define SPR_TFMR_SYNC_BIT_SEL_MASK     PPC_BITMASK(11,13)
> -#define SPR_TFMR_SYNC_BIT_SEL_LSH      PPC_BITLSHIFT(13)
> +#define SPR_TFMR_SYNC_BIT_SEL          PPC_BITMASK(11,13)
>  #define SPR_TFMR_TB_ECLIPZ             PPC_BIT(14)
>  #define SPR_TFMR_LOAD_TOD_MOD          PPC_BIT(16)
>  #define SPR_TFMR_MOVE_CHIP_TOD_TO_TB   PPC_BIT(18)
> @@ -109,10 +106,8 @@
>  /* Bits in TFMR - thread indep. status bits */
>  #define SPR_TFMR_HDEC_PARITY_ERROR     PPC_BIT(26)
>  #define SPR_TFMR_TBST_CORRUPT          PPC_BIT(27)
> -#define SPR_TFMR_TBST_ENCODED_MASK     PPC_BITMASK(28,31)
> -#define SPR_TFMR_TBST_ENCODED_LSH      PPC_BITLSHIFT(31)
> -#define SPR_TFMR_TBST_LAST_MASK                PPC_BITMASK(32,35)
> -#define SPR_TFMR_TBST_LAST_LSH         PPC_BITLSHIFT(35)
> +#define SPR_TFMR_TBST_ENCODED          PPC_BITMASK(28,31)
> +#define SPR_TFMR_TBST_LAST             PPC_BITMASK(32,35)
>  #define SPR_TFMR_TB_ENABLED            PPC_BIT(40)
>  #define SPR_TFMR_TB_VALID              PPC_BIT(41)
>  #define SPR_TFMR_TB_SYNC_OCCURED       PPC_BIT(42)
> @@ -120,8 +115,7 @@
>  #define SPR_TFMR_TB_MISSING_STEP       PPC_BIT(44)
>  #define SPR_TFMR_TB_RESIDUE_ERR                PPC_BIT(45)
>  #define SPR_TFMR_FW_CONTROL_ERR                PPC_BIT(46)
> -#define SPR_TFMR_CHIP_TOD_STATUS_MASK  PPC_BITMASK(47,50)
> -#define SPR_TFMR_CHIP_TOD_STATUS_LSH   PPC_BITLSHIFT(50)
> +#define SPR_TFMR_CHIP_TOD_STATUS       PPC_BITMASK(47,50)
>  #define SPR_TFMR_CHIP_TOD_INTERRUPT    PPC_BIT(51)
>  #define SPR_TFMR_CHIP_TOD_TIMEOUT      PPC_BIT(54)
>  #define SPR_TFMR_CHIP_TOD_PARITY_ERR   PPC_BIT(56)
> @@ -147,8 +141,7 @@
>  #define SPR_HMER_SCOM_FIR_HMI          PPC_BIT(16)
>  #define SPR_HMER_TRIG_FIR_HMI          PPC_BIT(17)
>  #define SPR_HMER_HYP_RESOURCE_ERR      PPC_BIT(20)
> -#define SPR_HMER_XSCOM_STATUS_MASK     PPC_BITMASK(21,23)
> -#define SPR_HMER_XSCOM_STATUS_LSH      PPC_BITLSHIFT(23)
> +#define SPR_HMER_XSCOM_STATUS          PPC_BITMASK(21,23)
>
>  /*
>   * HMEER: initial bits for HMI interrupt enable mask.
> @@ -167,12 +160,9 @@
>  #define SPR_HID0_ENABLE_ATTN           PPC_BIT(31)
>
>  /* PVR bits */
> -#define SPR_PVR_TYPE_MASK              0xffff0000
> -#define SPR_PVR_TYPE_LSH               16
> -#define SPR_PVR_VERS_MAJ_MASK          0x00000f00
> -#define SPR_PVR_VERS_MAJ_LSH           8
> -#define SPR_PVR_VERS_MIN_MASK          0x000000ff
> -#define SPR_PVR_VERS_MIN_LSH           0
> +#define SPR_PVR_TYPE                   0xffff0000
> +#define SPR_PVR_VERS_MAJ               0x00000f00
> +#define SPR_PVR_VERS_MIN               0x000000ff
>
>  #define PVR_TYPE(_pvr)         GETFIELD(SPR_PVR_TYPE, _pvr)
>  #define PVR_VERS_MAJ(_pvr)     GETFIELD(SPR_PVR_VERS_MAJ, _pvr)
> diff --git a/include/psi.h b/include/psi.h
> index 85057f5..7e7a24c 100644
> --- a/include/psi.h
> +++ b/include/psi.h
> @@ -92,14 +92,12 @@
>  #define PSIHB_XIVR_LOCAL_ERR           0x78
>  #define PSIHB_XIVR_HOST_ERR            0x80
>  #define PSIHB_ISRN                     0x88
> -#define PSIHB_ISRN_COMP_MASK           PPC_BITMASK(0,18)
> -#define PSIHB_ISRN_COMP_LSH            PPC_BITLSHIFT(18)
> +#define PSIHB_ISRN_COMP                        PPC_BITMASK(0,18)
>  #define PSIHB_ISRN_IRQ_MUX             PPC_BIT(28)
>  #define PSIHB_ISRN_IRQ_RESET           PPC_BIT(29)
>  #define PSIHB_ISRN_DOWNSTREAM_EN       PPC_BIT(30)
>  #define PSIHB_ISRN_UPSTREAM_EN         PPC_BIT(31)
> -#define PSIHB_ISRN_MASK_MASK           PPC_BITMASK(32,50)
> -#define PSIHB_ISRN_MASK_LSH            PPC_BITLSHIFT(50)
> +#define PSIHB_ISRN_MASK                        PPC_BITMASK(32,50)
>
>  #define PSIHB_IRQ_STATUS               0x58
>  #define   PSIHB_IRQ_STAT_OCC           PPC_BIT(27)
> --
> 1.8.3.1
>


More information about the Skiboot mailing list