All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 6/6] p1021mds: add QE and UEC support
Date: Mon, 31 Jan 2011 21:11:54 +0100	[thread overview]
Message-ID: <20110131201154.B982BD4D67C@gemini.denx.de> (raw)
In-Reply-To: <1296499317-26616-7-git-send-email-Haiying.Wang@freescale.com>

Dear Haiying.Wang at freescale.com,

In message <1296499317-26616-7-git-send-email-Haiying.Wang@freescale.com> you wrote:
> From: Haiying Wang <Haiying.Wang@freescale.com>
> 
> P1021 has some QE pins which need to be set in pmuxcr register before using QE
> functions. In this patch, pin QE0 and QE3 are set for UCC1 and UCC5 in Eth mode.
> QE9 and QE12 are set for MII management. QE12 needs to be released after MII
> access because QE12 pin is muxed with LBCTL signal.
> 
> P1021MDS has to load the microcode from NAND flash, this patch defines
> misc_init_r() for loading ucode and initializing qe.
...
> diff --git a/drivers/qe/uec.c b/drivers/qe/uec.c
> index 282ab23..04d7987 100644
> --- a/drivers/qe/uec.c
> +++ b/drivers/qe/uec.c
...
> +#ifdef CONFIG_P1021
> +#define BCSR11_ENET_MICRST	0x20
> +#endif
>  
>  /* Default UTBIPAR SMI address */
>  #ifndef CONFIG_UTBIPAR_INIT_TBIPA
> @@ -588,9 +591,25 @@ static void phy_change(struct eth_device *dev)
>  {
>  	uec_private_t	*uec = (uec_private_t *)dev->priv;
>  
> +#ifdef CONFIG_P1021
> +	ccsr_gur_t *gur = (void *)(CONFIG_SYS_MPC85xx_GUTS_ADDR);
> +
> +	/* QE9 and QE12 need to be set for enabling QE MII managment signals */
> +	setbits_be32(&gur->pmuxcr, MPC85xx_PMUXCR_QE9);
> +	setbits_be32(&gur->pmuxcr, MPC85xx_PMUXCR_QE12);
> +#endif
...

Can we please avoid having board specific code in common files?

If this is really necessary, it shoud be a feature-specific #define,
not a board specific one.


> @@ -425,6 +469,8 @@
>  #define CONFIG_PCI_PNP                 /* do pci plug-and-play */
>  #endif
>  
> +#define CONFIG_E1000

In which way is this change related to this commit?

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
It is better to marry than to burn.
                                - Bible ``I Corinthians'' ch. 7, v. 9

  reply	other threads:[~2011-01-31 20:11 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-31 18:41 [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board Haiying.Wang at freescale.com
2011-01-31 18:41 ` [U-Boot] [PATCH 1/6] Introduce the Tertiary Program loader Haiying.Wang at freescale.com
2011-01-31 18:41 ` [U-Boot] [PATCH 2/6] powerpc/85xx: add TPL support Haiying.Wang at freescale.com
2011-01-31 18:41 ` [U-Boot] [PATCH 3/6] P1021: add P1021MDS board support Haiying.Wang at freescale.com
2011-01-31 20:03   ` Wolfgang Denk
2011-01-31 20:08     ` Scott Wood
2011-01-31 20:18       ` Wolfgang Denk
2011-01-31 20:23         ` Scott Wood
2011-01-31 21:39     ` Haiying Wang
2011-01-31 22:05       ` Kumar Gala
2011-01-31 22:16         ` Wolfgang Denk
2011-01-31 22:14       ` Wolfgang Denk
2011-01-31 21:40     ` Kumar Gala
2011-01-31 18:41 ` [U-Boot] [PATCH 4/6] powerpc/p1021: add more P1021 defines Haiying.Wang at freescale.com
2011-01-31 20:07   ` Wolfgang Denk
2011-01-31 21:08   ` Kumar Gala
2011-01-31 23:36     ` Timur Tabi
2011-02-01  4:04       ` Kumar Gala
2011-01-31 18:41 ` [U-Boot] [PATCH 5/6] powerpc/85xx: do not initialize QE if QE's firmware is in nand flash Haiying.Wang at freescale.com
2011-01-31 20:08   ` Wolfgang Denk
2011-01-31 21:02     ` Haiying Wang
2011-01-31 21:37       ` Wolfgang Denk
2011-02-02  4:29         ` Kumar Gala
2011-01-31 18:41 ` [U-Boot] [PATCH 6/6] p1021mds: add QE and UEC support Haiying.Wang at freescale.com
2011-01-31 20:11   ` Wolfgang Denk [this message]
2011-01-31 20:50     ` Haiying Wang
2011-01-31 21:28       ` Kumar Gala
2011-02-01  3:14         ` Haiying Wang
2011-02-01 16:50           ` Scott Wood
2011-02-01 17:01             ` Haiying Wang
2011-02-01 19:15               ` Kumar Gala
2011-02-01 19:46                 ` Haiying Wang
2011-02-02  4:25                   ` Kumar Gala
2011-01-31 19:39 ` [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board Wolfgang Denk
2011-01-31 20:13   ` Scott Wood
2011-01-31 20:22     ` Wolfgang Denk
2011-01-31 20:31       ` Scott Wood
2011-01-31 20:39         ` Kumar Gala
2011-01-31 20:50           ` Wolfgang Denk
2011-01-31 20:50         ` Wolfgang Denk
2011-01-31 21:15           ` Scott Wood
2011-01-31 21:34             ` Wolfgang Denk
2011-01-31 22:07               ` Scott Wood
2011-01-31 22:40                 ` Wolfgang Denk
2011-01-31 22:55                   ` Scott Wood
2011-02-01  5:26 ` Kumar Gala

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=20110131201154.B982BD4D67C@gemini.denx.de \
    --to=wd@denx.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.