linux-kernel - Re: [PATCH] mmc: core: Extend sysfs with OCR register

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAPDyKFo7wVfRroVLui2b_oUBbMjYGFg8FMMW6vXF3pXtSw1a7Q@mail.gmail.com>
Date:	Tue, 12 Jul 2016 11:53:50 +0200
From:	Ulf Hansson <ulf.hansson@...aro.org>
To:	Bojan Prtvar <prtvar.b@...il.com>
Cc:	linux-mmc <linux-mmc@...r.kernel.org>,
	Jonathan Corbet <corbet@....net>,
	Linus Walleij <linus.walleij@...aro.org>,
	Adrian Hunter <adrian.hunter@...el.com>,
	Dong Aisheng <aisheng.dong@....com>,
	Wolfram Sang <wsa+renesas@...g-engineering.com>,
	Gwendal Grignou <gwendal@...omium.org>,
	Chaotian Jing <chaotian.jing@...iatek.com>,
	"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] mmc: core: Extend sysfs with OCR register

On 6 July 2016 at 18:21, Ulf Hansson <ulf.hansson@...aro.org> wrote:
> On 4 July 2016 at 13:56, Bojan Prtvar <prtvar.b@...il.com> wrote:
>> Make operation conditions register (OCR) easily accessible from user space.
>>
>> Signed-off-by: Bojan Prtvar <prtvar.b@...il.com>
>
> Thanks, applied for next! Amended the changelog with the explanation
> why this change is wanted.

I just realized that the OCR register is also available for SD cards.

Could you perhaps cook a patch which adds this to sysfs for SD cards as well!?

Kind regards
Uffe

>
> Kind regards
> Uffe
>
>> ---
>>  Documentation/mmc/mmc-dev-attrs.txt | 1 +
>>  drivers/mmc/core/mmc.c              | 2 ++
>>  2 files changed, 3 insertions(+)
>>
>> diff --git a/Documentation/mmc/mmc-dev-attrs.txt b/Documentation/mmc/mmc-dev-attrs.txt
>> index caa5557..8163eca 100644
>> --- a/Documentation/mmc/mmc-dev-attrs.txt
>> +++ b/Documentation/mmc/mmc-dev-attrs.txt
>> @@ -28,6 +28,7 @@ All attributes are read-only.
>>         preferred_erase_size    Preferred erase size
>>         raw_rpmb_size_mult      RPMB partition size
>>         rel_sectors             Reliable write sector count
>> +       ocr                     Operation Conditions Register
>>
>>  Note on Erase Size and Preferred Erase Size:
>>
>> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
>> index 5d438ad..1848334 100644
>> --- a/drivers/mmc/core/mmc.c
>> +++ b/drivers/mmc/core/mmc.c
>> @@ -727,6 +727,7 @@ MMC_DEV_ATTR(enhanced_area_offset, "%llu\n",
>>  MMC_DEV_ATTR(enhanced_area_size, "%u\n", card->ext_csd.enhanced_area_size);
>>  MMC_DEV_ATTR(raw_rpmb_size_mult, "%#x\n", card->ext_csd.raw_rpmb_size_mult);
>>  MMC_DEV_ATTR(rel_sectors, "%#x\n", card->ext_csd.rel_sectors);
>> +MMC_DEV_ATTR(ocr, "%08x\n", card->ocr);
>>
>>  static ssize_t mmc_fwrev_show(struct device *dev,
>>                               struct device_attribute *attr,
>> @@ -762,6 +763,7 @@ static struct attribute *mmc_std_attrs[] = {
>>         &dev_attr_enhanced_area_size.attr,
>>         &dev_attr_raw_rpmb_size_mult.attr,
>>         &dev_attr_rel_sectors.attr,
>> +       &dev_attr_ocr.attr,
>>         NULL,
>>  };
>>  ATTRIBUTE_GROUPS(mmc_std);
>> --
>> 2.8.0.GIT
>>