Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1615052
| From | Chanwoo Choi <cw00.choi@samsung.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v2 2/2] extcon: max77843: support USB accessories as external USB hosts |
| Date | 2017-04-03 12:10 +0200 |
| Message-ID | <ts75T-5t7-11@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> <ts6Mx-55V-5@gated-at.bofh.it> |
| Organization | Samsung Electronics |
On 2017년 04월 03일 18:41, Andi Shyti wrote:
> 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'm not sure that other MAX77843_MUIC_ADC_RESERVED_ACC_1/2/3/5 are always
same with MAX77843_MUIC_ADC_RESERVED_ACC_4. Also, we don't have the
any test case for MAX77843_MUIC_ADC_RESERVED_ACC_1/2/3/5.
>
> 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
>
>
> .
>
--
Best Regards,
Chanwoo Choi
Samsung Electronics
Back to linux.kernel | Previous | Next — Previous 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