On 9/10/20 12:01 AM, Krzysztof Kozlowski wrote: > The register addresses are not continuous, so use simple defines for > them. This also makes it easier to find the address for register. > > No functional change. > > Signed-off-by: Krzysztof Kozlowski <k...@kernel.org> > --- > drivers/extcon/extcon-ptn5150.c | 22 ++++++++++------------ > 1 file changed, 10 insertions(+), 12 deletions(-) > > diff --git a/drivers/extcon/extcon-ptn5150.c b/drivers/extcon/extcon-ptn5150.c > index dda5b3a3a908..1b68f56d8372 100644 > --- a/drivers/extcon/extcon-ptn5150.c > +++ b/drivers/extcon/extcon-ptn5150.c > @@ -19,18 +19,16 @@ > #include <linux/gpio/consumer.h> > > /* PTN5150 registers */ > -enum ptn5150_reg { > - PTN5150_REG_DEVICE_ID = 0x01, > - PTN5150_REG_CONTROL, > - PTN5150_REG_INT_STATUS, > - PTN5150_REG_CC_STATUS, > - PTN5150_REG_CON_DET = 0x09, > - PTN5150_REG_VCONN_STATUS, > - PTN5150_REG_RESET, > - PTN5150_REG_INT_MASK = 0x18, > - PTN5150_REG_INT_REG_STATUS, > - PTN5150_REG_END, > -}; > +#define PTN5150_REG_DEVICE_ID 0x01 > +#define PTN5150_REG_CONTROL 0x02 > +#define PTN5150_REG_INT_STATUS 0x03 > +#define PTN5150_REG_CC_STATUS 0x04 > +#define PTN5150_REG_CON_DET 0x09 > +#define PTN5150_REG_VCONN_STATUS 0x0a > +#define PTN5150_REG_RESET 0x0b > +#define PTN5150_REG_INT_MASK 0x18 > +#define PTN5150_REG_INT_REG_STATUS 0x19 > +#define PTN5150_REG_END > PTN5150_REG_INT_REG_STATUS
PTN5150_REG_END should be (PTN5150_REG_INT_REG_STATUS + 1) for regmap operation of PTN5150_REG_INT_REG_STATUS. > > #define PTN5150_DFP_ATTACHED 0x1 > #define PTN5150_UFP_ATTACHED 0x2 > -- Best Regards, Chanwoo Choi Samsung Electronics