All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] msr-tools: new package
@ 2016-01-29 17:11 Vincent Stehlé
  2016-02-22 14:40 ` Yegor Yefremov
  0 siblings, 1 reply; 8+ messages in thread
From: Vincent Stehlé @ 2016-01-29 17:11 UTC (permalink / raw)
  To: buildroot

The MSR Tools project provides utilities to access x86 processor MSRs
and CPU ID directly.

https://01.org/msr-tools

Signed-off-by: Vincent Stehl? <vincent.stehle@intel.com>
---
 package/Config.in                |  1 +
 package/msr-tools/Config.in      | 12 ++++++++++++
 package/msr-tools/msr-tools.hash |  2 ++
 package/msr-tools/msr-tools.mk   | 22 ++++++++++++++++++++++
 4 files changed, 37 insertions(+)
 create mode 100644 package/msr-tools/Config.in
 create mode 100644 package/msr-tools/msr-tools.hash
 create mode 100644 package/msr-tools/msr-tools.mk

diff --git a/package/Config.in b/package/Config.in
index 09c2b40..5e2451c 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -402,6 +402,7 @@ endif
 	source "package/memtest86/Config.in"
 	source "package/memtester/Config.in"
 	source "package/minicom/Config.in"
+	source "package/msr-tools/Config.in"
 	source "package/nanocom/Config.in"
 	source "package/neard/Config.in"
 	source "package/nvidia-driver/Config.in"
diff --git a/package/msr-tools/Config.in b/package/msr-tools/Config.in
new file mode 100644
index 0000000..cc7ac2d
--- /dev/null
+++ b/package/msr-tools/Config.in
@@ -0,0 +1,12 @@
+config BR2_PACKAGE_MSR_TOOLS
+	bool "msr-tools"
+	depends on BR2_i386 || BR2_x86_64
+	help
+	  The MSR Tools project provides utilities to access x86 processor MSRs
+	  and CPU ID directly.
+
+	  Those tools access the hardware through the /dev/cpu/<cpu#>/msr and
+	  /dev/cpu/<cpu#>/cpuid devices. This necessitates msr and cpuid
+	  support in the kernel.
+
+	  https://01.org/msr-tools
diff --git a/package/msr-tools/msr-tools.hash b/package/msr-tools/msr-tools.hash
new file mode 100644
index 0000000..18146df
--- /dev/null
+++ b/package/msr-tools/msr-tools.hash
@@ -0,0 +1,2 @@
+# Locally computed:
+sha256 9b87245ee091a798184e447066e9e0d7709b7c81f5e6ad55f2b958c1aa50c4a3 msr-tools-1.3.tar.gz
diff --git a/package/msr-tools/msr-tools.mk b/package/msr-tools/msr-tools.mk
new file mode 100644
index 0000000..91a783a
--- /dev/null
+++ b/package/msr-tools/msr-tools.mk
@@ -0,0 +1,22 @@
+################################################################################
+#
+# msr-tools
+#
+################################################################################
+
+MSR_TOOLS_VERSION = 1.3
+MSR_TOOLS_SITE = $(call github,01org,msr-tools,msr-tools-$(MSR_TOOLS_VERSION))
+MSR_TOOLS_SOURCE = msr-tools-$(MSR_TOOLS_VERSION).tar.gz
+MSR_TOOLS_LICENSE = GPLv2+
+
+define MSR_TOOLS_BUILD_CMDS
+	$(TARGET_CONFIGURE_OPTS) \
+	$(MAKE) -C $(@D) CC="$(CCACHE) $(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)"
+endef
+
+define MSR_TOOLS_INSTALL_TARGET_CMDS
+	$(TARGET_CONFIGURE_OPTS) \
+	$(MAKE) -C $(@D) install sbindir="$(TARGET_DIR)/usr/sbin"
+endef
+
+$(eval $(generic-package))
-- 
2.7.0.rc3

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [Buildroot] [PATCH] msr-tools: new package
  2016-01-29 17:11 [Buildroot] [PATCH] msr-tools: new package Vincent Stehlé
@ 2016-02-22 14:40 ` Yegor Yefremov
  2016-02-22 22:21   ` Vincent Stehlé
  2016-02-23  0:13   ` [Buildroot] [PATCH] " Arnout Vandecappelle
  0 siblings, 2 replies; 8+ messages in thread
