[PATCH] drm: don't push static constants on stack for %*ph

2013-08-02 Thread David Herrmann
Hi On Fri, Aug 2, 2013 at 1:09 PM, Andy Shevchenko wrote: > There is no need to pass constants via stack. The width may be explicitly > specified in the format. Yupp, all 3 make sense and actually speed up "format_decode()". Reviewed-by: David Herrmann Regards David > Signed-off-by: Andy

[PATCH] drm: don't push static constants on stack for %*ph

2013-08-02 Thread Andy Shevchenko
There is no need to pass constants via stack. The width may be explicitly specified in the format. Signed-off-by: Andy Shevchenko --- drivers/gpu/drm/nouveau/core/engine/disp/dport.c | 2 +- drivers/gpu/drm/radeon/atombios_dp.c | 2 +- drivers/gpu/drm/udl/udl_main.c

[PATCH] drm: don't push static constants on stack for %*ph

2013-08-02 Thread Andy Shevchenko
There is no need to pass constants via stack. The width may be explicitly specified in the format. Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com --- drivers/gpu/drm/nouveau/core/engine/disp/dport.c | 2 +- drivers/gpu/drm/radeon/atombios_dp.c | 2 +-

Re: [PATCH] drm: don't push static constants on stack for %*ph

2013-08-02 Thread David Herrmann
Hi On Fri, Aug 2, 2013 at 1:09 PM, Andy Shevchenko andriy.shevche...@linux.intel.com wrote: There is no need to pass constants via stack. The width may be explicitly specified in the format. Yupp, all 3 make sense and actually speed up format_decode(). Reviewed-by: David Herrmann