From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH v7 5/8] Input: silead - Switch to firmware_request_platform for retreiving the fw Date: Thu, 10 Oct 2019 17:47:24 -0700 Message-ID: <20191011004724.GC229325@dtor-ws> References: <20191004145056.43267-1-hdegoede@redhat.com> <20191004145056.43267-6-hdegoede@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20191004145056.43267-6-hdegoede@redhat.com> Sender: linux-kernel-owner@vger.kernel.org To: Hans de Goede Cc: Ard Biesheuvel , Darren Hart , Andy Shevchenko , Luis Chamberlain , Greg Kroah-Hartman , "Rafael J . Wysocki" , Thomas Gleixner , Ingo Molnar , Borislav Petkov , "H . Peter Anvin" , Jonathan Corbet , Peter Jones , Dave Olsthoorn , x86@kernel.org, platform-driver-x86@vger.kernel.org, linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, linux-input@vger.kernel.org List-Id: platform-driver-x86.vger.kernel.org On Fri, Oct 04, 2019 at 04:50:53PM +0200, Hans de Goede wrote: > Unfortunately sofar we have been unable to get permission to redistribute > Silead touchscreen firmwares in linux-firmware. This means that people > need to find and install the firmware themselves before the touchscreen > will work > > Some UEFI/x86 tablets with a Silead touchscreen have a copy of the fw > embedded in their UEFI boot-services code. > > This commit makes the silead driver use the new firmware_request_platform > function, which will fallback to looking for such an embedded copy when > direct filesystem lookup fails. This will make the touchscreen work OOTB > on devices where there is a fw copy embedded in the UEFI code. > > Signed-off-by: Hans de Goede Acked-by: Dmitry Torokhov > --- > drivers/input/touchscreen/silead.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/input/touchscreen/silead.c b/drivers/input/touchscreen/silead.c > index ad8b6a2bfd36..8fa2f3b7cfd8 100644 > --- a/drivers/input/touchscreen/silead.c > +++ b/drivers/input/touchscreen/silead.c > @@ -288,7 +288,7 @@ static int silead_ts_load_fw(struct i2c_client *client) > > dev_dbg(dev, "Firmware file name: %s", data->fw_name); > > - error = request_firmware(&fw, data->fw_name, dev); > + error = firmware_request_platform(&fw, data->fw_name, dev); > if (error) { > dev_err(dev, "Firmware request error %d\n", error); > return error; > -- > 2.23.0 > -- Dmitry