Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Randy Li]

On 08/09/2016 06:58 PM, Heiko Stübner wrote:

Am Dienstag, 9. August 2016, 18:06:33 schrieb Randy Li:

從我的 iPad 傳送


陈豪  於 2016年8月9日 下午6:02 寫道：

well，it has already been added

The root cause is not act8846. Firefly have a bug with sdmmc and it
seems they didn't fix it in firefly-reload.

http://bbs.t-firefly.com/forum.php?mod=viewthread=256

Thanks Jacob for pointing out this issue - I was always wondering why it
wasn't working :-) .



Yes, Jacob is right. Those high speed options need the voltage down to 1.8v.
They should be removed.

Randy, can you provide a follow-up patch that removes these excess properties
please?

I will send a patch in weekend. I need some time to confirm whether
the sdio wifi would meet the same problem in firefly reload.
I found those patches are not been merged yet, should I just send
a new version or just a patch remove those incorrect excess properties?



Thanks
Heiko



--
Randy Li
The third produce department

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Randy Li]

On 08/09/2016 06:58 PM, Heiko Stübner wrote:

Am Dienstag, 9. August 2016, 18:06:33 schrieb Randy Li:

從我的 iPad 傳送


陈豪  於 2016年8月9日 下午6:02 寫道：

well，it has already been added

The root cause is not act8846. Firefly have a bug with sdmmc and it
seems they didn't fix it in firefly-reload.

http://bbs.t-firefly.com/forum.php?mod=viewthread=256

Thanks Jacob for pointing out this issue - I was always wondering why it
wasn't working :-) .



Yes, Jacob is right. Those high speed options need the voltage down to 1.8v.
They should be removed.

Randy, can you provide a follow-up patch that removes these excess properties
please?

I will send a patch in weekend. I need some time to confirm whether
the sdio wifi would meet the same problem in firefly reload.
I found those patches are not been merged yet, should I just send
a new version or just a patch remove those incorrect excess properties?



Thanks
Heiko



--
Randy Li
The third produce department

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Heiko Stübner]

Am Dienstag, 9. August 2016, 18:06:33 schrieb Randy Li:
> 從我的 iPad 傳送
> 
> > 陈豪  於 2016年8月9日 下午6:02 寫道：
> > 
> > well，it has already been added
> > 
> > The root cause is not act8846. Firefly have a bug with sdmmc and it
> > seems they didn't fix it in firefly-reload.
> > 
> > http://bbs.t-firefly.com/forum.php?mod=viewthread=256

Thanks Jacob for pointing out this issue - I was always wondering why it 
wasn't working :-) .


> Yes, Jacob is right. Those high speed options need the voltage down to 1.8v.
> They should be removed.

Randy, can you provide a follow-up patch that removes these excess properties 
please?


Thanks
Heiko

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Heiko Stübner]

Am Dienstag, 9. August 2016, 18:06:33 schrieb Randy Li:
> 從我的 iPad 傳送
> 
> > 陈豪  於 2016年8月9日 下午6:02 寫道：
> > 
> > well，it has already been added
> > 
> > The root cause is not act8846. Firefly have a bug with sdmmc and it
> > seems they didn't fix it in firefly-reload.
> > 
> > http://bbs.t-firefly.com/forum.php?mod=viewthread=256

Thanks Jacob for pointing out this issue - I was always wondering why it 
wasn't working :-) .


> Yes, Jacob is right. Those high speed options need the voltage down to 1.8v.
> They should be removed.

Randy, can you provide a follow-up patch that removes these excess properties 
please?


Thanks
Heiko

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Randy Li]

從我的 iPad 傳送

> 陈豪  於 2016年8月9日 下午6:02 寫道：
> 
> well，it has already been added
> 
> The root cause is not act8846. Firefly have a bug with sdmmc and it
> seems they didn't fix it in firefly-reload.
> 
> http://bbs.t-firefly.com/forum.php?mod=viewthread=256
Yes, Jacob is right. Those high speed options need the voltage down to 1.8v. 
They should be removed.

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Randy Li]

