subject:"Re\: \[PATCH v3 10\/15\] h8300\: Build scripts"

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-18 Thread Yoshinori Sato

At Tue, 17 Feb 2015 10:46:21 +0100,
Paul Bolle wrote:
> 
> On Tue, 2015-02-17 at 16:39 +0900, Yoshinori Sato wrote:
> > At Sun, 15 Feb 2015 20:23:04 +0100,
> > Paul Bolle wrote:
> > > On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
> > > > diff --git a/arch/h8300/Kconfig.debug b/arch/h8300/Kconfig.debug
> > > > new file mode 100644
> > > > index 000..eb72b01
> > > > --- /dev/null
> > > > +++ b/arch/h8300/Kconfig.debug
> > > > @@ -0,0 +1,23 @@
> > > > +menu "Kernel hacking"
> > > > +
> > > > +source "lib/Kconfig.debug"
> > > > +
> > > > +config FULLDEBUG
> > > > +   bool "Full Symbolic/Source Debugging support"
> > > > +   help
> > > > + Enable debugging symbols on kernel build.
> > > > +
> > > > +config HIGHPROFILE
> > > > +   bool "Use fast second timer for profiling"
> > > > +   help
> > > > + Use a fast secondary clock to produce profiling information.
> > > > +
> > > > +config NO_KERNEL_MSG
> > > > +   bool "Suppress Kernel BUG Messages"
> > > > +   help
> > > > + Do not output any debug BUG messages within the kernel.
> > > > +
> > > > +config SH_STANDARD_BIOS
> > > > +   def_bool n
> > > > +
> > > 
> > > I don't think this entry is needed.
> > 
> > This entry using sh-sci. 
> 
> This entry will always set SH_STANDARD_BIOS to 'n', and a line reading
> # CONFIG_SH_STANDARD_BIOS is not set
> 
> will always be included in the generated .config for h8300. But for the
> three lines in drivers/tty/serial/sh-sci.c reading
> #ifdef CONFIG_SH_STANDARD_BIOS
> sh_bios_gdb_detach();
> #endif
> 
> that line has no effect. Even if there's no entry for SH_STANDARD_BIOS
> the preprocessor will still remove that call of sh_bios_gdb_detach().
> 
> > > > +endmenu
> 
> Thanks,

OK
It was used few years ago, but it was removed.
Removed this entry.

> 
> 
> Paul Bolle
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 
Yoshinori Sato

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-18 Thread Yoshinori Sato

At Tue, 17 Feb 2015 10:46:21 +0100,
Paul Bolle wrote:
 
 On Tue, 2015-02-17 at 16:39 +0900, Yoshinori Sato wrote:
  At Sun, 15 Feb 2015 20:23:04 +0100,
  Paul Bolle wrote:
   On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
diff --git a/arch/h8300/Kconfig.debug b/arch/h8300/Kconfig.debug
new file mode 100644
index 000..eb72b01
--- /dev/null
+++ b/arch/h8300/Kconfig.debug
@@ -0,0 +1,23 @@
+menu Kernel hacking
+
+source lib/Kconfig.debug
+
+config FULLDEBUG
+   bool Full Symbolic/Source Debugging support
+   help
+ Enable debugging symbols on kernel build.
+
+config HIGHPROFILE
+   bool Use fast second timer for profiling
+   help
+ Use a fast secondary clock to produce profiling information.
+
+config NO_KERNEL_MSG
+   bool Suppress Kernel BUG Messages
+   help
+ Do not output any debug BUG messages within the kernel.
+
+config SH_STANDARD_BIOS
+   def_bool n
+
   
   I don't think this entry is needed.
  
  This entry using sh-sci. 
 
 This entry will always set SH_STANDARD_BIOS to 'n', and a line reading
 # CONFIG_SH_STANDARD_BIOS is not set
 
 will always be included in the generated .config for h8300. But for the
 three lines in drivers/tty/serial/sh-sci.c reading
 #ifdef CONFIG_SH_STANDARD_BIOS
 sh_bios_gdb_detach();
 #endif
 
 that line has no effect. Even if there's no entry for SH_STANDARD_BIOS
 the preprocessor will still remove that call of sh_bios_gdb_detach().
 
+endmenu
 
 Thanks,

OK
It was used few years ago, but it was removed.
Removed this entry.

 
 
 Paul Bolle
 
 --
 To unsubscribe from this list: send the line unsubscribe linux-kernel in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
 Please read the FAQ at  http://www.tux.org/lkml/

-- 
Yoshinori Sato
ys...@users.sourceforge.jp
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-17 Thread Paul Bolle

On Tue, 2015-02-17 at 16:39 +0900, Yoshinori Sato wrote:
> At Sun, 15 Feb 2015 20:23:04 +0100,
> Paul Bolle wrote:
> > On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
> > > diff --git a/arch/h8300/Kconfig.debug b/arch/h8300/Kconfig.debug
> > > new file mode 100644
> > > index 000..eb72b01
> > > --- /dev/null
> > > +++ b/arch/h8300/Kconfig.debug
> > > @@ -0,0 +1,23 @@
> > > +menu "Kernel hacking"
> > > +
> > > +source "lib/Kconfig.debug"
> > > +
> > > +config FULLDEBUG
> > > + bool "Full Symbolic/Source Debugging support"
> > > + help
> > > +   Enable debugging symbols on kernel build.
> > > +
> > > +config HIGHPROFILE
> > > + bool "Use fast second timer for profiling"
> > > + help
> > > +   Use a fast secondary clock to produce profiling information.
> > > +
> > > +config NO_KERNEL_MSG
> > > + bool "Suppress Kernel BUG Messages"
> > > + help
> > > +   Do not output any debug BUG messages within the kernel.
> > > +
> > > +config SH_STANDARD_BIOS
> > > +   def_bool n
> > > +
> > 
> > I don't think this entry is needed.
> 
> This entry using sh-sci. 

This entry will always set SH_STANDARD_BIOS to 'n', and a line reading
# CONFIG_SH_STANDARD_BIOS is not set

will always be included in the generated .config for h8300. But for the
three lines in drivers/tty/serial/sh-sci.c reading
#ifdef CONFIG_SH_STANDARD_BIOS
sh_bios_gdb_detach();
#endif

that line has no effect. Even if there's no entry for SH_STANDARD_BIOS
the preprocessor will still remove that call of sh_bios_gdb_detach().

> > > +endmenu

Thanks,


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-17 Thread Paul Bolle

On Tue, 2015-02-17 at 16:39 +0900, Yoshinori Sato wrote:
 At Sun, 15 Feb 2015 20:23:04 +0100,
 Paul Bolle wrote:
  On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
   diff --git a/arch/h8300/Kconfig.debug b/arch/h8300/Kconfig.debug
   new file mode 100644
   index 000..eb72b01
   --- /dev/null
   +++ b/arch/h8300/Kconfig.debug
   @@ -0,0 +1,23 @@
   +menu Kernel hacking
   +
   +source lib/Kconfig.debug
   +
   +config FULLDEBUG
   + bool Full Symbolic/Source Debugging support
   + help
   +   Enable debugging symbols on kernel build.
   +
   +config HIGHPROFILE
   + bool Use fast second timer for profiling
   + help
   +   Use a fast secondary clock to produce profiling information.
   +
   +config NO_KERNEL_MSG
   + bool Suppress Kernel BUG Messages
   + help
   +   Do not output any debug BUG messages within the kernel.
   +
   +config SH_STANDARD_BIOS
   +   def_bool n
   +
  
  I don't think this entry is needed.
 
 This entry using sh-sci. 

This entry will always set SH_STANDARD_BIOS to 'n', and a line reading
# CONFIG_SH_STANDARD_BIOS is not set

will always be included in the generated .config for h8300. But for the
three lines in drivers/tty/serial/sh-sci.c reading
#ifdef CONFIG_SH_STANDARD_BIOS
sh_bios_gdb_detach();
#endif

that line has no effect. Even if there's no entry for SH_STANDARD_BIOS
the preprocessor will still remove that call of sh_bios_gdb_detach().

   +endmenu

Thanks,


Paul Bolle

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-16 Thread Yoshinori Sato

At Sun, 15 Feb 2015 20:23:04 +0100,
Paul Bolle wrote:
> 
> On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
> > Signed-off-by: Yoshinori Sato 
> > ---
> >  arch/h8300/Kconfig  | 119 +++
> >  arch/h8300/Kconfig.cpu  | 136 
> > 
> >  arch/h8300/Kconfig.debug|  23 +++
> >  arch/h8300/Makefile |  45 +
> >  arch/h8300/kernel/Makefile  |  16 +
> >  arch/h8300/kernel/vmlinux.lds.S |  85 +
> >  6 files changed, 424 insertions(+)
> >  create mode 100644 arch/h8300/Kconfig
> >  create mode 100644 arch/h8300/Kconfig.cpu
> >  create mode 100644 arch/h8300/Kconfig.debug
> >  create mode 100644 arch/h8300/Makefile
> >  create mode 100644 arch/h8300/kernel/Makefile
> >  create mode 100644 arch/h8300/kernel/vmlinux.lds.S
> > 
> > diff --git a/arch/h8300/Kconfig b/arch/h8300/Kconfig
> > new file mode 100644
> > index 000..828c356
> > --- /dev/null
> > +++ b/arch/h8300/Kconfig
> > @@ -0,0 +1,119 @@
> > +config H8300
> > +   bool
> > +   default y
> > +   select HAVE_IDE
> > +   select GENERIC_ATOMIC64
> > +   select HAVE_UID16
> > +   select VIRT_TO_BUS
> > +   select ARCH_WANT_IPC_PARSE_VERSION
> > +   select GENERIC_IRQ_SHOW
> > +   select FRAME_POINTER
> > +   select GENERIC_CPU_DEVICES
> > +   select MODULES_USE_ELF_RELA
> > +   select GENERIC_CLOCKEVENTS
> > +   select CLKDEV_LOOKUP
> > +   select COMMON_CLK
> > +   select HAVE_ARCH_TRACEHOOK
> > +   select ARCH_WANT_FRAME_POINTERS
> > +
> > +config MMU
> > +   bool
> > +   default n
> > +
> > +config SWAP
> > +   bool
> > +   default n
> > +
> 
> MMU will always be 'n'. (So the entry for MMU is not strictly needed,
> but I guess it is clearer to explicitly set this to 'n'.) But that means
> there's no reason here to mask the global SWAP config (see
> init/Kconfig), as that will also always be 'n' for H8300.

OK.
remove it.

> > +config ZONE_DMA
> > +   bool
> > +   default y
> > +
> > +config FPU
> > +   bool
> > +   default n
> 
> Why is this needed?

Inneeded.
Remove it.

> > +
> > +config RWSEM_GENERIC_SPINLOCK
> > +   bool
> > +   default y
> > +
> > +config RWSEM_XCHGADD_ALGORITHM
> > +   bool
> > +   default n
> 
> Not strictly needed.

OK.

> 
> > +config ARCH_HAS_ILOG2_U32
> > +   bool
> > +   default n
> > +
> 
> Ditto.
> 
> > +config ARCH_HAS_ILOG2_U64
> > +   bool
> > +   default n
> > +
> 
> Ditto.
> 
> > +config GENERIC_HWEIGHT
> > +   bool
> > +   default y
> > +
> > +config GENERIC_CALIBRATE_DELAY
> > +   bool
> > +   default y
> > +
> > +config GENERIC_BUG
> > +bool
> > +depends on BUG
> > +
> 
> GENERIC_BUG can never be set for H8300, so this entry is not needed.

OK.
Remove.

> > +config TIME_LOW_RES
> > +   bool
> > +   default y
> > +
> > +config NO_IOPORT_MAP
> > +   def_bool y
> > +
> > +config NO_DMA
> > +   def_bool n
> 
> Not needed (please note that only a few architectures have an entry for
> NO_DMA).

OK.
Remove.

> > +config ISA
> > +def_bool n
> > +
> > +config PCI
> > +   def_bool n
> > +
> 
> Neither ISA nor PCI will ever be set, won't they? So you might as well
> drop these too.
> 
> > +config HZ
> > +   int
> > +   default 100
> > +
> > +config NR_CPUS
> > +int
> 
> Eight spaces instead of one tab.

OK.

> > +   default 1
> > +
> > +source "init/Kconfig"
> > +
> > +source "kernel/Kconfig.freezer"
> > +
> > +source "arch/h8300/Kconfig.cpu"
> > +
> > +menu "Kernel Features"
> > +
> > +source "kernel/Kconfig.preempt"
> > +
> > +source "mm/Kconfig"
> > +
> > +endmenu
> > +
> > +menu "Executable file formats"
> > +
> > +source "fs/Kconfig.binfmt"
> > +
> > +endmenu
> > +
> > +source "net/Kconfig"
> > +
> > +source "drivers/Kconfig"
> > +
> > +source "fs/Kconfig"
> > +
> > +source "arch/h8300/Kconfig.debug"
> > +
> > +source "security/Kconfig"
> > +
> > +source "crypto/Kconfig"
> > +
> > +source "lib/Kconfig"
> > diff --git a/arch/h8300/Kconfig.cpu b/arch/h8300/Kconfig.cpu
> > new file mode 100644
> > index 000..2eaa797
> > --- /dev/null
> > +++ b/arch/h8300/Kconfig.cpu
> > @@ -0,0 +1,136 @@
> > +menu "Processor type and features"
> > +
> > +choice
> > +   prompt "H8/300 platform"
> > +
> > +config H8300_AE3068
> > +   bool "AE-3068/69"
> > +   select H83069
> > +   select RAMKERNEL
> > +   help
> > + AKI-H8/3068F / AKI-H8/3069F Flashmicom LAN Board Support
> > + More Information. (Japanese Only)
> > + 
> > + AE-3068/69 Evaluation Board Support
> > + More Information.
> > + 
> > +
> > +config H8300_H8MAX
> > +   bool "H8MAX"
> > +   select H83069
> > +   select RAMKERNEL
> > +   help
> > + H8MAX Evaluation Board Support
> > + More Information. (Japanese Only)
> > + 
> > +
> > +config H8300_KANEBEBE
> > +   bool "KaneBebe"
> > +   select H83069
> > +   select RAMKERNEL

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-16 Thread Yoshinori Sato

At Sun, 15 Feb 2015 20:23:04 +0100,
Paul Bolle wrote:
 
 On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
  Signed-off-by: Yoshinori Sato ys...@users.sourceforge.jp
  ---
   arch/h8300/Kconfig  | 119 +++
   arch/h8300/Kconfig.cpu  | 136 
  
   arch/h8300/Kconfig.debug|  23 +++
   arch/h8300/Makefile |  45 +
   arch/h8300/kernel/Makefile  |  16 +
   arch/h8300/kernel/vmlinux.lds.S |  85 +
   6 files changed, 424 insertions(+)
   create mode 100644 arch/h8300/Kconfig
   create mode 100644 arch/h8300/Kconfig.cpu
   create mode 100644 arch/h8300/Kconfig.debug
   create mode 100644 arch/h8300/Makefile
   create mode 100644 arch/h8300/kernel/Makefile
   create mode 100644 arch/h8300/kernel/vmlinux.lds.S
  
  diff --git a/arch/h8300/Kconfig b/arch/h8300/Kconfig
  new file mode 100644
  index 000..828c356
  --- /dev/null
  +++ b/arch/h8300/Kconfig
  @@ -0,0 +1,119 @@
  +config H8300
  +   bool
  +   default y
  +   select HAVE_IDE
  +   select GENERIC_ATOMIC64
  +   select HAVE_UID16
  +   select VIRT_TO_BUS
  +   select ARCH_WANT_IPC_PARSE_VERSION
  +   select GENERIC_IRQ_SHOW
  +   select FRAME_POINTER
  +   select GENERIC_CPU_DEVICES
  +   select MODULES_USE_ELF_RELA
  +   select GENERIC_CLOCKEVENTS
  +   select CLKDEV_LOOKUP
  +   select COMMON_CLK
  +   select HAVE_ARCH_TRACEHOOK
  +   select ARCH_WANT_FRAME_POINTERS
  +
  +config MMU
  +   bool
  +   default n
  +
  +config SWAP
  +   bool
  +   default n
  +
 
 MMU will always be 'n'. (So the entry for MMU is not strictly needed,
 but I guess it is clearer to explicitly set this to 'n'.) But that means
 there's no reason here to mask the global SWAP config (see
 init/Kconfig), as that will also always be 'n' for H8300.

OK.
remove it.

  +config ZONE_DMA
  +   bool
  +   default y
  +
  +config FPU
  +   bool
  +   default n
 
 Why is this needed?

Inneeded.
Remove it.

  +
  +config RWSEM_GENERIC_SPINLOCK
  +   bool
  +   default y
  +
  +config RWSEM_XCHGADD_ALGORITHM
  +   bool
  +   default n
 
 Not strictly needed.

OK.

 
  +config ARCH_HAS_ILOG2_U32
  +   bool
  +   default n
  +
 
 Ditto.
 
  +config ARCH_HAS_ILOG2_U64
  +   bool
  +   default n
  +
 
 Ditto.
 
  +config GENERIC_HWEIGHT
  +   bool
  +   default y
  +
  +config GENERIC_CALIBRATE_DELAY
  +   bool
  +   default y
  +
  +config GENERIC_BUG
  +bool
  +depends on BUG
  +
 
 GENERIC_BUG can never be set for H8300, so this entry is not needed.

OK.
Remove.

  +config TIME_LOW_RES
  +   bool
  +   default y
  +
  +config NO_IOPORT_MAP
  +   def_bool y
  +
  +config NO_DMA
  +   def_bool n
 
 Not needed (please note that only a few architectures have an entry for
 NO_DMA).

OK.
Remove.

  +config ISA
  +def_bool n
  +
  +config PCI
  +   def_bool n
  +
 
 Neither ISA nor PCI will ever be set, won't they? So you might as well
 drop these too.
 
  +config HZ
  +   int
  +   default 100
  +
  +config NR_CPUS
  +int
 
 Eight spaces instead of one tab.

OK.

  +   default 1
  +
  +source init/Kconfig
  +
  +source kernel/Kconfig.freezer
  +
  +source arch/h8300/Kconfig.cpu
  +
  +menu Kernel Features
  +
  +source kernel/Kconfig.preempt
  +
  +source mm/Kconfig
  +
  +endmenu
  +
  +menu Executable file formats
  +
  +source fs/Kconfig.binfmt
  +
  +endmenu
  +
  +source net/Kconfig
  +
  +source drivers/Kconfig
  +
  +source fs/Kconfig
  +
  +source arch/h8300/Kconfig.debug
  +
  +source security/Kconfig
  +
  +source crypto/Kconfig
  +
  +source lib/Kconfig
  diff --git a/arch/h8300/Kconfig.cpu b/arch/h8300/Kconfig.cpu
  new file mode 100644
  index 000..2eaa797
  --- /dev/null
  +++ b/arch/h8300/Kconfig.cpu
  @@ -0,0 +1,136 @@
  +menu Processor type and features
  +
  +choice
  +   prompt H8/300 platform
  +
  +config H8300_AE3068
  +   bool AE-3068/69
  +   select H83069
  +   select RAMKERNEL
  +   help
  + AKI-H8/3068F / AKI-H8/3069F Flashmicom LAN Board Support
  + More Information. (Japanese Only)
  + http://akizukidenshi.com/catalog/default.aspx
  + AE-3068/69 Evaluation Board Support
  + More Information.
  + http://www.microtronique.com/ae3069lan.htm
  +
  +config H8300_H8MAX
  +   bool H8MAX
  +   select H83069
  +   select RAMKERNEL
  +   help
  + H8MAX Evaluation Board Support
  + More Information. (Japanese Only)
  + http://strawberry-linux.com/h8/index.html
  +
  +config H8300_KANEBEBE
  +   bool KaneBebe
  +   select H83069
  +   select RAMKERNEL
  +   help
  + KaneBebe Evalition Board Support
  +
  +config H8300H_SIM
  +   bool H8/300H GDB Simulator
  +   select H83069
  +   select ROMKERNEL
  +   help
  + GDB Simulator Support
  + More Information.
  + http://sourceware.org/sid/
  +
  +config H8S_EDOSK2674
  +   bool EDOSK-2674
  +   select H8S2678
  +   select RAMKERNEL
  +   help
  + Renesas EDOSK-2674 Evaluation

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-15 Thread Paul Bolle

On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
> Signed-off-by: Yoshinori Sato 
> ---
>  arch/h8300/Kconfig  | 119 +++
>  arch/h8300/Kconfig.cpu  | 136 
> 
>  arch/h8300/Kconfig.debug|  23 +++
>  arch/h8300/Makefile |  45 +
>  arch/h8300/kernel/Makefile  |  16 +
>  arch/h8300/kernel/vmlinux.lds.S |  85 +
>  6 files changed, 424 insertions(+)
>  create mode 100644 arch/h8300/Kconfig
>  create mode 100644 arch/h8300/Kconfig.cpu
>  create mode 100644 arch/h8300/Kconfig.debug
>  create mode 100644 arch/h8300/Makefile
>  create mode 100644 arch/h8300/kernel/Makefile
>  create mode 100644 arch/h8300/kernel/vmlinux.lds.S
> 
> diff --git a/arch/h8300/Kconfig b/arch/h8300/Kconfig
> new file mode 100644
> index 000..828c356
> --- /dev/null
> +++ b/arch/h8300/Kconfig
> @@ -0,0 +1,119 @@
> +config H8300
> + bool
> + default y
> + select HAVE_IDE
> + select GENERIC_ATOMIC64
> + select HAVE_UID16
> + select VIRT_TO_BUS
> + select ARCH_WANT_IPC_PARSE_VERSION
> + select GENERIC_IRQ_SHOW
> + select FRAME_POINTER
> + select GENERIC_CPU_DEVICES
> + select MODULES_USE_ELF_RELA
> + select GENERIC_CLOCKEVENTS
> + select CLKDEV_LOOKUP
> + select COMMON_CLK
> + select HAVE_ARCH_TRACEHOOK
> + select ARCH_WANT_FRAME_POINTERS
> +
> +config MMU
> + bool
> + default n
> +
> +config SWAP
> + bool
> + default n
> +

MMU will always be 'n'. (So the entry for MMU is not strictly needed,
but I guess it is clearer to explicitly set this to 'n'.) But that means
there's no reason here to mask the global SWAP config (see
init/Kconfig), as that will also always be 'n' for H8300.

> +config ZONE_DMA
> + bool
> + default y
> +
> +config FPU
> + bool
> + default n

Why is this needed?

> +
> +config RWSEM_GENERIC_SPINLOCK
> + bool
> + default y
> +
> +config RWSEM_XCHGADD_ALGORITHM
> + bool
> + default n

Not strictly needed.

> +config ARCH_HAS_ILOG2_U32
> + bool
> + default n
> +

Ditto.

> +config ARCH_HAS_ILOG2_U64
> + bool
> + default n
> +

Ditto.

> +config GENERIC_HWEIGHT
> + bool
> + default y
> +
> +config GENERIC_CALIBRATE_DELAY
> + bool
> + default y
> +
> +config GENERIC_BUG
> +bool
> +depends on BUG
> +

GENERIC_BUG can never be set for H8300, so this entry is not needed.

> +config TIME_LOW_RES
> + bool
> + default y
> +
> +config NO_IOPORT_MAP
> + def_bool y
> +
> +config NO_DMA
> + def_bool n

Not needed (please note that only a few architectures have an entry for
NO_DMA).

> +config ISA
> +def_bool n
> +
> +config PCI
> + def_bool n
> +

Neither ISA nor PCI will ever be set, won't they? So you might as well
drop these too.

> +config HZ
> + int
> + default 100
> +
> +config NR_CPUS
> +int

Eight spaces instead of one tab.

> + default 1
> +
> +source "init/Kconfig"
> +
> +source "kernel/Kconfig.freezer"
> +
> +source "arch/h8300/Kconfig.cpu"
> +
> +menu "Kernel Features"
> +
> +source "kernel/Kconfig.preempt"
> +
> +source "mm/Kconfig"
> +
> +endmenu
> +
> +menu "Executable file formats"
> +
> +source "fs/Kconfig.binfmt"
> +
> +endmenu
> +
> +source "net/Kconfig"
> +
> +source "drivers/Kconfig"
> +
> +source "fs/Kconfig"
> +
> +source "arch/h8300/Kconfig.debug"
> +
> +source "security/Kconfig"
> +
> +source "crypto/Kconfig"
> +
> +source "lib/Kconfig"
> diff --git a/arch/h8300/Kconfig.cpu b/arch/h8300/Kconfig.cpu
> new file mode 100644
> index 000..2eaa797
> --- /dev/null
> +++ b/arch/h8300/Kconfig.cpu
> @@ -0,0 +1,136 @@
> +menu "Processor type and features"
> +
> +choice
> + prompt "H8/300 platform"
> +
> +config H8300_AE3068
> + bool "AE-3068/69"
> + select H83069
> + select RAMKERNEL
> + help
> +   AKI-H8/3068F / AKI-H8/3069F Flashmicom LAN Board Support
> +   More Information. (Japanese Only)
> +   
> +   AE-3068/69 Evaluation Board Support
> +   More Information.
> +   
> +
> +config H8300_H8MAX
> + bool "H8MAX"
> + select H83069
> + select RAMKERNEL
> + help
> +   H8MAX Evaluation Board Support
> +   More Information. (Japanese Only)
> +   
> +
> +config H8300_KANEBEBE
> + bool "KaneBebe"
> + select H83069
> + select RAMKERNEL
> + help
> +   KaneBebe Evalition Board Support
> +
> +config H8300H_SIM
> + bool "H8/300H GDB Simulator"
> + select H83069
> + select ROMKERNEL
> + help
> +   GDB Simulator Support
> +   More Information.
> +   
> +
> +config H8S_EDOSK2674
> + bool "EDOSK-2674"
> + select H8S2678
> + select RAMKERNEL
> + help
> +   Renesas

Re: [PATCH v3 10/15] h8300: Build scripts

2015-02-15 Thread Paul Bolle

On Sun, 2015-02-15 at 16:49 +0900, Yoshinori Sato wrote:
 Signed-off-by: Yoshinori Sato ys...@users.sourceforge.jp
 ---
  arch/h8300/Kconfig  | 119 +++
  arch/h8300/Kconfig.cpu  | 136 
 
  arch/h8300/Kconfig.debug|  23 +++
  arch/h8300/Makefile |  45 +
  arch/h8300/kernel/Makefile  |  16 +
  arch/h8300/kernel/vmlinux.lds.S |  85 +
  6 files changed, 424 insertions(+)
  create mode 100644 arch/h8300/Kconfig
  create mode 100644 arch/h8300/Kconfig.cpu
  create mode 100644 arch/h8300/Kconfig.debug
  create mode 100644 arch/h8300/Makefile
  create mode 100644 arch/h8300/kernel/Makefile
  create mode 100644 arch/h8300/kernel/vmlinux.lds.S
 
 diff --git a/arch/h8300/Kconfig b/arch/h8300/Kconfig
 new file mode 100644
 index 000..828c356
 --- /dev/null
 +++ b/arch/h8300/Kconfig
 @@ -0,0 +1,119 @@
 +config H8300
 + bool
 + default y
 + select HAVE_IDE
 + select GENERIC_ATOMIC64
 + select HAVE_UID16
 + select VIRT_TO_BUS
 + select ARCH_WANT_IPC_PARSE_VERSION
 + select GENERIC_IRQ_SHOW
 + select FRAME_POINTER
 + select GENERIC_CPU_DEVICES
 + select MODULES_USE_ELF_RELA
 + select GENERIC_CLOCKEVENTS
 + select CLKDEV_LOOKUP
 + select COMMON_CLK
 + select HAVE_ARCH_TRACEHOOK
 + select ARCH_WANT_FRAME_POINTERS
 +
 +config MMU
 + bool
 + default n
 +
 +config SWAP
 + bool
 + default n
 +

MMU will always be 'n'. (So the entry for MMU is not strictly needed,
but I guess it is clearer to explicitly set this to 'n'.) But that means
there's no reason here to mask the global SWAP config (see
init/Kconfig), as that will also always be 'n' for H8300.

 +config ZONE_DMA
 + bool
 + default y
 +
 +config FPU
 + bool
 + default n

Why is this needed?

 +
 +config RWSEM_GENERIC_SPINLOCK
 + bool
 + default y
 +
 +config RWSEM_XCHGADD_ALGORITHM
 + bool
 + default n

