2012-02-27 16:11:00

by Ben Collins

[permalink] [raw]
Subject: [Regression] powerpc/usb: fix issue of CPU halt when missing USB PHY clock

Hello,

The patch that checks the CTRL_PHY_CLK_VALID bit is failing incorrectly on my system (P4080DS). If I revert it, my system goes back to working correctly, but as is, it fails and wont allow my USB to work properly.

I've tried adding a loop to poll this bit 10,000 times with a udelay(1) on each loop and it still fails, so it doesn't appear to be a timing issue.

Is this check valid for all systems?

--
Bluecherry: http://www.bluecherrydvr.com/
SwissDisk : http://www.swissdisk.com/
Ubuntu : http://www.ubuntu.com/
My Blog : http://ben-collins.blogspot.com/


2012-02-28 10:10:03

by Liu Shengzhou-B36685

[permalink] [raw]
Subject: RE: [Regression] powerpc/usb: fix issue of CPU halt when missing USB PHY clock


> -----Original Message-----
> From: Ben Collins [mailto:[email protected]]
> Sent: Tuesday, February 28, 2012 12:11 AM
> To: Liu Shengzhou-B36685
> Cc: [email protected]
> Subject: [Regression] powerpc/usb: fix issue of CPU halt when missing
> USB PHY clock
>
> Hello,
>
> The patch that checks the CTRL_PHY_CLK_VALID bit is failing incorrectly
> on my system (P4080DS). If I revert it, my system goes back to working
> correctly, but as is, it fails and wont allow my USB to work properly.
>
> I've tried adding a loop to poll this bit 10,000 times with a udelay(1)
> on each loop and it still fails, so it doesn't appear to be a timing
> issue.
>
> Is this check valid for all systems?
>

This check is valid for P3060/P3041/P5020/P1023 platform.
on some platforms, there is this bit, but not on other some platforms.
For P4080, just revert this patch, then USB is able to work.
I'll post a new patch to fix it after getting some confirmations from silicon team.

Thanks,
Shengzhou