Blackfin: bf533-ezkit: shuffle flash defines a little
PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
Some of the flash defines weren't in the correct location and caused build problems in some configurations, so let's move types and defines to better local locations.
--- board/bf533-ezkit/bf533-ezkit.c | 25 ++++--------------------- board/bf533-ezkit/flash-defines.h | 26 ++++---------------------- board/bf533-ezkit/flash.c | 24 ++++++------------------ 3 files changed, 14 insertions(+), 61 deletions(-)
board/bf533-ezkit/bf533-ezkit.c +4 -21
@@ -1,34 +1,17 @@/* -* U-boot - ezkit533.c +* U-Boot - bf533-ezkit.c * -* Copyright (c) 2005-2007 Analog Devices Inc. +* Copyright (c) 2005-2008 Analog Devices Inc. * * (C) Copyright 2000-2004 * Wolfgang Denk, DENX Software Engineering, wd at denx.de. * -* 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., 51 Franklin St, Fifth Floor, Boston, -* MA 02110-1301 USA +* Licensed under the GPL-2 or later. */ #include <common.h> -#if defined(CONFIG_MISC_INIT_R )#include "psd4256.h" -#endif +#include "flash-defines.h" DECLARE_GLOBAL_DATA_PTR ;
board/bf533-ezkit/flash-defines.h +4 -22
@@ -1,28 +1,12 @@/* -* U-boot - flash-defines.h +* U-Boot - flash-defines.h * -* Copyright (c) 2005-2007 Analog Devices Inc. +* Copyright (c) 2005-2008 Analog Devices Inc. * * (C) Copyright 2000-2004 * Wolfgang Denk, DENX Software Engineering, wd at denx.de. * -* 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., 51 Franklin St, Fifth Floor, Boston, -* MA 02110-1301 USA +* Licensed under the GPL-2 or later. */ #ifndef __FLASHDEFINES_H__
@@ -50,6 +34,7 @@#define FLASH_SIZE 0x220000#define FLASH_MAN_ST 2#define CFG_FLASH0_BASE 0x20000000 +#define CFG_FLASH1_BASE 0x20200000#define RESET_VAL 0xF0flash_info_t flash_info[CFG_MAX_FLASH_BANKS ];
@@ -68,9 +53,6 @@ int write_flash(long nOffset, int nValue);void get_sector_number (long lOffset,int *pnSector);int GetSectorProtectionStatus (flash_info_t * info,int nSector);int GetOffset (int nBlock); -int AFP_NumSectors =40 ; -long AFP_SectorSize1 =0x10000 ; -int AFP_SectorSize2 =0x4000 ;#define WRITESEQ1 0x0AAA#define WRITESEQ2 0x0554
board/bf533-ezkit/flash.c +6 -18
@@ -1,34 +1,22 @@/* * U-boot - flash.c Flash driver for PSD4256GV * -* Copyright (c) 2005-2007 Analog Devices Inc. +* Copyright (c) 2005-2008 Analog Devices Inc. * This file is based on BF533EzFlash.c originally written by Analog Devices, Inc. * * (C) Copyright 2000-2004 * Wolfgang Denk, DENX Software Engineering, wd at denx.de. * -* 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., 51 Franklin St, Fifth Floor, Boston, -* MA 02110-1301 USA +* Licensed under the GPL-2 or later. */ #include <asm/io.h> #include "flash-defines.h" +int AFP_NumSectors =40 ; +long AFP_SectorSize1 =0x10000 ; +int AFP_SectorSize2 =0x4000 ; +void flash_reset (void ) {reset_flash ();
Signature · 1 line
1.5.5.3
PATCH 1/1 Blackfin: unify common ADI board settings
Rather than duplicate the same ADI settings in every ADI board, create a common ADI config header and have all ADI boards start using that. This will also make merging the ~10 boards I have to forward port a lot easier.
--- include/asm-blackfin/blackfin-config-post.h | 74 +++++- include/configs/bf533-ezkit.h | 246 ++++++----------- include/configs/bf533-stamp.h | 381 +++++++-------------------- include/configs/bf537-stamp.h | 359 ++++++++------------------ include/configs/bf561-ezkit.h | 283 +++++++------------- include/configs/bfin_adi_common.h | 138 ++++++++++ 6 files changed, 599 insertions(+), 882 deletions(-) create mode 100644 include/configs/bfin_adi_common.h
include/asm-blackfin/blackfin-config-post.h +68 -6
@@ -1,7 +1,7 @@/* * blackfin-config-post.h - setup common defines for Blackfin boards based on config.h * -* Copyright (c) 2007 Analog Devices Inc. +* Copyright (c) 2007-2008 Analog Devices Inc. * * Licensed under the GPL-2 or later. */
@@ -9,11 +9,6 @@#ifndef __ASM_BLACKFIN_CONFIG_POST_H__#define __ASM_BLACKFIN_CONFIG_POST_H__ -/* Check to make sure everything fits in external RAM */ -#if ((CFG_MONITOR_BASE +CFG_MONITOR_LEN ) >CFG_MAX_RAM_SIZE ) -# error Memory Map does not fit into configuration -#endif -/* Sanity check CONFIG_BFIN_CPU */ #ifndef CONFIG_BFIN_CPU # error CONFIG_BFIN_CPU: your board config needs to define this
@@ -69,4 +64,71 @@# define CMD_LINE_ADDR L1_SRAM_SCRATCH#endif +/* Default/common Blackfin memory layout */ +#ifndef CFG_SDRAM_BASE +# define CFG_SDRAM_BASE 0 +#endif +#ifndef CFG_MAX_RAM_SIZE +# define CFG_MAX_RAM_SIZE (CONFIG_MEM_SIZE * 1024 * 1024) +#endif +#ifndef CFG_MEMTEST_START +# define CFG_MEMTEST_START 0 +#endif +#ifndef CFG_MEMTEST_END +# define CFG_MEMTEST_END ((CONFIG_MEM_SIZE - 1) * 1024 * 1024) +#endif +#ifndef CFG_MONITOR_BASE +# define CFG_MONITOR_BASE (CFG_MAX_RAM_SIZE - CFG_MONITOR_LEN) +#endif +#ifndef CFG_MALLOC_BASE +# define CFG_MALLOC_BASE (CFG_MONITOR_BASE - CFG_MALLOC_LEN) +#endif +#ifndef CFG_GBL_DATA_ADDR +# define CFG_GBL_DATA_ADDR (CFG_MALLOC_BASE - CFG_GBL_DATA_SIZE) +#endif +#ifndef CONFIG_STACKBASE +# define CONFIG_STACKBASE (CFG_GBL_DATA_ADDR - 4) +#endif + +/* Check to make sure everything fits in external RAM */ +#if ((CFG_MONITOR_BASE +CFG_MONITOR_LEN ) >CFG_MAX_RAM_SIZE ) +# error Memory Map does not fit into configuration +#endif + +/* Default/common Blackfin environment settings */ +#ifndef CONFIG_LOADADDR +# define CONFIG_LOADADDR 0x1000000 +#endif +#ifndef CFG_LOAD_ADDR +# define CFG_LOAD_ADDR CONFIG_LOADADDR +#endif +#ifndef CFG_BOOTM_LEN +# define CFG_BOOTM_LEN 0x4000000 +#endif +#ifndef CFG_PROMPT +# define CFG_PROMPT "bfin> " +#endif +#ifndef CFG_CBSIZE +# if (CONFIG_COMMANDS &CFG_CMD_KGDB ) +# define CFG_CBSIZE 1024 +# else +# define CFG_CBSIZE 256 +# endif +#endif +#ifndef CFG_BARGSIZE +# define CFG_BARGSIZE CFG_CBSIZE +#endif +#ifndef CFG_PBSIZE +# define CFG_PBSIZE (CFG_CBSIZE + sizeof(CFG_PROMPT) + 16) +#endif +#ifndef CFG_MAXARGS +# define CFG_MAXARGS 16 +#endif +#ifndef CFG_HZ +# define CFG_HZ 1000 +#endif +#ifndef CFG_BAUDRATE_TABLE +# define CFG_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, 115200 } +#endif +#endif
include/configs/bf533-ezkit.h +86 -160
298 lines not on this page · raw message
include/configs/bf533-stamp.h +99 -282
456 lines not on this page · raw message
include/configs/bf537-stamp.h +108 -251
440 lines not on this page · raw message
include/configs/bf561-ezkit.h +100 -183
329 lines not on this page · raw message
include/configs/bfin_adi_common.h new file +138 -0
138 lines not on this page · raw message
Signature · 1 line
1.5.5.3
PATCH 1/1 Blackfin: unify common ADI board settings
Quoted 3 lines · In message <[email protected]> you wrote:
Rather than duplicate the same ADI settings in every ADI board, create a common ADI config header and have all ADI boards start using that. This will also make merging the ~10 boards I have to forward port a lot easier.
...
Quoted 1 line
+#define
SMC91111_EEPROM_INIT(){*pFIO_DIR=0x01;*pFIO_FLAG_S=0x01;SSYNC(); }
...
Quoted 2 lines
(quoted diff) +2
-0+#define I2C_ACTIVE do { *pFIO_DIR |= PF_SDA; *pFIO_INEN &= ~PF_SDA; SSYNC(); } while (0) +#define I2C_TRISTATE do { *pFIO_DIR &= ~PF_SDA; *pFIO_INEN |= PF_SDA; SSYNC(); } while (0)
...
Review
Quoted 2 lines
(quoted diff) +2
-0+#define CFG_MONITOR_LEN (256 * 1024) /* Reserve 256 kB for monitor */ +#define CFG_MALLOC_LEN (384 * 1024) /* Reserve 384 kB for malloc() (video/spi are big) */
etc. Many lines are much too long...
Best regards,
Wolfgang Denk
Signature · 5 lines
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 Peace was the way. -- Kirk, "The City on the Edge of Forever", stardate unknown
PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
In reply to Mike FrysingerQuoted 78 lines · In message <[email protected]> you wrote:
Some of the flash defines weren't in the correct location and caused build problems in some configurations, so let's move types and defines to better local locations.
Signed-off-by Mike Frysinger <[email protected]>--- board/bf533-ezkit/bf533-ezkit.c | 25 ++++--------------------- board/bf533-ezkit/flash-defines.h | 26 ++++---------------------- board/bf533-ezkit/flash.c | 24 ++++++------------------ 3 files changed, 14 insertions(+), 61 deletions(-)board/bf533-ezkit/bf533-ezkit.c +4
-21@@ -1,34 +1,17 @@/* -* U-boot - ezkit533.c +* U-Boot - bf533-ezkit.c * -* Copyright (c) 2005-2007 Analog Devices Inc. +* Copyright (c) 2005-2008 Analog Devices Inc. * * (C) Copyright 2000-2004 * Wolfgang Denk, DENX Software Engineering, wd at denx.de. * -* 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., 51 Franklin St, Fifth Floor, Boston, -* MA 02110-1301 USA +* Licensed under the GPL-2 or later. */ #include <common.h> -#if defined(CONFIG_MISC_INIT_R )#include "psd4256.h" -#endif +#include "flash-defines.h" DECLARE_GLOBAL_DATA_PTR ;board/bf533-ezkit/flash-defines.h +2
-11@@ -1,28 +1,12 @@/* -* U-boot - flash-defines.h +* U-Boot - flash-defines.h * -* Copyright (c) 2005-2007 Analog Devices Inc. +* Copyright (c) 2005-2008 Analog Devices Inc. * * (C) Copyright 2000-2004 * Wolfgang Denk, DENX Software Engineering, wd at denx.de. * -* 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,
board/bf533-ezkit/flash-defines.h line 10
Quoted 10 lines
board/bf533-ezkit/flash-defines.h +1
-8-* 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., 51 Franklin St, Fifth Floor, Boston, -* MA 02110-1301 USA +* Licensed under the GPL-2 or later. */
Please STOP messing with license headers.
I will reject all such edits.
Quoted 43 lines
board/bf533-ezkit/flash-defines.h +1
-3#ifndef __FLASHDEFINES_H__@@ -50,6 +34,7 @@#define FLASH_SIZE 0x220000#define FLASH_MAN_ST 2#define CFG_FLASH0_BASE 0x20000000 +#define CFG_FLASH1_BASE 0x20200000#define RESET_VAL 0xF0flash_info_t flash_info[CFG_MAX_FLASH_BANKS ];@@ -68,9 +53,6 @@ int write_flash(long nOffset, int nValue);void get_sector_number (long lOffset,int *pnSector);int GetSectorProtectionStatus (flash_info_t * info,int nSector);int GetOffset (int nBlock); -int AFP_NumSectors =40 ; -long AFP_SectorSize1 =0x10000 ; -int AFP_SectorSize2 =0x4000 ;#define WRITESEQ1 0x0AAA#define WRITESEQ2 0x0554board/bf533-ezkit/flash.c +1
-10@@ -1,34 +1,22 @@/* * U-boot - flash.c Flash driver for PSD4256GV * -* Copyright (c) 2005-2007 Analog Devices Inc. +* Copyright (c) 2005-2008 Analog Devices Inc. * This file is based on BF533EzFlash.c originally written by Analog Devices, Inc. * * (C) Copyright 2000-2004 * Wolfgang Denk, DENX Software Engineering, wd at denx.de. * -* 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,
board/bf533-ezkit/flash.c line 11
Quoted 10 lines
board/bf533-ezkit/flash.c +1
-8-* 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., 51 Franklin St, Fifth Floor, Boston, -* MA 02110-1301 USA +* Licensed under the GPL-2 or later. */
Rejected.
Best regards,
Wolfgang Denk
Signature · 6 lines
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
If you want strict real-time behavior, run in the real time schedu-
ling class. But there are no seatbelts or airbags; main(){for(;;);}
can hard hang your system. -- Bart SmaaldersPATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
Quoted 27 lines · Wolfgang Denk
Quoted 26 lines · In message <[email protected]> you wrote:
board/bf533-ezkit/flash-defines.h +2
-15@@ -1,28 +1,12 @@/* -* U-boot - flash-defines.h +* U-Boot - flash-defines.h * -* Copyright (c) 2005-2007 Analog Devices Inc. +* Copyright (c) 2005-2008 Analog Devices Inc. * * (C) Copyright 2000-2004 * Wolfgang Denk, DENX Software Engineering, wd at denx.de. * -* 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. -*
Quoted 8 lines
board/bf533-ezkit/flash-defines.h line 10
Quoted 6 lines
board/bf533-ezkit/flash-defines.h +1
-4-* 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., 51 Franklin St, Fifth Floor, Boston, -* MA 02110-1301 USA +* Licensed under the GPL-2 or later. */ Please STOP messing with license headers.
so you want it in a separate commit ?
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20080601/0346ad62/attachment.pgp
PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
Quoted 4 lines · In message <[email protected]> you wrote:
Quoted 2 lines
Please STOP messing with license headers.
so you want it in a separate commit ?
I do not want to see any of these one-line GPL "headers. Please use the regular, full heade. And especially do never remove these from any files where they are present.
Best regards,
Wolfgang Denk
Signature · 4 lines
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 How can you tell when sour cream goes bad?
PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
Quoted 8 lines · Wolfgang Denk
Quoted 3 lines · In message <[email protected]> you wrote:
Quoted 1 line
Please STOP messing with license headers.
so you want it in a separate commit ?
I do not want to see any of these one-line GPL "headers. Please use the regular, full heade. And especially do never remove these from any files where they are present.
having to scroll a page to see anything actually useful seems pretty wasteful to me. plus the full header breaks when the FSF moves around (and in fact, there are a ton of such outdated headers in u-boot right now).
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20080601/acd6a31e/attachment.pgp
PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
Quoted 4 lines · In message <[email protected]> you wrote:
having to scroll a page to see anything actually useful seems pretty wasteful to me. plus the full header breaks when the FSF moves around (and in fact, there are a ton of such outdated headers in u-boot right now).
Patches for this are welcome.
Best regards,
Wolfgang Denk
Signature · 4 lines
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
Nail here --X-- for new monitor.PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
Quoted 7 lines · Wolfgang Denk
Quoted 4 lines · In message <[email protected]> you wrote:
having to scroll a page to see anything actually useful seems pretty wasteful to me. plus the full header breaks when the FSF moves around (and in fact, there are a ton of such outdated headers in u-boot right now).
Patches for this are welcome.
i posted patches to fix it in straight forward manner. going through files to update addresses is a waste of time.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20080601/c2f15a2b/attachment.pgp
PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little
In reply to Mike FrysingerHi Mike,
Quoted 12 lines · Wolfgang Denk
Quoted 8 lines
Quoted 3 lines · In message <[email protected]> you wrote:
Quoted 1 line
Please STOP messing with license headers.
so you want it in a separate commit ?
I do not want to see any of these one-line GPL "headers. Please use the regular, full heade. And especially do never remove these from any files where they are present.
having to scroll a page to see anything actually useful seems pretty wasteful to me.
Many burdens the legal system put upon software seem wasteful to me, but I still cannot simply erase it because of my personal taste.
Quoted 2 lines
plus the full header breaks when the FSF moves around (and in fact, there are a ton of such outdated headers in u-boot right now).
I am not sure whether simply pointing to something is valid in legal context.
What I am sure however is that your patches are *not* valid, because they point into nothingness. It is neccessary to include the full licences with the source code, even URLs are not ok. Handwaving (even if it is totally clear to you) does not hold up in legal contexts.
So the most I could think of is to include a file called GPL-2 and directly point to that file.
But still, this is only my personal interpretation.
Cheers
Detlev
Signature · 8 lines
I've been examining the existing [linux] kernel configuration system, and I
have about concluded that the best favor we could do everybody involved with
it is to take it out behind the barn and shoot it through the head.
-- Eric S. Raymond on linux-kbuild Mar 2000
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de