Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964852AbbHJKhr (ORCPT ); Mon, 10 Aug 2015 06:37:47 -0400 Received: from mail-wi0-f173.google.com ([209.85.212.173]:35616 "EHLO mail-wi0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933932AbbHJKhi (ORCPT ); Mon, 10 Aug 2015 06:37:38 -0400 Message-ID: <55C87EEF.6020609@linaro.org> Date: Mon, 10 Aug 2015 11:37:35 +0100 From: Srinivas Kandagatla User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Andrew Lunn CC: 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. References: <1437995567-11203-1-git-send-email-srinivas.kandagatla@linaro.org> <1438693367.3793.29.camel@pengutronix.de> <20150809151916.GF14271@lunn.ch> In-Reply-To: <20150809151916.GF14271@lunn.ch> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1574 Lines: 46 Hi Andrew, On 09/08/15 16:19, Andrew Lunn wrote: > 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 > Yes, nvmem by default sets the read permission for everyone S_IRUGO. Which was the most common use case. > Has this been considered and discussed? Thanks for bringing this up, No, we did not discuss this explicitly, but it should not be an issue to accommodate this. Provider still have flexibility to configure such things. As a first patchset we wanted to keep the framework simple and add features as we move on. Thanks, srini > > Thanks > Andrew > -- 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/