從我的 iPad 傳送

> 陈豪  於 2016年8月9日 下午6:02 寫道：
> 
> well，it has already been added
> 
> The root cause is not act8846. Firefly have a bug with sdmmc and it
> seems they didn't fix it in firefly-reload.
> 
> http://bbs.t-firefly.com/forum.php?mod=viewthread=256
Yes, Jacob is right. Those high speed options need the voltage down to 1.8v. 
They should be removed.

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[陈豪]

well，it has already been added

The root cause is not act8846. Firefly have a bug with sdmmc and it
seems they didn't fix it in firefly-reload.

http://bbs.t-firefly.com/forum.php?mod=viewthread=256

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[陈豪]

well，it has already been added

The root cause is not act8846. Firefly have a bug with sdmmc and it
seems they didn't fix it in firefly-reload.

http://bbs.t-firefly.com/forum.php?mod=viewthread=256

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[陈豪]

2016-08-07 22:44 GMT+08:00 Heiko Stuebner :
> Hi Shawn,
>
> Am Sonntag, 7. August 2016, 11:40:21 schrieb Shawn Lin:
>> On 2016/7/19 3:46, Heiko Stübner wrote:
>> > Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:
>> >> The Firefly RK3288 Reload is a combination Firefly rk3288 core board
>> >> with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
>> >> which can be included into the dts for the various baseboards
>> >> in the future and dts for Reload base board.
>> >>
>> >> Currently supported are serial console, wired networking, eMMC and
>> >> SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
>> >> the OTG could work on the host mode now, the other USB host can't
>> >> work now, additional patches are required.
>> >>
>> >> Signed-off-by: Randy Li 
>> >> Acked-by: Rob Herring 
>> >
>> > applied [0], after some minor reordering - please double check.
>> >
>> > Also, while the branch is named 4.8 this will only go into 4.9, as the
>> > merge- window supposedly opens on the weekend.
>>
>> Sorry for the noise here:), but I find there is some mmc stuff
>> which confuse me. Maybe Randy could elaborate more?
>>
>> Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
>> will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.
>>
>> But the vqmmc is vccio_sd which is fixed as 3V3? :)
>
> so far we have only been able to achieve UHS speeds on devices based on the
> rk808 pmic (I had sucess on veyron and popmetal). The act8846-based boards
> seem to have some issue when trying to do the switch to 1.8V, resulting in
> mmc errors and the mmc-core switching back to 3.3V.
>
> I think this might be caused by a bug in the act8846 driver or some dts
> mistake. But so far nobody had the time and enthusiasm to investigate :-)
>
>
> Heiko

Hi all,
sdcard-supply must be specifed  in io-domains node to enable uhs.

io-domains {
compatible = "rockchip,rk3288-io-voltage-domain";
rockchip,grf = <>;
sdcard-supply = <_sd>;
wifi-supply = <_18>;
};

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[陈豪]

2016-08-07 22:44 GMT+08:00 Heiko Stuebner :
> Hi Shawn,
>
> Am Sonntag, 7. August 2016, 11:40:21 schrieb Shawn Lin:
>> On 2016/7/19 3:46, Heiko Stübner wrote:
>> > Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:
>> >> The Firefly RK3288 Reload is a combination Firefly rk3288 core board
>> >> with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
>> >> which can be included into the dts for the various baseboards
>> >> in the future and dts for Reload base board.
>> >>
>> >> Currently supported are serial console, wired networking, eMMC and
>> >> SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
>> >> the OTG could work on the host mode now, the other USB host can't
>> >> work now, additional patches are required.
>> >>
>> >> Signed-off-by: Randy Li 
>> >> Acked-by: Rob Herring 
>> >
>> > applied [0], after some minor reordering - please double check.
>> >
>> > Also, while the branch is named 4.8 this will only go into 4.9, as the
>> > merge- window supposedly opens on the weekend.
>>
>> Sorry for the noise here:), but I find there is some mmc stuff
>> which confuse me. Maybe Randy could elaborate more?
>>
>> Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
>> will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.
>>
>> But the vqmmc is vccio_sd which is fixed as 3V3? :)
>
> so far we have only been able to achieve UHS speeds on devices based on the
> rk808 pmic (I had sucess on veyron and popmetal). The act8846-based boards
> seem to have some issue when trying to do the switch to 1.8V, resulting in
> mmc errors and the mmc-core switching back to 3.3V.
>
> I think this might be caused by a bug in the act8846 driver or some dts
> mistake. But so far nobody had the time and enthusiasm to investigate :-)
>
>
> Heiko

