All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.