| Message ID | 1432446213-4886-7-git-send-email-k.kozlowski.k@gmail.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
On Sun, 24 May 2015, Krzysztof Kozlowski wrote: > Add MAX77693 prefix to some of the defines used in max77693 extcon > driver so the max77693-private.h can be included simultaneously with > max77843-private.h. > > Additionally use BIT() macro in header. > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski.k@gmail.com> > --- > drivers/extcon/extcon-max77693.c | 72 +++++++++++++------------ > include/linux/mfd/max77693-private.h | 102 +++++++++++++++++------------------ > 2 files changed, 89 insertions(+), 85 deletions(-) Acked-by: Lee Jones <lee.jones@linaro.org> > diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c > index 4ef45f257e6e..96f1545be4ab 100644 > --- a/drivers/extcon/extcon-max77693.c > +++ b/drivers/extcon/extcon-max77693.c > @@ -43,7 +43,7 @@ static struct max77693_reg_data default_init_data[] = { > { > /* STATUS2 - [3]ChgDetRun */ > .addr = MAX77693_MUIC_REG_STATUS2, > - .data = STATUS2_CHGDETRUN_MASK, > + .data = MAX77693_STATUS2_CHGDETRUN_MASK, > }, { > /* INTMASK1 - Unmask [3]ADC1KM,[0]ADCM */ > .addr = MAX77693_MUIC_REG_INTMASK1, > @@ -236,7 +236,7 @@ static int max77693_muic_set_debounce_time(struct max77693_muic_info *info, > */ > ret = regmap_write(info->max77693->regmap_muic, > MAX77693_MUIC_REG_CTRL3, > - time << CONTROL3_ADCDBSET_SHIFT); > + time << MAX77693_CONTROL3_ADCDBSET_SHIFT); > if (ret) { > dev_err(info->dev, "failed to set ADC debounce time\n"); > return ret; > @@ -269,7 +269,7 @@ static int max77693_muic_set_path(struct max77693_muic_info *info, > if (attached) > ctrl1 = val; > else > - ctrl1 = CONTROL1_SW_OPEN; > + ctrl1 = MAX77693_CONTROL1_SW_OPEN; > > ret = regmap_update_bits(info->max77693->regmap_muic, > MAX77693_MUIC_REG_CTRL1, COMP_SW_MASK, ctrl1); > @@ -279,13 +279,14 @@ static int max77693_muic_set_path(struct max77693_muic_info *info, > } > > if (attached) > - ctrl2 |= CONTROL2_CPEN_MASK; /* LowPwr=0, CPEn=1 */ > + ctrl2 |= MAX77693_CONTROL2_CPEN_MASK; /* LowPwr=0, CPEn=1 */ > else > - ctrl2 |= CONTROL2_LOWPWR_MASK; /* LowPwr=1, CPEn=0 */ > + ctrl2 |= MAX77693_CONTROL2_LOWPWR_MASK; /* LowPwr=1, CPEn=0 */ > > ret = regmap_update_bits(info->max77693->regmap_muic, > MAX77693_MUIC_REG_CTRL2, > - CONTROL2_LOWPWR_MASK | CONTROL2_CPEN_MASK, ctrl2); > + MAX77693_CONTROL2_LOWPWR_MASK | MAX77693_CONTROL2_CPEN_MASK, > + ctrl2); > if (ret < 0) { > dev_err(info->dev, "failed to update MUIC register\n"); > return ret; > @@ -327,8 +328,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read ADC value to check cable type and decide cable state > * according to cable type > */ > - adc = info->status[0] & STATUS1_ADC_MASK; > - adc >>= STATUS1_ADC_SHIFT; > + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; > + adc >>= MAX77693_STATUS1_ADC_SHIFT; > > /* > * Check current cable state/cable type and store cable type > @@ -351,8 +352,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read ADC value to check cable type and decide cable state > * according to cable type > */ > - adc = info->status[0] & STATUS1_ADC_MASK; > - adc >>= STATUS1_ADC_SHIFT; > + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; > + adc >>= MAX77693_STATUS1_ADC_SHIFT; > > /* > * Check current cable state/cable type and store cable type > @@ -367,13 +368,13 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > } else { > *attached = true; > > - adclow = info->status[0] & STATUS1_ADCLOW_MASK; > - adclow >>= STATUS1_ADCLOW_SHIFT; > - adc1k = info->status[0] & STATUS1_ADC1K_MASK; > - adc1k >>= STATUS1_ADC1K_SHIFT; > + adclow = info->status[0] & MAX77693_STATUS1_ADCLOW_MASK; > + adclow >>= MAX77693_STATUS1_ADCLOW_SHIFT; > + adc1k = info->status[0] & MAX77693_STATUS1_ADC1K_MASK; > + adc1k >>= MAX77693_STATUS1_ADC1K_SHIFT; > > - vbvolt = info->status[1] & STATUS2_VBVOLT_MASK; > - vbvolt >>= STATUS2_VBVOLT_SHIFT; > + vbvolt = info->status[1] & MAX77693_STATUS2_VBVOLT_MASK; > + vbvolt >>= MAX77693_STATUS2_VBVOLT_SHIFT; > > /** > * [0x1|VBVolt|ADCLow|ADC1K] > @@ -398,8 +399,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read charger type to check cable type and decide cable state > * according to type of charger cable. > */ > - chg_type = info->status[1] & STATUS2_CHGTYP_MASK; > - chg_type >>= STATUS2_CHGTYP_SHIFT; > + chg_type = info->status[1] & MAX77693_STATUS2_CHGTYP_MASK; > + chg_type >>= MAX77693_STATUS2_CHGTYP_SHIFT; > > if (chg_type == MAX77693_CHARGER_TYPE_NONE) { > *attached = false; > @@ -423,10 +424,10 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read ADC value to check cable type and decide cable state > * according to cable type > */ > - adc = info->status[0] & STATUS1_ADC_MASK; > - adc >>= STATUS1_ADC_SHIFT; > - chg_type = info->status[1] & STATUS2_CHGTYP_MASK; > - chg_type >>= STATUS2_CHGTYP_SHIFT; > + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; > + adc >>= MAX77693_STATUS1_ADC_SHIFT; > + chg_type = info->status[1] & MAX77693_STATUS2_CHGTYP_MASK; > + chg_type >>= MAX77693_STATUS2_CHGTYP_SHIFT; > > if (adc == MAX77693_MUIC_ADC_OPEN > && chg_type == MAX77693_CHARGER_TYPE_NONE) > @@ -438,8 +439,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read vbvolt field, if vbvolt is 1, > * this cable is used for charging. > */ > - vbvolt = info->status[1] & STATUS2_VBVOLT_MASK; > - vbvolt >>= STATUS2_VBVOLT_SHIFT; > + vbvolt = info->status[1] & MAX77693_STATUS2_VBVOLT_MASK; > + vbvolt >>= MAX77693_STATUS2_VBVOLT_SHIFT; > > cable_type = vbvolt; > break; > @@ -521,7 +522,8 @@ static int max77693_muic_dock_handler(struct max77693_muic_info *info, > } > > /* Dock-Car/Desk/Audio, PATH:AUDIO */ > - ret = max77693_muic_set_path(info, CONTROL1_SW_AUDIO, attached); > + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_AUDIO, > + attached); > if (ret < 0) > return ret; > extcon_set_cable_state_(info->edev, dock_id, attached); > @@ -586,14 +588,16 @@ static int max77693_muic_adc_ground_handler(struct max77693_muic_info *info) > case MAX77693_MUIC_GND_USB_HOST: > case MAX77693_MUIC_GND_USB_HOST_VB: > /* USB_HOST, PATH: AP_USB */ > - ret = max77693_muic_set_path(info, CONTROL1_SW_USB, attached); > + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_USB, > + attached); > if (ret < 0) > return ret; > extcon_set_cable_state_(info->edev, EXTCON_USB_HOST, attached); > break; > case MAX77693_MUIC_GND_AV_CABLE_LOAD: > /* Audio Video Cable with load, PATH:AUDIO */ > - ret = max77693_muic_set_path(info, CONTROL1_SW_AUDIO, attached); > + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_AUDIO, > + attached); > if (ret < 0) > return ret; > extcon_set_cable_state_(info->edev, EXTCON_USB, attached); > @@ -616,7 +620,7 @@ static int max77693_muic_jig_handler(struct max77693_muic_info *info, > int cable_type, bool attached) > { > int ret = 0; > - u8 path = CONTROL1_SW_OPEN; > + u8 path = MAX77693_CONTROL1_SW_OPEN; > > dev_info(info->dev, > "external connector is %s (adc:0x%02x)\n", > @@ -626,12 +630,12 @@ static int max77693_muic_jig_handler(struct max77693_muic_info *info, > case MAX77693_MUIC_ADC_FACTORY_MODE_USB_OFF: /* ADC_JIG_USB_OFF */ > case MAX77693_MUIC_ADC_FACTORY_MODE_USB_ON: /* ADC_JIG_USB_ON */ > /* PATH:AP_USB */ > - path = CONTROL1_SW_USB; > + path = MAX77693_CONTROL1_SW_USB; > break; > case MAX77693_MUIC_ADC_FACTORY_MODE_UART_OFF: /* ADC_JIG_UART_OFF */ > case MAX77693_MUIC_ADC_FACTORY_MODE_UART_ON: /* ADC_JIG_UART_ON */ > /* PATH:AP_UART */ > - path = CONTROL1_SW_UART; > + path = MAX77693_CONTROL1_SW_UART; > break; > default: > dev_err(info->dev, "failed to detect %s jig cable\n", > @@ -1181,12 +1185,12 @@ static int max77693_muic_probe(struct platform_device *pdev) > if (muic_pdata->path_uart) > info->path_uart = muic_pdata->path_uart; > else > - info->path_uart = CONTROL1_SW_UART; > + info->path_uart = MAX77693_CONTROL1_SW_UART; > > if (muic_pdata->path_usb) > info->path_usb = muic_pdata->path_usb; > else > - info->path_usb = CONTROL1_SW_USB; > + info->path_usb = MAX77693_CONTROL1_SW_USB; > > /* > * Default delay time for detecting cable state > @@ -1198,8 +1202,8 @@ static int max77693_muic_probe(struct platform_device *pdev) > else > delay_jiffies = msecs_to_jiffies(DELAY_MS_DEFAULT); > } else { > - info->path_usb = CONTROL1_SW_USB; > - info->path_uart = CONTROL1_SW_UART; > + info->path_usb = MAX77693_CONTROL1_SW_USB; > + info->path_uart = MAX77693_CONTROL1_SW_UART; > delay_jiffies = msecs_to_jiffies(DELAY_MS_DEFAULT); > } > > diff --git a/include/linux/mfd/max77693-private.h b/include/linux/mfd/max77693-private.h > index 8c4143c0c651..3c7a63b98ad6 100644 > --- a/include/linux/mfd/max77693-private.h > +++ b/include/linux/mfd/max77693-private.h > @@ -310,30 +310,30 @@ enum max77693_muic_reg { > #define INTMASK2_CHGTYP_MASK (1 << INTMASK2_CHGTYP_SHIFT) > > /* MAX77693 MUIC - STATUS1~3 Register */ > -#define STATUS1_ADC_SHIFT (0) > -#define STATUS1_ADCLOW_SHIFT (5) > -#define STATUS1_ADCERR_SHIFT (6) > -#define STATUS1_ADC1K_SHIFT (7) > -#define STATUS1_ADC_MASK (0x1f << STATUS1_ADC_SHIFT) > -#define STATUS1_ADCLOW_MASK (0x1 << STATUS1_ADCLOW_SHIFT) > -#define STATUS1_ADCERR_MASK (0x1 << STATUS1_ADCERR_SHIFT) > -#define STATUS1_ADC1K_MASK (0x1 << STATUS1_ADC1K_SHIFT) > - > -#define STATUS2_CHGTYP_SHIFT (0) > -#define STATUS2_CHGDETRUN_SHIFT (3) > -#define STATUS2_DCDTMR_SHIFT (4) > -#define STATUS2_DXOVP_SHIFT (5) > -#define STATUS2_VBVOLT_SHIFT (6) > -#define STATUS2_VIDRM_SHIFT (7) > -#define STATUS2_CHGTYP_MASK (0x7 << STATUS2_CHGTYP_SHIFT) > -#define STATUS2_CHGDETRUN_MASK (0x1 << STATUS2_CHGDETRUN_SHIFT) > -#define STATUS2_DCDTMR_MASK (0x1 << STATUS2_DCDTMR_SHIFT) > -#define STATUS2_DXOVP_MASK (0x1 << STATUS2_DXOVP_SHIFT) > -#define STATUS2_VBVOLT_MASK (0x1 << STATUS2_VBVOLT_SHIFT) > -#define STATUS2_VIDRM_MASK (0x1 << STATUS2_VIDRM_SHIFT) > - > -#define STATUS3_OVP_SHIFT (2) > -#define STATUS3_OVP_MASK (0x1 << STATUS3_OVP_SHIFT) > +#define MAX77693_STATUS1_ADC_SHIFT 0 > +#define MAX77693_STATUS1_ADCLOW_SHIFT 5 > +#define MAX77693_STATUS1_ADCERR_SHIFT 6 > +#define MAX77693_STATUS1_ADC1K_SHIFT 7 > +#define MAX77693_STATUS1_ADC_MASK (0x1f << MAX77693_STATUS1_ADC_SHIFT) > +#define MAX77693_STATUS1_ADCLOW_MASK BIT(MAX77693_STATUS1_ADCLOW_SHIFT) > +#define MAX77693_STATUS1_ADCERR_MASK BIT(MAX77693_STATUS1_ADCERR_SHIFT) > +#define MAX77693_STATUS1_ADC1K_MASK BIT(MAX77693_STATUS1_ADC1K_SHIFT) > + > +#define MAX77693_STATUS2_CHGTYP_SHIFT 0 > +#define MAX77693_STATUS2_CHGDETRUN_SHIFT 3 > +#define MAX77693_STATUS2_DCDTMR_SHIFT 4 > +#define MAX77693_STATUS2_DXOVP_SHIFT 5 > +#define MAX77693_STATUS2_VBVOLT_SHIFT 6 > +#define MAX77693_STATUS2_VIDRM_SHIFT 7 > +#define MAX77693_STATUS2_CHGTYP_MASK (0x7 << MAX77693_STATUS2_CHGTYP_SHIFT) > +#define MAX77693_STATUS2_CHGDETRUN_MASK BIT(MAX77693_STATUS2_CHGDETRUN_SHIFT) > +#define MAX77693_STATUS2_DCDTMR_MASK BIT(MAX77693_STATUS2_DCDTMR_SHIFT) > +#define MAX77693_STATUS2_DXOVP_MASK BIT(MAX77693_STATUS2_DXOVP_SHIFT) > +#define MAX77693_STATUS2_VBVOLT_MASK BIT(MAX77693_STATUS2_VBVOLT_SHIFT) > +#define MAX77693_STATUS2_VIDRM_MASK BIT(MAX77693_STATUS2_VIDRM_SHIFT) > + > +#define MAX77693_STATUS3_OVP_SHIFT 2 > +#define MAX77693_STATUS3_OVP_MASK BIT(MAX77693_STATUS3_OVP_SHIFT) > > /* MAX77693 CDETCTRL1~2 register */ > #define CDETCTRL1_CHGDETEN_SHIFT (0) > @@ -362,38 +362,38 @@ enum max77693_muic_reg { > #define COMN1SW_MASK (0x7 << COMN1SW_SHIFT) > #define COMP2SW_MASK (0x7 << COMP2SW_SHIFT) > #define COMP_SW_MASK (COMP2SW_MASK | COMN1SW_MASK) > -#define CONTROL1_SW_USB ((1 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_USB ((1 << COMP2SW_SHIFT) \ > | (1 << COMN1SW_SHIFT)) > -#define CONTROL1_SW_AUDIO ((2 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_AUDIO ((2 << COMP2SW_SHIFT) \ > | (2 << COMN1SW_SHIFT)) > -#define CONTROL1_SW_UART ((3 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_UART ((3 << COMP2SW_SHIFT) \ > | (3 << COMN1SW_SHIFT)) > -#define CONTROL1_SW_OPEN ((0 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_OPEN ((0 << COMP2SW_SHIFT) \ > | (0 << COMN1SW_SHIFT)) > > -#define CONTROL2_LOWPWR_SHIFT (0) > -#define CONTROL2_ADCEN_SHIFT (1) > -#define CONTROL2_CPEN_SHIFT (2) > -#define CONTROL2_SFOUTASRT_SHIFT (3) > -#define CONTROL2_SFOUTORD_SHIFT (4) > -#define CONTROL2_ACCDET_SHIFT (5) > -#define CONTROL2_USBCPINT_SHIFT (6) > -#define CONTROL2_RCPS_SHIFT (7) > -#define CONTROL2_LOWPWR_MASK (0x1 << CONTROL2_LOWPWR_SHIFT) > -#define CONTROL2_ADCEN_MASK (0x1 << CONTROL2_ADCEN_SHIFT) > -#define CONTROL2_CPEN_MASK (0x1 << CONTROL2_CPEN_SHIFT) > -#define CONTROL2_SFOUTASRT_MASK (0x1 << CONTROL2_SFOUTASRT_SHIFT) > -#define CONTROL2_SFOUTORD_MASK (0x1 << CONTROL2_SFOUTORD_SHIFT) > -#define CONTROL2_ACCDET_MASK (0x1 << CONTROL2_ACCDET_SHIFT) > -#define CONTROL2_USBCPINT_MASK (0x1 << CONTROL2_USBCPINT_SHIFT) > -#define CONTROL2_RCPS_MASK (0x1 << CONTROL2_RCPS_SHIFT) > - > -#define CONTROL3_JIGSET_SHIFT (0) > -#define CONTROL3_BTLDSET_SHIFT (2) > -#define CONTROL3_ADCDBSET_SHIFT (4) > -#define CONTROL3_JIGSET_MASK (0x3 << CONTROL3_JIGSET_SHIFT) > -#define CONTROL3_BTLDSET_MASK (0x3 << CONTROL3_BTLDSET_SHIFT) > -#define CONTROL3_ADCDBSET_MASK (0x3 << CONTROL3_ADCDBSET_SHIFT) > +#define MAX77693_CONTROL2_LOWPWR_SHIFT 0 > +#define MAX77693_CONTROL2_ADCEN_SHIFT 1 > +#define MAX77693_CONTROL2_CPEN_SHIFT 2 > +#define MAX77693_CONTROL2_SFOUTASRT_SHIFT 3 > +#define MAX77693_CONTROL2_SFOUTORD_SHIFT 4 > +#define MAX77693_CONTROL2_ACCDET_SHIFT 5 > +#define MAX77693_CONTROL2_USBCPINT_SHIFT 6 > +#define MAX77693_CONTROL2_RCPS_SHIFT 7 > +#define MAX77693_CONTROL2_LOWPWR_MASK BIT(MAX77693_CONTROL2_LOWPWR_SHIFT) > +#define MAX77693_CONTROL2_ADCEN_MASK BIT(MAX77693_CONTROL2_ADCEN_SHIFT) > +#define MAX77693_CONTROL2_CPEN_MASK BIT(MAX77693_CONTROL2_CPEN_SHIFT) > +#define MAX77693_CONTROL2_SFOUTASRT_MASK BIT(MAX77693_CONTROL2_SFOUTASRT_SHIFT) > +#define MAX77693_CONTROL2_SFOUTORD_MASK BIT(MAX77693_CONTROL2_SFOUTORD_SHIFT) > +#define MAX77693_CONTROL2_ACCDET_MASK BIT(MAX77693_CONTROL2_ACCDET_SHIFT) > +#define MAX77693_CONTROL2_USBCPINT_MASK BIT(MAX77693_CONTROL2_USBCPINT_SHIFT) > +#define MAX77693_CONTROL2_RCPS_MASK BIT(MAX77693_CONTROL2_RCPS_SHIFT) > + > +#define MAX77693_CONTROL3_JIGSET_SHIFT 0 > +#define MAX77693_CONTROL3_BTLDSET_SHIFT 2 > +#define MAX77693_CONTROL3_ADCDBSET_SHIFT 4 > +#define MAX77693_CONTROL3_JIGSET_MASK (0x3 << MAX77693_CONTROL3_JIGSET_SHIFT) > +#define MAX77693_CONTROL3_BTLDSET_MASK (0x3 << MAX77693_CONTROL3_BTLDSET_SHIFT) > +#define MAX77693_CONTROL3_ADCDBSET_MASK (0x3 << MAX77693_CONTROL3_ADCDBSET_SHIFT) > > /* Slave addr = 0x90: Haptic */ > enum max77693_haptic_reg {
Hi Krzysztof, I reviewed it and tested it for kernel build. Acked-by: Chanwoo Choi <cw00.choi@samsung.com> Thanks, Chanwoo Choi On Sun, May 24, 2015 at 2:43 PM, Krzysztof Kozlowski <k.kozlowski.k@gmail.com> wrote: > Add MAX77693 prefix to some of the defines used in max77693 extcon > driver so the max77693-private.h can be included simultaneously with > max77843-private.h. > > Additionally use BIT() macro in header. > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski.k@gmail.com> > --- > drivers/extcon/extcon-max77693.c | 72 +++++++++++++------------ > include/linux/mfd/max77693-private.h | 102 +++++++++++++++++------------------ > 2 files changed, 89 insertions(+), 85 deletions(-) > > diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c > index 4ef45f257e6e..96f1545be4ab 100644 > --- a/drivers/extcon/extcon-max77693.c > +++ b/drivers/extcon/extcon-max77693.c > @@ -43,7 +43,7 @@ static struct max77693_reg_data default_init_data[] = { > { > /* STATUS2 - [3]ChgDetRun */ > .addr = MAX77693_MUIC_REG_STATUS2, > - .data = STATUS2_CHGDETRUN_MASK, > + .data = MAX77693_STATUS2_CHGDETRUN_MASK, > }, { > /* INTMASK1 - Unmask [3]ADC1KM,[0]ADCM */ > .addr = MAX77693_MUIC_REG_INTMASK1, > @@ -236,7 +236,7 @@ static int max77693_muic_set_debounce_time(struct max77693_muic_info *info, > */ > ret = regmap_write(info->max77693->regmap_muic, > MAX77693_MUIC_REG_CTRL3, > - time << CONTROL3_ADCDBSET_SHIFT); > + time << MAX77693_CONTROL3_ADCDBSET_SHIFT); > if (ret) { > dev_err(info->dev, "failed to set ADC debounce time\n"); > return ret; > @@ -269,7 +269,7 @@ static int max77693_muic_set_path(struct max77693_muic_info *info, > if (attached) > ctrl1 = val; > else > - ctrl1 = CONTROL1_SW_OPEN; > + ctrl1 = MAX77693_CONTROL1_SW_OPEN; > > ret = regmap_update_bits(info->max77693->regmap_muic, > MAX77693_MUIC_REG_CTRL1, COMP_SW_MASK, ctrl1); > @@ -279,13 +279,14 @@ static int max77693_muic_set_path(struct max77693_muic_info *info, > } > > if (attached) > - ctrl2 |= CONTROL2_CPEN_MASK; /* LowPwr=0, CPEn=1 */ > + ctrl2 |= MAX77693_CONTROL2_CPEN_MASK; /* LowPwr=0, CPEn=1 */ > else > - ctrl2 |= CONTROL2_LOWPWR_MASK; /* LowPwr=1, CPEn=0 */ > + ctrl2 |= MAX77693_CONTROL2_LOWPWR_MASK; /* LowPwr=1, CPEn=0 */ > > ret = regmap_update_bits(info->max77693->regmap_muic, > MAX77693_MUIC_REG_CTRL2, > - CONTROL2_LOWPWR_MASK | CONTROL2_CPEN_MASK, ctrl2); > + MAX77693_CONTROL2_LOWPWR_MASK | MAX77693_CONTROL2_CPEN_MASK, > + ctrl2); > if (ret < 0) { > dev_err(info->dev, "failed to update MUIC register\n"); > return ret; > @@ -327,8 +328,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read ADC value to check cable type and decide cable state > * according to cable type > */ > - adc = info->status[0] & STATUS1_ADC_MASK; > - adc >>= STATUS1_ADC_SHIFT; > + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; > + adc >>= MAX77693_STATUS1_ADC_SHIFT; > > /* > * Check current cable state/cable type and store cable type > @@ -351,8 +352,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read ADC value to check cable type and decide cable state > * according to cable type > */ > - adc = info->status[0] & STATUS1_ADC_MASK; > - adc >>= STATUS1_ADC_SHIFT; > + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; > + adc >>= MAX77693_STATUS1_ADC_SHIFT; > > /* > * Check current cable state/cable type and store cable type > @@ -367,13 +368,13 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > } else { > *attached = true; > > - adclow = info->status[0] & STATUS1_ADCLOW_MASK; > - adclow >>= STATUS1_ADCLOW_SHIFT; > - adc1k = info->status[0] & STATUS1_ADC1K_MASK; > - adc1k >>= STATUS1_ADC1K_SHIFT; > + adclow = info->status[0] & MAX77693_STATUS1_ADCLOW_MASK; > + adclow >>= MAX77693_STATUS1_ADCLOW_SHIFT; > + adc1k = info->status[0] & MAX77693_STATUS1_ADC1K_MASK; > + adc1k >>= MAX77693_STATUS1_ADC1K_SHIFT; > > - vbvolt = info->status[1] & STATUS2_VBVOLT_MASK; > - vbvolt >>= STATUS2_VBVOLT_SHIFT; > + vbvolt = info->status[1] & MAX77693_STATUS2_VBVOLT_MASK; > + vbvolt >>= MAX77693_STATUS2_VBVOLT_SHIFT; > > /** > * [0x1|VBVolt|ADCLow|ADC1K] > @@ -398,8 +399,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read charger type to check cable type and decide cable state > * according to type of charger cable. > */ > - chg_type = info->status[1] & STATUS2_CHGTYP_MASK; > - chg_type >>= STATUS2_CHGTYP_SHIFT; > + chg_type = info->status[1] & MAX77693_STATUS2_CHGTYP_MASK; > + chg_type >>= MAX77693_STATUS2_CHGTYP_SHIFT; > > if (chg_type == MAX77693_CHARGER_TYPE_NONE) { > *attached = false; > @@ -423,10 +424,10 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read ADC value to check cable type and decide cable state > * according to cable type > */ > - adc = info->status[0] & STATUS1_ADC_MASK; > - adc >>= STATUS1_ADC_SHIFT; > - chg_type = info->status[1] & STATUS2_CHGTYP_MASK; > - chg_type >>= STATUS2_CHGTYP_SHIFT; > + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; > + adc >>= MAX77693_STATUS1_ADC_SHIFT; > + chg_type = info->status[1] & MAX77693_STATUS2_CHGTYP_MASK; > + chg_type >>= MAX77693_STATUS2_CHGTYP_SHIFT; > > if (adc == MAX77693_MUIC_ADC_OPEN > && chg_type == MAX77693_CHARGER_TYPE_NONE) > @@ -438,8 +439,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, > * Read vbvolt field, if vbvolt is 1, > * this cable is used for charging. > */ > - vbvolt = info->status[1] & STATUS2_VBVOLT_MASK; > - vbvolt >>= STATUS2_VBVOLT_SHIFT; > + vbvolt = info->status[1] & MAX77693_STATUS2_VBVOLT_MASK; > + vbvolt >>= MAX77693_STATUS2_VBVOLT_SHIFT; > > cable_type = vbvolt; > break; > @@ -521,7 +522,8 @@ static int max77693_muic_dock_handler(struct max77693_muic_info *info, > } > > /* Dock-Car/Desk/Audio, PATH:AUDIO */ > - ret = max77693_muic_set_path(info, CONTROL1_SW_AUDIO, attached); > + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_AUDIO, > + attached); > if (ret < 0) > return ret; > extcon_set_cable_state_(info->edev, dock_id, attached); > @@ -586,14 +588,16 @@ static int max77693_muic_adc_ground_handler(struct max77693_muic_info *info) > case MAX77693_MUIC_GND_USB_HOST: > case MAX77693_MUIC_GND_USB_HOST_VB: > /* USB_HOST, PATH: AP_USB */ > - ret = max77693_muic_set_path(info, CONTROL1_SW_USB, attached); > + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_USB, > + attached); > if (ret < 0) > return ret; > extcon_set_cable_state_(info->edev, EXTCON_USB_HOST, attached); > break; > case MAX77693_MUIC_GND_AV_CABLE_LOAD: > /* Audio Video Cable with load, PATH:AUDIO */ > - ret = max77693_muic_set_path(info, CONTROL1_SW_AUDIO, attached); > + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_AUDIO, > + attached); > if (ret < 0) > return ret; > extcon_set_cable_state_(info->edev, EXTCON_USB, attached); > @@ -616,7 +620,7 @@ static int max77693_muic_jig_handler(struct max77693_muic_info *info, > int cable_type, bool attached) > { > int ret = 0; > - u8 path = CONTROL1_SW_OPEN; > + u8 path = MAX77693_CONTROL1_SW_OPEN; > > dev_info(info->dev, > "external connector is %s (adc:0x%02x)\n", > @@ -626,12 +630,12 @@ static int max77693_muic_jig_handler(struct max77693_muic_info *info, > case MAX77693_MUIC_ADC_FACTORY_MODE_USB_OFF: /* ADC_JIG_USB_OFF */ > case MAX77693_MUIC_ADC_FACTORY_MODE_USB_ON: /* ADC_JIG_USB_ON */ > /* PATH:AP_USB */ > - path = CONTROL1_SW_USB; > + path = MAX77693_CONTROL1_SW_USB; > break; > case MAX77693_MUIC_ADC_FACTORY_MODE_UART_OFF: /* ADC_JIG_UART_OFF */ > case MAX77693_MUIC_ADC_FACTORY_MODE_UART_ON: /* ADC_JIG_UART_ON */ > /* PATH:AP_UART */ > - path = CONTROL1_SW_UART; > + path = MAX77693_CONTROL1_SW_UART; > break; > default: > dev_err(info->dev, "failed to detect %s jig cable\n", > @@ -1181,12 +1185,12 @@ static int max77693_muic_probe(struct platform_device *pdev) > if (muic_pdata->path_uart) > info->path_uart = muic_pdata->path_uart; > else > - info->path_uart = CONTROL1_SW_UART; > + info->path_uart = MAX77693_CONTROL1_SW_UART; > > if (muic_pdata->path_usb) > info->path_usb = muic_pdata->path_usb; > else > - info->path_usb = CONTROL1_SW_USB; > + info->path_usb = MAX77693_CONTROL1_SW_USB; > > /* > * Default delay time for detecting cable state > @@ -1198,8 +1202,8 @@ static int max77693_muic_probe(struct platform_device *pdev) > else > delay_jiffies = msecs_to_jiffies(DELAY_MS_DEFAULT); > } else { > - info->path_usb = CONTROL1_SW_USB; > - info->path_uart = CONTROL1_SW_UART; > + info->path_usb = MAX77693_CONTROL1_SW_USB; > + info->path_uart = MAX77693_CONTROL1_SW_UART; > delay_jiffies = msecs_to_jiffies(DELAY_MS_DEFAULT); > } > > diff --git a/include/linux/mfd/max77693-private.h b/include/linux/mfd/max77693-private.h > index 8c4143c0c651..3c7a63b98ad6 100644 > --- a/include/linux/mfd/max77693-private.h > +++ b/include/linux/mfd/max77693-private.h > @@ -310,30 +310,30 @@ enum max77693_muic_reg { > #define INTMASK2_CHGTYP_MASK (1 << INTMASK2_CHGTYP_SHIFT) > > /* MAX77693 MUIC - STATUS1~3 Register */ > -#define STATUS1_ADC_SHIFT (0) > -#define STATUS1_ADCLOW_SHIFT (5) > -#define STATUS1_ADCERR_SHIFT (6) > -#define STATUS1_ADC1K_SHIFT (7) > -#define STATUS1_ADC_MASK (0x1f << STATUS1_ADC_SHIFT) > -#define STATUS1_ADCLOW_MASK (0x1 << STATUS1_ADCLOW_SHIFT) > -#define STATUS1_ADCERR_MASK (0x1 << STATUS1_ADCERR_SHIFT) > -#define STATUS1_ADC1K_MASK (0x1 << STATUS1_ADC1K_SHIFT) > - > -#define STATUS2_CHGTYP_SHIFT (0) > -#define STATUS2_CHGDETRUN_SHIFT (3) > -#define STATUS2_DCDTMR_SHIFT (4) > -#define STATUS2_DXOVP_SHIFT (5) > -#define STATUS2_VBVOLT_SHIFT (6) > -#define STATUS2_VIDRM_SHIFT (7) > -#define STATUS2_CHGTYP_MASK (0x7 << STATUS2_CHGTYP_SHIFT) > -#define STATUS2_CHGDETRUN_MASK (0x1 << STATUS2_CHGDETRUN_SHIFT) > -#define STATUS2_DCDTMR_MASK (0x1 << STATUS2_DCDTMR_SHIFT) > -#define STATUS2_DXOVP_MASK (0x1 << STATUS2_DXOVP_SHIFT) > -#define STATUS2_VBVOLT_MASK (0x1 << STATUS2_VBVOLT_SHIFT) > -#define STATUS2_VIDRM_MASK (0x1 << STATUS2_VIDRM_SHIFT) > - > -#define STATUS3_OVP_SHIFT (2) > -#define STATUS3_OVP_MASK (0x1 << STATUS3_OVP_SHIFT) > +#define MAX77693_STATUS1_ADC_SHIFT 0 > +#define MAX77693_STATUS1_ADCLOW_SHIFT 5 > +#define MAX77693_STATUS1_ADCERR_SHIFT 6 > +#define MAX77693_STATUS1_ADC1K_SHIFT 7 > +#define MAX77693_STATUS1_ADC_MASK (0x1f << MAX77693_STATUS1_ADC_SHIFT) > +#define MAX77693_STATUS1_ADCLOW_MASK BIT(MAX77693_STATUS1_ADCLOW_SHIFT) > +#define MAX77693_STATUS1_ADCERR_MASK BIT(MAX77693_STATUS1_ADCERR_SHIFT) > +#define MAX77693_STATUS1_ADC1K_MASK BIT(MAX77693_STATUS1_ADC1K_SHIFT) > + > +#define MAX77693_STATUS2_CHGTYP_SHIFT 0 > +#define MAX77693_STATUS2_CHGDETRUN_SHIFT 3 > +#define MAX77693_STATUS2_DCDTMR_SHIFT 4 > +#define MAX77693_STATUS2_DXOVP_SHIFT 5 > +#define MAX77693_STATUS2_VBVOLT_SHIFT 6 > +#define MAX77693_STATUS2_VIDRM_SHIFT 7 > +#define MAX77693_STATUS2_CHGTYP_MASK (0x7 << MAX77693_STATUS2_CHGTYP_SHIFT) > +#define MAX77693_STATUS2_CHGDETRUN_MASK BIT(MAX77693_STATUS2_CHGDETRUN_SHIFT) > +#define MAX77693_STATUS2_DCDTMR_MASK BIT(MAX77693_STATUS2_DCDTMR_SHIFT) > +#define MAX77693_STATUS2_DXOVP_MASK BIT(MAX77693_STATUS2_DXOVP_SHIFT) > +#define MAX77693_STATUS2_VBVOLT_MASK BIT(MAX77693_STATUS2_VBVOLT_SHIFT) > +#define MAX77693_STATUS2_VIDRM_MASK BIT(MAX77693_STATUS2_VIDRM_SHIFT) > + > +#define MAX77693_STATUS3_OVP_SHIFT 2 > +#define MAX77693_STATUS3_OVP_MASK BIT(MAX77693_STATUS3_OVP_SHIFT) > > /* MAX77693 CDETCTRL1~2 register */ > #define CDETCTRL1_CHGDETEN_SHIFT (0) > @@ -362,38 +362,38 @@ enum max77693_muic_reg { > #define COMN1SW_MASK (0x7 << COMN1SW_SHIFT) > #define COMP2SW_MASK (0x7 << COMP2SW_SHIFT) > #define COMP_SW_MASK (COMP2SW_MASK | COMN1SW_MASK) > -#define CONTROL1_SW_USB ((1 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_USB ((1 << COMP2SW_SHIFT) \ > | (1 << COMN1SW_SHIFT)) > -#define CONTROL1_SW_AUDIO ((2 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_AUDIO ((2 << COMP2SW_SHIFT) \ > | (2 << COMN1SW_SHIFT)) > -#define CONTROL1_SW_UART ((3 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_UART ((3 << COMP2SW_SHIFT) \ > | (3 << COMN1SW_SHIFT)) > -#define CONTROL1_SW_OPEN ((0 << COMP2SW_SHIFT) \ > +#define MAX77693_CONTROL1_SW_OPEN ((0 << COMP2SW_SHIFT) \ > | (0 << COMN1SW_SHIFT)) > > -#define CONTROL2_LOWPWR_SHIFT (0) > -#define CONTROL2_ADCEN_SHIFT (1) > -#define CONTROL2_CPEN_SHIFT (2) > -#define CONTROL2_SFOUTASRT_SHIFT (3) > -#define CONTROL2_SFOUTORD_SHIFT (4) > -#define CONTROL2_ACCDET_SHIFT (5) > -#define CONTROL2_USBCPINT_SHIFT (6) > -#define CONTROL2_RCPS_SHIFT (7) > -#define CONTROL2_LOWPWR_MASK (0x1 << CONTROL2_LOWPWR_SHIFT) > -#define CONTROL2_ADCEN_MASK (0x1 << CONTROL2_ADCEN_SHIFT) > -#define CONTROL2_CPEN_MASK (0x1 << CONTROL2_CPEN_SHIFT) > -#define CONTROL2_SFOUTASRT_MASK (0x1 << CONTROL2_SFOUTASRT_SHIFT) > -#define CONTROL2_SFOUTORD_MASK (0x1 << CONTROL2_SFOUTORD_SHIFT) > -#define CONTROL2_ACCDET_MASK (0x1 << CONTROL2_ACCDET_SHIFT) > -#define CONTROL2_USBCPINT_MASK (0x1 << CONTROL2_USBCPINT_SHIFT) > -#define CONTROL2_RCPS_MASK (0x1 << CONTROL2_RCPS_SHIFT) > - > -#define CONTROL3_JIGSET_SHIFT (0) > -#define CONTROL3_BTLDSET_SHIFT (2) > -#define CONTROL3_ADCDBSET_SHIFT (4) > -#define CONTROL3_JIGSET_MASK (0x3 << CONTROL3_JIGSET_SHIFT) > -#define CONTROL3_BTLDSET_MASK (0x3 << CONTROL3_BTLDSET_SHIFT) > -#define CONTROL3_ADCDBSET_MASK (0x3 << CONTROL3_ADCDBSET_SHIFT) > +#define MAX77693_CONTROL2_LOWPWR_SHIFT 0 > +#define MAX77693_CONTROL2_ADCEN_SHIFT 1 > +#define MAX77693_CONTROL2_CPEN_SHIFT 2 > +#define MAX77693_CONTROL2_SFOUTASRT_SHIFT 3 > +#define MAX77693_CONTROL2_SFOUTORD_SHIFT 4 > +#define MAX77693_CONTROL2_ACCDET_SHIFT 5 > +#define MAX77693_CONTROL2_USBCPINT_SHIFT 6 > +#define MAX77693_CONTROL2_RCPS_SHIFT 7 > +#define MAX77693_CONTROL2_LOWPWR_MASK BIT(MAX77693_CONTROL2_LOWPWR_SHIFT) > +#define MAX77693_CONTROL2_ADCEN_MASK BIT(MAX77693_CONTROL2_ADCEN_SHIFT) > +#define MAX77693_CONTROL2_CPEN_MASK BIT(MAX77693_CONTROL2_CPEN_SHIFT) > +#define MAX77693_CONTROL2_SFOUTASRT_MASK BIT(MAX77693_CONTROL2_SFOUTASRT_SHIFT) > +#define MAX77693_CONTROL2_SFOUTORD_MASK BIT(MAX77693_CONTROL2_SFOUTORD_SHIFT) > +#define MAX77693_CONTROL2_ACCDET_MASK BIT(MAX77693_CONTROL2_ACCDET_SHIFT) > +#define MAX77693_CONTROL2_USBCPINT_MASK BIT(MAX77693_CONTROL2_USBCPINT_SHIFT) > +#define MAX77693_CONTROL2_RCPS_MASK BIT(MAX77693_CONTROL2_RCPS_SHIFT) > + > +#define MAX77693_CONTROL3_JIGSET_SHIFT 0 > +#define MAX77693_CONTROL3_BTLDSET_SHIFT 2 > +#define MAX77693_CONTROL3_ADCDBSET_SHIFT 4 > +#define MAX77693_CONTROL3_JIGSET_MASK (0x3 << MAX77693_CONTROL3_JIGSET_SHIFT) > +#define MAX77693_CONTROL3_BTLDSET_MASK (0x3 << MAX77693_CONTROL3_BTLDSET_SHIFT) > +#define MAX77693_CONTROL3_ADCDBSET_MASK (0x3 << MAX77693_CONTROL3_ADCDBSET_SHIFT) > > /* Slave addr = 0x90: Haptic */ > enum max77693_haptic_reg { > -- > 2.1.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c index 4ef45f257e6e..96f1545be4ab 100644 --- a/drivers/extcon/extcon-max77693.c +++ b/drivers/extcon/extcon-max77693.c @@ -43,7 +43,7 @@ static struct max77693_reg_data default_init_data[] = { { /* STATUS2 - [3]ChgDetRun */ .addr = MAX77693_MUIC_REG_STATUS2, - .data = STATUS2_CHGDETRUN_MASK, + .data = MAX77693_STATUS2_CHGDETRUN_MASK, }, { /* INTMASK1 - Unmask [3]ADC1KM,[0]ADCM */ .addr = MAX77693_MUIC_REG_INTMASK1, @@ -236,7 +236,7 @@ static int max77693_muic_set_debounce_time(struct max77693_muic_info *info, */ ret = regmap_write(info->max77693->regmap_muic, MAX77693_MUIC_REG_CTRL3, - time << CONTROL3_ADCDBSET_SHIFT); + time << MAX77693_CONTROL3_ADCDBSET_SHIFT); if (ret) { dev_err(info->dev, "failed to set ADC debounce time\n"); return ret; @@ -269,7 +269,7 @@ static int max77693_muic_set_path(struct max77693_muic_info *info, if (attached) ctrl1 = val; else - ctrl1 = CONTROL1_SW_OPEN; + ctrl1 = MAX77693_CONTROL1_SW_OPEN; ret = regmap_update_bits(info->max77693->regmap_muic, MAX77693_MUIC_REG_CTRL1, COMP_SW_MASK, ctrl1); @@ -279,13 +279,14 @@ static int max77693_muic_set_path(struct max77693_muic_info *info, } if (attached) - ctrl2 |= CONTROL2_CPEN_MASK; /* LowPwr=0, CPEn=1 */ + ctrl2 |= MAX77693_CONTROL2_CPEN_MASK; /* LowPwr=0, CPEn=1 */ else - ctrl2 |= CONTROL2_LOWPWR_MASK; /* LowPwr=1, CPEn=0 */ + ctrl2 |= MAX77693_CONTROL2_LOWPWR_MASK; /* LowPwr=1, CPEn=0 */ ret = regmap_update_bits(info->max77693->regmap_muic, MAX77693_MUIC_REG_CTRL2, - CONTROL2_LOWPWR_MASK | CONTROL2_CPEN_MASK, ctrl2); + MAX77693_CONTROL2_LOWPWR_MASK | MAX77693_CONTROL2_CPEN_MASK, + ctrl2); if (ret < 0) { dev_err(info->dev, "failed to update MUIC register\n"); return ret; @@ -327,8 +328,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, * Read ADC value to check cable type and decide cable state * according to cable type */ - adc = info->status[0] & STATUS1_ADC_MASK; - adc >>= STATUS1_ADC_SHIFT; + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; + adc >>= MAX77693_STATUS1_ADC_SHIFT; /* * Check current cable state/cable type and store cable type @@ -351,8 +352,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, * Read ADC value to check cable type and decide cable state * according to cable type */ - adc = info->status[0] & STATUS1_ADC_MASK; - adc >>= STATUS1_ADC_SHIFT; + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; + adc >>= MAX77693_STATUS1_ADC_SHIFT; /* * Check current cable state/cable type and store cable type @@ -367,13 +368,13 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, } else { *attached = true; - adclow = info->status[0] & STATUS1_ADCLOW_MASK; - adclow >>= STATUS1_ADCLOW_SHIFT; - adc1k = info->status[0] & STATUS1_ADC1K_MASK; - adc1k >>= STATUS1_ADC1K_SHIFT; + adclow = info->status[0] & MAX77693_STATUS1_ADCLOW_MASK; + adclow >>= MAX77693_STATUS1_ADCLOW_SHIFT; + adc1k = info->status[0] & MAX77693_STATUS1_ADC1K_MASK; + adc1k >>= MAX77693_STATUS1_ADC1K_SHIFT; - vbvolt = info->status[1] & STATUS2_VBVOLT_MASK; - vbvolt >>= STATUS2_VBVOLT_SHIFT; + vbvolt = info->status[1] & MAX77693_STATUS2_VBVOLT_MASK; + vbvolt >>= MAX77693_STATUS2_VBVOLT_SHIFT; /** * [0x1|VBVolt|ADCLow|ADC1K] @@ -398,8 +399,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, * Read charger type to check cable type and decide cable state * according to type of charger cable. */ - chg_type = info->status[1] & STATUS2_CHGTYP_MASK; - chg_type >>= STATUS2_CHGTYP_SHIFT; + chg_type = info->status[1] & MAX77693_STATUS2_CHGTYP_MASK; + chg_type >>= MAX77693_STATUS2_CHGTYP_SHIFT; if (chg_type == MAX77693_CHARGER_TYPE_NONE) { *attached = false; @@ -423,10 +424,10 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, * Read ADC value to check cable type and decide cable state * according to cable type */ - adc = info->status[0] & STATUS1_ADC_MASK; - adc >>= STATUS1_ADC_SHIFT; - chg_type = info->status[1] & STATUS2_CHGTYP_MASK; - chg_type >>= STATUS2_CHGTYP_SHIFT; + adc = info->status[0] & MAX77693_STATUS1_ADC_MASK; + adc >>= MAX77693_STATUS1_ADC_SHIFT; + chg_type = info->status[1] & MAX77693_STATUS2_CHGTYP_MASK; + chg_type >>= MAX77693_STATUS2_CHGTYP_SHIFT; if (adc == MAX77693_MUIC_ADC_OPEN && chg_type == MAX77693_CHARGER_TYPE_NONE) @@ -438,8 +439,8 @@ static int max77693_muic_get_cable_type(struct max77693_muic_info *info, * Read vbvolt field, if vbvolt is 1, * this cable is used for charging. */ - vbvolt = info->status[1] & STATUS2_VBVOLT_MASK; - vbvolt >>= STATUS2_VBVOLT_SHIFT; + vbvolt = info->status[1] & MAX77693_STATUS2_VBVOLT_MASK; + vbvolt >>= MAX77693_STATUS2_VBVOLT_SHIFT; cable_type = vbvolt; break; @@ -521,7 +522,8 @@ static int max77693_muic_dock_handler(struct max77693_muic_info *info, } /* Dock-Car/Desk/Audio, PATH:AUDIO */ - ret = max77693_muic_set_path(info, CONTROL1_SW_AUDIO, attached); + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_AUDIO, + attached); if (ret < 0) return ret; extcon_set_cable_state_(info->edev, dock_id, attached); @@ -586,14 +588,16 @@ static int max77693_muic_adc_ground_handler(struct max77693_muic_info *info) case MAX77693_MUIC_GND_USB_HOST: case MAX77693_MUIC_GND_USB_HOST_VB: /* USB_HOST, PATH: AP_USB */ - ret = max77693_muic_set_path(info, CONTROL1_SW_USB, attached); + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_USB, + attached); if (ret < 0) return ret; extcon_set_cable_state_(info->edev, EXTCON_USB_HOST, attached); break; case MAX77693_MUIC_GND_AV_CABLE_LOAD: /* Audio Video Cable with load, PATH:AUDIO */ - ret = max77693_muic_set_path(info, CONTROL1_SW_AUDIO, attached); + ret = max77693_muic_set_path(info, MAX77693_CONTROL1_SW_AUDIO, + attached); if (ret < 0) return ret; extcon_set_cable_state_(info->edev, EXTCON_USB, attached); @@ -616,7 +620,7 @@ static int max77693_muic_jig_handler(struct max77693_muic_info *info, int cable_type, bool attached) { int ret = 0; - u8 path = CONTROL1_SW_OPEN; + u8 path = MAX77693_CONTROL1_SW_OPEN; dev_info(info->dev, "external connector is %s (adc:0x%02x)\n", @@ -626,12 +630,12 @@ static int max77693_muic_jig_handler(struct max77693_muic_info *info, case MAX77693_MUIC_ADC_FACTORY_MODE_USB_OFF: /* ADC_JIG_USB_OFF */ case MAX77693_MUIC_ADC_FACTORY_MODE_USB_ON: /* ADC_JIG_USB_ON */ /* PATH:AP_USB */ - path = CONTROL1_SW_USB; + path = MAX77693_CONTROL1_SW_USB; break; case MAX77693_MUIC_ADC_FACTORY_MODE_UART_OFF: /* ADC_JIG_UART_OFF */ case MAX77693_MUIC_ADC_FACTORY_MODE_UART_ON: /* ADC_JIG_UART_ON */ /* PATH:AP_UART */ - path = CONTROL1_SW_UART; + path = MAX77693_CONTROL1_SW_UART; break; default: dev_err(info->dev, "failed to detect %s jig cable\n", @@ -1181,12 +1185,12 @@ static int max77693_muic_probe(struct platform_device *pdev) if (muic_pdata->path_uart) info->path_uart = muic_pdata->path_uart; else - info->path_uart = CONTROL1_SW_UART; + info->path_uart = MAX77693_CONTROL1_SW_UART; if (muic_pdata->path_usb) info->path_usb = muic_pdata->path_usb; else - info->path_usb = CONTROL1_SW_USB; + info->path_usb = MAX77693_CONTROL1_SW_USB; /* * Default delay time for detecting cable state @@ -1198,8 +1202,8 @@ static int max77693_muic_probe(struct platform_device *pdev) else delay_jiffies = msecs_to_jiffies(DELAY_MS_DEFAULT); } else { - info->path_usb = CONTROL1_SW_USB; - info->path_uart = CONTROL1_SW_UART; + info->path_usb = MAX77693_CONTROL1_SW_USB; + info->path_uart = MAX77693_CONTROL1_SW_UART; delay_jiffies = msecs_to_jiffies(DELAY_MS_DEFAULT); } diff --git a/include/linux/mfd/max77693-private.h b/include/linux/mfd/max77693-private.h index 8c4143c0c651..3c7a63b98ad6 100644 --- a/include/linux/mfd/max77693-private.h +++ b/include/linux/mfd/max77693-private.h @@ -310,30 +310,30 @@ enum max77693_muic_reg { #define INTMASK2_CHGTYP_MASK (1 << INTMASK2_CHGTYP_SHIFT) /* MAX77693 MUIC - STATUS1~3 Register */ -#define STATUS1_ADC_SHIFT (0) -#define STATUS1_ADCLOW_SHIFT (5) -#define STATUS1_ADCERR_SHIFT (6) -#define STATUS1_ADC1K_SHIFT (7) -#define STATUS1_ADC_MASK (0x1f << STATUS1_ADC_SHIFT) -#define STATUS1_ADCLOW_MASK (0x1 << STATUS1_ADCLOW_SHIFT) -#define STATUS1_ADCERR_MASK (0x1 << STATUS1_ADCERR_SHIFT) -#define STATUS1_ADC1K_MASK (0x1 << STATUS1_ADC1K_SHIFT) - -#define STATUS2_CHGTYP_SHIFT (0) -#define STATUS2_CHGDETRUN_SHIFT (3) -#define STATUS2_DCDTMR_SHIFT (4) -#define STATUS2_DXOVP_SHIFT (5) -#define STATUS2_VBVOLT_SHIFT (6) -#define STATUS2_VIDRM_SHIFT (7) -#define STATUS2_CHGTYP_MASK (0x7 << STATUS2_CHGTYP_SHIFT) -#define STATUS2_CHGDETRUN_MASK (0x1 << STATUS2_CHGDETRUN_SHIFT) -#define STATUS2_DCDTMR_MASK (0x1 << STATUS2_DCDTMR_SHIFT) -#define STATUS2_DXOVP_MASK (0x1 << STATUS2_DXOVP_SHIFT) -#define STATUS2_VBVOLT_MASK (0x1 << STATUS2_VBVOLT_SHIFT) -#define STATUS2_VIDRM_MASK (0x1 << STATUS2_VIDRM_SHIFT) - -#define STATUS3_OVP_SHIFT (2) -#define STATUS3_OVP_MASK (0x1 << STATUS3_OVP_SHIFT) +#define MAX77693_STATUS1_ADC_SHIFT 0 +#define MAX77693_STATUS1_ADCLOW_SHIFT 5 +#define MAX77693_STATUS1_ADCERR_SHIFT 6 +#define MAX77693_STATUS1_ADC1K_SHIFT 7 +#define MAX77693_STATUS1_ADC_MASK (0x1f << MAX77693_STATUS1_ADC_SHIFT) +#define MAX77693_STATUS1_ADCLOW_MASK BIT(MAX77693_STATUS1_ADCLOW_SHIFT) +#define MAX77693_STATUS1_ADCERR_MASK BIT(MAX77693_STATUS1_ADCERR_SHIFT) +#define MAX77693_STATUS1_ADC1K_MASK BIT(MAX77693_STATUS1_ADC1K_SHIFT) + +#define MAX77693_STATUS2_CHGTYP_SHIFT 0 +#define MAX77693_STATUS2_CHGDETRUN_SHIFT 3 +#define MAX77693_STATUS2_DCDTMR_SHIFT 4 +#define MAX77693_STATUS2_DXOVP_SHIFT 5 +#define MAX77693_STATUS2_VBVOLT_SHIFT 6 +#define MAX77693_STATUS2_VIDRM_SHIFT 7 +#define MAX77693_STATUS2_CHGTYP_MASK (0x7 << MAX77693_STATUS2_CHGTYP_SHIFT) +#define MAX77693_STATUS2_CHGDETRUN_MASK BIT(MAX77693_STATUS2_CHGDETRUN_SHIFT) +#define MAX77693_STATUS2_DCDTMR_MASK BIT(MAX77693_STATUS2_DCDTMR_SHIFT) +#define MAX77693_STATUS2_DXOVP_MASK BIT(MAX77693_STATUS2_DXOVP_SHIFT) +#define MAX77693_STATUS2_VBVOLT_MASK BIT(MAX77693_STATUS2_VBVOLT_SHIFT) +#define MAX77693_STATUS2_VIDRM_MASK BIT(MAX77693_STATUS2_VIDRM_SHIFT) + +#define MAX77693_STATUS3_OVP_SHIFT 2 +#define MAX77693_STATUS3_OVP_MASK BIT(MAX77693_STATUS3_OVP_SHIFT) /* MAX77693 CDETCTRL1~2 register */ #define CDETCTRL1_CHGDETEN_SHIFT (0) @@ -362,38 +362,38 @@ enum max77693_muic_reg { #define COMN1SW_MASK (0x7 << COMN1SW_SHIFT) #define COMP2SW_MASK (0x7 << COMP2SW_SHIFT) #define COMP_SW_MASK (COMP2SW_MASK | COMN1SW_MASK) -#define CONTROL1_SW_USB ((1 << COMP2SW_SHIFT) \ +#define MAX77693_CONTROL1_SW_USB ((1 << COMP2SW_SHIFT) \ | (1 << COMN1SW_SHIFT)) -#define CONTROL1_SW_AUDIO ((2 << COMP2SW_SHIFT) \ +#define MAX77693_CONTROL1_SW_AUDIO ((2 << COMP2SW_SHIFT) \ | (2 << COMN1SW_SHIFT)) -#define CONTROL1_SW_UART ((3 << COMP2SW_SHIFT) \ +#define MAX77693_CONTROL1_SW_UART ((3 << COMP2SW_SHIFT) \ | (3 << COMN1SW_SHIFT)) -#define CONTROL1_SW_OPEN ((0 << COMP2SW_SHIFT) \ +#define MAX77693_CONTROL1_SW_OPEN ((0 << COMP2SW_SHIFT) \ | (0 << COMN1SW_SHIFT)) -#define CONTROL2_LOWPWR_SHIFT (0) -#define CONTROL2_ADCEN_SHIFT (1) -#define CONTROL2_CPEN_SHIFT (2) -#define CONTROL2_SFOUTASRT_SHIFT (3) -#define CONTROL2_SFOUTORD_SHIFT (4) -#define CONTROL2_ACCDET_SHIFT (5) -#define CONTROL2_USBCPINT_SHIFT (6) -#define CONTROL2_RCPS_SHIFT (7) -#define CONTROL2_LOWPWR_MASK (0x1 << CONTROL2_LOWPWR_SHIFT) -#define CONTROL2_ADCEN_MASK (0x1 << CONTROL2_ADCEN_SHIFT) -#define CONTROL2_CPEN_MASK (0x1 << CONTROL2_CPEN_SHIFT) -#define CONTROL2_SFOUTASRT_MASK (0x1 << CONTROL2_SFOUTASRT_SHIFT) -#define CONTROL2_SFOUTORD_MASK (0x1 << CONTROL2_SFOUTORD_SHIFT) -#define CONTROL2_ACCDET_MASK (0x1 << CONTROL2_ACCDET_SHIFT) -#define CONTROL2_USBCPINT_MASK (0x1 << CONTROL2_USBCPINT_SHIFT) -#define CONTROL2_RCPS_MASK (0x1 << CONTROL2_RCPS_SHIFT) - -#define CONTROL3_JIGSET_SHIFT (0) -#define CONTROL3_BTLDSET_SHIFT (2) -#define CONTROL3_ADCDBSET_SHIFT (4) -#define CONTROL3_JIGSET_MASK (0x3 << CONTROL3_JIGSET_SHIFT) -#define CONTROL3_BTLDSET_MASK (0x3 << CONTROL3_BTLDSET_SHIFT) -#define CONTROL3_ADCDBSET_MASK (0x3 << CONTROL3_ADCDBSET_SHIFT) +#define MAX77693_CONTROL2_LOWPWR_SHIFT 0 +#define MAX77693_CONTROL2_ADCEN_SHIFT 1 +#define MAX77693_CONTROL2_CPEN_SHIFT 2 +#define MAX77693_CONTROL2_SFOUTASRT_SHIFT 3 +#define MAX77693_CONTROL2_SFOUTORD_SHIFT 4 +#define MAX77693_CONTROL2_ACCDET_SHIFT 5 +#define MAX77693_CONTROL2_USBCPINT_SHIFT 6 +#define MAX77693_CONTROL2_RCPS_SHIFT 7 +#define MAX77693_CONTROL2_LOWPWR_MASK BIT(MAX77693_CONTROL2_LOWPWR_SHIFT) +#define MAX77693_CONTROL2_ADCEN_MASK BIT(MAX77693_CONTROL2_ADCEN_SHIFT) +#define MAX77693_CONTROL2_CPEN_MASK BIT(MAX77693_CONTROL2_CPEN_SHIFT) +#define MAX77693_CONTROL2_SFOUTASRT_MASK BIT(MAX77693_CONTROL2_SFOUTASRT_SHIFT) +#define MAX77693_CONTROL2_SFOUTORD_MASK BIT(MAX77693_CONTROL2_SFOUTORD_SHIFT) +#define MAX77693_CONTROL2_ACCDET_MASK BIT(MAX77693_CONTROL2_ACCDET_SHIFT) +#define MAX77693_CONTROL2_USBCPINT_MASK BIT(MAX77693_CONTROL2_USBCPINT_SHIFT) +#define MAX77693_CONTROL2_RCPS_MASK BIT(MAX77693_CONTROL2_RCPS_SHIFT) + +#define MAX77693_CONTROL3_JIGSET_SHIFT 0 +#define MAX77693_CONTROL3_BTLDSET_SHIFT 2 +#define MAX77693_CONTROL3_ADCDBSET_SHIFT 4 +#define MAX77693_CONTROL3_JIGSET_MASK (0x3 << MAX77693_CONTROL3_JIGSET_SHIFT) +#define MAX77693_CONTROL3_BTLDSET_MASK (0x3 << MAX77693_CONTROL3_BTLDSET_SHIFT) +#define MAX77693_CONTROL3_ADCDBSET_MASK (0x3 << MAX77693_CONTROL3_ADCDBSET_SHIFT) /* Slave addr = 0x90: Haptic */ enum max77693_haptic_reg {
Add MAX77693 prefix to some of the defines used in max77693 extcon driver so the max77693-private.h can be included simultaneously with max77843-private.h. Additionally use BIT() macro in header. Signed-off-by: Krzysztof Kozlowski <k.kozlowski.k@gmail.com> --- drivers/extcon/extcon-max77693.c | 72 +++++++++++++------------ include/linux/mfd/max77693-private.h | 102 +++++++++++++++++------------------ 2 files changed, 89 insertions(+), 85 deletions(-)