From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on archive.lwn.net X-Spam-Level: X-Spam-Status: No, score=-5.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham autolearn_force=no version=3.4.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by archive.lwn.net (Postfix) with ESMTP id 0C28B7D57F for ; Mon, 10 Sep 2018 12:18:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728495AbeIJRMM (ORCPT ); Mon, 10 Sep 2018 13:12:12 -0400 Received: from mail.bootlin.com ([62.4.15.54]:37311 "EHLO mail.bootlin.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727649AbeIJRMM (ORCPT ); Mon, 10 Sep 2018 13:12:12 -0400 Received: by mail.bootlin.com (Postfix, from userid 110) id 018382074F; Mon, 10 Sep 2018 14:18:23 +0200 (CEST) Received: from bbrezillon (AAubervilliers-681-1-30-219.w90-88.abo.wanadoo.fr [90.88.15.219]) by mail.bootlin.com (Postfix) with ESMTPSA id 7F99E206EE; Mon, 10 Sep 2018 14:18:22 +0200 (CEST) Date: Mon, 10 Sep 2018 14:18:20 +0200 From: Boris Brezillon To: Srinivas Kandagatla Cc: Bartosz Golaszewski , "David S . Miller" , Mauro Carvalho Chehab , Greg Kroah-Hartman , Andrew Morton , Arnd Bergmann , Jonathan Corbet , Sekhar Nori , Kevin Hilman , David Lechner , Andrew Lunn , Alban Bedel , Maxime Ripard , Chen-Yu Tsai , linux-doc , Linux Kernel Mailing List , Linux ARM , Bartosz Golaszewski Subject: Re: [PATCH v2 01/16] nvmem: remove unused APIs Message-ID: <20180910141820.6c715895@bbrezillon> In-Reply-To: References: <20180907100750.14564-1-brgl@bgdev.pl> <20180907100750.14564-2-brgl@bgdev.pl> <7e138142-8161-3646-4daf-619995baf395@linaro.org> <3dbd65e5-c948-a784-0362-930fc41b840e@linaro.org> X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-doc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org Hi Srinivas, On Mon, 10 Sep 2018 12:47:19 +0100 Srinivas Kandagatla wrote: > 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. I'm just sharing my (and probably other maintainers/developers) PoV here, so please don't take this as an attempt to force you to change your mind, but rather an attempt at explaining why APIs usually stay private when they have no users. Kernel maintainers tend to reject APIs that have no users because adding something that nobody needs yet is hard to get right. I mean, you can design an API on what you think will be needed/appropriate, and then, when people actually start needing this feature, they realize it does not quite match what they need, and they have to adjust/rework this API. > > All it boils down to if we support device based apis using cell info or > not? But it looks like nobody is actually using it, and the first potential user (Bart) proposed a different approach with the nvmem cell table declaration. > IMO, I see no harm in leaving these apis as it is, unless there is a > strong reason to do so. It's harmless, but it's also unused. If people start using it and you realize the API is not working for some use cases, then you'll have to patch all existing users. All this being said, it's your call to make, and if you think it makes sense to keep these functions around for any reason, then be it. Regards, Boris