Received: by 10.192.165.148 with SMTP id m20csp4462078imm; Tue, 8 May 2018 08:49:50 -0700 (PDT) X-Google-Smtp-Source: AB8JxZqZbMZFfJlIsaCnHPIehcd+fSb/fRzFA7V/0VjbuBQunjmP25giF+W5ox5hnA6LnWa2BTGb X-Received: by 2002:a65:4d07:: with SMTP id i7-v6mr19303218pgt.149.1525794589957; Tue, 08 May 2018 08:49:49 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1525794589; cv=none; d=google.com; s=arc-20160816; b=WxoXmZ1sk1yTwJqMJzq187DurIc7mI/4m+XOgH5VAYwWewsb+arPUBXJSoKQFFSRED RzW4MXl4UmH/x2AB0AWvpxUCFF1fcxyioZKFeQMLeZENkQCEgtYTmMSZICJWxsUFW6aB ARDtzAEy0AqiUQhjFZKUiLqR263fKO/pLFETabxM2vayzEA38M+vhDcG+ArjL05EgJOE bw6zin7F0FNxkwt1oF9rNGmw+bsMOsbAIyuyy9GuwCB0qfHwiRNkl9Z6UwqjUBd8OZYZ D9Csj7XRayJlqw5uos3V0W4rzlFC5o758I3mj4CQs0/55QpzIRjdK5rCyz0xXqL2a2Ge yDZA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=WX5QG6Al44mtWpI1RPfIgVmWivkiXReXTVRRbU02LhA=; b=wGUG5idvwRPaJ3EaFjUjbkxWQYXZbHM0MhgHLhaOzY54OltsNID6UFgpfhMohIaiUG vlL8MApx3xkScJf+K5HMxwT67bEDLW89up+mc7qeRn3z3/zG9vWl4P1wknci6ioPSz0d BY4QM7WDibo4f8bKUsONtxFMINe2p+4EIvd/IHNKG9k2ZCbHuIFK0JQ9sN90K95rFx97 Hs9aDtNWUSUbC2/FEfeOh5d4e/o9U6+NcQuxLXnb6zbW3yNZt6TB2c3LXAa32i8FAqqx vljeVfOuQ4GyFNaow+kvQmkKxq7AAQuZy+jvF1pL0TiQud8cdSP4w0iVcvC9o3z724Pt r3uQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 92-v6si9082459pli.280.2018.05.08.08.49.35; Tue, 08 May 2018 08:49:49 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932544AbeEHPtM (ORCPT + 99 others); Tue, 8 May 2018 11:49:12 -0400 Received: from mx2.suse.de ([195.135.220.15]:51562 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932127AbeEHPtK (ORCPT ); Tue, 8 May 2018 11:49:10 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id EC00BAD21; Tue, 8 May 2018 15:49:08 +0000 (UTC) Date: Tue, 8 May 2018 15:49:08 +0000 From: "Luis R. Rodriguez" To: Mauro Carvalho Chehab Cc: Linux Doc Mailing List , Mauro Carvalho Chehab , linux-kernel@vger.kernel.org, Jonathan Corbet , "Rafael J. Wysocki" , Len Brown , Pavel Machek , "Luis R. Rodriguez" , Greg Kroah-Hartman , Masanari Iida , linux-pm@vger.kernel.org, Kees Cook Subject: Re: [PATCH 02/18] docs: fix location of request_firmware & friends Message-ID: <20180508154908.GD27853@wotan.suse.de> References: <9e40d0b3a085a937b42ec60a3b2409cf820ca713.1525684985.git.mchehab+samsung@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9e40d0b3a085a937b42ec60a3b2409cf820ca713.1525684985.git.mchehab+samsung@kernel.org> User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 07, 2018 at 06:35:38AM -0300, Mauro Carvalho Chehab wrote: > commit 5d6d1ddd2730 ("firmware: move firmware loader into its own directory") > and other commits renamed the old firmware_class.c file and split it > into separate files, but documentation was not changed accordingly, > causing Sphinx errors. > > Change the location accordingly at the documentation files. > > While here, add a missing entry at request_firmware.rst for > release_firmware() function. > > Fixes: 5d6d1ddd2730 ("firmware: move firmware loader into its own directory") > Cc: Kees Cook > Cc: Luis R. Rodriguez > Cc: Greg Kroah-Hartman > Signed-off-by: Mauro Carvalho Chehab > --- > Documentation/dell_rbu.txt | 4 ++-- > .../driver-api/firmware/fallback-mechanisms.rst | 2 +- > .../driver-api/firmware/request_firmware.rst | 17 +++++++++++------ > Documentation/driver-api/infrastructure.rst | 2 +- > Documentation/power/suspend-and-cpuhotplug.txt | 2 +- > 5 files changed, 16 insertions(+), 11 deletions(-) > > diff --git a/Documentation/dell_rbu.txt b/Documentation/dell_rbu.txt > index 0fdb6aa2704c..befeff80e7ec 100644 > --- a/Documentation/dell_rbu.txt > +++ b/Documentation/dell_rbu.txt > @@ -121,8 +121,8 @@ read back the image downloaded. > > .. note:: > > - This driver requires a patch for firmware_class.c which has the modified > - request_firmware_nowait function. > + This driver requires a patch for drivers/base/firmware_loader/main.c > + which has the modified request_firmware_nowait() function. > > Also after updating the BIOS image a user mode application needs to execute > code which sends the BIOS update request to the BIOS. So on the next reboot This part looks good and is needed. > diff --git a/Documentation/driver-api/firmware/fallback-mechanisms.rst b/Documentation/driver-api/firmware/fallback-mechanisms.rst > index f353783ae0be..7aed31b5a439 100644 > --- a/Documentation/driver-api/firmware/fallback-mechanisms.rst > +++ b/Documentation/driver-api/firmware/fallback-mechanisms.rst > @@ -72,7 +72,7 @@ the firmware requested, and establishes it in the device hierarchy by > associating the device used to make the request as the device's parent. > The sysfs directory's file attributes are defined and controlled through > the new device's class (firmware_class) and group (fw_dev_attr_groups). > -This is actually where the original firmware_class.c file name comes from, > +This is actually where drivers/base/firmware_loader/fallback.c comes from, Not this part. What I meant to keep well documented here was not just only the old firmware file name for the code, but also the module name firmware_class, and its respective sysfs class name which is registered. From what I recall testing, we could not rename the module now because of this. I believe it had to do with the modular case, given the sysfs class could still be registered. The fact that I forget the exact *issue* which prevented the module rename shows how important it is to document this. Folks 10 years from now may wonder why the hell that name stuck, and the point was to document that the *original* loader was the sysfs fallback mechanism. > as originally the only firmware loading mechanism available was the > mechanism we now use as a fallback mechanism. > > diff --git a/Documentation/driver-api/firmware/request_firmware.rst b/Documentation/driver-api/firmware/request_firmware.rst > index cf4516dfbf96..8e34d29ea02d 100644 > --- a/Documentation/driver-api/firmware/request_firmware.rst > +++ b/Documentation/driver-api/firmware/request_firmware.rst > @@ -17,19 +17,24 @@ an error is returned. > > request_firmware > ---------------- > -.. kernel-doc:: drivers/base/firmware_class.c > +.. kernel-doc:: drivers/base/firmware_loader/main.c > :functions: request_firmware This is fixed on Hans de Goede's patch already merged on Greg's own tree. > request_firmware_direct > ----------------------- > -.. kernel-doc:: drivers/base/firmware_class.c > +.. kernel-doc:: drivers/base/firmware_loader/main.c > :functions: request_firmware_direct This is fixed on Hans de Goede's patch already merged on Greg's own tree. > request_firmware_into_buf > ------------------------- > -.. kernel-doc:: drivers/base/firmware_class.c > +.. kernel-doc:: drivers/base/firmware_loader/main.c > :functions: request_firmware_into_buf This is fixed on Hans de Goede's patch already merged on Greg's own tree. > +release_firmware > +---------------- > +.. kernel-doc:: drivers/base/firmware_loader/main.c > + :functions: release_firmware This is fixed on Hans de Goede's patch already merged on Greg's own tree. > + > Asynchronous firmware requests > ============================== > > @@ -41,7 +46,7 @@ in atomic contexts. > > request_firmware_nowait > ----------------------- > -.. kernel-doc:: drivers/base/firmware_class.c > +.. kernel-doc:: drivers/base/firmware_loader/main.c > :functions: request_firmware_nowait This is fixed on Hans de Goede's patch already merged on Greg's own tree. > > Special optimizations on reboot > @@ -54,8 +59,8 @@ this can be done with firmware_request_cache() insted of requesting for the > firmare to be loaded. > > firmware_request_cache() > ------------------------ > -.. kernel-doc:: drivers/base/firmware_class.c > +------------------------ > +.. kernel-doc:: drivers/base/firmware_loader/main.c > :functions: firmware_request_cache This is fixed on Hans de Goede's patch already merged on Greg's own tree. > > request firmware API expected driver use > diff --git a/Documentation/driver-api/infrastructure.rst b/Documentation/driver-api/infrastructure.rst > index 6d9ff316b608..bee1b9a1702f 100644 > --- a/Documentation/driver-api/infrastructure.rst > +++ b/Documentation/driver-api/infrastructure.rst > @@ -28,7 +28,7 @@ Device Drivers Base > .. kernel-doc:: drivers/base/node.c > :internal: > > -.. kernel-doc:: drivers/base/firmware_class.c > +.. kernel-doc:: drivers/base/firmware_loader/main.c This is fixed on Hans de Goede's patch already merged on Greg's own tree. > :export: > > .. kernel-doc:: drivers/base/transport_class.c > diff --git a/Documentation/power/suspend-and-cpuhotplug.txt b/Documentation/power/suspend-and-cpuhotplug.txt > index 31abd04b9572..6f55eb960a6d 100644 > --- a/Documentation/power/suspend-and-cpuhotplug.txt > +++ b/Documentation/power/suspend-and-cpuhotplug.txt > @@ -168,7 +168,7 @@ update on the CPUs, as discussed below: > > [Please bear in mind that the kernel requests the microcode images from > userspace, using the request_firmware() function defined in > -drivers/base/firmware_class.c] > +drivers/base/firmware_loader/main.c] This is fixed on Hans de Goede's patch already merged on Greg's own tree. Luis