Hi all,
sdcard-supply must be specifed  in io-domains node to enable uhs.

io-domains {
compatible = "rockchip,rk3288-io-voltage-domain";
rockchip,grf = <>;
sdcard-supply = <_sd>;
wifi-supply = <_18>;
};

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Shawn Lin]

在 2016/8/7 22:44, Heiko Stuebner 写道:

Hi Shawn,

Am Sonntag, 7. August 2016, 11:40:21 schrieb Shawn Lin:

On 2016/7/19 3:46, Heiko Stübner wrote:

Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:

The Firefly RK3288 Reload is a combination Firefly rk3288 core board
with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
which can be included into the dts for the various baseboards
in the future and dts for Reload base board.

Currently supported are serial console, wired networking, eMMC and
SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
the OTG could work on the host mode now, the other USB host can't
work now, additional patches are required.

Signed-off-by: Randy Li 
Acked-by: Rob Herring 


applied [0], after some minor reordering - please double check.

Also, while the branch is named 4.8 this will only go into 4.9, as the
merge- window supposedly opens on the weekend.


Sorry for the noise here:), but I find there is some mmc stuff
which confuse me. Maybe Randy could elaborate more?

Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.

But the vqmmc is vccio_sd which is fixed as 3V3? :)


so far we have only been able to achieve UHS speeds on devices based on the
rk808 pmic (I had sucess on veyron and popmetal). The act8846-based boards
seem to have some issue when trying to do the switch to 1.8V, resulting in
mmc errors and the mmc-core switching back to 3.3V.


Interesting I will have a look at it this weekend with Randy face 2
face



I think this might be caused by a bug in the act8846 driver or some dts
mistake. But so far nobody had the time and enthusiasm to investigate :-)


Heiko

___
Linux-rockchip mailing list
linux-rockc...@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip




--
Best Regards
Shawn Lin

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Shawn Lin]

在 2016/8/7 22:44, Heiko Stuebner 写道:

Hi Shawn,

Am Sonntag, 7. August 2016, 11:40:21 schrieb Shawn Lin:

On 2016/7/19 3:46, Heiko Stübner wrote:

Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:

The Firefly RK3288 Reload is a combination Firefly rk3288 core board
with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
which can be included into the dts for the various baseboards
in the future and dts for Reload base board.

Currently supported are serial console, wired networking, eMMC and
SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
the OTG could work on the host mode now, the other USB host can't
work now, additional patches are required.

Signed-off-by: Randy Li 
Acked-by: Rob Herring 


applied [0], after some minor reordering - please double check.

Also, while the branch is named 4.8 this will only go into 4.9, as the
merge- window supposedly opens on the weekend.


Sorry for the noise here:), but I find there is some mmc stuff
which confuse me. Maybe Randy could elaborate more?

Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.

But the vqmmc is vccio_sd which is fixed as 3V3? :)


so far we have only been able to achieve UHS speeds on devices based on the
rk808 pmic (I had sucess on veyron and popmetal). The act8846-based boards
seem to have some issue when trying to do the switch to 1.8V, resulting in
mmc errors and the mmc-core switching back to 3.3V.


