From: Wolfram Sang <w.sang@pengutronix.de>
To: Lars Poeschel <poeschel@lemonage.de>
Cc: gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org,
linux-i2c@vger.kernel.org
Subject: Re: [PATCH RFC] misc/at24: distinguish between eeprom and fram chips
Date: Thu, 24 Jan 2013 08:27:01 +0100 [thread overview]
Message-ID: <20130124072701.GG8364@nekote.pengutronix.de> (raw)
In-Reply-To: <201212071114.29034.poeschel@lemonage.de>
On Fri, Dec 07, 2012 at 11:14:28AM +0100, Lars Poeschel wrote:
> > > > > I wanted to use a fm24c04 i2c fram chip with linux. I grepped the
> > > > > source and found nothing. I later found that my chip can be handled
> > > > > by at24 eeprom driver. It creates a sysfs file called eeprom to
> > > > > read from and write to the chip. Userspace has no chance to
> > > > > distinguish if it is writing an eeprom or a fram chip.
> > > >
> > > > Why should it?
> > >
> > > Because writes are much faster and it doesn't have to take care on erase
> > > cycles. It could use other write strategies on such devices and update
> > > informations that have to survive power downs more often.
> >
> > I agree. I think that a seperate attribute named e.g. 'page_size' would
> > be more helpful than renaming the binary file to fram?
>
> Yes, this is a much better solution! Adding a seperate sysfs file page_size
> and a file for the type of device which would read eeprom, fram, etc then.
> If you also think this is the way to go, I would spent one of my next free
> timeslots to this.
Oops, this mail seems to have dropped off :(
I am all for the 'page_size' attribute, but still not convinced what
gain the 'type' attribute would allow. For FRAM, the page size will be
large. Isn't this enough information?
Regards,
Wolfram
next prev parent reply other threads:[~2013-01-24 7:27 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-04 16:58 [PATCH RFC] misc/at24: distinguish between eeprom and fram chips Lars Poeschel
2012-12-04 18:32 ` Wolfram Sang
2012-12-05 9:43 ` Lars Poeschel
2012-12-05 16:41 ` Wolfram Sang
2012-12-07 10:14 ` Lars Poeschel
2013-01-24 7:27 ` Wolfram Sang [this message]
2013-01-28 10:40 ` Lars Poeschel
2013-02-06 18:19 ` Lars Poeschel
2013-02-10 16:30 ` Wolfram Sang
2013-02-11 8:51 ` Lars Poeschel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130124072701.GG8364@nekote.pengutronix.de \
--to=w.sang@pengutronix.de \
--cc=gregkh@linuxfoundation.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=poeschel@lemonage.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).