Re: [patch] update CFI URI n mtd kconfig

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu 13 Sep 2007 14:28, Josh Boyer pondered:
> On 9/12/07, Mike Frysinger <[email protected]> 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 <[email protected]>
> > ---
> > 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
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux