Re: [PATCH] add arm64 UEFI Linux loader

2013-12-19 Thread Leif Lindholm
On Thu, Dec 19, 2013 at 08:06:03PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > >> if grub_errno is set you shouldn't override it. And please use the same > >> message verbatim (unexpected end of file): it decreases work for > >> translators > > > > Well, since I have already checked th

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-19 Thread Vladimir 'φ-coder/phcoder' Serbinenko
>> if grub_errno is set you shouldn't override it. And please use the same >> message verbatim (unexpected end of file): it decreases work for translators > > Well, since I have already checked the file size before this, any > failure will be an i/o error - so I'll just goto fail instead. > If th

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-19 Thread Leif Lindholm
On Wed, Dec 18, 2013 at 06:23:06PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > >>> + if (!orig_fdt) > >>> +{ > >>> + fdt_loaded = 0; > >>> + /* Look for FDT in UEFI config tables. */ > >>> + tables = grub_efi_system_table->configuration_table; > >>> + > >>> + for

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-18 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 18.12.2013 17:54, Leif Lindholm wrote: > On Mon, Dec 16, 2013 at 10:34:51PM +0100, Vladimir 'φ-coder/phcoder' > Serbinenko wrote: >>> +static void >>> +get_fdt (void) >>> +{ >>> + grub_efi_configuration_table_t *tables; >>> + unsigned int i; >>> + int fdt_loaded; >>> + int size; >>> + >>> +

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-18 Thread Andrey Borzenkov
В Wed, 18 Dec 2013 17:54:39 +0100 Leif Lindholm пишет: > > > > + if (!loaded) > > > +{ > > > + grub_error (GRUB_ERR_BAD_ARGUMENT, > > > + N_("you need to load the kernel first")); > > > + goto fail; > > > +} > > > + > > > + files = grub_zalloc (argc * sizeof (files[

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-18 Thread Leif Lindholm
On Mon, Dec 16, 2013 at 10:34:51PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > > +static void > > +get_fdt (void) > > +{ > > + grub_efi_configuration_table_t *tables; > > + unsigned int i; > > + int fdt_loaded; > > + int size; > > + > > + if (!orig_fdt) > > +{ > > + fdt_loa

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-16 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 16.12.2013 17:24, Leif Lindholm wrote: > On Mon, Dec 16, 2013 at 08:13:01PM +0400, Andrey Borzenkov wrote: >> > В Mon, 16 Dec 2013 14:55:51 +0100 >> > Leif Lindholm пишет: >> > >>> > > This loader supports booting using the UEFI stub method only. >> > >> > And where is patch? :) > Doh! > Here

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-16 Thread Vladimir 'φ-coder/phcoder' Serbinenko
On 16.12.2013 14:55, Leif Lindholm wrote: > This loader supports booting using the UEFI stub method only. > I don't see any loader attached to mail at all. > Currently debugging an issue with using the load_image method, > so usurping the current GRUB image context instead. > > __

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-16 Thread Leif Lindholm
On Mon, Dec 16, 2013 at 08:13:01PM +0400, Andrey Borzenkov wrote: > В Mon, 16 Dec 2013 14:55:51 +0100 > Leif Lindholm пишет: > > > This loader supports booting using the UEFI stub method only. > > And where is patch? :) Doh! Here. >From 03ebc609f33df1ddcdf10668cacf69f1861a5acf Mon Sep 17 00:00:

Re: [PATCH] add arm64 UEFI Linux loader

2013-12-16 Thread Andrey Borzenkov
В Mon, 16 Dec 2013 14:55:51 +0100 Leif Lindholm пишет: > This loader supports booting using the UEFI stub method only. > And where is patch? :) ___ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel