From mboxrd@z Thu Jan 1 00:00:00 1970 From: Richard Retanubun Date: Wed, 19 Oct 2011 15:33:47 -0400 Subject: [U-Boot] [PATCH 2/2] disk: part_efi: fix **pgpt_pte == NULL In-Reply-To: <1319047487-9926-2-git-send-email-dianders@chromium.org> References: <1319047487-9926-1-git-send-email-dianders@chromium.org> <1319047487-9926-2-git-send-email-dianders@chromium.org> Message-ID: <4E9F261B.70609@RuggedCom.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 10/19/11 14:04, Doug Anderson wrote: > Code was setting **pgpt_pte == NULL, which meant that the pointer > to the gpt_pte would be stored at RAM address 00000000. This 'worked' > on T20 (SDRAM starts @ 0x00000000), but hung gpt/EFI access on T30 > (SDRAM starts @ 0x80000000). > > Signed-off-by: Tom Warren > Signed-off-by: Doug Anderson > --- > This was inspired by a change that I obtained through Tom Warren > at nVidia. > > disk/part_efi.c | 36 ++++++++++++++++++------------------ > 1 files changed, 18 insertions(+), 18 deletions(-) > > diff --git a/disk/part_efi.c b/disk/part_efi.c > index 6a139a9..08bd560 100644 > --- a/disk/part_efi.c > +++ b/disk/part_efi.c > @@ -121,7 +121,7 @@ static char *print_efiname(gpt_entry *pte) > void print_part_efi(block_dev_desc_t * dev_desc) > { > gpt_header gpt_head; > - gpt_entry **pgpt_pte = NULL; > + gpt_entry *gpt_pte = NULL; > int i = 0; ACK, good find! - Richard