Re: [PATCH] Fix the meaning of EDAC abbreviation

Re: [PATCH] Fix the meaning of EDAC abbreviation

[Doug Thompson]

--- On Wed, 6/3/09, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:

> From: Alan Cox <alan@lxorguk.ukuu.org.uk>
> Subject: Re: [PATCH] Fix the meaning of EDAC abbreviation
> To: leemgs1@gmail.com
> Cc: "Jiri Kosina" <trivial@kernel.org>, "linux-kernel" <linux-kernel@vger.kernel.org>, dougthompson@xmission.com
> Date: Wednesday, June 3, 2009, 1:42 AM
> > Fix the meaning of EDAC(Error
> Detection And Correction) correctly.
> >  - http://bluesmoke.sourceforge.net
> > 
> >         Signed-off-by:
> GeunSik Lim <geunsik.lim@samsung.com>
> > ---
> >  drivers/edac/Kconfig |    2 +-
> >  1 files changed, 1 insertions(+), 1
> deletions(-)
> > 
> > diff --git a/drivers/edac/Kconfig
> b/drivers/edac/Kconfig
> > index 956982f..2a81915 100644
> > --- a/drivers/edac/Kconfig
> > +++ b/drivers/edac/Kconfig
> > @@ -5,7 +5,7 @@
> >  #
> >  
> >  menuconfig EDAC
> > -    bool "EDAC - error detection and
> reporting"
> > +    bool "EDAC - Error Detection And
> Correction"
> 
> The code started off as one thing and acquired its name but
> a primary
> function of it now is indeed reporting rather than just
> correction. The
> name has stuck but the functionality doesn't quite match -
> so the
> existing menuconfig entry is actually more helpful.
> 

Alan has been "connected" with "bluesmoke"/EDAC far longer than I have and probably a better overall history of it.

Yet the EDAC modules harvest the events generated by the Error Detection and Correction hardware and present it, via reporting and he points out. But in spite of just being the "reporters", the modules manage the "detectors" in a way and deal with most of the "EDAC" system

I like the new patch mod

doug t

Re: [PATCH] Fix the meaning of EDAC abbreviation

[Doug Thompson]

--- On Tue, 6/2/09, GeunSik Lim <leemgs1@gmail.com> wrote:

> From: GeunSik Lim <leemgs1@gmail.com>
> Subject: [PATCH] Fix the meaning of EDAC abbreviation
> To: "Jiri Kosina" <trivial@kernel.org>
> Cc: "linux-kernel" <linux-kernel@vger.kernel.org>, dougthompson@xmission.com
> Date: Tuesday, June 2, 2009, 6:04 PM
> 
> 
> Dear Jiri Kosina,
> 
> This is trivial patch about meaning of EDAC in the
> driver/edac/Kconfig file.
>  * Screenshot after patch :
>    http://blogfiles16.naver.net/data42/2009/6/3/31/edac-before-after-invain.png
> 
> 
> Thanks, 
> 
> 
> Subject: [PATCH] Fix the meaning of EDAC abbreviation
> 
> Fix the meaning of EDAC(Error Detection And Correction)
> correctly.
>  - http://bluesmoke.sourceforge.net
> 
>         Signed-off-by: GeunSik Lim
> <geunsik.lim@samsung.com>
> ---
>  drivers/edac/Kconfig |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
> index 956982f..2a81915 100644
> --- a/drivers/edac/Kconfig
> +++ b/drivers/edac/Kconfig
> @@ -5,7 +5,7 @@
>  #
>  
>  menuconfig EDAC
> -    bool "EDAC - error detection and
> reporting"
> +    bool "EDAC - Error Detection And
> Correction"
>      depends on HAS_IOMEM
>      depends on X86 || PPC
>      help
> -- 
> 1.6.3.1


Good catch. All these years and I didn't see it. Always need more eyes


Acked-by: Doug Thompson <dougthompson@xmission.com>

[PATCH] Fix the meaning of EDAC abbreviation

[GeunSik Lim]

Dear Jiri Kosina,

This is trivial patch about meaning of EDAC in the driver/edac/Kconfig file.
 * Screenshot after patch :
   http://blogfiles16.naver.net/data42/2009/6/3/31/edac-before-after-invain.png


Thanks, 


Subject: [PATCH] Fix the meaning of EDAC abbreviation

Fix the meaning of EDAC(Error Detection And Correction) correctly.
 - http://bluesmoke.sourceforge.net

        Signed-off-by: GeunSik Lim <geunsik.lim@samsung.com>
---
 drivers/edac/Kconfig |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
index 956982f..2a81915 100644
--- a/drivers/edac/Kconfig
+++ b/drivers/edac/Kconfig
@@ -5,7 +5,7 @@
 #
 
 menuconfig EDAC
-	bool "EDAC - error detection and reporting"
+	bool "EDAC - Error Detection And Correction"
 	depends on HAS_IOMEM
 	depends on X86 || PPC
 	help
-- 
1.6.3.1



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

GeunSik Lim ( Samsung Electronics )
e-Mail  :1) geunsik.lim@samsung.com
         2) leemgs@gmail.com , leemgs1@gmail.com
HomePage: http://blog.naver.com/invain/
-----------------------------------------------

Re: [PATCH] Fix the meaning of EDAC abbreviation

[Alan Cox]

> Fix the meaning of EDAC(Error Detection And Correction) correctly.
>  - http://bluesmoke.sourceforge.net
> 
>         Signed-off-by: GeunSik Lim <geunsik.lim@samsung.com>
> ---
>  drivers/edac/Kconfig |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
> index 956982f..2a81915 100644
> --- a/drivers/edac/Kconfig
> +++ b/drivers/edac/Kconfig
> @@ -5,7 +5,7 @@
>  #
>  
>  menuconfig EDAC
> -	bool "EDAC - error detection and reporting"
> +	bool "EDAC - Error Detection And Correction"

The code started off as one thing and acquired its name but a primary
function of it now is indeed reporting rather than just correction. The
name has stuck but the functionality doesn't quite match - so the
existing menuconfig entry is actually more helpful.

Re: [PATCH] Fix the meaning of EDAC abbreviation

[GeunSik Lim]

On Wed, Jun 3, 2009 at 4:42 PM, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
>> Fix the meaning of EDAC(Error Detection And Correction) correctly.
>>  - http://bluesmoke.sourceforge.net
>>
>>         Signed-off-by: GeunSik Lim <geunsik.lim@samsung.com>
>> ---
>>  drivers/edac/Kconfig |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
>> index 956982f..2a81915 100644
>> --- a/drivers/edac/Kconfig
>> +++ b/drivers/edac/Kconfig
>> @@ -5,7 +5,7 @@
>>  #
>>
>>  menuconfig EDAC
>> -     bool "EDAC - error detection and reporting"
>> +     bool "EDAC - Error Detection And Correction"
>
> The code started off as one thing and acquired its name but a primary
> function of it now is indeed reporting rather than just correction. The
Thank you for explanation and advice about EDAC meaning.

> name has stuck but the functionality doesn't quite match - so the
> existing menuconfig entry is actually more helpful.
Um.
How about  we change contents with "EDAC (Error Detection And
Correction) reporting"  to understand EDAC abbreviation normally?
This is Just my thinking.

Thks,


-- 
Regards,
GeunSik Lim ( Samsung Electronics )
Blog : http://blog.naver.com/invain/
e-Mail: geunsik.lim@samsung.com
           leemgs@gmail.com , leemgs1@gmail.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/