Not strictly needed.

 +config ARCH_HAS_ILOG2_U32
 + bool
 + default n
 +

Ditto.

 +config ARCH_HAS_ILOG2_U64
 + bool
 + default n
 +

Ditto.

 +config GENERIC_HWEIGHT
 + bool
 + default y
 +
 +config GENERIC_CALIBRATE_DELAY
 + bool
 + default y
 +
 +config GENERIC_BUG
 +bool
 +depends on BUG
 +

GENERIC_BUG can never be set for H8300, so this entry is not needed.

 +config TIME_LOW_RES
 + bool
 + default y
 +
 +config NO_IOPORT_MAP
 + def_bool y
 +
 +config NO_DMA
 + def_bool n

Not needed (please note that only a few architectures have an entry for
NO_DMA).

 +config ISA
 +def_bool n
 +
 +config PCI
 + def_bool n
 +

Neither ISA nor PCI will ever be set, won't they? So you might as well
drop these too.

 +config HZ
 + int
 + default 100
 +
 +config NR_CPUS
 +int

Eight spaces instead of one tab.

 + default 1
 +
 +source init/Kconfig
 +
 +source kernel/Kconfig.freezer
 +
 +source arch/h8300/Kconfig.cpu
 +
 +menu Kernel Features
 +
 +source kernel/Kconfig.preempt
 +
 +source mm/Kconfig
 +
 +endmenu
 +
 +menu Executable file formats
 +
 +source fs/Kconfig.binfmt
 +
 +endmenu
 +
 +source net/Kconfig
 +
 +source drivers/Kconfig
 +
 +source fs/Kconfig
 +
 +source arch/h8300/Kconfig.debug
 +
 +source security/Kconfig
 +
 +source crypto/Kconfig
 +
 +source lib/Kconfig
 diff --git a/arch/h8300/Kconfig.cpu b/arch/h8300/Kconfig.cpu
 new file mode 100644
 index 000..2eaa797
 --- /dev/null
 +++ b/arch/h8300/Kconfig.cpu
 @@ -0,0 +1,136 @@
 +menu Processor type and features
 +
 +choice
 + prompt H8/300 platform
 +
 +config H8300_AE3068
 + bool AE-3068/69
 + select H83069
 + select RAMKERNEL
 + help
 +   AKI-H8/3068F / AKI-H8/3069F Flashmicom LAN Board Support
 +   More Information. (Japanese Only)
 +   http://akizukidenshi.com/catalog/default.aspx
 +   AE-3068/69 Evaluation Board Support
 +   More Information.
 +   http://www.microtronique.com/ae3069lan.htm
 +
 +config H8300_H8MAX
 + bool H8MAX
 + select H83069
 + select RAMKERNEL
 + help
 +   H8MAX Evaluation Board Support
 +   More Information. (Japanese Only)
 +   http://strawberry-linux.com/h8/index.html
 +
 +config H8300_KANEBEBE
 + bool KaneBebe
 + select H83069
 + select RAMKERNEL
 + help
 +   KaneBebe Evalition Board Support
 +
 +config H8300H_SIM
 + bool H8/300H GDB Simulator
 + select H83069
 + select ROMKERNEL
 + help
 +   GDB Simulator Support
 +   More Information.
 +   http://sourceware.org/sid/
 +
 +config H8S_EDOSK2674
 + bool EDOSK-2674
 + select H8S2678
 + select RAMKERNEL
 + help
 +   Renesas EDOSK-2674 Evaluation Board Support
 +   More Information.
 +   http://www.azpower.com/H8-uClinux/index.html
 +

Re: [PATCH v3 10/15] h8300: Build scripts

Re: [PATCH v3 10/15] h8300: Build scripts

Re: [PATCH v3 10/15] h8300: Build scripts

Re: [PATCH v3 10/15] h8300: Build scripts

Re: [PATCH v3 10/15] h8300: Build scripts

Re: [PATCH v3 10/15] h8300: Build scripts

Re: [PATCH v3 10/15] h8300: Build scripts

Re: [PATCH v3 10/15] h8300: Build scripts

8 matches

Site Navigation

Mail list logo

Footer information