public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 17/31] Enable XAUI interface for B4860QDS
Date: Fri, 22 Mar 2013 22:02:15 +0100	[thread overview]
Message-ID: <20130322210215.BE5092014CE@gemini.denx.de> (raw)
In-Reply-To: <1363973369-26110-17-git-send-email-yorksun@freescale.com>

Dear York Sun,

In message <1363973369-26110-17-git-send-email-yorksun@freescale.com> you wrote:
> From: Suresh Gupta <suresh.gupta@freescale.com>
> 
> - Added SERDES2 PRTCLs = 0x98, 0x9E
> - Default Phy Addresses for Teranetics PHY on XAUI card
> 	The PHY addresses of Teranetics PHY on XAUI riser card are assigned
> 	based on the slot it is in. Switches SW4[2:4] and SW6[2:4] on
> 	AMC2PEX-2S On B4860QDS, AMC2PEX card decide the PHY addresses on slot1
>         and slot2
> - Configure MDIO for 10Gig Mac
> 
> Signed-off-by: Suresh Gupta <suresh.gupta@freescale.com>
> ---
>  board/freescale/b4860qds/eth_b4860qds.c |   38 +++++++++++++++++++++++++++++++
>  drivers/net/fm/b4860.c                  |    4 +++-
>  include/configs/B4860QDS.h              |    6 ++++-
>  3 files changed, 46 insertions(+), 2 deletions(-)

WARNING: quoted string split across lines
#134: FILE: board/freescale/b4860qds/eth_b4860qds.c:281:
+               debug("Setting phy addresses on B4860 QDS AMC2PEX-2S"
+                               "for FM1_10GEC1: %x\n",

CHECK: Alignment should match open parenthesis
#137: FILE: board/freescale/b4860qds/eth_b4860qds.c:284:
+               fm_info_set_phy_address(FM1_10GEC1,
+                               CONFIG_SYS_FM1_10GEC1_PHY_ADDR);

WARNING: quoted string split across lines
#139: FILE: board/freescale/b4860qds/eth_b4860qds.c:286:
+               debug("Setting phy addresses on B4860 QDS AMC2PEX-2S"
+                               "for FM1_10GEC2: %x\n",

CHECK: Alignment should match open parenthesis
#142: FILE: board/freescale/b4860qds/eth_b4860qds.c:289:
+               fm_info_set_phy_address(FM1_10GEC2,
+                               CONFIG_SYS_FM1_10GEC2_PHY_ADDR);

WARNING: quoted string split across lines
#147: FILE: board/freescale/b4860qds/eth_b4860qds.c:294:
+               debug("Setting phy addresses on B4860 QDS AMC2PEX-2S"
+                               "for FM1_10GEC2: %x\n",

CHECK: Alignment should match open parenthesis
#150: FILE: board/freescale/b4860qds/eth_b4860qds.c:297:
+               fm_info_set_phy_address(FM1_10GEC2,
+                               CONFIG_SYS_FM1_10GEC2_PHY_ADDR);

CHECK: Alignment should match open parenthesis
#165: FILE: board/freescale/b4860qds/eth_b4860qds.c:330:
+                       fm_info_set_mdio(i,
+                            miiphy_get_dev_by_name(DEFAULT_FM_TGEC_MDIO_NAME));

CHECK: Alignment should match open parenthesis
#169: FILE: board/freescale/b4860qds/eth_b4860qds.c:334:
+                       printf("Fman1: 10GSEC%u set to unknown interface %i\n",
+                                       idx + 1, fm_info_get_enet_if(i));

CHECK: Logical continuations should be on the previous line
#190: FILE: drivers/net/fm/b4860.c:60:
        if ((port == FM1_10GEC1 || port == FM1_10GEC2)
+                       && ((is_serdes_configured(XAUI_FM1_MAC9))

CHECK: Logical continuations should be on the previous line
#191: FILE: drivers/net/fm/b4860.c:61:
+                       && ((is_serdes_configured(XAUI_FM1_MAC9))
+                       || (is_serdes_configured(XAUI_FM1_MAC10))))


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
Conceptual integrity in turn dictates that the  design  must  proceed
from  one  mind,  or  from  a  very small number of agreeing resonant
minds.               - Frederick Brooks Jr., "The Mythical Man Month" 

  reply	other threads:[~2013-03-22 21:02 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-22 17:28 [U-Boot] [PATCH 01/31] 83xx/pcie: fix build error for 83xx pcie York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 02/31] Enable L2 cache parity/ECC error checking York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 03/31] net/phy: fix select line for TN80xx York Sun
2013-03-22 20:59   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 04/31] powerpc/mpc85xx: check if core is disabled for showing status York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 05/31] powerpc/t4240qds: Add VDD override York Sun
2013-03-22 21:00   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 06/31] T4240/ramboot: enable PBL tool for T4240 York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 07/31] Add e6500 L2 replacement policy selection York Sun
2013-03-22 21:00   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 08/31] powerpc/p5040: enable NAND, SD, SPI boot support York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 09/31] powerpc/T4160: Merge T4160 and T4240 in config_mpc85xx.h York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 10/31] powerpc/mpc8xxx: Add T1040 and variant SoCs York Sun
2013-03-22 21:00   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 11/31] powerpc/chassis2: Change core numbering scheme York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 12/31] powerpc/mpc8xxx: Allow DDR overclock York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 13/31] powerpc/qixis: add clock measurement registers York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 14/31] powerpc/t4qds: use clock measurement for sysclk and ddr clock York Sun
2013-03-22 21:01   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 15/31] powerpc/p5040: enable PBL tool support York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 16/31] board/t4240qds, b4860qds: LAW/TLB for DCSR set to size 32M York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 17/31] Enable XAUI interface for B4860QDS York Sun
2013-03-22 21:02   ` Wolfgang Denk [this message]
2013-03-22 17:29 ` [U-Boot] [PATCH 18/31] powerpc/t4qds: Add SW7[4] in the DIP switch display York Sun
2013-03-22 21:02   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 19/31] powerpc/t4qds: Enable master module for Boot from SRIO and PCIE York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 20/31] powerpc/t4qds: Slave module for boot " York Sun
2013-03-22 21:02   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 21/31] powerpc/t4240qds: fix PHY reset timeout issue York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 22/31] powerpc/doc: Fix the misalignment of document README.srio-pcie-boot-corenet York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 23/31] powerpc/b4860: fix for Serdes connectivity to SFP's York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 24/31] powerpc/mpc85xx: add setting of clock-frequency for mpic node York Sun
2013-03-22 21:03   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 25/31] powerpc/p2041: fix serdes reference clock frequency display for PC board York Sun
2013-03-22 21:03   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 26/31] sf: spansion: Add support for S25FL128S York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 27/31] powerpc/B4: Merge B4420 and B4860 in config_mpc85xx.h York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 28/31] powerpc/p5040: fix mdio mux for 10G port York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 29/31] powerpc/usb: Fix usb device-tree fix-up York Sun
2013-03-22 21:04   ` Wolfgang Denk
2013-03-22 17:29 ` [U-Boot] [PATCH 30/31] fman/mEMAC: set SETSP bit in IF_MODE regisgter for RGMII speed York Sun
2013-03-22 17:29 ` [U-Boot] [PATCH 31/31] powerpc/b4860qds: Add LAW Target ID and Create LAW entry for Maple York Sun

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=20130322210215.BE5092014CE@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox