Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1292001

Re: [PATCH 0/6] Convert existing EEPROM drivers to NVMEM

Path csiph.com!eternal-september.org!feeder.eternal-september.org!weretis.net!feeder4.news.weretis.net!storethat.news.telefonica.de!telefonica.de!news.panservice.it!bofh.it!news.nic.it!robomod
From Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Newsgroups linux.kernel
Subject Re: [PATCH 0/6] Convert existing EEPROM drivers to NVMEM
Date Tue, 15 Dec 2015 11:10:02 +0100
Message-ID <qFUIq-2DP-11@gated-at.bofh.it> (permalink)
References <qDr7P-3nh-3@gated-at.bofh.it>
X-Original-To Andrew Lunn <andrew@lunn.ch>, GregKH <greg@kroah.com>, maxime.ripard@free-electrons.com, wsa@the-dreams.de, broonie@kernel.org, vz@mleia.com
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=2ZLkn91KY5uKsE5W6s0BOynAn41uyjjzk46dYfwhcg4=; b=ePv6CBP44WURVMTjpzdhFdpZZZhImxKgiP1VJtbZsJQsxeUMO6tWMroQ4xEv2LFqd/ mtT5s8sBDp6yt6a7tHyLr6eP4Qb5m4WkLFc2qec63NeQz3hbxS7nNVV/1lB8DdlpcbZf qSYnDpbM2OClUKGBibCjyQBK3KRaqhUOPv83s=
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=2ZLkn91KY5uKsE5W6s0BOynAn41uyjjzk46dYfwhcg4=; b=dUQSmehPP+fKVWoLBKWVKVF/c/8eMZn4TdjlmVmkrQjr1ObU4WUdqVuZjMGNBOtVl/ FOeOM2IR/8K3tUJ45r/9wDJJhaKAEjt5QrZ0Bx04st7adpD6k6DovKvrO8fRaPvtLvNw b7He5+msOTRnM7+LkG5X1BYYctda7TwFxcLi0LDtvRX1nlNl9CpK94DR092OQwNtYs81 WRX7wESOZ/Qirocs2WBZzZ1Cv1d40jFZpRNhLHtYLhXgIvwn4W9rxPGn+ydCnpjWObIt pun0TpvFjRpThoupcg477p/Ve5Ux2hWBYFwyJOpDvcYLS+bYBuOQ8WQvecCea+awzZqC KiBA==
X-Gm-Message-State ALoCoQmVxXYkElqEoJplIEpWb1OqNuBO5ONOWAnKOcWknEYpwwmTcZdq8bUhUV6U/Q2WhKQ3GNdhvaFZdYjvKFjFZrX7FmQ6Ng==
X-Received by 10.28.50.70 with SMTP id y67mr3643829wmy.91.1450173997339; Tue, 15 Dec 2015 02:06:37 -0800 (PST)
User-Agent Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
MIME-Version 1.0
Content-Type text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding 7bit
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 51
Organization linux.* mail to news gateway
X-Original-Cc afd@ti.com, linux-kernel@vger.kernel.org
X-Original-Date Tue, 15 Dec 2015 10:06:35 +0000
X-Original-Message-ID <566FE62B.6060500@linaro.org>
X-Original-References <1449583511-22521-1-git-send-email-andrew@lunn.ch>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1292001

Show key headers only | View raw


Thanks Andrew for looking into this.

On 08/12/15 14:05, Andrew Lunn wrote:
> This patches converts the old EEPROM drivers in driver/misc/eeprom to
> use the NVMEM framework. These drivers export there content in /sys as
> read only to root, since the EEPROM may contain sensitive information.
> So the first patch adds a flag so the NVMEM framework will create its
> file in /sys as root read only.
>
> To keep backwards compatibility with these older drivers, the contents
> of the EEPROM must be exports in sysfs in a file called eeprom in the
> devices node in sys, where as the NVMEM places them under class/nvmem.
> So add this optional backwards compatible to the framework.
>
> Then convert the at24, at25 and 93xx46 by adding regmap support,
> removing each drivers own /sys code and registering with the NVMEM
> framework.
>
> AT24 and 93xx46 has been boot tested, at25 compile tested only.
>
> Andrew Lunn (6):
>    nvmem: Add flag to export NVMEM to root only
>    nvmem: Add backwards compatibility support for older EEPROM drivers.
>    eeprom: at24: extend driver to plug into the NVMEM framework
>    eeprom: at25: Remove in kernel API for accessing the EEPROM
>    eeprom: at25: extend driver to plug into the NVMEM framework
>    eeprom: 93xx46: extend driver to plug into the NVMEM framework
>
>   drivers/misc/eeprom/Kconfig         |   9 +++
>   drivers/misc/eeprom/at24.c          | 119 +++++++++++++++++++----------
>   drivers/misc/eeprom/at25.c          | 147 ++++++++++++++++--------------------
>   drivers/misc/eeprom/eeprom_93xx46.c | 121 ++++++++++++++++++++++-------
>   drivers/nvmem/Kconfig               |   7 ++
>   drivers/nvmem/core.c                | 132 ++++++++++++++++++++++++++++++--
>   include/linux/nvmem-provider.h      |  11 +++
>   include/linux/spi/eeprom.h          |   2 -
>   8 files changed, 393 insertions(+), 155 deletions(-)


I did test this patchset on my board with at24, series looks good.
Other than some comments on few  patches.


--srini
>
--
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/

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH 0/6] Convert existing EEPROM drivers to NVMEM Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2015-12-15 11:10 +0100

csiph-web