All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Stancek <jstancek@redhat.com>
To: Alexey Kodanev <alexey.kodanev@oracle.com>
Cc: vasily isaenko <vasily.isaenko@oracle.com>,
	ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH] fw_load: new test of device firmware loading
Date: Wed, 5 Jun 2013 11:37:23 -0400 (EDT)	[thread overview]
Message-ID: <271133424.4337328.1370446643439.JavaMail.root@redhat.com> (raw)
In-Reply-To: <1370433977-6944-1-git-send-email-alexey.kodanev@oracle.com>



----- Original Message -----
> From: "Alexey Kodanev" <alexey.kodanev@oracle.com>
> To: ltp-list@lists.sourceforge.net
> Cc: "vasily isaenko" <vasily.isaenko@oracle.com>, "Alexey Kodanev" <alexey.kodanev@oracle.com>
> Sent: Wednesday, 5 June, 2013 2:06:17 PM
> Subject: [LTP] [PATCH] fw_load: new test of device firmware loading
> 
> This test checks that from kernel 3.7 firmware can be loaded directly
> (by-pass udev) or as usual. The test consists of the two parts: userspace
> and kernelspace.
> 
> Signed-off-by: Alexey Kodanev <alexey.kodanev@oracle.com>

Hi,

This patch tries to build kernel module also on kernels < 3.7, which seems
unnecessary, because the test won't run. And for very old kernels it won't
build at all. 

I know some people still build/run LTP on kernels as old as 2.6.18, which is why
I would rather go with WITH_MODULES="no" as default, if more kernel modules
become part of the build.

Regards,
Jan

------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. A cloud service to automate IT design, transition and operations
2. Dashboards that offer high-level views of enterprise services
3. A single system of record for all IT processes
http://p.sf.net/sfu/servicenow-d2d-j
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2013-06-05 15:37 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-05 12:06 [LTP] [PATCH] fw_load: new test of device firmware loading Alexey Kodanev
2013-06-05 15:37 ` Jan Stancek [this message]
2013-06-05 15:54   ` chrubis
2013-06-06 11:01     ` chrubis
     [not found]       ` <51B706F8.9070303@oracle.com>
2013-06-11 12:11         ` chrubis
     [not found]           ` <716795963.1251428.1370952887038.JavaMail.root@redhat.com>
2013-06-11 12:37             ` chrubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=271133424.4337328.1370446643439.JavaMail.root@redhat.com \
    --to=jstancek@redhat.com \
    --cc=alexey.kodanev@oracle.com \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=vasily.isaenko@oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.