Re: [PATCH v2 01/16] nvmem: remove unused APIs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Bartosz Golaszewski <brgl@xxxxxxxx>
Subject: Re: [PATCH v2 01/16] nvmem: remove unused APIs
From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
Date: Mon, 10 Sep 2018 12:47:19 +0100
Cc: "David S . Miller" <davem@xxxxxxxxxxxxx>, Mauro Carvalho Chehab <mchehab+samsung@xxxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, Jonathan Corbet <corbet@xxxxxxx>, Sekhar Nori <nsekhar@xxxxxx>, Kevin Hilman <khilman@xxxxxxxxxx>, David Lechner <david@xxxxxxxxxxxxxx>, Boris Brezillon <boris.brezillon@xxxxxxxxxxx>, Andrew Lunn <andrew@xxxxxxx>, Alban Bedel <albeu@xxxxxxx>, Maxime Ripard <maxime.ripard@xxxxxxxxxxx>, Chen-Yu Tsai <wens@xxxxxxxx>, linux-doc <linux-doc@xxxxxxxxxxxxxxx>, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, Linux ARM <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>, Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx>
In-reply-to: <CAMRc=Mck=rAJ6jbm57ZecJWNvOYR5pjLy0JnyxFzA8kFvoK7Ew@mail.gmail.com>
References: <20180907100750.14564-1-brgl@bgdev.pl> <20180907100750.14564-2-brgl@bgdev.pl> <c0b0c411-65f6-62f9-6d6d-2884e5a17250@linaro.org> <CAMRc=MfO01EQAfY4Uu3UwzzvhVOMF7_BpCOPVz2yVXqDgpb0AA@mail.gmail.com> <7e138142-8161-3646-4daf-619995baf395@linaro.org> <CAMRc=Mduu4CmLX5DFtJd8NSbzs5g65U9AmM=rbscit0JyQKVAQ@mail.gmail.com> <3dbd65e5-c948-a784-0362-930fc41b840e@linaro.org> <CAMRc=Mck=rAJ6jbm57ZecJWNvOYR5pjLy0JnyxFzA8kFvoK7Ew@mail.gmail.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1



On 10/09/18 12:31, Bartosz Golaszewski wrote:

About that: there are no users of nvmem_device_cell_read/write()
currently and with the new API I'm not sure this is still needed. Are
you alright with removing those two?


Why do you want to remove them? Other than reason of no users.

All it boils down to if we support device based apis using cell info ornot?IMO, I see no harm in leaving these apis as it is, unless there is astrong reason to do so.


thanks,
srini

Follow-Ups:
- Re: [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Boris Brezillon

References:
- [PATCH v2 00/16] nvmem: rework of the subsystem for non-DT users
  - From: Bartosz Golaszewski
- [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Bartosz Golaszewski
- Re: [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Srinivas Kandagatla
- Re: [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Bartosz Golaszewski
- Re: [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Srinivas Kandagatla
- Re: [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Bartosz Golaszewski
- Re: [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Srinivas Kandagatla
- Re: [PATCH v2 01/16] nvmem: remove unused APIs
  - From: Bartosz Golaszewski

Prev by Date: Re: [PATCH v2 01/16] nvmem: remove unused APIs
Next by Date: Re: [PATCH v7] Add udmabuf misc device
Previous by thread: Re: [PATCH v2 01/16] nvmem: remove unused APIs
Next by thread: Re: [PATCH v2 01/16] nvmem: remove unused APIs
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite Forum] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Video 4 Linux] [Device Mapper] [Linux Resources]