Hi Kever,

On 19 May 2017 at 20:29, Simon Glass <s...@chromium.org> wrote:
> On 16 May 2017 at 21:44, Kever Yang <kever.y...@rock-chips.com> wrote:
>> SDMMC-PWREN is a pin to control voltage for SDMMC IO, it may
>> be high active or low active, the dwmmc driver always assume
>> the sdmmc-pwren as high active.
>>
>> Kernel treat this pin as fixed regulator instead of a pin from
>> controller, and then it can set in dts file upon board schematic,
>> that's a good solution, we can also do this in u-boot.
>>
>> Signed-off-by: Kever Yang <kever.y...@rock-chips.com>
>> ---
>>
>>  drivers/pinctrl/rockchip/pinctrl_rk3328.c | 6 ++----
>>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> Acked-by: Simon Glass <s...@chromium.org>

Can you please rebase this patch and the following ones? I have marked
them as 'changes requested' in patchwork.

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to