* [PATCH] SA1100: fix building of h3100
@ 2009-10-05 17:50 Dmitry Artamonow
2009-10-06 10:48 ` Russell King - ARM Linux
2009-10-06 10:53 ` Kristoffer Ericson
0 siblings, 2 replies; 8+ messages in thread
From: Dmitry Artamonow @ 2009-10-05 17:50 UTC (permalink / raw)
To: linux-arm-kernel
Both iPaqs h3100 and h3600 currently share the same source
file - h3600.c But Makefile builds it only if CONFIG_SA1100_H3600
selected, so selecting just CONFIG_SA1100_H3100 results in
"no machine record defined" message and aborted compilation.
Fix it.
Signed-off-by: Dmitry Artamonow <mad_soft@inbox.ru>
---
Well, it left unnoticed by me for a while, just because I always build
kernels with _both_ machines compiled in.
arch/arm/mach-sa1100/Makefile | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-sa1100/Makefile b/arch/arm/mach-sa1100/Makefile
index 8a5546e..bb7b819 100644
--- a/arch/arm/mach-sa1100/Makefile
+++ b/arch/arm/mach-sa1100/Makefile
@@ -25,6 +25,7 @@ led-$(CONFIG_SA1100_CERF) += leds-cerf.o
obj-$(CONFIG_SA1100_COLLIE) += collie.o
+obj-$(CONFIG_SA1100_H3100) += h3600.o
obj-$(CONFIG_SA1100_H3600) += h3600.o
obj-$(CONFIG_SA1100_HACKKIT) += hackkit.o
--
1.6.3.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] SA1100: fix building of h3100
2009-10-05 17:50 [PATCH] SA1100: fix building of h3100 Dmitry Artamonow
@ 2009-10-06 10:48 ` Russell King - ARM Linux
2009-10-06 14:31 ` Dmitry Artamonow
2009-10-06 10:53 ` Kristoffer Ericson
1 sibling, 1 reply; 8+ messages in thread
From: Russell King - ARM Linux @ 2009-10-06 10:48 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Oct 05, 2009 at 09:50:34PM +0400, Dmitry Artamonow wrote:
> Both iPaqs h3100 and h3600 currently share the same source
> file - h3600.c But Makefile builds it only if CONFIG_SA1100_H3600
> selected, so selecting just CONFIG_SA1100_H3100 results in
> "no machine record defined" message and aborted compilation.
Maybe this is what SA1100_H3XXX was meant for? Since that symbol's now
gone, we might as well fix it this way.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] SA1100: fix building of h3100
2009-10-05 17:50 [PATCH] SA1100: fix building of h3100 Dmitry Artamonow
2009-10-06 10:48 ` Russell King - ARM Linux
@ 2009-10-06 10:53 ` Kristoffer Ericson
2009-10-06 11:28 ` Russell King - ARM Linux
1 sibling, 1 reply; 8+ messages in thread
From: Kristoffer Ericson @ 2009-10-06 10:53 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 5 Oct 2009 21:50:34 +0400
Dmitry Artamonow <mad_soft@inbox.ru> wrote:
> Both iPaqs h3100 and h3600 currently share the same source
> file - h3600.c But Makefile builds it only if CONFIG_SA1100_H3600
> selected, so selecting just CONFIG_SA1100_H3100 results in
> "no machine record defined" message and aborted compilation.
>
> Fix it.
Acked
>
> Signed-off-by: Dmitry Artamonow <mad_soft@inbox.ru>
> ---
> Well, it left unnoticed by me for a while, just because I always build
> kernels with _both_ machines compiled in.
>
> arch/arm/mach-sa1100/Makefile | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-sa1100/Makefile b/arch/arm/mach-sa1100/Makefile
> index 8a5546e..bb7b819 100644
> --- a/arch/arm/mach-sa1100/Makefile
> +++ b/arch/arm/mach-sa1100/Makefile
> @@ -25,6 +25,7 @@ led-$(CONFIG_SA1100_CERF) += leds-cerf.o
>
> obj-$(CONFIG_SA1100_COLLIE) += collie.o
>
> +obj-$(CONFIG_SA1100_H3100) += h3600.o
> obj-$(CONFIG_SA1100_H3600) += h3600.o
>
> obj-$(CONFIG_SA1100_HACKKIT) += hackkit.o
> --
> 1.6.3.4
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Kristoffer Ericson <kristoffer.ericson@gmail.com>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] SA1100: fix building of h3100
2009-10-06 10:53 ` Kristoffer Ericson
@ 2009-10-06 11:28 ` Russell King - ARM Linux
2009-10-06 12:51 ` Kristoffer Ericson
2009-10-06 14:38 ` Dmitry Artamonow
0 siblings, 2 replies; 8+ messages in thread
From: Russell King - ARM Linux @ 2009-10-06 11:28 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 06, 2009 at 12:53:18PM +0200, Kristoffer Ericson wrote:
> On Mon, 5 Oct 2009 21:50:34 +0400
> Dmitry Artamonow <mad_soft@inbox.ru> wrote:
>
> > Both iPaqs h3100 and h3600 currently share the same source
> > file - h3600.c But Makefile builds it only if CONFIG_SA1100_H3600
> > selected, so selecting just CONFIG_SA1100_H3100 results in
> > "no machine record defined" message and aborted compilation.
> >
> > Fix it.
>
> Acked
If there are people seriously looking at H3[16]00 stuff, it would be
a good idea to move it over to use the proper GPIO support rather
than continue to rely upon its own version.
Also maybe splitting it into h3100.c and h3600.c, with (if any) common
bits in h3xxx.c might be an idea?
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] SA1100: fix building of h3100
2009-10-06 11:28 ` Russell King - ARM Linux
@ 2009-10-06 12:51 ` Kristoffer Ericson
2009-10-06 13:05 ` Albert ARIBAUD
2009-10-06 14:38 ` Dmitry Artamonow
1 sibling, 1 reply; 8+ messages in thread
From: Kristoffer Ericson @ 2009-10-06 12:51 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, 6 Oct 2009 12:28:31 +0100
Russell King - ARM Linux <linux@arm.linux.org.uk> wrote:
> On Tue, Oct 06, 2009 at 12:53:18PM +0200, Kristoffer Ericson wrote:
> > On Mon, 5 Oct 2009 21:50:34 +0400
> > Dmitry Artamonow <mad_soft@inbox.ru> wrote:
> >
> > > Both iPaqs h3100 and h3600 currently share the same source
> > > file - h3600.c But Makefile builds it only if CONFIG_SA1100_H3600
> > > selected, so selecting just CONFIG_SA1100_H3100 results in
> > > "no machine record defined" message and aborted compilation.
> > >
> > > Fix it.
> >
> > Acked
>
> If there are people seriously looking at H3[16]00 stuff, it would be
> a good idea to move it over to use the proper GPIO support rather
> than continue to rely upon its own version.
Im not looking specificly at the IPAQ stuff, but it comes in
contact with my jornada work. I agree that moving it into general
GPIO support should be the priority.
>
> Also maybe splitting it into h3100.c and h3600.c, with (if any) common
> bits in h3xxx.c might be an idea?
--
Kristoffer Ericson <kristoffer.ericson@gmail.com>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] SA1100: fix building of h3100
2009-10-06 12:51 ` Kristoffer Ericson
@ 2009-10-06 13:05 ` Albert ARIBAUD
0 siblings, 0 replies; 8+ messages in thread
From: Albert ARIBAUD @ 2009-10-06 13:05 UTC (permalink / raw)
To: linux-arm-kernel
Kristoffer Ericson a ?crit :
>> If there are people seriously looking at H3[16]00 stuff, it would be
>> a good idea to move it over to use the proper GPIO support rather
>> than continue to rely upon its own version.
>
> Im not looking specificly at the IPAQ stuff, but it comes in
> contact with my jornada work. I agree that moving it into general
> GPIO support should be the priority.
I do have an iPAQ 3630 and am also interested in moving it to general
GPIO support. Maybe I can help there?
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] SA1100: fix building of h3100
2009-10-06 10:48 ` Russell King - ARM Linux
@ 2009-10-06 14:31 ` Dmitry Artamonow
0 siblings, 0 replies; 8+ messages in thread
From: Dmitry Artamonow @ 2009-10-06 14:31 UTC (permalink / raw)
To: linux-arm-kernel
On 11:48 Tue 06 Oct , Russell King - ARM Linux wrote:
> On Mon, Oct 05, 2009 at 09:50:34PM +0400, Dmitry Artamonow wrote:
> > Both iPaqs h3100 and h3600 currently share the same source
> > file - h3600.c But Makefile builds it only if CONFIG_SA1100_H3600
> > selected, so selecting just CONFIG_SA1100_H3100 results in
> > "no machine record defined" message and aborted compilation.
>
> Maybe this is what SA1100_H3XXX was meant for? Since that symbol's now
> gone, we might as well fix it this way.
Yes, thanks. To be clear - this is a fix meant for 2.6.32.
I think h3600.c should be split in the future anyway, so there's really
no need to revive SA1100_H3XXX.
--
Best regards,
Dmitry "MAD" Artamonow
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] SA1100: fix building of h3100
2009-10-06 11:28 ` Russell King - ARM Linux
2009-10-06 12:51 ` Kristoffer Ericson
@ 2009-10-06 14:38 ` Dmitry Artamonow
1 sibling, 0 replies; 8+ messages in thread
From: Dmitry Artamonow @ 2009-10-06 14:38 UTC (permalink / raw)
To: linux-arm-kernel
On 12:28 Tue 06 Oct , Russell King - ARM Linux wrote:
> If there are people seriously looking at H3[16]00 stuff, it would be
> a good idea to move it over to use the proper GPIO support rather
> than continue to rely upon its own version.
Yes, that's that I'm working on - the plan is to get rid of those horrible
abstracted ipaq egpios, and use gpiolib functions instead (reusing
htc-egpio driver for gpio expander)
I'll try to finish and clean up my patches soon, and submit them for
review, so hopefully they could get into 2.6.33
> Also maybe splitting it into h3100.c and h3600.c, with (if any) common
> bits in h3xxx.c might be an idea?
Yes, as I said, that's the direction I'd like to go too.
After conversion to gpiolib and real gpios there probably will left
not much common at all.
--
Best regards,
Dmitry "MAD" Artamonow
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2009-10-06 14:38 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-10-05 17:50 [PATCH] SA1100: fix building of h3100 Dmitry Artamonow
2009-10-06 10:48 ` Russell King - ARM Linux
2009-10-06 14:31 ` Dmitry Artamonow
2009-10-06 10:53 ` Kristoffer Ericson
2009-10-06 11:28 ` Russell King - ARM Linux
2009-10-06 12:51 ` Kristoffer Ericson
2009-10-06 13:05 ` Albert ARIBAUD
2009-10-06 14:38 ` Dmitry Artamonow
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.