public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
* [patch] update CFI URI n mtd kconfig
@ 2007-09-13  3:39 Mike Frysinger
  2007-09-13 18:28 ` Josh Boyer
  0 siblings, 1 reply; 6+ messages in thread
From: Mike Frysinger @ 2007-09-13  3:39 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linux-mtd, LKML

Since AMD shunted its flash memory division, the URI in the mtd Kconfig is now
broken, so the attached patch points people to Wikipedia.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
diff --git a/drivers/mtd/chips/Kconfig b/drivers/mtd/chips/Kconfig
index 479d32b..980117a 100644
--- a/drivers/mtd/chips/Kconfig
+++ b/drivers/mtd/chips/Kconfig
@@ -11,8 +11,8 @@ config MTD_CFI
 	  AMD and other flash manufactures that provides a universal method
 	  for probing the capabilities of flash devices. If you wish to
 	  support any device that is CFI-compliant, you need to enable this
-	  option. Visit <http://www.amd.com/products/nvd/overview/cfi.html>
-	  for more information on CFI.
+	  option. For more information on CFI, visit:
+	  <http://wikipedia.org/wiki/Common_Flash_Memory_Interface>
 
 config MTD_JEDECPROBE
 	tristate "Detect non-CFI AMD/JEDEC-compatible flash chips"

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [patch] update CFI URI n mtd kconfig
  2007-09-13  3:39 [patch] update CFI URI n mtd kconfig Mike Frysinger
@ 2007-09-13 18:28 ` Josh Boyer
  2007-09-13 20:31   ` Robin Getz
  0 siblings, 1 reply; 6+ messages in thread
From: Josh Boyer @ 2007-09-13 18:28 UTC (permalink / raw)
  To: Mike Frysinger; +Cc: Andrew Morton, linux-mtd, LKML

On 9/12/07, Mike Frysinger <vapier@gentoo.org> wrote:
> Since AMD shunted its flash memory division, the URI in the mtd Kconfig is now
> broken, so the attached patch points people to Wikipedia.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---
> diff --git a/drivers/mtd/chips/Kconfig b/drivers/mtd/chips/Kconfig
> index 479d32b..980117a 100644
> --- a/drivers/mtd/chips/Kconfig
> +++ b/drivers/mtd/chips/Kconfig
> @@ -11,8 +11,8 @@ config MTD_CFI
>           AMD and other flash manufactures that provides a universal method
>           for probing the capabilities of flash devices. If you wish to
>           support any device that is CFI-compliant, you need to enable this
> -         option. Visit <http://www.amd.com/products/nvd/overview/cfi.html>
> -         for more information on CFI.
> +         option. For more information on CFI, visit:
> +         <http://wikipedia.org/wiki/Common_Flash_Memory_Interface>

Erm, I'd rather not point to wikipedia.  Can you change it to point to:

http://www.intel.com/design/flash/swb/cfi.htm

instead?

josh

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [patch] update CFI URI n mtd kconfig
  2007-09-13 18:28 ` Josh Boyer
@ 2007-09-13 20:31   ` Robin Getz
  2007-09-13 21:01     ` Mike Frysinger
  2007-09-13 23:09     ` Josh Boyer
  0 siblings, 2 replies; 6+ messages in thread
From: Robin Getz @ 2007-09-13 20:31 UTC (permalink / raw)
  To: Josh Boyer; +Cc: linux-mtd, Andrew Morton, Mike Frysinger, LKML

On Thu 13 Sep 2007 14:28, Josh Boyer pondered:
> On 9/12/07, Mike Frysinger <vapier@gentoo.org> wrote:
> > Since AMD shunted its flash memory division, the URI in the mtd Kconfig is now
> > broken, so the attached patch points people to Wikipedia.
> >
> > Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> > ---
> > diff --git a/drivers/mtd/chips/Kconfig b/drivers/mtd/chips/Kconfig
> > index 479d32b..980117a 100644
> > --- a/drivers/mtd/chips/Kconfig
> > +++ b/drivers/mtd/chips/Kconfig
> > @@ -11,8 +11,8 @@ config MTD_CFI
> >           AMD and other flash manufactures that provides a universal method

Even on the Intel page, it gives "Intel, AMD, Sharp, and Fujitsu" credit - why
limit Sharp, and Fujitsu to "others"? (I would just take out everyone's name).

> >           for probing the capabilities of flash devices. If you wish to
> >           support any device that is CFI-compliant, you need to enable this
> > -         option. Visit <http://www.amd.com/products/nvd/overview/cfi.html>
> > -         for more information on CFI.
> > +         option. For more information on CFI, visit:
> > +         <http://wikipedia.org/wiki/Common_Flash_Memory_Interface>
> 
> Erm, I'd rather not point to wikipedia.  Can you change it to point to:
> 
> http://www.intel.com/design/flash/swb/cfi.htm

Why not from the place that you can get the standard? http://jedec.org/

or http://www.jedec.org/download/search/jesd68-01.pdf 

There isn't any point to pick one vendor - unless you work there, own
stock, or one provides outstanding support (free hardware) for driver 
development. :)

-Robin

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [patch] update CFI URI n mtd kconfig
  2007-09-13 20:31   ` Robin Getz
@ 2007-09-13 21:01     ` Mike Frysinger
  2007-09-14  9:07       ` Clemens Koller
  2007-09-13 23:09     ` Josh Boyer
  1 sibling, 1 reply; 6+ messages in thread
From: Mike Frysinger @ 2007-09-13 21:01 UTC (permalink / raw)
  To: Robin Getz; +Cc: Josh Boyer, linux-mtd, Andrew Morton, LKML

[-- Attachment #1: Type: text/plain, Size: 260 bytes --]

On Thursday 13 September 2007, Robin Getz wrote:
> Why not from the place that you can get the standard? http://jedec.org/

i found jedec.org a nightmare to navigate

> or http://www.jedec.org/download/search/jesd68-01.pdf

i'll add that to wikipedia ;)
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 827 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [patch] update CFI URI n mtd kconfig
  2007-09-13 20:31   ` Robin Getz
  2007-09-13 21:01     ` Mike Frysinger
@ 2007-09-13 23:09     ` Josh Boyer
  1 sibling, 0 replies; 6+ messages in thread
From: Josh Boyer @ 2007-09-13 23:09 UTC (permalink / raw)
  To: Robin Getz; +Cc: linux-mtd, Andrew Morton, Mike Frysinger, LKML

On 9/13/07, Robin Getz <rgetz@blackfin.uclinux.org> wrote:
> On Thu 13 Sep 2007 14:28, Josh Boyer pondered:
> > On 9/12/07, Mike Frysinger <vapier@gentoo.org> wrote:
> > > Since AMD shunted its flash memory division, the URI in the mtd Kconfig is now
> > > broken, so the attached patch points people to Wikipedia.
> > >
> > > Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> > > ---
> > > diff --git a/drivers/mtd/chips/Kconfig b/drivers/mtd/chips/Kconfig
> > > index 479d32b..980117a 100644
> > > --- a/drivers/mtd/chips/Kconfig
> > > +++ b/drivers/mtd/chips/Kconfig
> > > @@ -11,8 +11,8 @@ config MTD_CFI
> > >           AMD and other flash manufactures that provides a universal method
>
> Even on the Intel page, it gives "Intel, AMD, Sharp, and Fujitsu" credit - why
> limit Sharp, and Fujitsu to "others"? (I would just take out everyone's name).

He wasn't changing that part of the option.  I'm fairly certain that
description has been there for years.  If you want to change it to be
a bit more "fair", feel free.

> > >           for probing the capabilities of flash devices. If you wish to
> > >           support any device that is CFI-compliant, you need to enable this
> > > -         option. Visit <http://www.amd.com/products/nvd/overview/cfi.html>
> > > -         for more information on CFI.
> > > +         option. For more information on CFI, visit:
> > > +         <http://wikipedia.org/wiki/Common_Flash_Memory_Interface>
> >
> > Erm, I'd rather not point to wikipedia.  Can you change it to point to:
> >
> > http://www.intel.com/design/flash/swb/cfi.htm
>
> Why not from the place that you can get the standard? http://jedec.org/
>
> or http://www.jedec.org/download/search/jesd68-01.pdf
>
> There isn't any point to pick one vendor - unless you work there, own
> stock, or one provides outstanding support (free hardware) for driver
> development. :)

I wasn't picking a vendor.  I was pointing out a well laid out webpage
that describes what CFI is and provides the spec for it.  If you want
to link directly to jedec, ok.

And no, I don't work at Intel, own stock in it, or know of them giving
out free hardware on a regular basis.  I do happen to like what Intel
is doing in regards to opening their drivers, but that has nothing to
do with this :).

josh

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [patch] update CFI URI n mtd kconfig
  2007-09-13 21:01     ` Mike Frysinger
@ 2007-09-14  9:07       ` Clemens Koller
  0 siblings, 0 replies; 6+ messages in thread
From: Clemens Koller @ 2007-09-14  9:07 UTC (permalink / raw)
  To: Mike Frysinger; +Cc: Josh Boyer, linux-mtd, Andrew Morton, Robin Getz, LKML

Mike Frysinger schrieb:
> On Thursday 13 September 2007, Robin Getz wrote:
>> Why not from the place that you can get the standard? http://jedec.org/
> 
> i found jedec.org a nightmare to navigate
> 
>> or http://www.jedec.org/download/search/jesd68-01.pdf
> 
> i'll add that to wikipedia ;)
> -mike

...done.

Regards,

Clemens Koller
__________________________________
R&D Imaging Devices
Anagramm GmbH
Rupert-Mayer-Straße 45/1
Linhof Werksgelände
D-81379 München
Tel.089-741518-50
Fax 089-741518-19
http://www.anagramm-technology.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2007-09-14  9:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-09-13  3:39 [patch] update CFI URI n mtd kconfig Mike Frysinger
2007-09-13 18:28 ` Josh Boyer
2007-09-13 20:31   ` Robin Getz
2007-09-13 21:01     ` Mike Frysinger
2007-09-14  9:07       ` Clemens Koller
2007-09-13 23:09     ` Josh Boyer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox