linux-stable.git/drivers/staging/panel, branch v4.0

drivers/staging: use current->state helpers

2015-01-28T19:25:28+00:00

Call __set_current_state() instead of assigning the new state directly.
These interfaces also aid CONFIG_DEBUG_ATOMIC_SLEEP environments,
keeping track of who changed the state.

Signed-off-by: Davidlohr Bueso 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Remove magic numbers in LCD commands

2015-01-12T14:39:33+00:00

Get rid of magic numbers in LCD commands and replace them with defined
values, so that it's more obvious that the commands are doing.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Remove unused variable

2015-01-12T14:39:33+00:00

Remove lcd.left_shift because it is only written to at some places but
never read from.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Move LCD-related state into struct lcd

2014-12-03T00:34:35+00:00

Move more or less all LCD-related state into struct lcd
in order to get better cohesion; use bool instead of int
where it makes sense.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Remove more magic number comparison

2014-12-03T00:34:35+00:00

Use a defined value instead of magic number comparison
for checking whether a module param value has been set.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Refactor LCD init code

2014-12-03T00:34:34+00:00

Rework lcd_init method to make it a little bit more clear about
the precedence of the params, move LCD geometry and pins layout
to the LCD struct and thus make the LCD-related module params
effectively read-only.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Make two more module params read-only

2014-12-03T00:34:34+00:00

Make keypad_type and lcd_type module params read-only.
This step also starts making it more clear what is
the precedence of device params coming from different
sources (device profile, runtime module param values etc).

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Start making module params read-only

2014-12-03T00:34:34+00:00

Start decoupling module params from the actual device state,
both for lcd and keypad, by keeping the params read-only
and moving the device state to related structs.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Use defined value or checking module params state

2014-12-03T00:34:34+00:00

Avoid magic number and use a comparison with a defined value instead
that checks whether module param has been set by the user to some
value at loading time.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman

staging: panel: Remove magic numbers

2014-12-03T00:34:34+00:00

Get rid of magic numbers indicating that the value of a module param
is not set. Use a defined value instead.

Signed-off-by: Mariusz Gorski 
Acked-by: Willy Tarreau 
Signed-off-by: Greg Kroah-Hartman