Interesting I will have a look at it this weekend with Randy face 2
face



I think this might be caused by a bug in the act8846 driver or some dts
mistake. But so far nobody had the time and enthusiasm to investigate :-)


Heiko

___
Linux-rockchip mailing list
linux-rockc...@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip




--
Best Regards
Shawn Lin

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Heiko Stuebner]

Hi Shawn,

Am Sonntag, 7. August 2016, 11:40:21 schrieb Shawn Lin:
> On 2016/7/19 3:46, Heiko Stübner wrote:
> > Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:
> >> The Firefly RK3288 Reload is a combination Firefly rk3288 core board
> >> with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
> >> which can be included into the dts for the various baseboards
> >> in the future and dts for Reload base board.
> >> 
> >> Currently supported are serial console, wired networking, eMMC and
> >> SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
> >> the OTG could work on the host mode now, the other USB host can't
> >> work now, additional patches are required.
> >> 
> >> Signed-off-by: Randy Li 
> >> Acked-by: Rob Herring 
> > 
> > applied [0], after some minor reordering - please double check.
> > 
> > Also, while the branch is named 4.8 this will only go into 4.9, as the
> > merge- window supposedly opens on the weekend.
> 
> Sorry for the noise here:), but I find there is some mmc stuff
> which confuse me. Maybe Randy could elaborate more?
> 
> Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
> will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.
> 
> But the vqmmc is vccio_sd which is fixed as 3V3? :)

so far we have only been able to achieve UHS speeds on devices based on the 
rk808 pmic (I had sucess on veyron and popmetal). The act8846-based boards 
seem to have some issue when trying to do the switch to 1.8V, resulting in 
mmc errors and the mmc-core switching back to 3.3V.

I think this might be caused by a bug in the act8846 driver or some dts 
mistake. But so far nobody had the time and enthusiasm to investigate :-)


Heiko

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Heiko Stuebner]

Hi Shawn,

Am Sonntag, 7. August 2016, 11:40:21 schrieb Shawn Lin:
> On 2016/7/19 3:46, Heiko Stübner wrote:
> > Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:
> >> The Firefly RK3288 Reload is a combination Firefly rk3288 core board
> >> with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
> >> which can be included into the dts for the various baseboards
> >> in the future and dts for Reload base board.
> >> 
> >> Currently supported are serial console, wired networking, eMMC and
> >> SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
> >> the OTG could work on the host mode now, the other USB host can't
> >> work now, additional patches are required.
> >> 
> >> Signed-off-by: Randy Li 
> >> Acked-by: Rob Herring 
> > 
> > applied [0], after some minor reordering - please double check.
> > 
> > Also, while the branch is named 4.8 this will only go into 4.9, as the
> > merge- window supposedly opens on the weekend.
> 
> Sorry for the noise here:), but I find there is some mmc stuff
> which confuse me. Maybe Randy could elaborate more?
> 
> Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
> will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.
> 
> But the vqmmc is vccio_sd which is fixed as 3V3? :)

so far we have only been able to achieve UHS speeds on devices based on the 
rk808 pmic (I had sucess on veyron and popmetal). The act8846-based boards 
seem to have some issue when trying to do the switch to 1.8V, resulting in 
mmc errors and the mmc-core switching back to 3.3V.

I think this might be caused by a bug in the act8846 driver or some dts 
mistake. But so far nobody had the time and enthusiasm to investigate :-)


Heiko

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Shawn Lin]

On 2016/7/19 3:46, Heiko Stübner wrote:

Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:

The Firefly RK3288 Reload is a combination Firefly rk3288 core board
with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
which can be included into the dts for the various baseboards
in the future and dts for Reload base board.

Currently supported are serial console, wired networking, eMMC and
SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
the OTG could work on the host mode now, the other USB host can't
work now, additional patches are required.

Signed-off-by: Randy Li 
Acked-by: Rob Herring 


