Commit 09593e38 authored by Antonio Ospite's avatar Antonio Ospite Committed by Jiri Kosina

HID: sony: fix errors from scripts/checkpatch.pl

  ./scripts/checkpatch.pl \
    --types "SPACING,TRAILING_WHITESPACE,POINTER_LOCATION,CODE_INDENT" \
    -f drivers/hid/hid-sony.c

  ERROR: trailing whitespace
  #933: FILE: drivers/hid/hid-sony.c:933:
  +^I * $

  ERROR: space prohibited after that open square bracket '['
  #947: FILE: drivers/hid/hid-sony.c:947:
  +	[ 1] = BTN_TRIGGER_HAPPY1,

  ERROR: space prohibited after that open square bracket '['
  #948: FILE: drivers/hid/hid-sony.c:948:
  +	[ 2] = BTN_TRIGGER_HAPPY2,

  ERROR: space prohibited after that open square bracket '['
  #949: FILE: drivers/hid/hid-sony.c:949:
  +	[ 3] = BTN_TRIGGER_HAPPY3,

  ERROR: space prohibited after that open square bracket '['
  #950: FILE: drivers/hid/hid-sony.c:950:
  +	[ 4] = BTN_TRIGGER_HAPPY4,

  ERROR: space prohibited after that open square bracket '['
  #951: FILE: drivers/hid/hid-sony.c:951:
  +	[ 5] = BTN_TRIGGER_HAPPY5,

  ERROR: space prohibited after that open square bracket '['
  #952: FILE: drivers/hid/hid-sony.c:952:
  +	[ 6] = BTN_TRIGGER_HAPPY6,

  ERROR: space prohibited after that open square bracket '['
  #953: FILE: drivers/hid/hid-sony.c:953:
  +	[ 7] = BTN_TRIGGER_HAPPY7,

  ERROR: space prohibited after that open square bracket '['
  #954: FILE: drivers/hid/hid-sony.c:954:
  +	[ 8] = BTN_TRIGGER_HAPPY8,

  ERROR: space prohibited after that open square bracket '['
  #955: FILE: drivers/hid/hid-sony.c:955:
  +	[ 9] = BTN_TRIGGER_HAPPY9,

  ERROR: "(foo*)" should be "(foo *)"
  #1032: FILE: drivers/hid/hid-sony.c:1032:
  +	void(*send_output_report)(struct sony_sc*);

  WARNING: missing space after return type
  #1032: FILE: drivers/hid/hid-sony.c:1032:
  +	void(*send_output_report)(struct sony_sc*);

  ERROR: "(foo*)" should be "(foo *)"
  #2261: FILE: drivers/hid/hid-sony.c:2261:
  +				void(*send_output_report)(struct sony_sc*))

  WARNING: missing space after return type
  #2261: FILE: drivers/hid/hid-sony.c:2261:
  +				void(*send_output_report)(struct sony_sc*))

  ERROR: code indent should use tabs where possible
  #2449: FILE: drivers/hid/hid-sony.c:2449:
  +         */$

  total: 13 errors, 2 warnings, 2570 lines checked
Signed-off-by: default avatarAntonio Ospite <ao2@ao2.it>
Acked-by: default avatarFrank Praznik <frank.praznik@gmail.com>
Signed-off-by: default avatarJiri Kosina <jkosina@suse.cz>
parent ad07b7a6
...@@ -930,7 +930,7 @@ static const unsigned int buzz_keymap[] = { ...@@ -930,7 +930,7 @@ static const unsigned int buzz_keymap[] = {
/* /*
* The controller has 4 remote buzzers, each with one LED and 5 * The controller has 4 remote buzzers, each with one LED and 5
* buttons. * buttons.
* *
* We use the mapping chosen by the controller, which is: * We use the mapping chosen by the controller, which is:
* *
* Key Offset * Key Offset
...@@ -944,15 +944,15 @@ static const unsigned int buzz_keymap[] = { ...@@ -944,15 +944,15 @@ static const unsigned int buzz_keymap[] = {
* So, for example, the orange button on the third buzzer is mapped to * So, for example, the orange button on the third buzzer is mapped to
* BTN_TRIGGER_HAPPY14 * BTN_TRIGGER_HAPPY14
*/ */
[ 1] = BTN_TRIGGER_HAPPY1, [1] = BTN_TRIGGER_HAPPY1,
[ 2] = BTN_TRIGGER_HAPPY2, [2] = BTN_TRIGGER_HAPPY2,
[ 3] = BTN_TRIGGER_HAPPY3, [3] = BTN_TRIGGER_HAPPY3,
[ 4] = BTN_TRIGGER_HAPPY4, [4] = BTN_TRIGGER_HAPPY4,
[ 5] = BTN_TRIGGER_HAPPY5, [5] = BTN_TRIGGER_HAPPY5,
[ 6] = BTN_TRIGGER_HAPPY6, [6] = BTN_TRIGGER_HAPPY6,
[ 7] = BTN_TRIGGER_HAPPY7, [7] = BTN_TRIGGER_HAPPY7,
[ 8] = BTN_TRIGGER_HAPPY8, [8] = BTN_TRIGGER_HAPPY8,
[ 9] = BTN_TRIGGER_HAPPY9, [9] = BTN_TRIGGER_HAPPY9,
[10] = BTN_TRIGGER_HAPPY10, [10] = BTN_TRIGGER_HAPPY10,
[11] = BTN_TRIGGER_HAPPY11, [11] = BTN_TRIGGER_HAPPY11,
[12] = BTN_TRIGGER_HAPPY12, [12] = BTN_TRIGGER_HAPPY12,
...@@ -1029,7 +1029,7 @@ struct sony_sc { ...@@ -1029,7 +1029,7 @@ struct sony_sc {
struct led_classdev *leds[MAX_LEDS]; struct led_classdev *leds[MAX_LEDS];
unsigned long quirks; unsigned long quirks;
struct work_struct state_worker; struct work_struct state_worker;
void(*send_output_report)(struct sony_sc*); void (*send_output_report)(struct sony_sc *);
struct power_supply *battery; struct power_supply *battery;
struct power_supply_desc battery_desc; struct power_supply_desc battery_desc;
int device_id; int device_id;
...@@ -2259,7 +2259,7 @@ static void sony_release_device_id(struct sony_sc *sc) ...@@ -2259,7 +2259,7 @@ static void sony_release_device_id(struct sony_sc *sc)
} }
static inline void sony_init_output_report(struct sony_sc *sc, static inline void sony_init_output_report(struct sony_sc *sc,
void(*send_output_report)(struct sony_sc*)) void (*send_output_report)(struct sony_sc *))
{ {
sc->send_output_report = send_output_report; sc->send_output_report = send_output_report;
...@@ -2447,7 +2447,7 @@ static int sony_suspend(struct hid_device *hdev, pm_message_t message) ...@@ -2447,7 +2447,7 @@ static int sony_suspend(struct hid_device *hdev, pm_message_t message)
/* /*
* On suspend save the current LED state, * On suspend save the current LED state,
* stop running force-feedback and blank the LEDS. * stop running force-feedback and blank the LEDS.
*/ */
if (SONY_LED_SUPPORT || SONY_FF_SUPPORT) { if (SONY_LED_SUPPORT || SONY_FF_SUPPORT) {
struct sony_sc *sc = hid_get_drvdata(hdev); struct sony_sc *sc = hid_get_drvdata(hdev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment