[uClinux-dev] [PATCH] unaligned.h fix for M68000 core

[Luis Alves]

Hi all,

This patch fixes unaligned memory access for the 68000 core based cpu's.

Some time ago, my cpu (68000) was raising address/bus error's when mounting cifs shares (didn't bother to debug it at the time).
After developing the MMC/SD card driver I was having the same issue when mounting the vfat fs.

I've traced the issue down to the 'unaligned.h' file. (I guess nobody has ever used unaligned.h back in the 68328 'era'.


Regards,
Luis Alves


Signed-off-by: Luis Alves <ljalvs at gmail.com>
---
arch/m68k/include/asm/unaligned.h | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/m68k/include/asm/unaligned.h b/arch/m68k/include/asm/unaligned.h
index 019caa7..f4043ae 100644
--- a/arch/m68k/include/asm/unaligned.h
+++ b/arch/m68k/include/asm/unaligned.h
@@ -2,7 +2,7 @@
#define _ASM_M68K_UNALIGNED_H


-#ifdef CONFIG_COLDFIRE
+#if defined(CONFIG_COLDFIRE) || defined(CONFIG_M68000)
#include <linux/unaligned/be_struct.h>
#include <linux/unaligned/le_byteshift.h>
#include <linux/unaligned/generic.h>
--
1.7.5.4

[Greg Ungerer]

Hi Luis,

On 14/05/12 02:39, Luis Alves wrote:
> This patch fixes unaligned memory access for the 68000 core based cpu's.
>
> Some time ago, my cpu (68000) was raising address/bus error's when mounting cifs shares (didn't bother to debug it at the time).
> After developing the MMC/SD card driver I was having the same issue when mounting the vfat fs.
>
> I've traced the issue down to the 'unaligned.h' file. (I guess nobody has ever used unaligned.h back in the 68328 'era'.

Looks good, thanks.
I'll apply it to the m68knommu git tree.

Regards
Greg


> Signed-off-by: Luis Alves<ljalvs at gmail.com>
> ---
> arch/m68k/include/asm/unaligned.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/m68k/include/asm/unaligned.h b/arch/m68k/include/asm/unaligned.h
> index 019caa7..f4043ae 100644
> --- a/arch/m68k/include/asm/unaligned.h
> +++ b/arch/m68k/include/asm/unaligned.h
> @@ -2,7 +2,7 @@
> #define _ASM_M68K_UNALIGNED_H
>
>
> -#ifdef CONFIG_COLDFIRE
> +#if defined(CONFIG_COLDFIRE) || defined(CONFIG_M68000)
> #include<linux/unaligned/be_struct.h>
> #include<linux/unaligned/le_byteshift.h>
> #include<linux/unaligned/generic.h>


--
------------------------------------------------------------------------
Greg Ungerer -- Principal Engineer EMAIL: gerg at snapgear.com
SnapGear Group, McAfee PHONE: [phone]
8 Gardner Close FAX: [phone]
Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com