aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/staging/fbtft/fb_hx8347d.c
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2017-02-02 15:43:42 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-02-03 13:01:36 +0100
commit22eb36b8142b1f07f23c365e87a5dad6c9f233f1 (patch)
treece2d04c6b75134688627a29ac28504dcf227cc5f /drivers/staging/fbtft/fb_hx8347d.c
parentstaging: rtl8192u: move stats_IndicateArray off stack (diff)
downloadlinux-dev-22eb36b8142b1f07f23c365e87a5dad6c9f233f1.tar.xz
linux-dev-22eb36b8142b1f07f23c365e87a5dad6c9f233f1.zip
staging: fbtft: change 'gamma' array to u32
Having a local variable of 1024 bytes on 64-bit architectures is a bit too much, and I ran into this warning while trying to see what functions use the largest stack: drivers/staging/fbtft/fbtft-sysfs.c: In function 'store_gamma_curve': drivers/staging/fbtft/fbtft-sysfs.c:132:1: warning: the frame size of 1032 bytes is larger than 1024 bytes [-Wframe-larger-than=] As there is no need for 64-bit gamma values (on 32-bit architectures, we don't use those either), I'm changing the type from 'unsigned long' to 'u32' here, which cuts the required space in half everywhere. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/fbtft/fb_hx8347d.c')
-rw-r--r--drivers/staging/fbtft/fb_hx8347d.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/fbtft/fb_hx8347d.c b/drivers/staging/fbtft/fb_hx8347d.c
index 450a61e3f99c..bbf78f8644a8 100644
--- a/drivers/staging/fbtft/fb_hx8347d.c
+++ b/drivers/staging/fbtft/fb_hx8347d.c
@@ -102,7 +102,7 @@ static void set_addr_win(struct fbtft_par *par, int xs, int ys, int xe, int ye)
* VRN0 VRN1 VRN2 VRN3 VRN4 VRN5 PRN0 PRN1 PKN0 PKN1 PKN2 PKN3 PKN4 CGM
*/
#define CURVE(num, idx) curves[num * par->gamma.num_values + idx]
-static int set_gamma(struct fbtft_par *par, unsigned long *curves)
+static int set_gamma(struct fbtft_par *par, u32 *curves)
{
unsigned long mask[] = {
0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x3f, 0x7f, 0x7f, 0x1f, 0x1f,