* [PATCH 0/6] sparc64 io refactoring
@ 2014-07-20 11:35 Sam Ravnborg
2014-07-20 11:43 ` Sam Ravnborg
2014-07-22 4:56 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Sam Ravnborg @ 2014-07-20 11:35 UTC (permalink / raw)
To: sparclinux
The following patchset refactor io_64.h. This was triggered by
another patchset by Thierry Reding that updates the the generic
io.h such that it may be used by sparc64 and thus make sparc
and sparc64 more equal in this area.
Before attempting to introduce the generic version it was
necessary to clean up the current state to avoid any mistakes.
The updates from Thierry needs to go in before I will attempt
introducing the generic io.h.
The sparcspkr used inb()/outb() primitives
for of_ioremap() memory. Update this driver to use sbus_ variants.
This change was triggered by a number of new warnings that would otherwise
have been seen when dropping the macro indirections.
Likewise PeeCeeI.c had some warnings that was fixed by
using another IO varaint - which simplified the code too.
The patchs has been generated with the --histogram option
to make the patch that re-order functions in io_64.h
more readable.
So to see the same patches locally you need this option too.
I have not yet any working sparc64 HW - so this is not tested.
But I brought a SUN NetraX1 that I am working on gettting
running. That may allow me to do some minimal tests in the future.
Sam
Sam Ravnborg (6):
sparcspkr: use sbus_*() primitives for IO
sparc64: update IO access functions in PeeCeeI
sparc64: remove macro indirection in io_64.h
sparc64: drop unused SLOW_DOWN_IO definitions
sparc64: reorder functions in io_64.h
sparc64: avoid code duplication in io_64.h
arch/sparc/include/asm/io_64.h | 381 ++++++++++++++++-------------------------
arch/sparc/lib/PeeCeeI.c | 36 ++--
drivers/input/misc/sparcspkr.c | 22 +--
3 files changed, 174 insertions(+), 265 deletions(-)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/6] sparc64 io refactoring
2014-07-20 11:35 [PATCH 0/6] sparc64 io refactoring Sam Ravnborg
@ 2014-07-20 11:43 ` Sam Ravnborg
2014-07-22 4:56 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Sam Ravnborg @ 2014-07-20 11:43 UTC (permalink / raw)
To: sparclinux
On Sun, Jul 20, 2014 at 01:35:02PM +0200, Sam Ravnborg wrote:
> The following patchset refactor io_64.h. This was triggered by
> another patchset by Thierry Reding that updates the the generic
> io.h such that it may be used by sparc64 and thus make sparc
> and sparc64 more equal in this area.
>
> Before attempting to introduce the generic version it was
> necessary to clean up the current state to avoid any mistakes.
> The updates from Thierry needs to go in before I will attempt
> introducing the generic io.h.
>
> The sparcspkr used inb()/outb() primitives
> for of_ioremap() memory. Update this driver to use sbus_ variants.
> This change was triggered by a number of new warnings that would otherwise
> have been seen when dropping the macro indirections.
> Likewise PeeCeeI.c had some warnings that was fixed by
> using another IO varaint - which simplified the code too.
>
> The patchs has been generated with the --histogram option
> to make the patch that re-order functions in io_64.h
> more readable.
> So to see the same patches locally you need this option too.
>
> I have not yet any working sparc64 HW - so this is not tested.
> But I brought a SUN NetraX1 that I am working on gettting
> running. That may allow me to do some minimal tests in the future.
I checked the size impact on these patches on a defconfig kernel:
Before:
$ size vmlinux
text data bss dec hex filename
5838466 478776 6876081 13193323 c9506b vmlinux
After:
$size vmlinux
text data bss dec hex filename
5837570 478776 6876081 13192427 c94ceb vmlinux
So in other words a reduction on 896 bytes in text.
This it not much but my main concern was if the kernel
grow bigger do to these changes - which was not the case.
I did not check but I assume the most of the reduction in
size come from the changes in PeeCeeI.c.
Sam
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/6] sparc64 io refactoring
2014-07-20 11:35 [PATCH 0/6] sparc64 io refactoring Sam Ravnborg
2014-07-20 11:43 ` Sam Ravnborg
@ 2014-07-22 4:56 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2014-07-22 4:56 UTC (permalink / raw)
To: sparclinux
From: Sam Ravnborg <sam@ravnborg.org>
Date: Sun, 20 Jul 2014 13:43:13 +0200
> On Sun, Jul 20, 2014 at 01:35:02PM +0200, Sam Ravnborg wrote:
>> The following patchset refactor io_64.h. This was triggered by
>> another patchset by Thierry Reding that updates the the generic
>> io.h such that it may be used by sparc64 and thus make sparc
>> and sparc64 more equal in this area.
>>
>> Before attempting to introduce the generic version it was
>> necessary to clean up the current state to avoid any mistakes.
>> The updates from Thierry needs to go in before I will attempt
>> introducing the generic io.h.
>>
>> The sparcspkr used inb()/outb() primitives
>> for of_ioremap() memory. Update this driver to use sbus_ variants.
>> This change was triggered by a number of new warnings that would otherwise
>> have been seen when dropping the macro indirections.
>> Likewise PeeCeeI.c had some warnings that was fixed by
>> using another IO varaint - which simplified the code too.
>>
>> The patchs has been generated with the --histogram option
>> to make the patch that re-order functions in io_64.h
>> more readable.
>> So to see the same patches locally you need this option too.
>>
>> I have not yet any working sparc64 HW - so this is not tested.
>> But I brought a SUN NetraX1 that I am working on gettting
>> running. That may allow me to do some minimal tests in the future.
>
> I checked the size impact on these patches on a defconfig kernel:
>
> Before:
> $ size vmlinux
> text data bss dec hex filename
> 5838466 478776 6876081 13193323 c9506b vmlinux
>
> After:
> $size vmlinux
> text data bss dec hex filename
> 5837570 478776 6876081 13192427 c94ceb vmlinux
>
> So in other words a reduction on 896 bytes in text.
> This it not much but my main concern was if the kernel
> grow bigger do to these changes - which was not the case.
>
> I did not check but I assume the most of the reduction in
> size come from the changes in PeeCeeI.c.
Looks great, applied to sparc-next, thanks Sam!
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-07-22 4:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-20 11:35 [PATCH 0/6] sparc64 io refactoring Sam Ravnborg
2014-07-20 11:43 ` Sam Ravnborg
2014-07-22 4:56 ` David Miller
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.