From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1XluuC-0000OH-BY for barebox@lists.infradead.org; Wed, 05 Nov 2014 07:21:25 +0000 Date: Wed, 5 Nov 2014 08:21:00 +0100 From: Sascha Hauer Message-ID: <20141105072100.GH30369@pengutronix.de> References: <1415094176-26435-1-git-send-email-l.stach@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1415094176-26435-1-git-send-email-l.stach@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 01/10] efi: add proper reset hook To: Lucas Stach Cc: barebox@lists.infradead.org On Tue, Nov 04, 2014 at 10:42:47AM +0100, Lucas Stach wrote: > This allows to actually reset the system from barebox > instead of dropping back into the EFI firmware. Now we can reset the system, but not go back into the EFI firmware anymore. Should we introduce a command to make that possible? Sascha > > Signed-off-by: Lucas Stach > --- > arch/efi/efi/efi.c | 2 +- > include/efi.h | 11 +++++++---- > 2 files changed, 8 insertions(+), 5 deletions(-) > > diff --git a/arch/efi/efi/efi.c b/arch/efi/efi/efi.c > index 7de8ec82e8fa..af3fc4366ba1 100644 > --- a/arch/efi/efi/efi.c > +++ b/arch/efi/efi/efi.c > @@ -248,7 +248,7 @@ console_initcall(efi_console_init); > > void reset_cpu(unsigned long addr) > { > - BS->exit(efi_parent_image, EFI_SUCCESS, 0, NULL); > + RT->reset_system(EFI_RESET_WARM, EFI_SUCCESS, 0, NULL); > > while(1); > } > diff --git a/include/efi.h b/include/efi.h > index 4ad9f69237e7..70fe254dbafa 100644 > --- a/include/efi.h > +++ b/include/efi.h > @@ -255,9 +255,11 @@ extern efi_boot_services_t *BS; > /* > * Types and defines for EFI ResetSystem > */ > -#define EFI_RESET_COLD 0 > -#define EFI_RESET_WARM 1 > -#define EFI_RESET_SHUTDOWN 2 > +typedef enum { > + EFI_RESET_COLD = 0, > + EFI_RESET_WARM = 1, > + EFI_RESET_SHUTDOWN = 2 > +} efi_reset_type_t; > > /* > * EFI Runtime Services table > @@ -279,7 +281,8 @@ typedef struct { > s16 *variable_name, efi_guid_t *vendor); > void *set_variable; > void *get_next_high_mono_count; > - void *reset_system; > + void (EFIAPI *reset_system)(efi_reset_type_t reset_type, efi_status_t reset_status, > + unsigned long data_size, void *reset_data); > void *update_capsule; > void *query_capsule_caps; > void *query_variable_info; > -- > 2.1.1 > > > _______________________________________________ > barebox mailing list > barebox@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/barebox > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox