Re: [PATCH 1/1] edac x38: new MC driver module

Hitoshi Mitake
2009-02-22T14:20:43Z

On Sat, 21 Feb 2009 14:09:37 +0100
Sam Ravnborg <sam@ravnborg.org> wrote:

> > Hi,
> >
> > Very sorry for long distance between my previous post and this...
> >
> > I wrote a patch to add HAVE_READQ and HAVE_WRITEQ to each architecture's Kconfig file
> > which have readq() and writeq().
> >
> > But there is problem.
> > I wrote helps for HAVE_READQ and HAVE_WRITEQ in Kconfig file
> > accodring to the advice by Russell King ( http://marc.info/?l=linux-kernel&m=122701161824218&w=2 ),
> > but these helps are invisible when I doing menuconfig.
> > (when type '/' and search readq string, HAVE_READQ found, but
> > help string is not printed...)
> >
> > Do you have some nice technique that make these helps visible easily?
>
> The options are not visible in menuconfig and therefore the is not
> much point in displaying help for them thre when you search for the symbol.
> But the help contained in the KConfig file is fully justified as it is
> now documented why/when to select these options.
>
> Sam

description of the patch: Adding HAVE_READQ, HAVE_WRITEQ and their help texts
to each architecture's Kconfig file which have readq() and writeq().

Signed-off-by: Hitoshi Mitake <h.mitake@gmail.com>

---
arch/sh/Kconfig | 18 ++++++++++++++++++
1 files changed, 18 insertions(+), 0 deletions(-)

diff --git a/arch/sh/Kconfig b/arch/sh/Kconfig
index ebabe51..7b1f664 100644
--- a/arch/sh/Kconfig
+++ b/arch/sh/Kconfig
@@ -699,3 +699,21 @@ source "security/Kconfig"
source "crypto/Kconfig"

source "lib/Kconfig"
+
+config HAVE_READQ
+ def_bool y
+ help
+ This is a sign to represent that this architecture provides
+ readq() function. readq() is a function to read 8 bytes from
+ I/O space. Each drivers use readq() must depend on this symbol.
+ Because lots of little private readq() implementations
+ all over the tree is sucks.
+
+config HAVE_WRITEQ
+ def_bool y
+ help
+ This is a sign to represent that this architecture provides
+ writeq() function. writeq() is a function to read 8 bytes from
+ I/O space. Each drivers use writeq() must depend on this symbol.
+ Because lots of little private writeq() implementations
+ all over the tree is sucks.
--
1.6.1.2