-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Somebody in the thread at some point said:
Hi Werner -
> - include/asm-arm/arch-s3c2410/irqs.h shouldn't contain device-specific
> changes (this is not a problem of the FIQ patch per se, but shows up
> here as well). To be tackled later.
I think maybe you misunderstood this -- the only edit in there
+/* offset for FIQ IRQ numbers - used by arm fiq.c */
+
+#define FIQ_START 0
is actually feeding the pre-existing ./arch/arm/kernel/fiq.c:
...
void enable_fiq(int fiq)
{
enable_irq(fiq + FIQ_START); <====
}
void disable_fiq(int fiq)
{
disable_irq(fiq + FIQ_START); <====
}
...
not any of my code. It's fixing an oversight that this define, required
to use arm arch fiq.c code on that machine, is not in there already. So
with this additional info I hope you agree this addition to that include
is totally correctamundo, although evidently busting it out into its own
patch would have been clearer.
> - "struct _fiq_ipc" in include/asm-arm/arch-s3c2410/fiq_ipc_gta02.h
> isn't really just the usual "internal use only" declaration, so it
> should probably be "struct _fiq_ipc". In general, I think we should
I think you missed an edit on that line :-)
> try to keep the number of _xxx or (better) __xxx items low and
> perhaps try to limit them to cases where there's a related xxx.
Fair enough, with the attached patch I changed it to use fiq_ipc_t which
is the method used in plenty of other places in the kernel (and indeed I
really ought to have used in the first place).
- -Andy
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org
iD8DBQFHouuoOjLpvpq7dMoRAldRAJ457GYzFRgu9UImKa3EBriraekjugCbB6TR
Z0VZR+L7+q6jVqTrNeEkzPA=
=mBEb
-----END PGP SIGNATURE-----
clean-fiq.patch
From: warmcat <[EMAIL PROTECTED]>
Change ipc struct type from _fiq_ipc to fiq_ipc_t
From: Andy Green <[EMAIL PROTECTED]>
Signed-off-by: Andy Green <[EMAIL PROTECTED]>
---
arch/arm/mach-s3c2440/mach-gta02.c | 2 +-
include/asm-arm/arch-s3c2410/fiq_ipc_gta02.h | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-s3c2440/mach-gta02.c b/arch/arm/mach-s3c2440/mach-gta02.c
index 763831b..4550e31 100644
--- a/arch/arm/mach-s3c2440/mach-gta02.c
+++ b/arch/arm/mach-s3c2440/mach-gta02.c
@@ -89,7 +89,7 @@
* the definition in there and include the same definition in your kernel
* module that wants to interoperate with your FIQ code.
*/
-struct _fiq_ipc fiq_ipc;
+struct fiq_ipc_t fiq_ipc;
EXPORT_SYMBOL(fiq_ipc);
#define DIVISOR_FROM_US(x) ((x) << 1)
diff --git a/include/asm-arm/arch-s3c2410/fiq_ipc_gta02.h b/include/asm-arm/arch-s3c2410/fiq_ipc_gta02.h
index b590c17..286ad34 100644
--- a/include/asm-arm/arch-s3c2410/fiq_ipc_gta02.h
+++ b/include/asm-arm/arch-s3c2410/fiq_ipc_gta02.h
@@ -20,7 +20,7 @@
#include <asm/plat-s3c/regs-timer.h>
-struct _fiq_ipc {
+struct fiq_ipc_t {
/* vibrator */
unsigned long vib_gpio_pin; /* which pin to meddle with */
u8 vib_pwm; /* 0 = OFF -- will ensure GPIO deasserted and stop FIQ */
@@ -28,7 +28,7 @@ struct _fiq_ipc {
};
/* actual definition lives in arch/arm/mach-s3c2440/fiq_c_isr.c */
-extern struct _fiq_ipc fiq_ipc;
+extern struct fiq_ipc_t fiq_ipc;
extern unsigned long _fiq_count_fiqs;
extern void fiq_kick(void); /* provoke a FIQ "immediately" */
