* Re: [PATCH 0/5] Eliminate most of sparc_ksyms_*.c
2009-01-06 14:58 [PATCH 0/5] Eliminate most of sparc_ksyms_*.c Julian Calaby
@ 2009-01-06 20:44 ` Sam Ravnborg
2009-01-06 20:50 ` David Miller
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Sam Ravnborg @ 2009-01-06 20:44 UTC (permalink / raw)
To: sparclinux
On Wed, Jan 07, 2009 at 01:58:04AM +1100, Julian Calaby wrote:
> Hi all,
>
> This patch set eliminates most of the definitions within
> arch/sparc/kernel/sparc_ksyms_*.c by moving them to their rightful places.
>
> This is done within patches 1-3.
>
> Note that this work was originally performed by Sam Ravnborg and these
> patches have been rebased and added to by myself.
>
> I've tried to reflect this in the commit messages but may have gotten it
> a little wrong.
>
> Patches 4 and 5 are cleanups that I had in my tree for an eventual
> oplib_*.h unification - but as this screws up my original patches and
> they're so small, I figured this'd be as good a time as any to send them.
Hi Julian.
I appreciate you took over this patch-set - thanks!
I have not yet looked over the patches. But at first glance it looked
like they were word-wrapped??
Anyway - I will let you know if/when I get to it.
Sam
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 0/5] Eliminate most of sparc_ksyms_*.c
2009-01-06 14:58 [PATCH 0/5] Eliminate most of sparc_ksyms_*.c Julian Calaby
2009-01-06 20:44 ` Sam Ravnborg
@ 2009-01-06 20:50 ` David Miller
2009-01-06 20:59 ` Sam Ravnborg
2009-01-06 22:44 ` Julian Calaby
3 siblings, 0 replies; 5+ messages in thread
From: David Miller @ 2009-01-06 20:50 UTC (permalink / raw)
To: sparclinux
From: Sam Ravnborg <sam@ravnborg.org>
Date: Tue, 6 Jan 2009 21:44:04 +0100
> I appreciate you took over this patch-set - thanks!
>
> I have not yet looked over the patches. But at first glance it looked
> like they were word-wrapped??
Also initial spaces were transformed to 2 spaces in the
patch.
This happened to all his previous patch postings too, but
I just silently fixed them up by hand. :-/
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/5] Eliminate most of sparc_ksyms_*.c
2009-01-06 14:58 [PATCH 0/5] Eliminate most of sparc_ksyms_*.c Julian Calaby
2009-01-06 20:44 ` Sam Ravnborg
2009-01-06 20:50 ` David Miller
@ 2009-01-06 20:59 ` Sam Ravnborg
2009-01-06 22:44 ` Julian Calaby
3 siblings, 0 replies; 5+ messages in thread
From: Sam Ravnborg @ 2009-01-06 20:59 UTC (permalink / raw)
To: sparclinux
On Tue, Jan 06, 2009 at 12:50:48PM -0800, David Miller wrote:
> From: Sam Ravnborg <sam@ravnborg.org>
> Date: Tue, 6 Jan 2009 21:44:04 +0100
>
> > I appreciate you took over this patch-set - thanks!
> >
> > I have not yet looked over the patches. But at first glance it looked
> > like they were word-wrapped??
>
> Also initial spaces were transformed to 2 spaces in the
> patch.
>
> This happened to all his previous patch postings too, but
> I just silently fixed them up by hand. :-/
Julian - can I ask you to read Documentation/email-clients.txt
and fix your setup of Thunderbird and resumit the serie.
You can then include your fix to one of your patches as
you already posted.
Thanks,
Sam
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/5] Eliminate most of sparc_ksyms_*.c
2009-01-06 14:58 [PATCH 0/5] Eliminate most of sparc_ksyms_*.c Julian Calaby
` (2 preceding siblings ...)
2009-01-06 20:59 ` Sam Ravnborg
@ 2009-01-06 22:44 ` Julian Calaby
3 siblings, 0 replies; 5+ messages in thread
From: Julian Calaby @ 2009-01-06 22:44 UTC (permalink / raw)
To: sparclinux
On Wed, Jan 7, 2009 at 07:50, David Miller <davem@davemloft.net> wrote:
> From: Sam Ravnborg <sam@ravnborg.org>
> Date: Tue, 6 Jan 2009 21:44:04 +0100
>
>> I appreciate you took over this patch-set - thanks!
>>
>> I have not yet looked over the patches. But at first glance it looked
>> like they were word-wrapped??
>
> Also initial spaces were transformed to 2 spaces in the
> patch.
>
> This happened to all his previous patch postings too, but
> I just silently fixed them up by hand. :-/
Sodding heck. I thought that as nobody had complained before,
everything was working as it should.
Ok, I'll resubmit with my fixes (and there's another hunk that needs
to be stuck into patch 3 now too, fecking build test failures) some
time today, hopefully at lunchtime.
Hopefully setting up Thunderbird (well, Icedove) will be pain free. =)
Thanks,
--
Julian Calaby
Email: julian.calaby@gmail.com
^ permalink raw reply [flat|nested] 5+ messages in thread