From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751445AbdA0V1r (ORCPT <rfc822;w@1wt.eu>);
        Fri, 27 Jan 2017 16:27:47 -0500
Received: from mx2.suse.de ([195.135.220.15]:48748 "EHLO mx2.suse.de"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1751178AbdA0V1P (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 27 Jan 2017 16:27:15 -0500
Date: Fri, 27 Jan 2017 22:27:08 +0100
From: "Luis R. Rodriguez" <mcgrof@kernel.org>
To: =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= <zajec5@gmail.com>
Cc: Greg KH <gregkh@linuxfoundation.org>,
        "Luis R. Rodriguez" <mcgrof@kernel.org>,
        Ming Lei <ming.lei@canonical.com>, Borislav Petkov <bp@alien8.de>,
        wagi@monom.org, Tom Gundersen <teg@jklm.no>,
        Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Vikram Mulukutla <markivx@codeaurora.org>,
        Stephen Boyd <stephen.boyd@linaro.org>,
        Mark Brown <broonie@kernel.org>, zohar@linux.vnet.ibm.com,
        Takashi Iwai <tiwai@suse.de>,
        Johannes Berg <johannes@sipsolutions.net>,
        Christian Lamparter <chunkeey@googlemail.com>,
        Hauke Mehrtens <hauke@hauke-m.de>,
        Josh Boyer <jwboyer@fedoraproject.org>,
        Dmitry Torokhov <dmitry.torokhov@gmail.com>,
        David Woodhouse <dwmw2@infradead.org>, jslaby@suse.com,
        Linus Torvalds <torvalds@linux-foundation.org>, luto@amacapital.net,
        Fengguang Wu <fengguang.wu@intel.com>,
        Richard Purdie <rpurdie@rpsys.net>,
        Jacek Anaszewski <j.anaszewski@samsung.com>, Abhay_Salunke@dell.com,
        Julia Lawall <Julia.Lawall@lip6.fr>, Gilles.Muller@lip6.fr,
        nicolas.palix@imag.fr, dhowells@redhat.com, bjorn.andersson@linaro.org,
        Arend Van Spriel <arend.vanspriel@broadcom.com>,
        Kalle Valo <kvalo@codeaurora.org>
Subject: Re: [PATCH v4 3/3] p54: convert to sysdata API
Message-ID: <20170127212708.GC24047@wotan.suse.de>
References: <20170112150244.12700-4-mcgrof@kernel.org>
 <20170119113857.GQ28024@kroah.com>
 <20170119162751.GJ13946@wotan.suse.de>
 <20170126215005.GU13946@wotan.suse.de>
 <20170127074706.GA31112@kroah.com>
 <CACna6ryHxFqGwY1d8yDWzmJwQYKZBzN7EwTcPyHRj-h-=sVaCw@mail.gmail.com>
 <20170127140731.GB8206@kroah.com>
 <CACna6rwWqq0y4KTY9ignCR5wryZQWj2-8mCqxVPemjmQokZLZg@mail.gmail.com>
 <20170127143052.GA18706@kroah.com>
 <CACna6rzdmkGfn54-_CyMKdft2dcj_sndFX4XFwyiKW0BhfBo7A@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CACna6rzdmkGfn54-_CyMKdft2dcj_sndFX4XFwyiKW0BhfBo7A@mail.gmail.com>
User-Agent: Mutt/1.6.0 (2016-04-01)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Jan 27, 2017 at 03:39:36PM +0100, Rafał Miłecki wrote:
> On 27 January 2017 at 15:30, Greg KH <gregkh@linuxfoundation.org> wrote:
> > On Fri, Jan 27, 2017 at 03:14:14PM +0100, Rafał Miłecki wrote:
> >> > Does that patch really "simplify" anything?  Anyway, resend it if the
> >> > maintainer of the subsystem ignores it (you did cc: the correct people,
> >> > right?)
> >>
> >> According to the MAINTAINERS there isn't firmware API tree /
> >> maintainer. Also this is just a cleanup so I don't know if I should
> >> expect some random maintainer (e.g. wireless tree) to pick it.
> >
> > I don't think you looked very hard:
> >
> >  $ ./scripts/get_maintainer.pl --file drivers/base/firmware_class.c
> >  Ming Lei <ming.lei@canonical.com> (maintainer:FIRMWARE LOADER (request_firmware))
> >  "Luis R. Rodriguez" <mcgrof@kernel.org> (maintainer:FIRMWARE LOADER (request_firmware))
> >  Greg Kroah-Hartman <gregkh@linuxfoundation.org> (supporter:DRIVER CORE, KOBJECTS, DEBUGFS, KERNFS AND SYSFS)
> >  linux-kernel@vger.kernel.org (open list:FIRMWARE LOADER (request_firmware))
> >
> > Please try again...
> 
> My memory totally failed me :( Sorry

I was eventually copied on the patch but there was also a side discussion on
you wanting FW_OPT_NO_WARN. Can you review if the proposed driver_data API does
what I think you wanted. Although request_firmware_direct() only avoids warning
on a sync all the diver_data API also supports this for async calls. If its
something else I'd prefer we evaluate extending the driver_data API before
loosely adding yet another API call using the old API.

  Luis