On Thursday, March 20, 2014 01:57:10 PM Bartlomiej Zolnierkiewicz wrote: > > > +#define DA8XX_SYSCFG1_VIRT(x) (da8xx_syscfg1_base + (x)) > > > +#define DA8XX_PWRDN_REG 0x18 > > > + > > > +/* SATA PHY Control Register offset from AHCI base */ > > > +#define SATA_P0PHYCR_REG 0x178 > > > + > > > +#define SATA_PHY_MPY(x) ((x) << 0) > > > +#define SATA_PHY_LOS(x) ((x) << 6) > > > +#define SATA_PHY_RXCDR(x) ((x) << 10) > > > +#define SATA_PHY_RXEQ(x) ((x) << 13) > > > +#define SATA_PHY_TXSWING(x) ((x) << 19) > > > +#define SATA_PHY_ENPLL(x) ((x) << 31) > > > > These can be replaced with BIT(N) > > OK, I'll fix it. Uh, no, we can't use BIT() here. BIT(N) does (1UL << (N)) and here we have ((N) << offset). Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html