summaryrefslogtreecommitdiffstats
path: root/drivers/staging/fbtft/fb_ssd1325.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_ssd1325.c
parent26190d41b97734eee9ec076c12d6defe42de7efc (diff)
downloadop-kernel-dev-22eb36b8142b1f07f23c365e87a5dad6c9f233f1.zip
op-kernel-dev-22eb36b8142b1f07f23c365e87a5dad6c9f233f1.tar.gz
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_ssd1325.c')
-rw-r--r--drivers/staging/fbtft/fb_ssd1325.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/fbtft/fb_ssd1325.c b/drivers/staging/fbtft/fb_ssd1325.c
index 15078bf..b7e40c2 100644
--- a/drivers/staging/fbtft/fb_ssd1325.c
+++ b/drivers/staging/fbtft/fb_ssd1325.c
@@ -116,7 +116,7 @@ static int blank(struct fbtft_par *par, bool on)
* 0 = Setting of GS1 < Setting of GS2 < Setting of GS3.....<
* Setting of GS14 < Setting of GS15
*/
-static int set_gamma(struct fbtft_par *par, unsigned long *curves)
+static int set_gamma(struct fbtft_par *par, u32 *curves)
{
int i;
OpenPOWER on IntegriCloud