From: Yegor Yefremov @ 2016-02-22 14:40 UTC (permalink / raw)
  To: buildroot

Hi Vincent,

On Fri, Jan 29, 2016 at 6:11 PM, Vincent Stehl?
<vincent.stehle@intel.com> wrote:
> The MSR Tools project provides utilities to access x86 processor MSRs
> and CPU ID directly.
>
> https://01.org/msr-tools
>
> Signed-off-by: Vincent Stehl? <vincent.stehle@intel.com>
> ---
>  package/Config.in                |  1 +
>  package/msr-tools/Config.in      | 12 ++++++++++++
>  package/msr-tools/msr-tools.hash |  2 ++
>  package/msr-tools/msr-tools.mk   | 22 ++++++++++++++++++++++
>  4 files changed, 37 insertions(+)
>  create mode 100644 package/msr-tools/Config.in
>  create mode 100644 package/msr-tools/msr-tools.hash
>  create mode 100644 package/msr-tools/msr-tools.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 09c2b40..5e2451c 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -402,6 +402,7 @@ endif
>         source "package/memtest86/Config.in"
>         source "package/memtester/Config.in"
>         source "package/minicom/Config.in"
> +       source "package/msr-tools/Config.in"
>         source "package/nanocom/Config.in"
>         source "package/neard/Config.in"
>         source "package/nvidia-driver/Config.in"
> diff --git a/package/msr-tools/Config.in b/package/msr-tools/Config.in
> new file mode 100644
> index 0000000..cc7ac2d
> --- /dev/null
> +++ b/package/msr-tools/Config.in
> @@ -0,0 +1,12 @@
> +config BR2_PACKAGE_MSR_TOOLS
> +       bool "msr-tools"
> +       depends on BR2_i386 || BR2_x86_64
> +       help
> +         The MSR Tools project provides utilities to access x86 processor MSRs
> +         and CPU ID directly.
> +
> +         Those tools access the hardware through the /dev/cpu/<cpu#>/msr and
> +         /dev/cpu/<cpu#>/cpuid devices. This necessitates msr and cpuid
> +         support in the kernel.
> +
> +         https://01.org/msr-tools

Config.in is missing a depends message like this:

comment "msr-tools runs only on x86 CPUs"
        depends on !BR2_i386 && !BR2_x86_64

> diff --git a/package/msr-tools/msr-tools.hash b/package/msr-tools/msr-tools.hash
> new file mode 100644
> index 0000000..18146df
> --- /dev/null
> +++ b/package/msr-tools/msr-tools.hash
> @@ -0,0 +1,2 @@
> +# Locally computed:
> +sha256 9b87245ee091a798184e447066e9e0d7709b7c81f5e6ad55f2b958c1aa50c4a3 msr-tools-1.3.tar.gz
> diff --git a/package/msr-tools/msr-tools.mk b/package/msr-tools/msr-tools.mk
> new file mode 100644
> index 0000000..91a783a
> --- /dev/null
> +++ b/package/msr-tools/msr-tools.mk
> @@ -0,0 +1,22 @@
> +################################################################################
> +#
> +# msr-tools
> +#
> +################################################################################
> +
> +MSR_TOOLS_VERSION = 1.3
> +MSR_TOOLS_SITE = $(call github,01org,msr-tools,msr-tools-$(MSR_TOOLS_VERSION))
> +MSR_TOOLS_SOURCE = msr-tools-$(MSR_TOOLS_VERSION).tar.gz

Do you really need MSR_TOOLS_SOURCE?

Other than that looks good to me.

Yegor

> +MSR_TOOLS_LICENSE = GPLv2+
> +
> +define MSR_TOOLS_BUILD_CMDS
> +       $(TARGET_CONFIGURE_OPTS) \
> +       $(MAKE) -C $(@D) CC="$(CCACHE) $(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)"
> +endef
> +
> +define MSR_TOOLS_INSTALL_TARGET_CMDS
> +       $(TARGET_CONFIGURE_OPTS) \
> +       $(MAKE) -C $(@D) install sbindir="$(TARGET_DIR)/usr/sbin"
> +endef
> +
> +$(eval $(generic-package))
> --
> 2.7.0.rc3
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [Buildroot] [PATCH] msr-tools: new package
  2016-02-22 14:40 ` Yegor Yefremov
@ 2016-02-22 22:21   ` Vincent Stehlé
  2016-02-22 22:31     ` [Buildroot] [PATCH v2] " Vincent Stehlé
  2016-02-23  0:13   ` [Buildroot] [PATCH] " Arnout Vandecappelle
  1 sibling, 1 reply; 8+ messages in thread
From: Vincent Stehlé @ 2016-02-22 22:21 UTC (permalink / raw)
  To: buildroot

Hi Yegor,

Thank you very much for your review!

On Mon, Feb 22, 2016 at 03:40:59PM +0100, Yegor Yefremov wrote:
> 
> Config.in is missing a depends message like this:
> 
> comment "msr-tools runs only on x86 CPUs"
>         depends on !BR2_i386 && !BR2_x86_64

Good idea; I'll add that.

..
> > +MSR_TOOLS_SOURCE = msr-tools-$(MSR_TOOLS_VERSION).tar.gz
> 
> Do you really need MSR_TOOLS_SOURCE?

And you are right again: this is definitely not necessary to repeat the default
source name :)

Thanks for spotting those; I'll send a v2 right away.

Best regards,

Vincent.

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [Buildroot] [PATCH v2] msr-tools: new package
  2016-02-22 22:21   ` Vincent Stehlé
@ 2016-02-22 22:31     ` Vincent Stehlé
  2016-02-23  0:22       ` Arnout Vandecappelle
  0 siblings, 1 reply; 8+ messages in thread
From: Vincent Stehlé @ 2016-02-22 22:31 UTC (permalink / raw)
  To: buildroot

The MSR Tools project provides utilities to access x86 processor MSRs
and CPU ID directly.

https://01.org/msr-tools

Signed-off-by: Vincent Stehl? <vincent.stehle@intel.com>
Cc: Yegor Yefremov <yegorslists@googlemail.com>

---

Changes since v1:
- Add comments in Config.in that msr-tools runs only on x86
- Drop redundant _SOURCE specification in msr-tools.mk

 package/Config.in                |  1 +
 package/msr-tools/Config.in      | 15 +++++++++++++++
 package/msr-tools/msr-tools.hash |  2 ++
 package/msr-tools/msr-tools.mk   | 21 +++++++++++++++++++++
 4 files changed, 39 insertions(+)
 create mode 100644 package/msr-tools/Config.in
 create mode 100644 package/msr-tools/msr-tools.hash
 create mode 100644 package/msr-tools/msr-tools.mk

diff --git a/package/Config.in b/package/Config.in
index 529ad33..f468b46 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -403,6 +403,7 @@ endif
 	source "package/memtest86/Config.in"
 	source "package/memtester/Config.in"
 	source "package/minicom/Config.in"
+	source "package/msr-tools/Config.in"
 	source "package/nanocom/Config.in"
 	source "package/neard/Config.in"
 	source "package/nvidia-driver/Config.in"
diff --git a/package/msr-tools/Config.in b/package/msr-tools/Config.in
new file mode 100644
index 0000000..b348499
--- /dev/null
+++ b/package/msr-tools/Config.in
@@ -0,0 +1,15 @@
+config BR2_PACKAGE_MSR_TOOLS
+	bool "msr-tools"
+	depends on BR2_i386 || BR2_x86_64
+	help
+	  The MSR Tools project provides utilities to access x86 processor MSRs
+	  and CPU ID directly.
+
+	  Those tools access the hardware through the /dev/cpu/<cpu#>/msr and
+	  /dev/cpu/<cpu#>/cpuid devices. This necessitates msr and cpuid
+	  support in the kernel.
+
+	  https://01.org/msr-tools
+
+comment "msr-tools runs only on x86 CPUs"
+	depends on !BR2_i386 && !BR2_x86_64
diff --git a/package/msr-tools/msr-tools.hash b/package/msr-tools/msr-tools.hash
new file mode 100644
index 0000000..18146df
--- /dev/null
+++ b/package/msr-tools/msr-tools.hash
@@ -0,0 +1,2 @@
+# Locally computed:
+sha256 9b87245ee091a798184e447066e9e0d7709b7c81f5e6ad55f2b958c1aa50c4a3 msr-tools-1.3.tar.gz
diff --git a/package/msr-tools/msr-tools.mk b/package/msr-tools/msr-tools.mk
new file mode 100644
index 0000000..f14684f
--- /dev/null
+++ b/package/msr-tools/msr-tools.mk
@@ -0,0 +1,21 @@
+################################################################################
+#
+# msr-tools
+#
+################################################################################
+
+MSR_TOOLS_VERSION = 1.3
+MSR_TOOLS_SITE = $(call github,01org,msr-tools,msr-tools-$(MSR_TOOLS_VERSION))
+MSR_TOOLS_LICENSE = GPLv2+
+
+define MSR_TOOLS_BUILD_CMDS
+	$(TARGET_CONFIGURE_OPTS) \
+	$(MAKE) -C $(@D) CC="$(CCACHE) $(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)"
+endef
+
+define MSR_TOOLS_INSTALL_TARGET_CMDS
+	$(TARGET_CONFIGURE_OPTS) \
+	$(MAKE) -C $(@D) install sbindir="$(TARGET_DIR)/usr/sbin"
+endef
+
+$(eval $(generic-package))
-- 
2.7.0

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [Buildroot] [PATCH] msr-tools: new package
  2016-02-22 14:40 ` Yegor Yefremov
  2016-02-22 22:21   ` Vincent Stehlé
@ 2016-02-23  0:13   ` Arnout Vandecappelle
  1 sibling, 0 replies; 8+ messages in thread
