On 22 April 2015 at 10:52, Arend van Spriel <arend@xxxxxxxxxxxx> wrote:
> - wireless list/maintainer
>
>
> On 04/22/15 09:32, Ulf Hansson wrote:
>>
>> On 14 April 2015 at 20:10, Arend van Spriel<arend@xxxxxxxxxxxx> wrote:
>>>
>>> commit 330b4e4be937 ("brcmfmac: Add wowl support for SDIO devices.")
>>> changed the behaviour by removing the MMC_PM_KEEP_POWER flag for
>>> non-wowl scenario, which needs to be restored. Another necessary
>>> change is to mark the card as being non-removable. With this in place
>>> the suspend resume test passes successfully doing:
>>>
>>> # echo devices> /sys/power/pm_test
>>> # echo mem> /sys/power/state
>>>
>>> Note that power may still be switched off when system is going
>>> in S3 state.
>>>
>>> Reported-by: Fu, Zhonghui<<zhonghui.fu@xxxxxxxxxxxxxxx>
>>> Reviewed-by: Pieter-Paul Giesberts<pieterpg@xxxxxxxxxxxx>
>>> Reviewed-by: Franky (Zhenhui) Lin<frankyl@xxxxxxxxxxxx>
>>> Signed-off-by: Arend van Spriel<arend@xxxxxxxxxxxx>
>>> ---
>>> drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c | 18
>>> +++++++++++++-----
>>> 1 file changed, 13 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c
>>> b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c
>>> index 9b508bd..8a69544 100644
>>> --- a/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c
>>> +++ b/drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c
>>> @@ -1011,6 +1011,14 @@ static int brcmf_sdiod_remove(struct
>>> brcmf_sdio_dev *sdiodev)
>>> return 0;
>>> }
>>>
>>> +static void brcmf_sdiod_host_fixup(struct mmc_host *host)
>>> +{
>>> + /* runtime-pm powers off the device */
>>> + pm_runtime_forbid(host->parent);
>>
>>
>> That you need this, clearly shows that something is broken in the mmc
>> core/host layer.
>
>
> This patch only moved this. The patch introducing this is here [1].
OK, thanks for sharing the information!
>
>> Could you elaborate a bit on what configuration you are using. Like
>> what mmc host, which SDIO bus speed mode.
>
>
> Not just one. My test setup is a dev board hooked up to a card reader slot
> using sdhci-pci driver. Another setup I have is a chromebook with our device
> integrated with dw_mmc-rockchip driver. It is an arm platform with dt entry:
>
> &sdio0 {
> broken-cd;
> bus-width = <4>;
> cap-sd-highspeed;
> sd-uhs-sdr12;
> sd-uhs-sdr25;
> sd-uhs-sdr50;
> sd-uhs-sdr104;
> cap-sdio-irq;
> card-external-vcc-supply = <&wifi_regulator>;
> clocks = <&cru HCLK_SDIO0>, <&cru SCLK_SDIO0>, <&cru
> SCLK_SDIO0_DRV>,
> <&cru SCLK_SDIO0_SAMPLE>, <&rk808 RK808_CLKOUT1>;
> clock-names = "biu", "ciu", "ciu_drv", "ciu_sample",
> "card_ext_clock";
> keep-power-in-suspend;
> non-removable;
> num-slots = <1>;
> default-sample-phase = <90>;
> pinctrl-names = "default";
> pinctrl-0 = <&sdio0_clk &sdio0_cmd &sdio0_bus4>;
> status = "okay";
> vmmc-supply = <&vcc33_sys>;
> vqmmc-supply = <&vcc18_wl>;
> };
>
> I think card-external-vcc-supply is property that chromeos kernel handles to
> power the device.
Should then likely be moved in a mmc pwrseq node and handled by the
mmc core, right?
The same might apply to "card_ext_clock"!?
>
>> And have you tested different configurations? Like what happens if you
>> use a different SDIO bus speed mode?
In the dw_mmc case, the host controller driver doesn't implement
runtime PM - only system PM (unless you are carrying some additional
out of tree patch :-) ).
So, are you sure the bug exists in this configuration at all?
Kind regards
Uffe
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
