* [PATCH 0/5] Eliminate most of sparc_ksyms_*.c
@ 2009-01-06 14:58 Julian Calaby
2009-01-06 20:44 ` Sam Ravnborg
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Julian Calaby @ 2009-01-06 14:58 UTC (permalink / raw)
To: sparclinux
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.
Thanks,
Julian Calaby
^ 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
` (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
end of thread, other threads:[~2009-01-06 22:44 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
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.