Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1615038
| From | Andi Shyti <andi.shyti@samsung.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v2 2/2] extcon: max77843: support USB accessories as external USB hosts |
| Date | 2017-04-03 11:50 +0200 |
| Message-ID | <ts6Mx-55V-5@gated-at.bofh.it> (permalink) |
| References | <trGp3-4Fr-5@gated-at.bofh.it> <ts5x7-4oz-13@gated-at.bofh.it> <trGp3-4Fr-3@gated-at.bofh.it> <ts5x7-4oz-11@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
Hi Chanwoo,
> > @@ -403,6 +417,11 @@ static int max77843_muic_adc_handler(struct max77843_muic_info *info)
> >
> > switch (cable_type) {
> > case MAX77843_MUIC_ADC_GROUND:
> > + case MAX77843_MUIC_ADC_RESERVED_ACC_1:
> > + case MAX77843_MUIC_ADC_RESERVED_ACC_2:
> > + case MAX77843_MUIC_ADC_RESERVED_ACC_3:
> > + case MAX77843_MUIC_ADC_RESERVED_ACC_4:
> > + case MAX77843_MUIC_ADC_RESERVED_ACC_5:
> > ret = max77843_muic_adc_gnd_handler(info);
> > if (ret < 0)
> > return ret;
> > @@ -427,11 +446,6 @@ static int max77843_muic_adc_handler(struct max77843_muic_info *info)
> > case MAX77843_MUIC_ADC_REMOTE_S10_BUTTON:
> > case MAX77843_MUIC_ADC_REMOTE_S11_BUTTON:
> > case MAX77843_MUIC_ADC_REMOTE_S12_BUTTON:
> > - case MAX77843_MUIC_ADC_RESERVED_ACC_1:
> > - case MAX77843_MUIC_ADC_RESERVED_ACC_2:
> > - case MAX77843_MUIC_ADC_RESERVED_ACC_3:
> > - case MAX77843_MUIC_ADC_RESERVED_ACC_4:
> > - case MAX77843_MUIC_ADC_RESERVED_ACC_5:
> > case MAX77843_MUIC_ADC_AUDIO_DEVICE_TYPE2:
> > case MAX77843_MUIC_ADC_PHONE_POWERED_DEV:
> > case MAX77843_MUIC_ADC_TTY_CONVERTER:
> >
>
> The extcon-max77843.c driver was implemented with real use-case
> by testing the various kind of external connectors.
>
> I want to apply the part for real use-case of this patch.
> As I knew, you tested this patch with only MAX77843_MUIC_ADC_RESERVED_ACC_4.
> So, I prefer to apply following patch only for MAX77843_MUIC_ADC_RESERVED_ACC_4.
OK, I will do as you say, although I don't agree (as we
discussed on review.tizen.org).
I will drop patch 1 and change only ACC_4.
> --
> drivers/extcon/extcon-max77843.c | 12 +++++++++++-
> 1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/extcon/extcon-max77843.c b/drivers/extcon/extcon-max77843.c
> index fcdabc4b4025..45367d74bc01 100644
> --- a/drivers/extcon/extcon-max77843.c
> +++ b/drivers/extcon/extcon-max77843.c
> @@ -271,6 +271,16 @@ static int max77843_muic_get_cable_type(struct max77843_muic_info *info,
> } else {
> *attached = true;
> switch (adc) {
> + case MAX77843_MUIC_ADC_RESERVED_ACC_4:
> + /*
> + * The MAX77843_MUIC_ADC_RESERVED_ACC_* type are
> + * external accessories and as such they don't
> + * need any charging action. In this case just
> + * return MAX77843_MUIC_CHG_NONE so that
> + * charging type interrupts are ignored.
> + */
> + info->prev_chg_type = MAX77843_MUIC_CHG_NONE;
> + break;
> case MAX77843_MUIC_ADC_GROUND:
> info->prev_chg_type = MAX77843_MUIC_CHG_GND;
> break;
> @@ -403,6 +413,7 @@ static int max77843_muic_adc_handler(struct max77843_muic_info *info)
>
> switch (cable_type) {
> case MAX77843_MUIC_ADC_GROUND:
> + case MAX77843_MUIC_ADC_RESERVED_ACC_4:
> ret = max77843_muic_adc_gnd_handler(info);
> if (ret < 0)
> return ret;
> @@ -430,7 +441,6 @@ static int max77843_muic_adc_handler(struct max77843_muic_info *info)
> case MAX77843_MUIC_ADC_RESERVED_ACC_1:
> case MAX77843_MUIC_ADC_RESERVED_ACC_2:
> case MAX77843_MUIC_ADC_RESERVED_ACC_3:
> - case MAX77843_MUIC_ADC_RESERVED_ACC_4:
> case MAX77843_MUIC_ADC_RESERVED_ACC_5:
> case MAX77843_MUIC_ADC_AUDIO_DEVICE_TYPE2:
> case MAX77843_MUIC_ADC_PHONE_POWERED_DEV:
This patch, of course, doesn't make sense because it's based on
top of the previous that you asked me to drop. I will send the
correct one.
Thanks,
Andi
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v2 2/2] extcon: max77843: support USB accessories as external USB hosts Andi Shyti <andi.shyti@gmail.com> - 2017-04-02 07:40 +0200
Re: [PATCH v2 2/2] extcon: max77843: support USB accessories as external USB hosts Chanwoo Choi <cw00.choi@samsung.com> - 2017-04-03 10:30 +0200
Re: [PATCH v2 2/2] extcon: max77843: support USB accessories as external USB hosts Andi Shyti <andi.shyti@samsung.com> - 2017-04-03 11:50 +0200
Re: [PATCH v2 2/2] extcon: max77843: support USB accessories as external USB hosts Chanwoo Choi <cw00.choi@samsung.com> - 2017-04-03 12:10 +0200
csiph-web