Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1421921 > unrolled thread

[PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO detection

Back to article view | Back to linux.kernel

Contents

  [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO detection Roger Quadros <rogerq@ti.com> - 2016-06-14 16:10 +0200
    Re: [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO  detection Chanwoo Choi <cw00.choi@samsung.com> - 2016-06-15 04:00 +0200
      Re: [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO  detection Roger Quadros <rogerq@ti.com> - 2016-06-15 08:50 +0200
        Re: [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO  detection Chanwoo Choi <cw00.choi@samsung.com> - 2016-06-15 10:00 +0200
    [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO  detection Roger Quadros <rogerq@ti.com> - 2016-06-15 10:20 +0200

#1421921 — [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO detection

If USB cable is connected prior to boot, we don't get any interrupts
so we must manually check the VBUS state and report it during probe.
If we don't do it then USB controller will never know that peripheral
cable was connected till the user unplugs and replugs the cable.

Fixes: b7aad8e2685b ("extcon: palmas: Add the support for VBUS detection by using GPIO")
Signed-off-by: Roger Quadros <rogerq@ti.com>
---
 drivers/extcon/extcon-palmas.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/extcon/extcon-palmas.c b/drivers/extcon/extcon-palmas.c
index 8b3226d..caff46c 100644
--- a/drivers/extcon/extcon-palmas.c
+++ b/drivers/extcon/extcon-palmas.c
@@ -360,6 +360,8 @@ static int palmas_usb_probe(struct platform_device *pdev)
 
 	palmas_enable_irq(palmas_usb);
 	/* perform initial detection */
+	if (palmas_usb->enable_gpio_vbus_detection)
+		palmas_vbus_irq_handler(palmas_usb->gpio_vbus_irq, palmas_usb);
 	palmas_gpio_id_detect(&palmas_usb->wq_detectid.work);
 	device_set_wakeup_capable(&pdev->dev, true);
 	return 0;
-- 
2.7.4

[toc] | [next] | [standalone]

#1422483 — Re: [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO detection

On 2016년 06월 14일 23:04, Roger Quadros wrote:
> If USB cable is connected prior to boot, we don't get any interrupts
> so we must manually check the VBUS state and report it during probe.
> If we don't do it then USB controller will never know that peripheral
> cable was connected till the user unplugs and replugs the cable.
> 
> Fixes: b7aad8e2685b ("extcon: palmas: Add the support for VBUS detection by using GPIO")
> Signed-off-by: Roger Quadros <rogerq@ti.com>
> ---
>  drivers/extcon/extcon-palmas.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/extcon/extcon-palmas.c b/drivers/extcon/extcon-palmas.c
> index 8b3226d..caff46c 100644
> --- a/drivers/extcon/extcon-palmas.c
> +++ b/drivers/extcon/extcon-palmas.c
> @@ -360,6 +360,8 @@ static int palmas_usb_probe(struct platform_device *pdev)
>  
>  	palmas_enable_irq(palmas_usb);
>  	/* perform initial detection */
> +	if (palmas_usb->enable_gpio_vbus_detection)
> +		palmas_vbus_irq_handler(palmas_usb->gpio_vbus_irq, palmas_usb);
>  	palmas_gpio_id_detect(&palmas_usb->wq_detectid.work);
>  	device_set_wakeup_capable(&pdev->dev, true);
>  	return 0;
> 

Applied it.

Thanks,
Chanwoo Choi

[toc] | [prev] | [next] | [standalone]

#1422672 — Re: [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO detection

On 15/06/16 04:57, Chanwoo Choi wrote:
> On 2016년 06월 14일 23:04, Roger Quadros wrote:
>> If USB cable is connected prior to boot, we don't get any interrupts
>> so we must manually check the VBUS state and report it during probe.
>> If we don't do it then USB controller will never know that peripheral
>> cable was connected till the user unplugs and replugs the cable.
>>
>> Fixes: b7aad8e2685b ("extcon: palmas: Add the support for VBUS detection by using GPIO")
>> Signed-off-by: Roger Quadros <rogerq@ti.com>
>> ---
>>  drivers/extcon/extcon-palmas.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/extcon/extcon-palmas.c b/drivers/extcon/extcon-palmas.c
>> index 8b3226d..caff46c 100644
>> --- a/drivers/extcon/extcon-palmas.c
>> +++ b/drivers/extcon/extcon-palmas.c
>> @@ -360,6 +360,8 @@ static int palmas_usb_probe(struct platform_device *pdev)
>>  
>>  	palmas_enable_irq(palmas_usb);
>>  	/* perform initial detection */
>> +	if (palmas_usb->enable_gpio_vbus_detection)
>> +		palmas_vbus_irq_handler(palmas_usb->gpio_vbus_irq, palmas_usb);
>>  	palmas_gpio_id_detect(&palmas_usb->wq_detectid.work);
>>  	device_set_wakeup_capable(&pdev->dev, true);
>>  	return 0;
>>
> 
> Applied it.

Thanks. But this is a bug fix and must go in v4.7-rc
as well as stable v4.6+.

Can you please add

Cc: stable@vger.kernel.org # v4.6+

--
cheers,
-roger

[toc] | [prev] | [next] | [standalone]

#1422755 — Re: [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO detection

On 2016년 06월 15일 15:47, Roger Quadros wrote:
> On 15/06/16 04:57, Chanwoo Choi wrote:
>> On 2016년 06월 14일 23:04, Roger Quadros wrote:
>>> If USB cable is connected prior to boot, we don't get any interrupts
>>> so we must manually check the VBUS state and report it during probe.
>>> If we don't do it then USB controller will never know that peripheral
>>> cable was connected till the user unplugs and replugs the cable.
>>>
>>> Fixes: b7aad8e2685b ("extcon: palmas: Add the support for VBUS detection by using GPIO")
>>> Signed-off-by: Roger Quadros <rogerq@ti.com>
>>> ---
>>>  drivers/extcon/extcon-palmas.c | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/drivers/extcon/extcon-palmas.c b/drivers/extcon/extcon-palmas.c
>>> index 8b3226d..caff46c 100644
>>> --- a/drivers/extcon/extcon-palmas.c
>>> +++ b/drivers/extcon/extcon-palmas.c
>>> @@ -360,6 +360,8 @@ static int palmas_usb_probe(struct platform_device *pdev)
>>>  
>>>  	palmas_enable_irq(palmas_usb);
>>>  	/* perform initial detection */
>>> +	if (palmas_usb->enable_gpio_vbus_detection)
>>> +		palmas_vbus_irq_handler(palmas_usb->gpio_vbus_irq, palmas_usb);
>>>  	palmas_gpio_id_detect(&palmas_usb->wq_detectid.work);
>>>  	device_set_wakeup_capable(&pdev->dev, true);
>>>  	return 0;
>>>
>>
>> Applied it.
> 
> Thanks. But this is a bug fix and must go in v4.7-rc
> as well as stable v4.6+.

OK. I'll send the pull request for this patch.

> 
> Can you please add
> 
> Cc: stable@vger.kernel.org # v4.6+

If we should add the 'stable@verger.kernel.org',
this patch should be sent to mailing list.
So, I think that you better to send this patch including stable mailing list.
And then, I'll send pull request which apply this patch to 4.7-rc.

Thanks,
Chanwoo Choi

[toc] | [prev] | [next] | [standalone]

#1422768 — [PATCH] extcon: palmas: Fix boot up state of VBUS when using GPIO detection

If USB cable is connected prior to boot, we don't get any interrupts
so we must manually check the VBUS state and report it during probe.
If we don't do it then USB controller will never know that peripheral
cable was connected till the user unplugs and replugs the cable.

Fixes: b7aad8e2685b ("extcon: palmas: Add the support for VBUS detection by using GPIO")
Cc: stable@vger.kernel.org # v4.6+
Signed-off-by: Roger Quadros <rogerq@ti.com>
---
 drivers/extcon/extcon-palmas.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/extcon/extcon-palmas.c b/drivers/extcon/extcon-palmas.c
index 8b3226d..caff46c 100644
--- a/drivers/extcon/extcon-palmas.c
+++ b/drivers/extcon/extcon-palmas.c
@@ -360,6 +360,8 @@ static int palmas_usb_probe(struct platform_device *pdev)
 
 	palmas_enable_irq(palmas_usb);
 	/* perform initial detection */
+	if (palmas_usb->enable_gpio_vbus_detection)
+		palmas_vbus_irq_handler(palmas_usb->gpio_vbus_irq, palmas_usb);
 	palmas_gpio_id_detect(&palmas_usb->wq_detectid.work);
 	device_set_wakeup_capable(&pdev->dev, true);
 	return 0;
-- 
2.7.4

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web