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 canuck.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1RCWM0-0008V1-02 for barebox@lists.infradead.org; Sat, 08 Oct 2011 12:50:13 +0000 Date: Sat, 8 Oct 2011 14:50:09 +0200 From: Sascha Hauer Message-ID: <20111008125009.GK31404@pengutronix.de> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: 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-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: pandaboard boot issue To: Fabian van der Werf Cc: barebox@lists.infradead.org On Sat, Oct 08, 2011 at 01:52:17PM +0200, Fabian van der Werf wrote: > Hi, > > > I ran into problems trying to boot the pandaboard, which I think is the same > others reported on this list. After printing the pandaboard revision number > execution simply stops. I traced the problem to an unaligned access in > disk_register_partitions(). Barebox loads first sector of the disk in > memory. The offset of the partition table is 446 (decimal) which causes the > entire partition table to be unaligned. > > Below is a simple patch. Though, I am not sure whether this is the right > fix, because this would affect all arm platforms and I would expect that > such a bug would be noted earlier. I fiddled a little with the alignment bit > in the System Control Register, but without success. I have to admit that I > don't yet fully understand the behaviour of this bit. > > Does someone else have any ideas? Or is my patch acceptable? The patch looks good to me. Unfortunately your mailer wraps the lines so your patch won't apply. Can you fix and resend this? Thanks Sascha > > > Kind regards, > > Fabian van der Werf > > > From f98e14b971e0269251d8694be4c7b0254a30bc52 Mon Sep 17 00:00:00 2001 > From: Fabian van der Werf > Date: Sat, 8 Oct 2011 13:34:30 +0200 > Subject: [PATCH] Fixed unaligned access > > --- > drivers/ata/disk_drive.c | 15 ++++++++++----- > 1 files changed, 10 insertions(+), 5 deletions(-) > > diff --git a/drivers/ata/disk_drive.c b/drivers/ata/disk_drive.c > index 14b5e66..6a5dc87 100644 > --- a/drivers/ata/disk_drive.c > +++ b/drivers/ata/disk_drive.c > @@ -38,6 +38,7 @@ > #include > #include > #include > +#include > > /** > * Description of one partition table entry (D*S type) > @@ -97,25 +98,29 @@ static int disk_register_partitions(struct device_d > *dev, struct partition_entry > int part_order[4] = {0, 1, 2, 3}; > int i, rc; > char drive_name[16], partition_name[19]; > + u32 partition_start, partition_size; > > /* TODO order the partitions */ > > for (i = 0; i < 4; i++) { > + partition_start = > get_unaligned(&table[part_order[i]].partition_start); > + partition_size = > get_unaligned(&table[part_order[i]].partition_size); > + > sprintf(drive_name, "%s%d", dev->name, dev->id); > sprintf(partition_name, "%s%d.%d", dev->name, dev->id, i); > - if (table[part_order[i]].partition_start != 0) { > + if (partition_start != 0) { > #if 1 > /* ignore partitions we can't handle due to 32 bit limits */ > - if (table[part_order[i]].partition_start > 0x7fffff) > + if (partition_start > 0x7fffff) > continue; > - if (table[part_order[i]].partition_size > 0x7fffff) > + if (partition_size > 0x7fffff) > continue; > #endif > dev_dbg(dev, "Registering partition %s to drive %s\n", > partition_name, drive_name); > rc = devfs_add_partition(drive_name, > - table[part_order[i]].partition_start * SECTOR_SIZE, > - table[part_order[i]].partition_size * SECTOR_SIZE, > + partition_start * SECTOR_SIZE, > + partition_size * SECTOR_SIZE, > DEVFS_PARTITION_FIXED, partition_name); > if (rc != 0) > dev_err(dev, "Failed to register partition %s (%d)\n", > partition_name, rc); > -- > 1.7.0.4 > _______________________________________________ > 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