]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
fsl: improve the PIXIS code and fix a few bugs
authorTimur Tabi <timur@freescale.com>
Wed, 31 Mar 2010 22:44:13 +0000 (17:44 -0500)
committerKumar Gala <galak@kernel.crashing.org>
Wed, 7 Apr 2010 05:21:27 +0000 (00:21 -0500)
Refactor and document the Freescale PIXIS code, used on most 85xx and 86xx
boards.  This makes the code easier to read and more flexible.

Delete pixis.h, because none of the exported functions were actually being
used by any other file.  Make all of the functions in pixis.c 'static'.
Remove "#include pixis.h" from every file that has it.

Remove some unnecessary #includes.

Make 'pixis_base' into a macro, so that we don't need to define it in every
function.

Add "while(1);" loops at the end of functions that reset the board, so that
execution doesn't continue while the reset is in progress.

Replace in_8/out_8 calls with clrbits_8, setbits_8, or clrsetbits_8, where
appropriate.

Replace ulong/uint with their spelled-out equivalents.  Remove unnecessary
typecasts, changing the types of some variables if necessary.

Add CONFIG_SYS_PIXIS_VCFGEN0_ENABLE and CONFIG_SYS_PIXIS_VBOOT_ENABLE to make
it easier for specific boards to support variations in the PIXIS registers
sets.  No current boards appears to need this feature.

Fix the definition of CONFIG_SYS_PIXIS_VBOOT_MASK for the MPC8610 HPCD.
Apparently, "pixis_reset altbank" has never worked on this board.

Signed-off-by: Timur Tabi <timur@freescale.com>
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
board/freescale/common/pixis.c
board/freescale/common/pixis.h [deleted file]
board/freescale/mpc8536ds/mpc8536ds.c
board/freescale/mpc8544ds/mpc8544ds.c
board/freescale/mpc8572ds/mpc8572ds.c
board/freescale/mpc8610hpcd/mpc8610hpcd.c
board/freescale/mpc8610hpcd/mpc8610hpcd_diu.c
board/freescale/mpc8641hpcn/mpc8641hpcn.c
board/freescale/p2020ds/p2020ds.c
cpu/mpc512x/diu.c
include/configs/MPC8610HPCD.h

index 7210512bfbcb686e369d3ba88b32f1291e118922..119eaf95af05c8f01063a2f6c6ae2390c8ca60cb 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2006 Freescale Semiconductor
+ * Copyright 2006,2010 Freescale Semiconductor
  * Jeff Brown
  * Srikanth Srinivasan (srikanth.srinivasan@freescale.com)
  *
 
 #include <common.h>
 #include <command.h>
-#include <watchdog.h>
-#include <asm/cache.h>
 #include <asm/io.h>
 
-#include "pixis.h"
-
-
-static ulong strfractoint(uchar *strptr);
-
+#define pixis_base (u8 *)PIXIS_BASE
 
 /*
  * Simple board reset.
  */
 void pixis_reset(void)
 {
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
        out_8(pixis_base + PIXIS_RST, 0);
-}
 
+       while (1);
+}
 
 /*
  * Per table 27, page 58 of MPC8641HPCN spec.
  */
