[Intel-gfx] [PATCH i-g-t] tools: print a warning for tools replaced by intel_reg
Daniel Vetter
daniel at ffwll.ch
Mon Jun 15 08:51:44 PDT 2015
On Wed, Jun 10, 2015 at 11:41:03AM +0100, Thomas Wood wrote:
> Cc: Jani Nikula <jani.nikula at intel.com>
> Signed-off-by: Thomas Wood <thomas.wood at intel.com>
I'd go even one step further and do an exec using the corresponding
intel_reg commandline. On top of this patch ofc. Then we could nuke a lot
of the existing boilerplate already and then perhaps nuke all the tools
one release later on or so.
-Daniel
> ---
> tools/intel_iosf_sb_read.c | 3 +++
> tools/intel_iosf_sb_write.c | 3 +++
> tools/intel_reg_dumper.c | 3 +++
> tools/intel_reg_read.c | 3 +++
> tools/intel_reg_snapshot.c | 4 ++++
> tools/intel_reg_write.c | 3 +++
> tools/intel_vga_read.c | 3 +++
> tools/intel_vga_write.c | 3 +++
> 8 files changed, 25 insertions(+)
>
> diff --git a/tools/intel_iosf_sb_read.c b/tools/intel_iosf_sb_read.c
> index f188cda..097b0b8 100644
> --- a/tools/intel_iosf_sb_read.c
> +++ b/tools/intel_iosf_sb_read.c
> @@ -94,6 +94,9 @@ int main(int argc, char *argv[])
> int i, nregs, count = 1, reg_stride;
> const char *name;
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> if (!IS_VALLEYVIEW(dev->device_id) &&
> !IS_CHERRYVIEW(dev->device_id)) {
> usage(argv[0]);
> diff --git a/tools/intel_iosf_sb_write.c b/tools/intel_iosf_sb_write.c
> index eb6700d..a091a0d 100644
> --- a/tools/intel_iosf_sb_write.c
> +++ b/tools/intel_iosf_sb_write.c
> @@ -89,6 +89,9 @@ int main(int argc, char** argv)
> int i, nregs;
> const char *name;
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> if (!IS_VALLEYVIEW(dev->device_id) &&
> !IS_CHERRYVIEW(dev->device_id)) {
> usage(argv[0]);
> diff --git a/tools/intel_reg_dumper.c b/tools/intel_reg_dumper.c
> index 3d32005..f9b4d06 100644
> --- a/tools/intel_reg_dumper.c
> +++ b/tools/intel_reg_dumper.c
> @@ -2931,6 +2931,9 @@ int main(int argc, char** argv)
> char *file = NULL, *reg_name = NULL;
> uint32_t reg_val, power_well;
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> while ((opt = getopt(argc, argv, "d:h")) != -1) {
> switch (opt) {
> case 'd':
> diff --git a/tools/intel_reg_read.c b/tools/intel_reg_read.c
> index 46fa664..39671ff 100644
> --- a/tools/intel_reg_read.c
> +++ b/tools/intel_reg_read.c
> @@ -74,6 +74,9 @@ int main(int argc, char** argv)
> int decode_bits = 0;
> int dwords = 1;
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> while ((ch = getopt(argc, argv, "dfhc:")) != -1) {
> switch(ch) {
> case 'd':
> diff --git a/tools/intel_reg_snapshot.c b/tools/intel_reg_snapshot.c
> index 50dafd6..8b097e6 100644
> --- a/tools/intel_reg_snapshot.c
> +++ b/tools/intel_reg_snapshot.c
> @@ -26,6 +26,7 @@
>
> #include <unistd.h>
> #include <assert.h>
> +#include <stdio.h>
> #include "intel_io.h"
> #include "intel_chipset.h"
>
> @@ -36,6 +37,9 @@ int main(int argc, char** argv)
> int mmio_bar;
> int ret;
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> pci_dev = intel_get_pci_device();
> devid = pci_dev->device_id;
> intel_mmio_use_pci_bar(pci_dev);
> diff --git a/tools/intel_reg_write.c b/tools/intel_reg_write.c
> index b0ddffe..127519c 100644
> --- a/tools/intel_reg_write.c
> +++ b/tools/intel_reg_write.c
> @@ -36,6 +36,9 @@ int main(int argc, char** argv)
> {
> uint32_t reg, value;
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> if (argc < 3) {
> printf("Usage: %s addr value\n", argv[0]);
> printf(" WARNING: This is dangerous to you and your system's health.\n");
> diff --git a/tools/intel_vga_read.c b/tools/intel_vga_read.c
> index ea50705..8305fb8 100644
> --- a/tools/intel_vga_read.c
> +++ b/tools/intel_vga_read.c
> @@ -57,6 +57,9 @@ int main(int argc, char *argv[])
> int i, ch;
> const char *cmdname = argv[0];
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> while ((ch = getopt(argc, argv, "m")) != -1) {
> switch(ch) {
> case 'm':
> diff --git a/tools/intel_vga_write.c b/tools/intel_vga_write.c
> index 8215962..cb1ed98 100644
> --- a/tools/intel_vga_write.c
> +++ b/tools/intel_vga_write.c
> @@ -57,6 +57,9 @@ int main(int argc, char *argv[])
> int ch;
> const char *cmdname = argv[0];
>
> + fprintf(stderr, "WARNING: Use of %s has been deprecated and replaced by"
> + " intel_reg.\n", argv[0]);
> +
> while ((ch = getopt(argc, argv, "m")) != -1) {
> switch(ch) {
> case 'm':
> --
> 2.4.2
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the Intel-gfx
mailing list