Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-10 Thread Mark Brown
On 10 Jun 2010, at 23:07, Grant Likely wrote: To me, this seems firmly in the realm of silicon bug workaround. Considering that the pins aren't relocatable (ie, the GPIO numbers never change), I've got no problem having the GPIO reset logic added to arch/powerpc/platforms/52xx and hard coding

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-10 Thread Grant Likely
On Wed, Jun 9, 2010 at 4:30 AM, Mark Brown broo...@opensource.wolfsonmicro.com wrote: On Wed, Jun 09, 2010 at 08:13:30AM +0200, Wolfgang Denk wrote: In message aanlktimis90kr5uqhdbq02osd94dn08soitm51ylr...@mail.gmail.com you wrote: Would making a change in uboot be a better solution? Eric,

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-10 Thread Grant Likely
On Thu, Jun 10, 2010 at 5:14 PM, Mark Brown broo...@opensource.wolfsonmicro.com wrote: On 10 Jun 2010, at 23:07, Grant Likely wrote: To me, this seems firmly in the realm of silicon bug workaround. Considering that the pins aren't relocatable (ie, the GPIO numbers never change), I've got no

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Wolfgang Denk
Dear Jon Smirl, In message aanlktimis90kr5uqhdbq02osd94dn08soitm51ylr...@mail.gmail.com you wrote: Would making a change in uboot be a better solution? Eric, can you verify if changing uboot also fixes the problem? To me it seems better if the driver itself does what needs to be done instead

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Mark Brown
On Wed, Jun 09, 2010 at 08:13:30AM +0200, Wolfgang Denk wrote: In message aanlktimis90kr5uqhdbq02osd94dn08soitm51ylr...@mail.gmail.com you wrote: Would making a change in uboot be a better solution? Eric, can you verify if changing uboot also fixes the problem? To me it seems better if

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Jon Smirl
On Wed, Jun 9, 2010 at 2:13 AM, Wolfgang Denk w...@denx.de wrote: Dear Jon Smirl, In message aanlktimis90kr5uqhdbq02osd94dn08soitm51ylr...@mail.gmail.com you wrote: Would making a change in uboot be a better solution? Eric, can you verify if changing uboot also fixes the problem? To me

RE: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Eric Millbrandt
-Original Message- From: Jon Smirl [mailto:jonsm...@gmail.com] Sent: Wednesday, June 09, 2010 08:22 To: Wolfgang Denk Cc: Eric Millbrandt; Mark Brown; linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH 0/2] mpc5200 ac97 gpio reset On Wed, Jun 9, 2010 at 2:13 AM, Wolfgang Denk w

RE: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Eric Millbrandt
-Original Message- From: Mark Brown [mailto:broo...@opensource.wolfsonmicro.com] Sent: Wednesday, June 09, 2010 06:31 To: Wolfgang Denk Cc: Jon Smirl; Eric Millbrandt; linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH 0/2] mpc5200 ac97 gpio reset --snip-- It might, however

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Mark Brown
On Wed, Jun 09, 2010 at 10:21:40AM -0400, Eric Millbrandt wrote: [Please fix your MUA to word wrap paragraphs to within 80 characters, I've reflowed the text below.] From the MPC5200B user manual: Some AC97 devices goes to a test mode, if the Sync line is high during the Res line is low

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Jon Smirl
On Wed, Jun 9, 2010 at 10:32 AM, Mark Brown broo...@opensource.wolfsonmicro.com wrote: On Wed, Jun 09, 2010 at 10:21:40AM -0400, Eric Millbrandt wrote: [Please fix your MUA to word wrap paragraphs to within 80 characters, I've reflowed the text below.] From the MPC5200B user manual: Some

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-09 Thread Mark Brown
On Wed, Jun 09, 2010 at 10:41:23AM -0400, Jon Smirl wrote: On Wed, Jun 9, 2010 at 10:32 AM, Mark Brown Please include this quote in the changelog for the patch, if this a documented workaround from the vendor that's a very different thing to something that you've found happens to work on

[PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-08 Thread Eric Millbrandt
These patches reimplement the reset fuction in the ac97 to use gpio pins instead of using the mpc5200 ac97 reset functionality in the psc. This avoids a problem in which attached ac97 devices go into test mode appear unresponsive. These patches were tested on a pcm030 baseboard and on custom

Re: [PATCH 0/2] mpc5200 ac97 gpio reset

2010-06-08 Thread Jon Smirl
On Tue, Jun 8, 2010 at 12:46 PM, Eric Millbrandt emillbra...@dekaresearch.com wrote: These patches reimplement the reset fuction in the ac97 to use gpio pins instead of using the mpc5200 ac97 reset functionality in the psc.  This avoids a problem in which attached ac97 devices go into test mode