Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1241390 > unrolled thread
| Started by | "Kweh, Hock Leong" <hock.leong.kweh@intel.com> |
|---|---|
| First post | 2015-10-07 13:20 +0200 |
| Last post | 2015-10-08 13:10 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH v8 0/2] Enable capsule loader interface for efi firmware updating "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-10-07 13:20 +0200
Re: [PATCH v8 0/2] Enable capsule loader interface for efi firmware updating Bryan O'Donoghue <pure.logic@nexus-software.ie> - 2015-10-07 17:00 +0200
RE: [PATCH v8 0/2] Enable capsule loader interface for efi firmware updating "Kweh, Hock Leong" <hock.leong.kweh@intel.com> - 2015-10-08 06:00 +0200
Re: [PATCH v8 0/2] Enable capsule loader interface for efi firmware updating Bryan O'Donoghue <pure.logic@nexus-software.ie> - 2015-10-08 13:10 +0200
| From | "Kweh, Hock Leong" <hock.leong.kweh@intel.com> |
|---|---|
| Date | 2015-10-07 13:20 +0200 |
| Subject | [PATCH v8 0/2] Enable capsule loader interface for efi firmware updating |
| Message-ID | <qgUVj-39p-5@gated-at.bofh.it> |
From: "Kweh, Hock Leong" <hock.leong.kweh@intel.com> Dear maintainers & communities, This patchset is created on top of Matt's patchset: 1.)https://lkml.org/lkml/2014/10/7/390 "[PATCH 1/2] efi: Move efi_status_to_err() to efi.h" 2.)https://lkml.org/lkml/2014/10/7/391 "[PATCH 2/2] efi: Capsule update support" It expose a misc char interface for user to upload the capsule binary and calling efi_capsule_update() API to pass the binary to EFI firmware. The steps to update efi firmware are: 1.) cat firmware.cap > /dev/efi_capsule_loader 2.) reboot Any failed upload error message will be returned while doing "cat" through file operation write() function call. Tested the code with Intel Quark Galileo GEN1 platform. Thanks. --- NOTE: If Matt agrees with this design, [PATCH v7 1/1] will be squash into his [PATCH 2/2]: https://lkml.org/lkml/2014/10/7/391 for submitting. changelog v8: * further clean up on kunmap() & efi_free_all_buff_pages() * design enhanced to support 1st few writes are less than efi header size * removed support to padding capsule and flag error once the upload size bigger than header defined size changelog v7: * add successful message printed in dmesg * shorten the code in efi_capsule_write() by splitting out efi_capsule_setup_info() & efi_capsule_submit_update() functions * design added capability to support multiple file open action * re-write those comments by following standard format * design added the "uncomplete" error return through flush() file operation changelog v6: * clean up on error handling for better code flow and review * clean up on pr_err() for critical error only * design taking care writing block that below PAGE_SIZE * once error has occurred, design will return -EIO until file close * document design expectations/scenarios in the code * change the dynamic allocation cap_info struct to statically allocated changelog v5: * changed to new design without leveraging firmware_class API * use misc_char device interface instead of sysfs * error return through file Write() function call Kweh, Hock Leong (2): efi: export efi_capsule_supported() function symbol efi: a misc char interface for user to update efi firmware drivers/firmware/efi/Kconfig | 10 drivers/firmware/efi/Makefile | 1 drivers/firmware/efi/capsule.c | 1 drivers/firmware/efi/efi-capsule-loader.c | 356 +++++++++++++++++++++++++++++ 4 files changed, 368 insertions(+) create mode 100644 drivers/firmware/efi/efi-capsule-loader.c -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Bryan O'Donoghue <pure.logic@nexus-software.ie> |
|---|---|
| Date | 2015-10-07 17:00 +0200 |
| Subject | Re: [PATCH v8 0/2] Enable capsule loader interface for efi firmware updating |
| Message-ID | <qgYme-7YI-21@gated-at.bofh.it> |
| In reply to | #1241390 |
On 07/10/15 20:13, Kweh, Hock Leong wrote: > From: "Kweh, Hock Leong" <hock.leong.kweh@intel.com> > > Dear maintainers & communities, > > This patchset is created on top of Matt's patchset: > 1.)https://lkml.org/lkml/2014/10/7/390 > "[PATCH 1/2] efi: Move efi_status_to_err() to efi.h" > 2.)https://lkml.org/lkml/2014/10/7/391 > "[PATCH 2/2] efi: Capsule update support" > > It expose a misc char interface for user to upload the capsule binary and > calling efi_capsule_update() API to pass the binary to EFI firmware. > > The steps to update efi firmware are: > 1.) cat firmware.cap > /dev/efi_capsule_loader > 2.) reboot > > Any failed upload error message will be returned while doing "cat" through > file operation write() function call. > > Tested the code with Intel Quark Galileo GEN1 platform. > > Thanks. > > --- > NOTE: > If Matt agrees with this design, [PATCH v7 1/1] will be squash into his > [PATCH 2/2]: https://lkml.org/lkml/2014/10/7/391 for submitting. > > changelog v8: > * further clean up on kunmap() & efi_free_all_buff_pages() > * design enhanced to support 1st few writes are less than efi header size > * removed support to padding capsule and flag error once the upload size > bigger than header defined size > > changelog v7: > * add successful message printed in dmesg > * shorten the code in efi_capsule_write() by splitting out > efi_capsule_setup_info() & efi_capsule_submit_update() functions > * design added capability to support multiple file open action > * re-write those comments by following standard format > * design added the "uncomplete" error return through flush() file operation > > changelog v6: > * clean up on error handling for better code flow and review > * clean up on pr_err() for critical error only > * design taking care writing block that below PAGE_SIZE > * once error has occurred, design will return -EIO until file close > * document design expectations/scenarios in the code > * change the dynamic allocation cap_info struct to statically allocated > > changelog v5: > * changed to new design without leveraging firmware_class API > * use misc_char device interface instead of sysfs > * error return through file Write() function call > > > Kweh, Hock Leong (2): > efi: export efi_capsule_supported() function symbol > efi: a misc char interface for user to update efi firmware > > drivers/firmware/efi/Kconfig | 10 > drivers/firmware/efi/Makefile | 1 > drivers/firmware/efi/capsule.c | 1 > drivers/firmware/efi/efi-capsule-loader.c | 356 +++++++++++++++++++++++++++++ > 4 files changed, 368 insertions(+) > create mode 100644 drivers/firmware/efi/efi-capsule-loader.c > Wilson. Same question as at V7. If you aren't supporting the MFH on Galileo then what capsule are you using with 0.75 BIOS ? From memory it won't work without the MFH included ... -- BOD -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | "Kweh, Hock Leong" <hock.leong.kweh@intel.com> |
|---|---|
| Date | 2015-10-08 06:00 +0200 |
| Subject | RE: [PATCH v8 0/2] Enable capsule loader interface for efi firmware updating |
| Message-ID | <qhax4-en-9@gated-at.bofh.it> |
| In reply to | #1241577 |
> -----Original Message----- > From: Bryan O'Donoghue [mailto:pure.logic@nexus-software.ie] > Sent: Wednesday, October 07, 2015 11:00 PM > > Wilson. > > Same question as at V7. If you aren't supporting the MFH on Galileo then > what capsule are you using with 0.75 BIOS ? From memory it won't work > without the MFH included ... > > -- > BOD As I mentioned before the Quark Security Header patch will not upstream to mainline and will be carried by Intel. So to test with Quark Platform, I will apply that patch into my kernel. Thanks & Regards, Wilson -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Bryan O'Donoghue <pure.logic@nexus-software.ie> |
|---|---|
| Date | 2015-10-08 13:10 +0200 |
| Subject | Re: [PATCH v8 0/2] Enable capsule loader interface for efi firmware updating |
| Message-ID | <qhhfc-1Eh-1@gated-at.bofh.it> |
| In reply to | #1241954 |
On 08/10/15 04:50, Kweh, Hock Leong wrote: >> -----Original Message----- >> From: Bryan O'Donoghue [mailto:pure.logic@nexus-software.ie] >> Sent: Wednesday, October 07, 2015 11:00 PM >> >> Wilson. >> >> Same question as at V7. If you aren't supporting the MFH on Galileo then >> what capsule are you using with 0.75 BIOS ? From memory it won't work >> without the MFH included ... >> >> -- >> BOD > > As I mentioned before the Quark Security Header patch will not upstream > to mainline and will be carried by Intel. So to test with Quark Platform, I will > apply that patch into my kernel. Ah OK gotcha. You're not actually going to provide support for capsule update in mainline for Quark then. I'll take that discussion to a separate thread. -- BOD -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web