applied [0], after some minor reordering - please double check.

Also, while the branch is named 4.8 this will only go into 4.9, as the merge-
window supposedly opens on the weekend.


Sorry for the noise here:), but I find there is some mmc stuff
which confuse me. Maybe Randy could elaborate more?

Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.

But the vqmmc is vccio_sd which is fixed as 3V3? :)





Heiko

[0] 
https://git.kernel.org/cgit/linux/kernel/git/mmind/linux-rockchip.git/commit/?id=e00fdd4931ea4c47f53cba59b7a3800677546bea


___
Linux-rockchip mailing list
linux-rockc...@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip




--
Best Regards
Shawn Lin

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Shawn Lin]

On 2016/7/19 3:46, Heiko Stübner wrote:

Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:

The Firefly RK3288 Reload is a combination Firefly rk3288 core board
with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
which can be included into the dts for the various baseboards
in the future and dts for Reload base board.

Currently supported are serial console, wired networking, eMMC and
SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
the OTG could work on the host mode now, the other USB host can't
work now, additional patches are required.

Signed-off-by: Randy Li 
Acked-by: Rob Herring 


applied [0], after some minor reordering - please double check.

Also, while the branch is named 4.8 this will only go into 4.9, as the merge-
window supposedly opens on the weekend.


Sorry for the noise here:), but I find there is some mmc stuff
which confuse me. Maybe Randy could elaborate more?

Your sdmmc claims to support UHS-I mode like sd-uhs-sdr104, etc, which
will ask dw_mmc to switch vqmmc from 3V3 to 1V8 after sending CMD11.

But the vqmmc is vccio_sd which is fixed as 3V3? :)





Heiko

[0] 
https://git.kernel.org/cgit/linux/kernel/git/mmind/linux-rockchip.git/commit/?id=e00fdd4931ea4c47f53cba59b7a3800677546bea


___
Linux-rockchip mailing list
linux-rockc...@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip




--
Best Regards
Shawn Lin

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Heiko Stübner]

Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:
> The Firefly RK3288 Reload is a combination Firefly rk3288 core board
> with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
> which can be included into the dts for the various baseboards
> in the future and dts for Reload base board.
> 
> Currently supported are serial console, wired networking, eMMC and
> SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
> the OTG could work on the host mode now, the other USB host can't
> work now, additional patches are required.
> 
> Signed-off-by: Randy Li 
> Acked-by: Rob Herring 

applied [0], after some minor reordering - please double check.

Also, while the branch is named 4.8 this will only go into 4.9, as the merge-
window supposedly opens on the weekend.


Heiko

[0] 
https://git.kernel.org/cgit/linux/kernel/git/mmind/linux-rockchip.git/commit/?id=e00fdd4931ea4c47f53cba59b7a3800677546bea

Re: [PATCH] ARM: dts: add rk3288-firefly-reload

[Heiko Stübner]

Am Montag, 18. Juli 2016, 23:32:32 schrieb Randy Li:
> The Firefly RK3288 Reload is a combination Firefly rk3288 core board
> with the Reload baseboard. Add a dtsi for the Firefly rk3288 core
> which can be included into the dts for the various baseboards
> in the future and dts for Reload base board.
> 
> Currently supported are serial console, wired networking, eMMC and
> SD storage, SPFIF, IR receiver, LEDs, SDIO wifi and USB. But only
> the OTG could work on the host mode now, the other USB host can't
> work now, additional patches are required.
> 
> Signed-off-by: Randy Li 
> Acked-by: Rob Herring 

applied [0], after some minor reordering - please double check.

Also, while the branch is named 4.8 this will only go into 4.9, as the merge-
window supposedly opens on the weekend.


Heiko

[0] 
https://git.kernel.org/cgit/linux/kernel/git/mmind/linux-rockchip.git/commit/?id=e00fdd4931ea4c47f53cba59b7a3800677546bea