Re: [PATCH v9 1/5] firmware: add extensible driver data params

Luis R. Rodriguez
2017-06-23T22:47:39Z

On Fri, Jun 23, 2017 at 11:59:36PM +0800, Greg KH wrote:
> On Mon, Jun 19, 2017 at 09:35:22PM +0200, Luis R. Rodriguez wrote:
> > You may argue that *one* upstream users is not sufficient to introduce a new
> > feature for, but I disagree given we have had new full *API* added for a new
> > feature on the firmware API even for drivers THAT ARE NOT UPSTREAM! For
> > instance request_firmware_into_buf() has no upstream users!!!
>
> That's not acceptable at all, I'll send a patch after this to remove
> that. We don't keep apis around with no in-kernel users, you know this.

I'm delighted to hear we can do away with the request_firmware_into_buf() crap.

> > Now, you might say that even though this is true that there many users of
> > out-of-tree drivers that need this. While true, if this is the bar we'd go
> > with, we can't then ignore the iwlwifi userbase, and the possible gains of
> > having a proper non-recursive use of the daisy chained requests.
>
> Nope, I don't care about out-of-tree drivers as we have no idea what is
> going on there at all. I've always had this position.

Beautiful. Music to my ears.

Luis