SPEAr : Support added for SPEAr300 board
PATCH v4 9/12 SPEAr : Support added for SPEAr300 board
SPEAr300 SoC support contains basic spear300 support along with the usage of following drivers
- serial driver(UART)
- i2c driver
- smi driver
- nand driver(FSMC)
- usbd driver
--- MAKEALL | 1 + Makefile | 1 + board/spear/spear300/Makefile | 51 ++++++++++++++++++++++++++++++++++ board/spear/spear300/config.mk | 39 ++++++++++++++++++++++++++ board/spear/spear300/spear300.c | 57 +++++++++++++++++++++++++++++++++++++++ include/configs/spear.h | 25 +++++++++++++++-- 6 files changed, 171 insertions(+), 3 deletions(-)
create mode 100755 board/spear/spear300/Makefile create mode 100755 board/spear/spear300/config.mk create mode 100755 board/spear/spear300/spear300.c
<snip>
include/configs/spear.h +5 -0
@@ -28,7 +28,12 @@* High Level Configuration Options * (easy to change) */ +#if defined(CONFIG_MK_spear600)#define CONFIG_SPEAR600 1 +#elif defined(CONFIG_MK_spear300) +#define CONFIG_SPEAR3XX 1 +#define CONFIG_SPEAR300 1 +#endif
There should be a config file per board.
The common parts can be kept in a separate file
So the should be 'spear300.h' etc.
Tom
PATCH v4 9/12 SPEAr : Support added for SPEAr300 board
Hello Tom,
Quoted 22 lines
include/configs/spear.h | 25 +++++++++++++++-- 6 files changed, 171 insertions(+), 3 deletions(-) create mode 100755 board/spear/spear300/Makefile create mode 100755 board/spear/spear300/config.mk create mode 100755 board/spear/spear300/spear300.c<snip>
include/configs/spear.h +5
-0@@ -28,7 +28,12 @@* High Level Configuration Options * (easy to change) */ +#if defined(CONFIG_MK_spear600)#define CONFIG_SPEAR600 1 +#elif defined(CONFIG_MK_spear300) +#define CONFIG_SPEAR3XX 1 +#define CONFIG_SPEAR300 1 +#endif
Quoted 3 lines
There should be a config file per board.
The common parts can be kept in a separate file
So the should be 'spear300.h' etc.
Is it mandatory ?
Since there are already some board configs in Makefile eg. P2020RDB_config and P1011RDB_config etc
In fact, I got this comment from Wolfgang on my first patch set that these are similar boards and can be combined
Regards
Vipin
PATCH v4 9/12 SPEAr : Support added for SPEAr300 board
Quoted 32 lines · Vipin KUMAR
Hello Tom,
Quoted 22 lines
include/configs/spear.h | 25 +++++++++++++++-- 6 files changed, 171 insertions(+), 3 deletions(-) create mode 100755 board/spear/spear300/Makefile create mode 100755 board/spear/spear300/config.mk create mode 100755 board/spear/spear300/spear300.c<snip>
include/configs/spear.h +5
-0@@ -28,7 +28,12 @@* High Level Configuration Options * (easy to change) */ +#if defined(CONFIG_MK_spear600)#define CONFIG_SPEAR600 1 +#elif defined(CONFIG_MK_spear300) +#define CONFIG_SPEAR3XX 1 +#define CONFIG_SPEAR300 1 +#endif Quoted 3 lines
There should be a config file per board.
The common parts can be kept in a separate file
So the should be 'spear300.h' etc.Is it mandatory ?
Since there are already some board configs inMakefileeg.P2020RDB_configandP1011RDB_configetc
Quoted 2 lines
In fact, I got this comment from Wolfgang on my first patch set that these are similar boards and can be combined
Yes similar boards can share the same config.
This is I believe is ok if the board has a slightly different hw configuration that its siblings. For example using NOR instead of NAND.
A good rule of thumb is, boards with the same machine id should have the same config file. Those with different should not.
The split for you would bespear3xx.h and spear6xx.h
or
spear300.h spear310.h spear320.h spear600.h
or
common-spear.h and then smaller spear*.h files.
Tom
Quoted 3 lines
Regards
Vipin