Re: [PATCH] firmware: remove request_firmware_into_buf()

From	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] firmware: remove request_firmware_into_buf()
Date	2017-07-13 09:50 +0200
Message-ID	<u2H2P-2IC-39@gated-at.bofh.it> (permalink)
References	<tVDnl-6P1-61@gated-at.bofh.it> <tWINY-6Zu-17@gated-at.bofh.it> <tWSDE-5aT-19@gated-at.bofh.it> <tX2D1-3Ko-37@gated-at.bofh.it> <u2zyh-6mj-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Jul 13, 2017 at 01:44:34AM +0200, Luis R. Rodriguez wrote:
> On Tue, Jun 27, 2017 at 10:37:11AM -0700, Bjorn Andersson wrote:
> > On Mon 26 Jun 23:52 PDT 2017, Greg Kroah-Hartman wrote:
> > > Why would we keep it if there is no in-tree user for it?  If you want it
> > > sometime in the future, great, we can revert the deletion then, but
> > > keeping it around for nothing isn't ok, you know that :)
> > > 
> > 
> > Of course I know that :)
> > 
> > I did put a patch in the tubes for this yesterday [1], it's late for
> > v4.13, but I would be happy to see the API stay and we would have a user
> > in v4.14 (and tick this off Qualcomm's "required" list).
> > 
> > [1] https://lkml.org/lkml/2017/6/26/693
> 
> Greg,
> 
> What have you decided to do?

Given that this didn't go in for 4.13-rc1, I figured it was kind of
obvious, I'm waiting to see if a user shows up for this release :)

And it's always trivial to revert a patch to add an api back if someone
does need/want it, so this shouldn't be a big deal to anyone.

> Also what is the threshold for number of drivers to use a new feature for us to
> add it? Note that there is a bundle of features queued up now and as per your
> own preference it would seem you want a new API call for each new feature...

Usually it's a good engineering practice to have 3 users of any
interface to prove that you got the interface correct, right?  But I'm
not making any hard and fast rule here, let's see the patches and judge
it from there...

thanks,

greg k-h

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] firmware: remove request_firmware_into_buf() "Luis R. Rodriguez" <mcgrof@kernel.org> - 2017-07-13 01:50 +0200
  Re: [PATCH] firmware: remove request_firmware_into_buf() Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-07-13 09:50 +0200

csiph-web