public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* white space fixes wanted?
@ 2005-11-22  0:24 Jens-Michael Hoffmann
  2005-11-22  1:29 ` Greg KH
  0 siblings, 1 reply; 5+ messages in thread
From: Jens-Michael Hoffmann @ 2005-11-22  0:24 UTC (permalink / raw)
  To: linux-kernel

When looking in drivers/ieee1394/pcilynx.c, I noticed that there are a lot of spaces instead of tabs.
I prepared a patch to fix that.

file size of pcilynx.c before: 53775 bytes
file size after:                         42870 bytes

patch size is 85356 bytes.

Should I send this patch to the mailing list?

Jens-Michael

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: white space fixes wanted?
  2005-11-22  0:24 white space fixes wanted? Jens-Michael Hoffmann
@ 2005-11-22  1:29 ` Greg KH
  2005-11-22  2:48   ` jensmh
  2005-11-22  4:36   ` Jody McIntyre
  0 siblings, 2 replies; 5+ messages in thread
From: Greg KH @ 2005-11-22  1:29 UTC (permalink / raw)
  To: Jens-Michael Hoffmann; +Cc: linux-kernel

On Tue, Nov 22, 2005 at 12:24:08AM +0000, Jens-Michael Hoffmann wrote:
> When looking in drivers/ieee1394/pcilynx.c, I noticed that there are a lot of spaces instead of tabs.
> I prepared a patch to fix that.
> 
> file size of pcilynx.c before: 53775 bytes
> file size after:                         42870 bytes
> 
> patch size is 85356 bytes.
> 
> Should I send this patch to the mailing list?

How about to the ieee1394 developers?  They would be the best people to
be able to accept this change.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: white space fixes wanted?
  2005-11-22  1:29 ` Greg KH
@ 2005-11-22  2:48   ` jensmh
  2005-11-22  4:36   ` Jody McIntyre
  1 sibling, 0 replies; 5+ messages in thread
From: jensmh @ 2005-11-22  2:48 UTC (permalink / raw)
  To: Greg KH; +Cc: linux-kernel

Greg KH writes:
> > patch size is 85356 bytes.
> > 
> > Should I send this patch to the mailing list?
> 
> How about to the ieee1394 developers?  They would be the best people to
> be able to accept this change.

Done.
Since this is my first contribution to the linux kernel I don't know if it is mandatory
to publish patches on lkml.

thanks

Jens-Michael

(sorry for my bad english)

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: white space fixes wanted?
  2005-11-22  1:29 ` Greg KH
  2005-11-22  2:48   ` jensmh
@ 2005-11-22  4:36   ` Jody McIntyre
  2005-11-22 10:42     ` Jens-Michael Hoffmann
  1 sibling, 1 reply; 5+ messages in thread
From: Jody McIntyre @ 2005-11-22  4:36 UTC (permalink / raw)
  To: Greg KH; +Cc: Jens-Michael Hoffmann, linux-kernel

On Mon, Nov 21, 2005 at 05:29:12PM -0800, Greg KH wrote:

> > Should I send this patch to the mailing list?
> 
> How about to the ieee1394 developers?  They would be the best people to
> be able to accept this change.

I second that.  I'll gladly take code cleanup patches for ieee1394 -
it needs them :)

Cheers,
Jody (1394 maintainer)

> 
> thanks,
> 
> greg k-h
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: white space fixes wanted?
  2005-11-22  4:36   ` Jody McIntyre
@ 2005-11-22 10:42     ` Jens-Michael Hoffmann
  0 siblings, 0 replies; 5+ messages in thread
From: Jens-Michael Hoffmann @ 2005-11-22 10:42 UTC (permalink / raw)
  To: Jody McIntyre; +Cc: Greg KH, linux-kernel

Jody McIntyre writes:
> On Mon, Nov 21, 2005 at 05:29:12PM -0800, Greg KH wrote:
> 
> > > Should I send this patch to the mailing list?
> > 
> > How about to the ieee1394 developers?  They would be the best people to
> > be able to accept this change.
> 
> I second that.  I'll gladly take code cleanup patches for ieee1394 -
> it needs them :)

Great, I'll send the patches directly to you because they are quite large.

Jens-Michael

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2005-11-22  9:42 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-11-22  0:24 white space fixes wanted? Jens-Michael Hoffmann
2005-11-22  1:29 ` Greg KH
2005-11-22  2:48   ` jensmh
2005-11-22  4:36   ` Jody McIntyre
2005-11-22 10:42     ` Jens-Michael Hoffmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox