LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: Kumar Gala <kumar.gala@freescale.com>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: Add PCI support for TQM834x Boards
Date: Mon, 24 Oct 2005 20:29:48 +0200	[thread overview]
Message-ID: <20051024182948.BA8D3353444@atlas.denx.de> (raw)
In-Reply-To: Your message of "Mon, 24 Oct 2005 10:20:24 CDT." <398F469F-357E-445D-B48E-59899230E001@freescale.com>

Dear Kumar,

in message <398F469F-357E-445D-B48E-59899230E001@freescale.com> you wrote:
>
> * The comment in mpc83xx_setup_hose() may not be 100% correct (see  
> inline)

It probably needs more explanation.

> * Updating of pci_ids.h should be done via http:// 
> pciids.sourceforge.net/.  I can handle doing this for 834x if you  
> want me to.

I'd appreciate if you could handle it this time. I will  remeber  how
to do it correctly from now on. Thanks.

...
> > +#ifdef CONFIG_TQM834x
> 
> Is the following comment 100% correct?  I'm not sure I follow the  
> STK85xx reference.
> 
> > +    /*
> > +     * WARNING! only PCI_CLK_OUTPUT1 is enabled for the TQM834x as this is
> > +     * the one line actually used for clocking all external PCI devices in
> > +     * current setup of the STK85xx.  Enabling other PCI_CLK_OUTPUT lines

This is correct. TQ uses the same eval  board  for  all  TQM83xx  and
TQM85xx  modules.  It  was originally developed for the TQM85xx so it
got named STK85xx (STarter Kit 85xx). Later,  it  was  impossible  to
reflect  the  use for the TQM83xx in the name of the eval board. Yes,
this is confusing...

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
"Have you lived in this village all your life?"        "No, not yet."

      reply	other threads:[~2005-10-24 18:30 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-22 22:36 Add PCI support for TQM834x Boards Wolfgang Denk
2005-10-24 15:20 ` Kumar Gala
2005-10-24 18:29   ` Wolfgang Denk [this message]

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=20051024182948.BA8D3353444@atlas.denx.de \
    --to=wd@denx.de \
    --cc=kumar.gala@freescale.com \
    --cc=linuxppc-dev@ozlabs.org \
    /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