Re: [PATCH] phy: phy-samsung-usb2: Change phy power on/power off sequence
From: Marek Szyprowski
Date: Wed Jun 25 2014 - 01:42:34 EST
Hello,
On 2014-06-24 17:35, Kamil Debski wrote:
On Tue, Jun 24, 2014 at 1:54 PM, Kamil Debski <k.debski@xxxxxxxxxxx>
wrote:
The Exynos4412 USB 2.0 PHY hardware differs from the description
provided in the documentation. Some register bits have different
function. This patch fixes the defines of register bits and changes
the way how phys are powered on and off.
I guess this replaces the patch titled "drivers: phy: exynos4x12-phy:
fix HSIC1 power on/off sequence"
Yes, indeed it replaces this patch. I did some more research on how the
hardware actually behaves.
My patch was a quick hack to get HSIC1 working. Kamil spent much more time
trying to figure out how the hardware actually behaves. Please drop my patch
and replace it with this one.
Tested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
Tested on ODROID-U2. Seems to be working as well as the previous patch:
Thank you very much for testing.
- Internal SMSC hub works on boot and after reboot, tested with USB
mouse
- Internal SMSC ethernet device works on boot, but disappears upon
reboot. (same as previous patch, also reproduced by Marek)
By reboot I guess that you mean typing "reboot" or by using SysRq magic
and not power cycling?
If so, I had experienced the same symptoms. I guess that the Ethernet
chip is not reset properly and fails to enumerate without power cycling
(it's nRESET pin is connected to P3V3).
I found that removing regulator-always-on from buck8_reg: BUCK8 in the
dts file fixes this problem.
Thanks for investigating this issue! I will add this fix to next version
of Odroid patches.
Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/