iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Input: ads7846 - fix sparse endian warnings

Browse Source 
Also remove the temporary pointer and use ->rx_buf directly.

Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
This commit is contained in:
Harvey Harrison 2008-07-23 14:16:19 -04:00 committed by Dmitry Torokhov
parent 53703659ab
commit 494f685775
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/input/touchscreen/ads7846.c
View File

@ -633,19 +633,17 @@ static void ads7846_rx_val(void *ads)
struct ads7846 *ts = ads; struct ads7846 *ts = ads;
struct spi_message *m; struct spi_message *m;
struct spi_transfer *t; struct spi_transfer *t;
u16 *rx_val;
int val; int val;
int action; int action;
int status; int status;
m = &ts->msg[ts->msg_idx]; m = &ts->msg[ts->msg_idx];
t = list_entry(m->transfers.prev, struct spi_transfer, transfer_list); t = list_entry(m->transfers.prev, struct spi_transfer, transfer_list);
rx_val = t->rx_buf;
/* adjust: on-wire is a must-ignore bit, a BE12 value, then padding; /* adjust: on-wire is a must-ignore bit, a BE12 value, then padding;
* built from two 8 bit values written msb-first. * built from two 8 bit values written msb-first.
*/ */
val = be16_to_cpu(*rx_val) >> 3; val = be16_to_cpup((__be16 *)t->rx_buf) >> 3;
action = ts->filter(ts->filter_data, ts->msg_idx, &val); action = ts->filter(ts->filter_data, ts->msg_idx, &val);
switch (action) { switch (action) {
@ -659,7 +657,7 @@ static void ads7846_rx_val(void *ads)
m = ts->last_msg; m = ts->last_msg;
break; break;
case ADS7846_FILTER_OK: case ADS7846_FILTER_OK:
*rx_val = val; *(u16 *)t->rx_buf = val;
ts->tc.ignore = 0; ts->tc.ignore = 0;
m = &ts->msg[++ts->msg_idx]; m = &ts->msg[++ts->msg_idx];
break; break;