From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
To: Andrew Lunn <andrew@lunn.ch>
Cc: GregKH <greg@kroah.com>,
maxime.ripard@free-electrons.com, wsa@the-dreams.de,
broonie@kernel.org, vz@mleia.com, afd@ti.com,
linux-kernel@vger.kernel.org, agust@denx.de
Subject: Re: [PATCH 6/6] eeprom: 93xx46: extend driver to plug into the NVMEM framework
Date: Tue, 15 Dec 2015 10:26:47 +0000 [thread overview]
Message-ID: <566FEAE7.10109@linaro.org> (raw)
In-Reply-To: <20151215101715.GC17792@lunn.ch>
+ adding Anatolij
On 15/12/15 10:17, Andrew Lunn wrote:
>> Is there a reason for this driver to be using the old style?
>> I can understand the issues with at24/at25 but does this driver also
>> suffer from such issues?
>
> In order to keep backwards compatibility, we need the older file in
> /sys. The only other option is to remove it and see if anybody
> complains about us breaking the ABI.
We should atleast attempt to pitch in this direction, and ask if
somebody really cares if the location of the eeprom/nvmem file matters
to them?
We should probably check with Anatolij Gustschin.
Anatolij, Do you see any issues if we totally move this driver to nvmem
framework? Which involves relocating and renameing the old eeprom file
to /sys/bus/nvmem/devices/*/nvmem
--srini
>
> Andrew
>
next prev parent reply other threads:[~2015-12-15 10:27 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-08 14:05 [PATCH 0/6] Convert existing EEPROM drivers to NVMEM Andrew Lunn
2015-12-08 14:05 ` [PATCH 1/6] nvmem: Add flag to export NVMEM to root only Andrew Lunn
2015-12-15 10:02 ` Srinivas Kandagatla
2015-12-08 14:05 ` [PATCH 2/6] nvmem: Add backwards compatibility support for older EEPROM drivers Andrew Lunn
2015-12-11 13:03 ` Wolfram Sang
2015-12-11 13:43 ` Andrew Lunn
2015-12-12 11:04 ` Wolfram Sang
2015-12-15 10:04 ` Srinivas Kandagatla
2015-12-15 10:04 ` Srinivas Kandagatla
2015-12-08 14:05 ` [PATCH 3/6] eeprom: at24: extend driver to plug into the NVMEM framework Andrew Lunn
2015-12-08 14:05 ` [PATCH 4/6] eeprom: at25: Remove in kernel API for accessing the EEPROM Andrew Lunn
2015-12-15 10:04 ` Srinivas Kandagatla
2015-12-08 14:05 ` [PATCH 5/6] eeprom: at25: extend driver to plug into the NVMEM framework Andrew Lunn
2015-12-08 14:05 ` [PATCH 6/6] eeprom: 93xx46: " Andrew Lunn
2015-12-15 10:05 ` Srinivas Kandagatla
2015-12-15 10:17 ` Andrew Lunn
2015-12-15 10:26 ` Srinivas Kandagatla [this message]
2015-12-15 10:37 ` Andrew Lunn
2015-12-15 10:47 ` Wolfram Sang
2015-12-15 10:51 ` Srinivas Kandagatla
2015-12-15 11:05 ` Anatolij Gustschin
2015-12-15 12:20 ` Srinivas Kandagatla
2015-12-15 10:06 ` [PATCH 0/6] Convert existing EEPROM drivers to NVMEM Srinivas Kandagatla
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=566FEAE7.10109@linaro.org \
--to=srinivas.kandagatla@linaro.org \
--cc=afd@ti.com \
--cc=agust@denx.de \
--cc=andrew@lunn.ch \
--cc=broonie@kernel.org \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=maxime.ripard@free-electrons.com \
--cc=vz@mleia.com \
--cc=wsa@the-dreams.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.