From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 19 Apr 2012 00:57:04 +0200 Subject: [U-Boot] [PATCH] powerpc/usb: fix bug of CPU halt when missing USB PHY clock In-Reply-To: References: <1328085078-3104-1-git-send-email-Shengzhou.Liu@freescale.com> <3F453DDFF675A64A89321A1F3528102170B538@039-SN1MPN1-004.039d.mgd.msft.net> Message-ID: <201204190057.04588.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Andy Fleming, > Pinging you on this, again. Also, I've now moved the patch to Marek's > queue, instead of mine. Thanks! Did the FSL silicon team respond already? > > On Sun, Feb 26, 2012 at 8:44 PM, Liu Shengzhou-B36685 > > wrote: > >> -----Original Message----- > >> From: Marek Vasut [mailto:marex at denx.de] > >> Sent: Monday, February 27, 2012 7:13 AM > >> To: u-boot at lists.denx.de > >> Cc: Liu Shengzhou-B36685 > >> Subject: Re: [U-Boot] [PATCH] powerpc/usb: fix bug of CPU halt when > >> missing USB PHY clock > >> > >> > when missing USB PHY clock and issuing "usb start" at u-boot prompt, > >> > writing to or_portsc register will cause CPU halt. We should check > >> > USBGP[PHY_CLK_VALID] bit at the first time in ehci_hcd_init() to > >> > >> avoid > >> > >> > CPU hang in this case. > >> > > >> > Signed-off-by: Shengzhou Liu > >> > --- > >> > drivers/usb/host/ehci-fsl.c | 22 +++++++++++++++++++--- > >> > 1 files changed, 19 insertions(+), 3 deletions(-) > >> > >> Hi, > >> > >> what's the status of this patch/patchset? > >> > >> Thanks > >> M > > > > Currently we found that usb CTRL_PHY_CLK_VALID bit breaks on P1022 > > platform, which not contains this bit. - P1023/P3041/P5020 etc, have > > this bit > > - P3060/4080/PSC913x do have this bit, but not mentioned in RM. > > - P1022(perhaps and other) has no this bit > > I'm waiting for the response from FSL silicon team to confirm whether > > there is other platform not including this bit or not, so this patch > > maybe have to be pending until I get confirmation. > > > > Thanks, > > Shengzhou > > > > > > _______________________________________________ > > U-Boot mailing list > > U-Boot at lists.denx.de > > http://lists.denx.de/mailman/listinfo/u-boot Best regards, Marek Vasut