Hi Michal, On Tue, Jun 14, 2016 at 3:46 AM, Michal Suchanek <hramrach@xxxxxxxxx> wrote: > SUNXI_CTL_ -> SUNXI_TFR_CTL_ > SUNXI_TFR_CTL_LMTF -> SUNXI_TFR_CTL_FBS I don't know these abbreviations, are they both referring to the same thing? > SUNXI_TFR_CTL_CS_ACTIVE_LOW -> SUNXI_TFR_CTL_SPOL It looks like you're making the constant name less descriptive here. Is the old version (CS_ACTIVE_LOW) incorrect? > and some SUNXI_???_CTL_ -> SUNXI_CTL_ > for constants migrated to different registers between sun4i and sun6i > > No functional change. > > Signed-off-by: Michal Suchanek <hramrach@xxxxxxxxx> > --- > drivers/spi/spi-sun4i.c | 68 ++++++++++++++++++++++++------------------------- > drivers/spi/spi-sun6i.c | 14 +++++----- > 2 files changed, 41 insertions(+), 41 deletions(-) > > diff --git a/drivers/spi/spi-sun4i.c b/drivers/spi/spi-sun4i.c > index 155d720..b7f8de1 100644 > --- a/drivers/spi/spi-sun4i.c > +++ b/drivers/spi/spi-sun4i.c > @@ -28,21 +28,21 @@ > > #define SUNXI_TXDATA_REG 0x04 > > -#define SUNXI_CTL_REG 0x08 > +#define SUNXI_TFR_CTL_REG 0x08 > #define SUNXI_CTL_ENABLE BIT(0) > #define SUNXI_CTL_MASTER BIT(1) > -#define SUNXI_CTL_CPHA BIT(2) > -#define SUNXI_CTL_CPOL BIT(3) > -#define SUNXI_CTL_CS_ACTIVE_LOW BIT(4) > -#define SUNXI_CTL_LMTF BIT(6) > +#define SUNXI_TFR_CTL_CPHA BIT(2) > +#define SUNXI_TFR_CTL_CPOL BIT(3) > +#define SUNXI_TFR_CTL_SPOL BIT(4) > +#define SUNXI_TFR_CTL_FBS BIT(6) > #define SUNXI_CTL_TF_RST BIT(8) > #define SUNXI_CTL_RF_RST BIT(9) > -#define SUNXI_CTL_XCH BIT(10) > -#define SUNXI_CTL_CS_MASK 0x3000 > -#define SUNXI_CTL_CS(cs) (((cs) << 12) & SUNXI_CTL_CS_MASK) > -#define SUNXI_CTL_DHB BIT(15) > -#define SUNXI_CTL_CS_MANUAL BIT(16) > -#define SUNXI_CTL_CS_LEVEL BIT(17) > +#define SUNXI_TFR_CTL_XCH BIT(10) > +#define SUNXI_TFR_CTL_CS_MASK 0x3000 > +#define SUNXI_TFR_CTL_CS(cs) (((cs) << 12) & SUNXI_TFR_CTL_CS_MASK) > +#define SUNXI_TFR_CTL_DHB BIT(15) > +#define SUNXI_TFR_CTL_CS_MANUAL BIT(16) > +#define SUNXI_TFR_CTL_CS_LEVEL BIT(17) > #define SUNXI_CTL_TP BIT(18) > > #define SUNXI_INT_CTL_REG 0x0c > diff --git a/drivers/spi/spi-sun6i.c b/drivers/spi/spi-sun6i.c > index a27bf8f..f26b52a 100644 > --- a/drivers/spi/spi-sun6i.c > +++ b/drivers/spi/spi-sun6i.c > @@ -26,9 +26,9 @@ > #define SUNXI_FIFO_DEPTH 128 > > #define SUNXI_GBL_CTL_REG 0x04 > -#define SUNXI_GBL_CTL_BUS_ENABLE BIT(0) > -#define SUNXI_GBL_CTL_MASTER BIT(1) > -#define SUNXI_GBL_CTL_TP BIT(7) > +#define SUNXI_CTL_ENABLE BIT(0) > +#define SUNXI_CTL_MASTER BIT(1) > +#define SUNXI_CTL_TP BIT(7) If these are bit definitions for the GBL register, why throw that information away? > #define SUNXI_GBL_CTL_RST BIT(31) > > #define SUNXI_TFR_CTL_REG 0x08 > @@ -50,8 +50,8 @@ > #define SUNXI_INT_STA_REG 0x14 > > #define SUNXI_FIFO_CTL_REG 0x18 > -#define SUNXI_FIFO_CTL_RF_RST BIT(15) > -#define SUNXI_FIFO_CTL_TF_RST BIT(31) > +#define SUNXI_CTL_RF_RST BIT(15) > +#define SUNXI_CTL_TF_RST BIT(31) Same here with FIFO. > > #define SUNXI_FIFO_STA_REG 0x1c > #define SUNXI_FIFO_STA_RF_CNT_MASK 0x7f My gut feeling on this is that we have a lot of cases of a definition of a register offset, then definitions of the bits in that register with that register encoded into the constant's name. You appear to be throwing a lot of that information away which makes me worry. Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html