* [PATCH 0/2] cleanup iosapic.c
@ 2006-03-24 6:43 Satoru Takeuchi
2006-03-24 10:26 ` Jes Sorensen
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Satoru Takeuchi @ 2006-03-24 6:43 UTC (permalink / raw)
To: linux-ia64
Hi,
These patches clean up arch/ia64/kernel/iosapic.c.
The changes are as follows:
- [PATCH 1/2] correct some messages and fixes some some minor typos.
- [PATCH 2/2] simplify some condition checks.
For more specifc information, please refer to the header of each patch.
These patches are against 2.6.16. I tested them on Tiger4.
Thanks,
Satoru Takeuchi
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH 0/2] cleanup iosapic.c
2006-03-24 6:43 [PATCH 0/2] cleanup iosapic.c Satoru Takeuchi
@ 2006-03-24 10:26 ` Jes Sorensen
2006-03-27 8:09 ` Satoru Takeuchi
2006-03-27 8:21 ` Jes Sorensen
2 siblings, 0 replies; 4+ messages in thread
From: Jes Sorensen @ 2006-03-24 10:26 UTC (permalink / raw)
To: linux-ia64
>>>>> "Satoru" = Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com> writes:
Satoru> Hi, These patches clean up arch/ia64/kernel/iosapic.c. The
Satoru> changes are as follows:
Satoru> - [PATCH 1/2] correct some messages and fixes some some minor
Satoru> typos.
Hi Satoru,
When you are in there fixing those types of issues, it would be nice
if you also fixed the parts you are touching to comply with
codingstyle at the same time (80 columns and tab indentation). Not a
big thing, just seemed a good time to resolve it.
Cheers,
Jes
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 0/2] cleanup iosapic.c
2006-03-24 6:43 [PATCH 0/2] cleanup iosapic.c Satoru Takeuchi
2006-03-24 10:26 ` Jes Sorensen
@ 2006-03-27 8:09 ` Satoru Takeuchi
2006-03-27 8:21 ` Jes Sorensen
2 siblings, 0 replies; 4+ messages in thread
From: Satoru Takeuchi @ 2006-03-27 8:09 UTC (permalink / raw)
To: linux-ia64
Hi Jes,
At 24 Mar 2006 05:26:02 -0500,
Jes Sorensen wrote:
>
> >>>>> "Satoru" = Satoru Takeuchi <takeuchi_satoru@jp.fujitsu.com> writes:
>
> Satoru> Hi, These patches clean up arch/ia64/kernel/iosapic.c. The
> Satoru> changes are as follows:
>
> Satoru> - [PATCH 1/2] correct some messages and fixes some some minor
> Satoru> typos.
>
> Hi Satoru,
>
> When you are in there fixing those types of issues, it would be nice
> if you also fixed the parts you are touching to comply with
> codingstyle at the same time (80 columns and tab indentation). Not a
> big thing, just seemed a good time to resolve it.
>
> Cheers,
> Jes
OK, I fixed as you said. New patch is here.
These patches clean up arch/ia64/kernel/iosapic.c.
The changes are as follows:
- [PATCH 1/2] correct some messages and fixes some minor things.
- [PATCH 2/2] simplify some condition checks.
For more specifc information, please refer to the header of each patch.
These patches are against 2.6.16. I tested them on Tiger4.
Thanks,
Satoru Takeuchi
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 0/2] cleanup iosapic.c
2006-03-24 6:43 [PATCH 0/2] cleanup iosapic.c Satoru Takeuchi
2006-03-24 10:26 ` Jes Sorensen
2006-03-27 8:09 ` Satoru Takeuchi
@ 2006-03-27 8:21 ` Jes Sorensen
2 siblings, 0 replies; 4+ messages in thread
From: Jes Sorensen @ 2006-03-27 8:21 UTC (permalink / raw)
To: linux-ia64
Satoru Takeuchi wrote:
> OK, I fixed as you said. New patch is here.
>
> These patches clean up arch/ia64/kernel/iosapic.c.
> The changes are as follows:
>
> - [PATCH 1/2] correct some messages and fixes some minor things.
Looks good to me!
Cheers,
Jes
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2006-03-27 8:21 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-03-24 6:43 [PATCH 0/2] cleanup iosapic.c Satoru Takeuchi
2006-03-24 10:26 ` Jes Sorensen
2006-03-27 8:09 ` Satoru Takeuchi
2006-03-27 8:21 ` Jes Sorensen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox