All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: How to test my patches for the (staging) android driver?
       [not found] <20131203151642.GB2264@fu>
@ 2013-12-03 15:57 ` Matthias Beyer
  2013-12-04 15:08   ` Matthias Beyer
  0 siblings, 1 reply; 4+ messages in thread
From: Matthias Beyer @ 2013-12-03 15:57 UTC (permalink / raw)
  To: linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1208 bytes --]

Hi,

Greg-KHs email bot tells me to send this message to a ML. It suggests
to send it to this list or kernelnewbies@nl.linux.org, but the latter
results in a failure notice. So I send this email to this list.

On 03-12-2013 16:16:42, Matthias Beyer wrote:
> Hi,
> 
> First: I'm really new to the kernel in manner of submitting patches,
> testing etc. Please, if you send me guidelines or pointers on
> what to do, explain how to do things!
> 
> I'm working on some cleanup patches for code in drivers/staging/android/.
> These patches are cleanup patches as suggested in the appropriate TODO
> file, including fixes for checkpatch, but also function splitting as
> there are some really long functions (mainly in binder.c).
> 
> How to test my code beside compiling it? Do I even have a possibility
> to test it or should I just send it to you?
> 
> Besides: Which ML should I post this (and the patches) to? Main Kernel
> ML?
> 
> -- 
> Mit freundlichen Grüßen,
> Kind regards,
> Matthias Beyer
> 
> Proudly sent with mutt.
> Happily signed with gnupg.



-- 
Mit freundlichen Grüßen,
Kind regards,
Matthias Beyer

Proudly sent with mutt.
Happily signed with gnupg.

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

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

* How to test my patches for the (staging) android driver?
  2013-12-03 15:57 ` How to test my patches for the (staging) android driver? Matthias Beyer
@ 2013-12-04 15:08   ` Matthias Beyer
  2013-12-04 15:17     ` Valdis.Kletnieks at vt.edu
  0 siblings, 1 reply; 4+ messages in thread
From: Matthias Beyer @ 2013-12-04 15:08 UTC (permalink / raw)
  To: kernelnewbies

Hi kernelnewbies!

As I do not get any answer from the other MLs, this goes here, too.
Maybe there is just too much noise in lkml... never mind!

On 03-12-2013 16:57:08, Matthias Beyer wrote:
> Hi,
> 
> Greg-KHs email bot tells me to send this message to a ML. It suggests
> to send it to this list or kernelnewbies at nl.linux.org, but the latter
> results in a failure notice. So I send this email to this list.
> 
> On 03-12-2013 16:16:42, Matthias Beyer wrote:
> > Hi,
> > 
> > First: I'm really new to the kernel in manner of submitting patches,
> > testing etc. Please, if you send me guidelines or pointers on
> > what to do, explain how to do things!
> > 
> > I'm working on some cleanup patches for code in drivers/staging/android/.
> > These patches are cleanup patches as suggested in the appropriate TODO
> > file, including fixes for checkpatch, but also function splitting as
> > there are some really long functions (mainly in binder.c).
> > 
> > How to test my code beside compiling it? Do I even have a possibility
> > to test it or should I just send it to you?
> > 
> > Besides: Which ML should I post this (and the patches) to? Main Kernel
> > ML?
> > 
> 

-- 
Mit freundlichen Gr??en,
Kind regards,
Matthias Beyer

Proudly sent with mutt.
Happily signed with gnupg.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
Url : http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20131204/c647d318/attachment.bin 

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

* How to test my patches for the (staging) android driver?
  2013-12-04 15:08   ` Matthias Beyer
@ 2013-12-04 15:17     ` Valdis.Kletnieks at vt.edu
  2013-12-04 18:53       ` Matthias Beyer
  0 siblings, 1 reply; 4+ messages in thread
From: Valdis.Kletnieks at vt.edu @ 2013-12-04 15:17 UTC (permalink / raw)
  To: kernelnewbies

On Wed, 04 Dec 2013 16:08:52 +0100, Matthias Beyer said:

> > > How to test my code beside compiling it? Do I even have a possibility
> > > to test it or should I just send it to you?

If at all possible, you should cross-compile for an appropriate architecture
and actually boot and test the code in question.  There's lots of ways
to write code in C that compiles cleanly, looks correct, but is subtly
wrong for some reason.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 865 bytes
Desc: not available
Url : http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20131204/6f77d06a/attachment.bin 

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

* How to test my patches for the (staging) android driver?
  2013-12-04 15:17     ` Valdis.Kletnieks at vt.edu
@ 2013-12-04 18:53       ` Matthias Beyer
  0 siblings, 0 replies; 4+ messages in thread
From: Matthias Beyer @ 2013-12-04 18:53 UTC (permalink / raw)
  To: kernelnewbies

On 04-12-2013 10:17:23, Valdis.Kletnieks at vt.edu wrote:
> On Wed, 04 Dec 2013 16:08:52 +0100, Matthias Beyer said:
> 
> > > > How to test my code beside compiling it? Do I even have a possibility
> > > > to test it or should I just send it to you?
> 
> If at all possible, you should cross-compile for an appropriate architecture
> and actually boot and test the code in question.  There's lots of ways
> to write code in C that compiles cleanly, looks correct, but is subtly
> wrong for some reason.

Thank you for your reply!

How to cross-compile and even more important: How to test? I
do not have devices for testing! Should I try it in a VM?

What I can do is checking if the object file is the same before and
after my stylefix patches. But I think this does not work for
functions I split up into several smaller ones...

-- 
Mit freundlichen Gr??en,
Kind regards,
Matthias Beyer

Proudly sent with mutt.
Happily signed with gnupg.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
Url : http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20131204/7e8fbbe1/attachment-0001.bin 

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

end of thread, other threads:[~2013-12-04 18:53 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20131203151642.GB2264@fu>
2013-12-03 15:57 ` How to test my patches for the (staging) android driver? Matthias Beyer
2013-12-04 15:08   ` Matthias Beyer
2013-12-04 15:17     ` Valdis.Kletnieks at vt.edu
2013-12-04 18:53       ` Matthias Beyer

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.