public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Rupjyoti Sarmah <rsarmah@apm.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ppc44x: config GPIOs for USB on canyonlands board
Date: Fri, 24 Sep 2010 17:06:48 +0530	[thread overview]
Message-ID: <11e252f5c85bf4cf5f8e7c0067c8bb5a@mail.gmail.com> (raw)
In-Reply-To: <20100923185627.0A76DB7164@gemini.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.

  reply	other threads:[~2010-09-24 11:36 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-23 14:21 [U-Boot] [PATCH] ppc44x: config GPIOs for USB on canyonlands board Rupjyoti Sarmah
2010-09-23 18:56 ` Wolfgang Denk
2010-09-24 11:36   ` Rupjyoti Sarmah [this message]
2010-09-24 17:26     ` Feng Kan
2010-09-24 17:43       ` Wolfgang Denk
2010-09-24 22:43         ` Feng Kan
2010-09-28 14:16           ` Detlev Zundel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=11e252f5c85bf4cf5f8e7c0067c8bb5a@mail.gmail.com \
    --to=rsarmah@apm.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox