* [PATCH] lib: fix #define max
@ 2014-10-21 15:49 Mika Kuoppala
2014-10-22 18:32 ` Jani Nikula
0 siblings, 1 reply; 4+ messages in thread
From: Mika Kuoppala @ 2014-10-21 15:49 UTC (permalink / raw)
To: intel-gfx
Regression from:
commit be4710a541b517b5f8663448bffed5656d59b47b
Author: Thomas Wood <thomas.wood@intel.com>
Date: Fri Oct 10 11:20:35 2014 +0100
lib: add common min and max macros
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85218
Tested-by: Guo Jinxian <jinxianx.guo@intel.com>
Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
---
lib/igt_aux.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index 415614b..9b42918 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -85,6 +85,6 @@ bool intel_check_memory(uint32_t count, uint32_t size, unsigned mode);
#define min(a, b) ((a) < (b) ? (a) : (b))
-#define max(a, b) ((a) < (b) ? (a) : (b))
+#define max(a, b) ((a) > (b) ? (a) : (b))
#endif /* IGT_AUX_H */
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] lib: fix #define max
2014-10-21 15:49 [PATCH] lib: fix #define max Mika Kuoppala
@ 2014-10-22 18:32 ` Jani Nikula
2014-10-23 12:19 ` Daniel Vetter
0 siblings, 1 reply; 4+ messages in thread
From: Jani Nikula @ 2014-10-22 18:32 UTC (permalink / raw)
To: Mika Kuoppala, intel-gfx
On Tue, 21 Oct 2014, Mika Kuoppala <mika.kuoppala@linux.intel.com> wrote:
> Regression from:
>
> commit be4710a541b517b5f8663448bffed5656d59b47b
> Author: Thomas Wood <thomas.wood@intel.com>
> Date: Fri Oct 10 11:20:35 2014 +0100
>
> lib: add common min and max macros
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85218
> Tested-by: Guo Jinxian <jinxianx.guo@intel.com>
> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
Push it already!
> ---
> lib/igt_aux.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/igt_aux.h b/lib/igt_aux.h
> index 415614b..9b42918 100644
> --- a/lib/igt_aux.h
> +++ b/lib/igt_aux.h
> @@ -85,6 +85,6 @@ bool intel_check_memory(uint32_t count, uint32_t size, unsigned mode);
>
>
> #define min(a, b) ((a) < (b) ? (a) : (b))
> -#define max(a, b) ((a) < (b) ? (a) : (b))
> +#define max(a, b) ((a) > (b) ? (a) : (b))
>
> #endif /* IGT_AUX_H */
> --
> 1.9.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Jani Nikula, Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] lib: fix #define max
2014-10-22 18:32 ` Jani Nikula
@ 2014-10-23 12:19 ` Daniel Vetter
0 siblings, 0 replies; 4+ messages in thread
From: Daniel Vetter @ 2014-10-23 12:19 UTC (permalink / raw)
To: Jani Nikula; +Cc: intel-gfx
On Wed, Oct 22, 2014 at 09:32:23PM +0300, Jani Nikula wrote:
> On Tue, 21 Oct 2014, Mika Kuoppala <mika.kuoppala@linux.intel.com> wrote:
> > Regression from:
> >
> > commit be4710a541b517b5f8663448bffed5656d59b47b
> > Author: Thomas Wood <thomas.wood@intel.com>
> > Date: Fri Oct 10 11:20:35 2014 +0100
> >
> > lib: add common min and max macros
> >
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85218
> > Tested-by: Guo Jinxian <jinxianx.guo@intel.com>
> > Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
>
> Push it already!
To clarify igt process: If you're patch has been sitting on the m-l for a
day or so without objection, just push it. If it's critical (like this
one) and you're confident it's good push right after sending.
Submitting to intel-gfx for igt patches is mostly so that other people can
watch what flies by and drop suggestions for later improvements. And of
course new contributors get the full review treatment to make sure they
use all the nice stuff the testsuite provides properly.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] lib: fix #define max
@ 2014-10-21 13:31 Mika Kuoppala
0 siblings, 0 replies; 4+ messages in thread
From: Mika Kuoppala @ 2014-10-21 13:31 UTC (permalink / raw)
To: intel-gfx
Regression from:
commit be4710a541b517b5f8663448bffed5656d59b47b
Author: Thomas Wood <thomas.wood@intel.com>
Date: Fri Oct 10 11:20:35 2014 +0100
lib: add common min and max macros
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85218
Tested-by: Guo Jinxian <jinxianx.guo@intel.com>
Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
---
lib/igt_aux.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index 415614b..9b42918 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -85,6 +85,6 @@ bool intel_check_memory(uint32_t count, uint32_t size, unsigned mode);
#define min(a, b) ((a) < (b) ? (a) : (b))
-#define max(a, b) ((a) < (b) ? (a) : (b))
+#define max(a, b) ((a) > (b) ? (a) : (b))
#endif /* IGT_AUX_H */
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-10-23 12:19 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-21 15:49 [PATCH] lib: fix #define max Mika Kuoppala
2014-10-22 18:32 ` Jani Nikula
2014-10-23 12:19 ` Daniel Vetter
-- strict thread matches above, loose matches on Subject: below --
2014-10-21 13:31 Mika Kuoppala
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.