-int set_px_sysclk(ulong sysclk)
+static int set_px_sysclk(unsigned long sysclk)
 {
        u8 sysclk_s, sysclk_r, sysclk_v, vclkh, vclkl, sysclk_aux;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
 
        switch (sysclk) {
        case 33:
@@ -117,13 +110,13 @@ int set_px_sysclk(ulong sysclk)
        return 1;
 }
 
-
-int set_px_mpxpll(ulong mpxpll)
+/* Set the CFG_SYSPLL bits
+ *
+ * This only has effect if PX_VCFGEN0[SYSPLL]=1, which is true if
+ * read_from_px_regs() is called.
+ */
+static int set_px_mpxpll(unsigned long mpxpll)
 {
-       u8 tmp;
-       u8 val;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
-
        switch (mpxpll) {
        case 2:
        case 4:
@@ -133,28 +126,19 @@ int set_px_mpxpll(ulong mpxpll)
        case 12:
        case 14:
        case 16:
-               val = (u8) mpxpll;
-               break;
-       default:
-               printf("Unsupported MPXPLL ratio.\n");
-               return 0;
+               clrsetbits_8(pixis_base + PIXIS_VSPEED1, 0x1F, mpxpll);
+               return 1;
        }
 
-       tmp = in_8(pixis_base + PIXIS_VSPEED1);
-       tmp = (tmp & 0xF0) | (val & 0x0F);
-       out_8(pixis_base + PIXIS_VSPEED1, tmp);
-
-       return 1;
+       printf("Unsupported MPXPLL ratio.\n");
+       return 0;
 }
 
-
-int set_px_corepll(ulong corepll)
+static int set_px_corepll(unsigned long corepll)
 {
-       u8 tmp;
        u8 val;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
 
-       switch ((int)corepll) {
+       switch (corepll) {
        case 20:
                val = 0x08;
                break;
@@ -178,113 +162,132 @@ int set_px_corepll(ulong corepll)
                return 0;
        }
 
-       tmp = in_8(pixis_base + PIXIS_VSPEED0);
-       tmp = (tmp & 0xE0) | (val & 0x1F);
-       out_8(pixis_base + PIXIS_VSPEED0, tmp);
-
+       clrsetbits_8(pixis_base + PIXIS_VSPEED0, 0x1F, val);
        return 1;
 }
 
+#ifndef CONFIG_SYS_PIXIS_VCFGEN0_ENABLE
+#define CONFIG_SYS_PIXIS_VCFGEN0_ENABLE                0x1C
+#endif
 
-void read_from_px_regs(int set)
+/* Tell the PIXIS where to find the COREPLL, MPXPLL, SYSCLK values
+ *
+ * The PIXIS can be programmed to look at either the on-board dip switches
+ * or various other PIXIS registers to determine the values for COREPLL,
+ * MPXPLL, and SYSCLK.
+ *
+ * CONFIG_SYS_PIXIS_VCFGEN0_ENABLE is the value to write to the PIXIS_VCFGEN0
+ * register that tells the pixis to use the various PIXIS register.
+ */
+static void read_from_px_regs(int set)
 {
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
-       u8 mask = 0x1C; /* COREPLL, MPXPLL, SYSCLK controlled by PIXIS */
        u8 tmp = in_8(pixis_base + PIXIS_VCFGEN0);
 
        if (set)
-               tmp = tmp | mask;
+               tmp = tmp | CONFIG_SYS_PIXIS_VCFGEN0_ENABLE;
        else
-               tmp = tmp & ~mask;
+               tmp = tmp & ~CONFIG_SYS_PIXIS_VCFGEN0_ENABLE;
+
        out_8(pixis_base + PIXIS_VCFGEN0, tmp);
 }
 
+/* CONFIG_SYS_PIXIS_VBOOT_ENABLE is the value to write to the PX_VCFGEN1
+ * register that tells the pixis to use the PX_VBOOT[LBMAP] register.
+ */
+#ifndef CONFIG_SYS_PIXIS_VBOOT_ENABLE
+#define CONFIG_SYS_PIXIS_VBOOT_ENABLE  0x04
+#endif
 
-void read_from_px_regs_altbank(int set)
+/* Configure the source of the boot location
+ *
+ * The PIXIS can be programmed to look at either the on-board dip switches
+ * or the PX_VBOOT[LBMAP] register to determine where we should boot.
+ *
+ * If we want to boot from the alternate boot bank, we need to tell the PIXIS
+ * to ignore the on-board dip switches and use the PX_VBOOT[LBMAP] instead.
+ */
+static void read_from_px_regs_altbank(int set)
 {
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
-       u8 mask = 0x04; /* FLASHBANK and FLASHMAP controlled by PIXIS */
        u8 tmp = in_8(pixis_base + PIXIS_VCFGEN1);
 
        if (set)
-               tmp = tmp | mask;
+               tmp = tmp | CONFIG_SYS_PIXIS_VBOOT_ENABLE;
        else
-               tmp = tmp & ~mask;
+               tmp = tmp & ~CONFIG_SYS_PIXIS_VBOOT_ENABLE;
+
        out_8(pixis_base + PIXIS_VCFGEN1, tmp);
 }
 
+/* CONFIG_SYS_PIXIS_VBOOT_MASK contains the bits to set in VBOOT register that
+ * tells the PIXIS what the alternate flash bank is.
+ *
+ * Note that it's not really a mask.  It contains the actual LBMAP bits that
+ * must be set to select the alternate bank.  This code assumes that the
+ * primary bank has these bits set to 0, and the alternate bank has these
+ * bits set to 1.
+ */
 #ifndef CONFIG_SYS_PIXIS_VBOOT_MASK
 #define CONFIG_SYS_PIXIS_VBOOT_MASK    (0x40)
 #endif
 
-void clear_altbank(void)
+/* Tell the PIXIS to boot from the default flash bank
+ *
+ * Program the default flash bank into the VBOOT register.  This register is
+ * used only if PX_VCFGEN1[FLASH]=1.
+ */
+static void clear_altbank(void)
 {
-       u8 tmp;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
-
-       tmp = in_8(pixis_base + PIXIS_VBOOT);
-       tmp &= ~CONFIG_SYS_PIXIS_VBOOT_MASK;
-
-       out_8(pixis_base + PIXIS_VBOOT, tmp);
+       clrbits_8(pixis_base + PIXIS_VBOOT, CONFIG_SYS_PIXIS_VBOOT_MASK);
 }
 
-
-void set_altbank(void)
+/* Tell the PIXIS to boot from the alternate flash bank
+ *
+ * Program the alternate flash bank into the VBOOT register.  This register is
+ * used only if PX_VCFGEN1[FLASH]=1.
+ */
+static void set_altbank(void)
 {
-       u8 tmp;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
-
-       tmp = in_8(pixis_base + PIXIS_VBOOT);
-       tmp |= CONFIG_SYS_PIXIS_VBOOT_MASK;
-
-       out_8(pixis_base + PIXIS_VBOOT, tmp);
+       setbits_8(pixis_base + PIXIS_VBOOT, CONFIG_SYS_PIXIS_VBOOT_MASK);
 }
 
-
-void set_px_go(void)
+/* Reset the board with watchdog disabled.
+ *
+ * This respects the altbank setting.
+ */
+static void set_px_go(void)
 {
-       u8 tmp;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
+       /* Disable the VELA sequencer and watchdog */
+       clrbits_8(pixis_base + PIXIS_VCTL, 9);
 
-       tmp = in_8(pixis_base + PIXIS_VCTL);
-       tmp = tmp & 0x1E;                       /* clear GO bit */
-       out_8(pixis_base + PIXIS_VCTL, tmp);
+       /* Reboot by starting the VELA sequencer */
+       setbits_8(pixis_base + PIXIS_VCTL, 0x1);
 
-       tmp = in_8(pixis_base + PIXIS_VCTL);
-       tmp = tmp | 0x01;       /* set GO bit - start reset sequencer */
-       out_8(pixis_base + PIXIS_VCTL, tmp);
+       while (1);
 }
 
-
-void set_px_go_with_watchdog(void)
+/* Reset the board with watchdog enabled.
+ *
+ * This respects the altbank setting.
+ */
+static void set_px_go_with_watchdog(void)
 {
-       u8 tmp;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
+       /* Disable the VELA sequencer */
+       clrbits_8(pixis_base + PIXIS_VCTL, 1);
 
-       tmp = in_8(pixis_base + PIXIS_VCTL);
-       tmp = tmp & 0x1E;
-       out_8(pixis_base + PIXIS_VCTL, tmp);
+       /* Enable the watchdog and reboot by starting the VELA sequencer */
+       setbits_8(pixis_base + PIXIS_VCTL, 0x9);
 
-       tmp = in_8(pixis_base + PIXIS_VCTL);
-       tmp = tmp | 0x09;
-       out_8(pixis_base + PIXIS_VCTL, tmp);
+       while (1);
 }
 
-
-int pixis_disable_watchdog_cmd(cmd_tbl_t *cmdtp,
-                              int flag, int argc, char *argv[])
+/* Disable the watchdog
+ *
+ */
+static int pixis_disable_watchdog_cmd(cmd_tbl_t *cmdtp, int flag, int argc,
+                                     char *argv[])
 {
-       u8 tmp;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
-
-       tmp = in_8(pixis_base + PIXIS_VCTL);
-       tmp = tmp & 0x1E;
-       out_8(pixis_base + PIXIS_VCTL, tmp);
-
-       /* setting VCTL[WDEN] to 0 to disable watch dog */
-       tmp = in_8(pixis_base + PIXIS_VCTL);
-       tmp &= ~0x08;
-       out_8(pixis_base + PIXIS_VCTL, tmp);
+       /* Disable the VELA sequencer and the watchdog */
+       clrbits_8(pixis_base + PIXIS_VCTL, 9);
 
        return 0;
 }
@@ -296,16 +299,17 @@ U_BOOT_CMD(
 );
 
 #ifdef CONFIG_PIXIS_SGMII_CMD
-int pixis_set_sgmii(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
+
+/* Enable or disable SGMII mode for a TSEC
+ */
+static int pixis_set_sgmii(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 {
        int which_tsec = -1;
-       u8 *pixis_base = (u8 *)PIXIS_BASE;
-       uchar mask;
-       uchar switch_mask;
+       unsigned char mask;
+       unsigned char switch_mask;
 
-       if (argc > 2)
-               if (strcmp(argv[1], "all") != 0)
-                       which_tsec = simple_strtoul(argv[1], NULL, 0);
+       if ((argc > 2) && (strcmp(argv[1], "all") != 0))
+               which_tsec = simple_strtoul(argv[1], NULL, 0);
 
        switch (which_tsec) {
 #ifdef CONFIG_TSEC1
@@ -363,6 +367,7 @@ U_BOOT_CMD(
        "    off - disables SGMII\n"
        "    switch - use switch settings"
 );
+
 #endif
 
 /*
@@ -371,14 +376,13 @@ U_BOOT_CMD(
  * FPGA register values.
  * input: strptr i.e. argv[2]
  */
-
-static ulong strfractoint(uchar *strptr)
+static unsigned long strfractoint(char *strptr)
 {
-       int i, j, retval;
+       int i, j;
        int mulconst;
-       int intarr_len = 0, decarr_len = 0, no_dec = 0;
-       ulong intval = 0, decval = 0;
-       uchar intarr[3], decarr[3];
+       int intarr_len, no_dec = 0;
+       unsigned long intval = 0, decval = 0;
+       char intarr[3], decarr[3];
 
        /* Assign the integer part to intarr[]
         * If there is no decimal point i.e.
@@ -412,26 +416,21 @@ static ulong strfractoint(uchar *strptr)
                        j++;
                }
 
-               decarr_len = j;
                decarr[j] = '\0';
 
                mulconst = 1;
-               for (i = 0; i < decarr_len; i++)
+               for (i = 0; i < j; i++)
                        mulconst *= 10;
-               decval = simple_strtoul((char *)decarr, NULL, 10);
+               decval = simple_strtoul(decarr, NULL, 10);
        }
 
-       intval = simple_strtoul((char *)intarr, NULL, 10);
+       intval = simple_strtoul(intarr, NULL, 10);
        intval = intval * mulconst;
 
-       retval = intval + decval;
-
-       return retval;
+       return intval + decval;
 }
 
-
-int
-pixis_reset_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
+static int pixis_reset_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 {
        unsigned int i;
        char *p_cf = NULL;
@@ -440,7 +439,7 @@ pixis_reset_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
        char *p_cf_mpxpll = NULL;
        char *p_altbank = NULL;
        char *p_wd = NULL;
-       unsigned int unknown_param = 0;
+       int unknown_param = 0;
 
        /*
         * No args is a simple reset request.
@@ -493,9 +492,9 @@ pixis_reset_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
         */
        read_from_px_regs(0);
 
-       if (p_altbank) {
+       if (p_altbank)
                read_from_px_regs_altbank(0);
-       }
+
        clear_altbank();
 
        /*
@@ -507,7 +506,7 @@ pixis_reset_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
                unsigned long mpxpll;
 
                sysclk = simple_strtoul(p_cf_sysclk, NULL, 10);
-               corepll = strfractoint((uchar *) p_cf_corepll);
+               corepll = strfractoint(p_cf_corepll);
                mpxpll = simple_strtoul(p_cf_mpxpll, NULL, 10);
 
                if (!(set_px_sysclk(sysclk)
@@ -536,11 +535,10 @@ pixis_reset_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
        /*
         * Reset with watchdog specified.
         */
-       if (p_wd) {
+       if (p_wd)
                set_px_go_with_watchdog();
-       } else {
+       else
                set_px_go();
-       }
 
        /*
         * Shouldn't be reached.
diff --git a/board/freescale/common/pixis.h b/board/freescale/common/pixis.h
deleted file mode 100644 (file)
index ff62a62..0000000
+++ /dev/null
@@ -1,31 +0,0 @@
-/*
- * Copyright 2006 Freescale Semiconductor
- *
- * See file CREDITS for list of people who contributed to this
- * project.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as
- * published by the Free Software Foundation; either version 2 of
- * the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
- * MA 02111-1307 USA
- */
-
-extern void pixis_reset(void);
-extern int set_px_sysclk(ulong sysclk);
-extern int set_px_mpxpll(ulong mpxpll);
-extern int set_px_corepll(ulong corepll);
-extern void read_from_px_regs(int set);
-extern void read_from_px_regs_altbank(int set);
-extern void set_altbank(void);
-extern void set_px_go(void);
-extern void set_px_go_with_watchdog(void);
index 81a56b55e0bdd16c71f141befce3cfd89098126a..253ed181fcfbe0fe7f612f754f4d47e056dac732 100644 (file)
@@ -39,7 +39,6 @@
 #include <netdev.h>
 #include <sata.h>
 
-#include "../common/pixis.h"
 #include "../common/sgmii_riser.h"
 
 phys_size_t fixed_sdram(void);
index b35e02f6855378049e656dce5da4f2fd50ade2ca..0be2d892d30b0793cb3f81d5b05ed117e39d69bf 100644 (file)
@@ -35,7 +35,6 @@
 #include <tsec.h>
 #include <netdev.h>
 
-#include "../common/pixis.h"
 #include "../common/sgmii_riser.h"
 
 int checkboard (void)
index 74085c3260ed2ddf1ea415d08c7c7cca29226ac2..6029a5185c2a7ded93e49afce4344b0224a276b0 100644 (file)
@@ -36,7 +36,6 @@
 #include <tsec.h>
 #include <netdev.h>
 
-#include "../common/pixis.h"
 #include "../common/sgmii_riser.h"
 
 long int fixed_sdram(void);
index ab5f800453b6043a42ee86df90ec8b5502a3994c..2ef7b2323da1cec2e51f0b6c0e459956aad28f85 100644 (file)
@@ -34,8 +34,6 @@
 #include <spd_sdram.h>
 #include <netdev.h>
 
-#include "../common/pixis.h"
-
 void sdram_init(void);
 phys_size_t fixed_sdram(void);
 void mpc8610hpcd_diu_init(void);
index 4186a2ecdadf59aceac44da1020d597266683e89..94fb1ebf8f8604c4f1d7fc5fd3ae7183451d3d98 100644 (file)
@@ -29,7 +29,6 @@
 
 #ifdef CONFIG_FSL_DIU_FB
 
-#include "../common/pixis.h"
 #include "../common/fsl_diu_fb.h"
 
 #if defined(CONFIG_VIDEO) || defined(CONFIG_CFB_CONSOLE)
index 443c9fd8ec4f036695f77f920d3fd170d19b8fe1..b352c334cf8eec912e2cf30a9d788085e95c61b9 100644 (file)
@@ -31,8 +31,6 @@
 #include <fdt_support.h>
 #include <netdev.h>
 
-#include "../common/pixis.h"
-
 phys_size_t fixed_sdram(void);
 
 int board_early_init_f(void)
index f6eae55d1d68dc25aff9dc19bb0d12000133e382..664135cf703d47aa1d6e175bff8029885798b849 100644 (file)
@@ -38,7 +38,6 @@
 #include <asm/mp.h>
 #include <netdev.h>
 
-#include "../common/pixis.h"
 #include "../common/sgmii_riser.h"
 
 DECLARE_GLOBAL_DATA_PTR;
index ca459a101e91739acc8a35224ded4b4c0e447005..c2453c68d148ac46534218e8d47ce13533a072e8 100644 (file)
@@ -27,7 +27,6 @@
 #include <command.h>
 #include <asm/io.h>
 
-#include "../../board/freescale/common/pixis.h"
 #include "../../board/freescale/common/fsl_diu_fb.h"
 
 #if defined(CONFIG_VIDEO) || defined(CONFIG_CFB_CONSOLE)
index fed441eb37c4914ea6a92b896ef845b2811a47a8..8382e3ca83e93acd67a4234fc947de1f80d9cad7 100644 (file)
 #define PIXIS_VSPEED1          0x18    /* VELA VSpeed 1 */
 #define PIXIS_VCLKH            0x19    /* VELA VCLKH register */
 #define PIXIS_VCLKL            0x1A    /* VELA VCLKL register */
-#define CONFIG_SYS_PIXIS_VBOOT_MASK    0x0C    /* Reset altbank mask*/
+#define CONFIG_SYS_PIXIS_VBOOT_MASK    0xC0    /* Reset altbank mask */
 
 #define CONFIG_SYS_MAX_FLASH_BANKS     2               /* number of banks */
 #define CONFIG_SYS_MAX_FLASH_SECT      1024            /* sectors per device */