From: Arnout Vandecappelle @ 2016-02-23  0:13 UTC (permalink / raw)
  To: buildroot

On 02/22/16 15:40, Yegor Yefremov wrote:
> Config.in is missing a depends message like this:
> 
> comment "msr-tools runs only on x86 CPUs"
>         depends on !BR2_i386 && !BR2_x86_64

 Actually, no: we don't add architecture dependencies. The user can't change his
configuration to a different architecture to be able to select the package, so
the comment isn't much use.

 Regards,
 Arnout

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [Buildroot] [PATCH v2] msr-tools: new package
  2016-02-22 22:31     ` [Buildroot] [PATCH v2] " Vincent Stehlé
@ 2016-02-23  0:22       ` Arnout Vandecappelle
  2016-02-28 21:12         ` [Buildroot] [PATCH v3] " Vincent Stehlé
  0 siblings, 1 reply; 8+ messages in thread
From: Arnout Vandecappelle @ 2016-02-23  0:22 UTC (permalink / raw)
  To: buildroot

On 02/22/16 23:31, Vincent Stehl? wrote:
> The MSR Tools project provides utilities to access x86 processor MSRs
> and CPU ID directly.
> 
> https://01.org/msr-tools
> 
> Signed-off-by: Vincent Stehl? <vincent.stehle@intel.com>
> Cc: Yegor Yefremov <yegorslists@googlemail.com>
> 
> ---
> 
> Changes since v1:
> - Add comments in Config.in that msr-tools runs only on x86
> - Drop redundant _SOURCE specification in msr-tools.mk
> 
>  package/Config.in                |  1 +
>  package/msr-tools/Config.in      | 15 +++++++++++++++
>  package/msr-tools/msr-tools.hash |  2 ++
>  package/msr-tools/msr-tools.mk   | 21 +++++++++++++++++++++
>  4 files changed, 39 insertions(+)
>  create mode 100644 package/msr-tools/Config.in
>  create mode 100644 package/msr-tools/msr-tools.hash
>  create mode 100644 package/msr-tools/msr-tools.mk
> 
> diff --git a/package/Config.in b/package/Config.in
> index 529ad33..f468b46 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -403,6 +403,7 @@ endif
>  	source "package/memtest86/Config.in"
>  	source "package/memtester/Config.in"
>  	source "package/minicom/Config.in"
> +	source "package/msr-tools/Config.in"
>  	source "package/nanocom/Config.in"
>  	source "package/neard/Config.in"
>  	source "package/nvidia-driver/Config.in"
> diff --git a/package/msr-tools/Config.in b/package/msr-tools/Config.in
> new file mode 100644
> index 0000000..b348499
> --- /dev/null
> +++ b/package/msr-tools/Config.in
> @@ -0,0 +1,15 @@
> +config BR2_PACKAGE_MSR_TOOLS
> +	bool "msr-tools"
> +	depends on BR2_i386 || BR2_x86_64
> +	help
> +	  The MSR Tools project provides utilities to access x86 processor MSRs
> +	  and CPU ID directly.
> +
> +	  Those tools access the hardware through the /dev/cpu/<cpu#>/msr and
> +	  /dev/cpu/<cpu#>/cpuid devices. This necessitates msr and cpuid
> +	  support in the kernel.

 Maybe specify which kernel options should be enabled, i.e. CONFIG_X86_CPUID and
CONFIG_X86_MSR.

> +
> +	  https://01.org/msr-tools
> +
> +comment "msr-tools runs only on x86 CPUs"
> +	depends on !BR2_i386 && !BR2_x86_64

 As I wrote, this shouldn't be there.

> diff --git a/package/msr-tools/msr-tools.hash b/package/msr-tools/msr-tools.hash
> new file mode 100644
> index 0000000..18146df
> --- /dev/null
> +++ b/package/msr-tools/msr-tools.hash
> @@ -0,0 +1,2 @@
> +# Locally computed:
> +sha256 9b87245ee091a798184e447066e9e0d7709b7c81f5e6ad55f2b958c1aa50c4a3 msr-tools-1.3.tar.gz
> diff --git a/package/msr-tools/msr-tools.mk b/package/msr-tools/msr-tools.mk
> new file mode 100644
> index 0000000..f14684f
> --- /dev/null
> +++ b/package/msr-tools/msr-tools.mk
> @@ -0,0 +1,21 @@
> +################################################################################
> +#
> +# msr-tools
> +#
> +################################################################################
> +
> +MSR_TOOLS_VERSION = 1.3
> +MSR_TOOLS_SITE = $(call github,01org,msr-tools,msr-tools-$(MSR_TOOLS_VERSION))
> +MSR_TOOLS_LICENSE = GPLv2+

 cpuid.c specifies GPLv2-only, so we should stick to that.

 Add a comment saying that there is no license file in the tarball. Or cpuid.c
could be used.

> +
> +define MSR_TOOLS_BUILD_CMDS
> +	$(TARGET_CONFIGURE_OPTS) \
> +	$(MAKE) -C $(@D) CC="$(CCACHE) $(TARGET_CC)" CFLAGS="$(TARGET_CFLAGS)"

 Is it possible to instead use

	$(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS)

? At least, LDFLAGS should be passed as well.

> +endef
> +
> +define MSR_TOOLS_INSTALL_TARGET_CMDS
> +	$(TARGET_CONFIGURE_OPTS) \

 I think this should be after the make.


 Regards,
 Arnout

> +	$(MAKE) -C $(@D) install sbindir="$(TARGET_DIR)/usr/sbin"
> +endef
> +
> +$(eval $(generic-package))
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [Buildroot] [PATCH v3] msr-tools: new package
  2016-02-23  0:22       ` Arnout Vandecappelle
@ 2016-02-28 21:12         ` Vincent Stehlé
  2016-02-28 21:32           ` Thomas Petazzoni
  0 siblings, 1 reply; 8+ messages in thread
From: Vincent Stehlé @ 2016-02-28 21:12 UTC (permalink / raw)
  To: buildroot

The MSR Tools project provides utilities to access x86 processor MSRs
and CPU ID directly.

https://01.org/msr-tools

Signed-off-by: Vincent Stehl? <vincent.stehle@intel.com>
Cc: Yegor Yefremov <yegorslists@googlemail.com>
Cc: Arnout Vandecappelle <arnout@mind.be>
---


Hi Arnout,

Thank you for your review.

All the items you pointed out should be addressed now. Do not hesitate to let
me know if you spot anything else.

Best regards,

Vincent.

Changes in v3:
- Drop comments in Config.in that msr-tools runs only on x86 after all
- Mention the necessary kernel options explicitly in Config.in help.
- Change license to GPLv2 as per cpuid.c
- Change makefile commands to be more in line with common usage.

Changes in v2:
- Add comments in Config.in that msr-tools runs only on x86
- Drop redundant _SOURCE specification in msr-tools.mk


 package/Config.in                |  1 +
 package/msr-tools/Config.in      | 13 +++++++++++++
 package/msr-tools/msr-tools.hash |  2 ++
 package/msr-tools/msr-tools.mk   | 21 +++++++++++++++++++++
 4 files changed, 37 insertions(+)
 create mode 100644 package/msr-tools/Config.in
 create mode 100644 package/msr-tools/msr-tools.hash
 create mode 100644 package/msr-tools/msr-tools.mk

diff --git a/package/Config.in b/package/Config.in
index 529ad33..f468b46 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -403,6 +403,7 @@ endif
 	source "package/memtest86/Config.in"
 	source "package/memtester/Config.in"
 	source "package/minicom/Config.in"
+	source "package/msr-tools/Config.in"
 	source "package/nanocom/Config.in"
 	source "package/neard/Config.in"
 	source "package/nvidia-driver/Config.in"
diff --git a/package/msr-tools/Config.in b/package/msr-tools/Config.in
new file mode 100644
index 0000000..50ae505
--- /dev/null
+++ b/package/msr-tools/Config.in
@@ -0,0 +1,13 @@
+config BR2_PACKAGE_MSR_TOOLS
+	bool "msr-tools"
+	depends on BR2_i386 || BR2_x86_64
+	help
+	  The MSR Tools project provides utilities to access x86 processor MSRs
+	  and CPU ID directly.
+
+	  Those tools access the hardware through the /dev/cpu/<cpu#>/msr and
+	  /dev/cpu/<cpu#>/cpuid devices. This necessitates msr and cpuid
+	  support in the kernel, which means you need a kernel compiled with
+	  configuration options CONFIG_X86_CPUID and CONFIG_X86_MSR enabled.
+
+	  https://01.org/msr-tools
diff --git a/package/msr-tools/msr-tools.hash b/package/msr-tools/msr-tools.hash
new file mode 100644
index 0000000..18146df
--- /dev/null
+++ b/package/msr-tools/msr-tools.hash
@@ -0,0 +1,2 @@
+# Locally computed:
+sha256 9b87245ee091a798184e447066e9e0d7709b7c81f5e6ad55f2b958c1aa50c4a3 msr-tools-1.3.tar.gz
diff --git a/package/msr-tools/msr-tools.mk b/package/msr-tools/msr-tools.mk
new file mode 100644
index 0000000..a5fc8a8
--- /dev/null
+++ b/package/msr-tools/msr-tools.mk
@@ -0,0 +1,21 @@
+################################################################################
+#
+# msr-tools
+#
+################################################################################
+
+MSR_TOOLS_VERSION = 1.3
+MSR_TOOLS_SITE = $(call github,01org,msr-tools,msr-tools-$(MSR_TOOLS_VERSION))
+MSR_TOOLS_LICENSE = GPLv2
+MSR_TOOLS_LICENSE_FILES = cpuid.c
+
+define MSR_TOOLS_BUILD_CMDS
+	$(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS)
+endef
+
+define MSR_TOOLS_INSTALL_TARGET_CMDS
+	$(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) install \
+		sbindir="$(TARGET_DIR)/usr/sbin"
+endef
+
+$(eval $(generic-package))
-- 
2.7.0

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [Buildroot] [PATCH v3] msr-tools: new package
  2016-02-28 21:12         ` [Buildroot] [PATCH v3] " Vincent Stehlé
@ 2016-02-28 21:32           ` Thomas Petazzoni
  0 siblings, 0 replies; 8+ messages in thread
From: Thomas Petazzoni @ 2016-02-28 21:32 UTC (permalink / raw)
  To: buildroot

Dear Vincent Stehl?,

On Sun, 28 Feb 2016 22:12:58 +0100, Vincent Stehl? wrote:
> The MSR Tools project provides utilities to access x86 processor MSRs
> and CPU ID directly.
> 
> https://01.org/msr-tools
> 
> Signed-off-by: Vincent Stehl? <vincent.stehle@intel.com>
> Cc: Yegor Yefremov <yegorslists@googlemail.com>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> ---

Applied to next with the following changes:

    [Thomas:
     - rewrap Config.in help text
     - use $(TARGET_MAKE_ENV) when calling $(MAKE).]

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-02-28 21:32 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-29 17:11 [Buildroot] [PATCH] msr-tools: new package Vincent Stehlé
2016-02-22 14:40 ` Yegor Yefremov
2016-02-22 22:21   ` Vincent Stehlé
2016-02-22 22:31     ` [Buildroot] [PATCH v2] " Vincent Stehlé
2016-02-23  0:22       ` Arnout Vandecappelle
2016-02-28 21:12         ` [Buildroot] [PATCH v3] " Vincent Stehlé
2016-02-28 21:32           ` Thomas Petazzoni
2016-02-23  0:13   ` [Buildroot] [PATCH] " Arnout Vandecappelle

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.