From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH v9 0/9] Add simple NVMEM Framework via regmap. Date: Sun, 9 Aug 2015 17:19:16 +0200 Message-ID: <20150809151916.GF14271@lunn.ch> References: <1437995567-11203-1-git-send-email-srinivas.kandagatla@linaro.org> <1438693367.3793.29.camel@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1438693367.3793.29.camel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: Srinivas Kandagatla , stefan.wahren-eS4NqCHxEME@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, arnd-r2nGTMty4D4@public.gmane.org, khilman-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Greg Kroah-Hartman , s.hauer-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org, sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, mporter-OWPKS81ov/FWk0Htik3J/w@public.gmane.org, pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org, Rob Herring , Mark Brown , linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, wxt-TNX95d0MmH7DzftRWevZcw@public.gmane.org List-Id: linux-api@vger.kernel.org Hi Srinivas The AT24 eeprom driver contains the comment: /* * Export the EEPROM bytes through sysfs, since that's convenient. * By default, only root should see the data (maybe passwords etc) */ and as you would expect from this: # ls -l ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/eeprom -rw------- 1 root root 512 Aug 9 10:16 ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/eeprom The AT25 and the MAX6875 driver are the same. However nvmem has different defaults: # ls -l ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem -rw-r--r-- 1 root root 0 Aug 9 10:16 ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem Has this been considered and discussed? Thanks Andrew From mboxrd@z Thu Jan 1 00:00:00 1970 From: andrew@lunn.ch (Andrew Lunn) Date: Sun, 9 Aug 2015 17:19:16 +0200 Subject: [PATCH v9 0/9] Add simple NVMEM Framework via regmap. In-Reply-To: <1438693367.3793.29.camel@pengutronix.de> References: <1437995567-11203-1-git-send-email-srinivas.kandagatla@linaro.org> <1438693367.3793.29.camel@pengutronix.de> Message-ID: <20150809151916.GF14271@lunn.ch> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Srinivas The AT24 eeprom driver contains the comment: /* * Export the EEPROM bytes through sysfs, since that's convenient. * By default, only root should see the data (maybe passwords etc) */ and as you would expect from this: # ls -l ./devices/platform/soc/soc:aips-bus at 40000000/40066000.i2c/i2c-0/0-0050/eeprom -rw------- 1 root root 512 Aug 9 10:16 ./devices/platform/soc/soc:aips-bus at 40000000/40066000.i2c/i2c-0/0-0050/eeprom The AT25 and the MAX6875 driver are the same. However nvmem has different defaults: # ls -l ./devices/platform/soc/soc:aips-bus at 40000000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem -rw-r--r-- 1 root root 0 Aug 9 10:16 ./devices/platform/soc/soc:aips-bus at 40000000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem Has this been considered and discussed? Thanks Andrew From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933058AbbHIP0f (ORCPT ); Sun, 9 Aug 2015 11:26:35 -0400 Received: from vps0.lunn.ch ([178.209.37.122]:56484 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752508AbbHIP0c (ORCPT ); Sun, 9 Aug 2015 11:26:32 -0400 Date: Sun, 9 Aug 2015 17:19:16 +0200 From: Andrew Lunn To: Srinivas Kandagatla Cc: Srinivas Kandagatla , stefan.wahren@i2se.com, devicetree@vger.kernel.org, arnd@arndb.de, khilman@kernel.org, Greg Kroah-Hartman , s.hauer@pengutronix.de, sboyd@codeaurora.org, linux-kernel@vger.kernel.org, mporter@konsulko.com, pantelis.antoniou@konsulko.com, Rob Herring , Mark Brown , linux-api@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, wxt@rock-chips.com Subject: Re: [PATCH v9 0/9] Add simple NVMEM Framework via regmap. Message-ID: <20150809151916.GF14271@lunn.ch> References: <1437995567-11203-1-git-send-email-srinivas.kandagatla@linaro.org> <1438693367.3793.29.camel@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1438693367.3793.29.camel@pengutronix.de> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Srinivas The AT24 eeprom driver contains the comment: /* * Export the EEPROM bytes through sysfs, since that's convenient. * By default, only root should see the data (maybe passwords etc) */ and as you would expect from this: # ls -l ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/eeprom -rw------- 1 root root 512 Aug 9 10:16 ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/eeprom The AT25 and the MAX6875 driver are the same. However nvmem has different defaults: # ls -l ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem -rw-r--r-- 1 root root 0 Aug 9 10:16 ./devices/platform/soc/soc:aips-bus@40000000/40066000.i2c/i2c-0/0-0050/0-00500/nvmem Has this been considered and discussed? Thanks Andrew