Re: [updated PATCH] remove 555 unneeded #includes of sched.h

2007-01-22 Thread Oleg Verych
On 2006-12-29, Tim Schmielau wrote:
[]
> OK, building 2.6.20-rc2-mm1 with all 59 configs from arch/arm/configs 
> with and w/o the patch indeed found one mysterious #include that may not 
> be removed. Thanks, Russell!
>
> Andrew, please use the attached patch instead of the previous one, it also 
> has a slightly better patch description.

Great job!

About patch. To make it smaller, i think, you better use less
"unified context" lines `diff -u1'.

Nice done! 


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [updated PATCH] remove 555 unneeded #includes of sched.h

2007-01-22 Thread Oleg Verych
On 2006-12-29, Tim Schmielau wrote:
[]
 OK, building 2.6.20-rc2-mm1 with all 59 configs from arch/arm/configs 
 with and w/o the patch indeed found one mysterious #include that may not 
 be removed. Thanks, Russell!

 Andrew, please use the attached patch instead of the previous one, it also 
 has a slightly better patch description.

Great job!

About patch. To make it smaller, i think, you better use less
unified context lines `diff -u1'.

Nice done! 


-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [updated PATCH] remove 555 unneeded #includes of sched.h

2006-12-30 Thread Russell King
On Fri, Dec 29, 2006 at 11:23:58AM +0100, Tim Schmielau wrote:
> On Thu, 28 Dec 2006, Tim Schmielau wrote:
> > On Thu, 28 Dec 2006, Russell King wrote:
> > 
> > > To cover these, you need to build at least rpc_defconfig, 
> > > lubbock_defconfig,
> > > netwinder_defconfig, badge4_defconfig, cerf_defconfig, ...etc...
> > 
> > OK, I'll try to do that.
> > Do I need to build all the configs in arch/arm/configs?
> 
> OK, building 2.6.20-rc2-mm1 with all 59 configs from arch/arm/configs 
> with and w/o the patch indeed found one mysterious #include that may not 
> be removed. Thanks, Russell!

Great, thanks for doing that.

Acked-by: Russell King <[EMAIL PROTECTED]>

-- 
Russell King
 Linux kernel2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [updated PATCH] remove 555 unneeded #includes of sched.h

2006-12-30 Thread Russell King
On Fri, Dec 29, 2006 at 11:23:58AM +0100, Tim Schmielau wrote:
 On Thu, 28 Dec 2006, Tim Schmielau wrote:
  On Thu, 28 Dec 2006, Russell King wrote:
  
   To cover these, you need to build at least rpc_defconfig, 
   lubbock_defconfig,
   netwinder_defconfig, badge4_defconfig, cerf_defconfig, ...etc...
  
  OK, I'll try to do that.
  Do I need to build all the configs in arch/arm/configs?
 
 OK, building 2.6.20-rc2-mm1 with all 59 configs from arch/arm/configs 
 with and w/o the patch indeed found one mysterious #include that may not 
 be removed. Thanks, Russell!

Great, thanks for doing that.

Acked-by: Russell King [EMAIL PROTECTED]

-- 
Russell King
 Linux kernel2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [updated PATCH] remove 555 unneeded #includes of sched.h

2006-12-29 Thread Tim Schmielau
On Fri, 29 Dec 2006, Tim Schmielau wrote:

> Andrew, please use the attached patch instead of the previous one, it also 
> has a slightly better patch description.

This time I actually attached something. Sorry.
(In case anybody wonders: yes, I did get some sleep while the builds were 
running)

Tim

remove-555-unneeded-includes-of-schedh.patch.bz2
Description: BZip2 compressed data


[updated PATCH] remove 555 unneeded #includes of sched.h

2006-12-29 Thread Tim Schmielau
On Thu, 28 Dec 2006, Tim Schmielau wrote:
> On Thu, 28 Dec 2006, Russell King wrote:
> 
> > To cover these, you need to build at least rpc_defconfig, lubbock_defconfig,
> > netwinder_defconfig, badge4_defconfig, cerf_defconfig, ...etc...
> 
> OK, I'll try to do that.
> Do I need to build all the configs in arch/arm/configs?

OK, building 2.6.20-rc2-mm1 with all 59 configs from arch/arm/configs 
with and w/o the patch indeed found one mysterious #include that may not 
be removed. Thanks, Russell!

Andrew, please use the attached patch instead of the previous one, it also 
has a slightly better patch description.

Thanks,
Tim
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[updated PATCH] remove 555 unneeded #includes of sched.h

2006-12-29 Thread Tim Schmielau
On Thu, 28 Dec 2006, Tim Schmielau wrote:
 On Thu, 28 Dec 2006, Russell King wrote:
 
  To cover these, you need to build at least rpc_defconfig, lubbock_defconfig,
  netwinder_defconfig, badge4_defconfig, cerf_defconfig, ...etc...
 
 OK, I'll try to do that.
 Do I need to build all the configs in arch/arm/configs?

OK, building 2.6.20-rc2-mm1 with all 59 configs from arch/arm/configs 
with and w/o the patch indeed found one mysterious #include that may not 
be removed. Thanks, Russell!

Andrew, please use the attached patch instead of the previous one, it also 
has a slightly better patch description.

Thanks,
Tim
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [updated PATCH] remove 555 unneeded #includes of sched.h

2006-12-29 Thread Tim Schmielau
On Fri, 29 Dec 2006, Tim Schmielau wrote:

 Andrew, please use the attached patch instead of the previous one, it also 
 has a slightly better patch description.

This time I actually attached something. Sorry.
(In case anybody wonders: yes, I did get some sleep while the builds were 
running)

Tim

remove-555-unneeded-includes-of-schedh.patch.bz2
Description: BZip2 compressed data