Re: [PATCH v2 05/12] mmc: pwrseq: Document optional exteral vcc for the simple power sequence

From: Javier Martinez Canillas
Date: Fri Jan 08 2016 - 07:13:53 EST


Hello Caesar,

On 01/07/2016 05:25 AM, Caesar Wang wrote:
> This patch adds the exteral vcc optional for the simple power sequence.
>
> Some WLAN chips attached to a SDIO interface, need an external vcc
> to be operational. Since this is very common, extend the simple MMC
> power sequence DT binding to support an optional exteral vcc.
>

s/exteral/external in the subject line and other places.

I think that makes sense to extend the simple power sequence provider
to add a regulator since many SDIO chips may needed it but it seems
from the commit message of patch 06/12 that you don't actually need
a regulator but to toggle a WL_REG_ON pin in your WiFi chip.

I'll answer to that patch though.

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America