From mboxrd@z Thu Jan  1 00:00:00 1970
From: Rupjyoti Sarmah <rsarmah@apm.com>
Date: Fri, 24 Sep 2010 17:06:48 +0530
Subject: [U-Boot] [PATCH] ppc44x: config GPIOs for USB on canyonlands
	board
In-Reply-To: <20100923185627.0A76DB7164@gemini.denx.de>
References: <201009231421.o8NELOE9022250@amcc.com>
	<20100923185627.0A76DB7164@gemini.denx.de>
Message-ID: <11e252f5c85bf4cf5f8e7c0067c8bb5a@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Wolfgang,


> So moved the reconfiguration to the USB init  function

Does the Linux kernel perform the same initiaalization of the GPIO
pins?  If not, then your change will most likely cause the USB is not
working in Linux unless you used USB in U-Boot.

RUP >> Yes, it does.

> --- a/board/amcc/canyonlands/canyonlands.c
> +

You drop the "if (pvr_460ex()) {" part here. Is this intentional?
RUP>> Yes

Also, when adding this code to usb_board_init(), would it not be
logical to undo this initialization in usb_board_stop()?

RUP >> Ok, I will include your suggestion and resubmit


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
In theory, there is no difference between  theory  and  practice.  In
practice, however, there is.