Skip to contentSkip to conversation

Blackfin: bf533-ezkit: shuffle flash defines a little

PATCH New Mike Frysinger 3 participants 10 messages Jun 1, 2008 – Jun 4, 2008

lore
Share & tools
lore
#0 lore raw

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
index 738f69c..b039811 100644
@@ -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
index 4e043e0..bb69fa8 100644
@@ -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		0xF0
 
 flash_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
index cdf4dc6..ba75d99 100644
@@ -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
#1 lore raw

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
index 6a1ffa1..d2a413f 100644
@@ -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
#2 lore raw

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
#3 lore raw

PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little

In reply to Mike Frysinger
Quoted 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.

---
 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
index 738f69c..b039811 100644
@@ -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
index 4e043e0..bb69fa8 100644
@@ -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		0xF0
 
 flash_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 +1 -10
index cdf4dc6..ba75d99 100644
@@ -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 Smaalders
#4 lore raw

PATCH 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

#5 lore raw

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?
#6 lore raw

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

#7 lore raw

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.
#8 lore raw

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

#9 lore raw

PATCH 1/1 Blackfin: bf533-ezkit: shuffle flash defines a little

In reply to Mike Frysinger

Hi 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