Re: [PATCH v2] staging: olpc_dcon: olpc_dcon_xo_1.c: Switch to the gpio descriptor interface
From: kbuild test robot
Date: Sun Nov 04 2018 - 00:37:21 EST
Hi Nishad,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on staging/staging-testing]
[also build test ERROR on v4.19 next-20181102]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Nishad-Kamdar/staging-olpc_dcon-olpc_dcon_xo_1-c-Switch-to-the-gpio-descriptor-interface/20181104-041335
config: i386-allyesconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All error/warnings (new ones prefixed by >>):
>> drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:23:2: error: expected identifier before numeric constant
DCON_IRQ,
^
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:34:50: error: 'GPIO_ASIS' undeclared here (not in a function); did you mean 'GPIOD_ASIS'?
[DCON_STAT0] = { .name = "dcon_stat0", .flags = GPIO_ASIS },
^~~~~~~~~
GPIOD_ASIS
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:37:3: error: 'DCON_LOAD' undeclared here (not in a function); did you mean 'DCON_STAT1'?
[DCON_LOAD] = { .name = "dcon_load", .flags = GPIO_ASIS },
^~~~~~~~~
DCON_STAT1
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:37:3: error: array index in initializer not of integer type
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:37:3: note: (near initialization for 'gpios_asis')
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:38:3: error: 'DCON_BLANK' undeclared here (not in a function); did you mean 'DCON_LOAD'?
[DCON_BLANK] = { .name = "dcon_blank", .flag = GPIO_ASIS },
^~~~~~~~~~
DCON_LOAD
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:38:3: error: array index in initializer not of integer type
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:38:3: note: (near initialization for 'gpios_asis')
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:38:42: error: 'const struct dcon_gpio' has no member named 'flag'; did you mean 'flags'?
[DCON_BLANK] = { .name = "dcon_blank", .flag = GPIO_ASIS },
^~~~
flags
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c: In function 'dcon_init_xo_1':
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:47:26: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
struct dcon_gpio *pin = &gpios_asis[0];
^
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:50:42: error: dereferencing pointer to incomplete type 'struct i2c_client'
gpios[i] = devm_gpiod_get(&dcon->client->dev, pin[i]->name,
^~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:50:42: error: request for member 'dev' in something not a structure or union
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:50:55: error: invalid type argument of '->' (have 'struct dcon_gpio')
gpios[i] = devm_gpiod_get(&dcon->client->dev, pin[i]->name,
^~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:50:55: error: request for member 'name' in something not a structure or union
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:51:14: error: invalid type argument of '->' (have 'struct dcon_gpio')
pin[i]->flags);
^~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:51:14: error: request for member 'flags' in something not a structure or union
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:50:29: error: passing argument 1 of 'devm_gpiod_get' from incompatible pointer type [-Werror=incompatible-pointer-types]
gpios[i] = devm_gpiod_get(&dcon->client->dev, pin[i]->name,
^
In file included from drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:14:0:
include/linux/gpio/consumer.h:87:55: note: expected 'struct device *' but argument is of type 'const struct dcon_gpio (*)[1]'
struct gpio_desc *__must_check devm_gpiod_get(struct device *dev,
^~~~~~~~~~~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:50:49: error: passing argument 2 of 'devm_gpiod_get' from incompatible pointer type [-Werror=incompatible-pointer-types]
gpios[i] = devm_gpiod_get(&dcon->client->dev, pin[i]->name,
^~~
In file included from drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:14:0:
include/linux/gpio/consumer.h:87:55: note: expected 'const char *' but argument is of type 'const struct dcon_gpio *'
struct gpio_desc *__must_check devm_gpiod_get(struct device *dev,
^~~~~~~~~~~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:51:8: error: incompatible type for argument 3 of 'devm_gpiod_get'
pin[i]->flags);
^~~
In file included from drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:14:0:
include/linux/gpio/consumer.h:87:55: note: expected 'enum gpiod_flags' but argument is of type 'const struct dcon_gpio *'
struct gpio_desc *__must_check devm_gpiod_get(struct device *dev,
^~~~~~~~~~~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:50:12: error: assignment from incompatible pointer type [-Werror=incompatible-pointer-types]
gpios[i] = devm_gpiod_get(&dcon->client->dev, pin[i]->name,
^
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:54:80: error: invalid type argument of '->' (have 'struct dcon_gpio')
pr_err("failed to request %s GPIO: %d\n", pin[i]->name,
^
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:54:80: error: request for member 'name' in something not a structure or union
>> drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:54:11: warning: format '%s' expects argument of type 'char *', but argument 2 has type 'const struct dcon_gpio *' [-Wformat=]
pr_err("failed to request %s GPIO: %d\n", pin[i]->name,
^~~~~~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:82:29: error: array subscript is not an integer
gpiod_direction_input(gpios[DCON_BLANK]);
^
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:82:24: error: passing argument 1 of 'gpiod_direction_input' from incompatible pointer type [-Werror=incompatible-pointer-types]
gpiod_direction_input(gpios[DCON_BLANK]);
^~~~~
In file included from drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:14:0:
include/linux/gpio/consumer.h:110:5: note: expected 'struct gpio_desc *' but argument is of type 'const struct dcon_gpio *'
int gpiod_direction_input(struct gpio_desc *desc);
^~~~~~~~~~~~~~~~~~~~~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:83:30: error: array subscript is not an integer
gpiod_direction_output(gpios[DCON_LOAD],
^
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:83:25: error: passing argument 1 of 'gpiod_direction_output' from incompatible pointer type [-Werror=incompatible-pointer-types]
gpiod_direction_output(gpios[DCON_LOAD],
^~~~~
In file included from drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:14:0:
include/linux/gpio/consumer.h:111:5: note: expected 'struct gpio_desc *' but argument is of type 'const struct dcon_gpio *'
int gpiod_direction_output(struct gpio_desc *desc, int value);
^~~~~~~~~~~~~~~~~~~~~~
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c: In function 'dcon_set_dconload_1':
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:186:23: error: array subscript is not an integer
gpiod_set_value(gpios[DCON_LOAD], val);
^
drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:186:18: error: passing argument 1 of 'gpiod_set_value' from incompatible pointer type [-Werror=incompatible-pointer-types]
gpiod_set_value(gpios[DCON_LOAD], val);
^~~~~
In file included from drivers/staging//olpc_dcon/olpc_dcon_xo_1.c:14:0:
include/linux/gpio/consumer.h:120:6: note: expected 'struct gpio_desc *' but argument is of type 'const struct dcon_gpio *'
void gpiod_set_value(struct gpio_desc *desc, int value);
^~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +54 drivers/staging//olpc_dcon/olpc_dcon_xo_1.c
19
20 enum dcon_gpios {
21 DCON_STAT0,
22 DCON_STAT1,
> 23 DCON_IRQ,
24 DCON_LOAD,
25 DCON_BLANK,
26 };
27
28 struct dcon_gpio {
29 const char *name;
30 unsigned long flags;
31 };
32
33 static const struct dcon_gpio gpios_asis[] = {
34 [DCON_STAT0] = { .name = "dcon_stat0", .flags = GPIO_ASIS },
35 [DCON_STAT1] = { .name = "dcon_stat1", .flags = GPIO_ASIS },
36 [DCON_IRQ] = { .name = "dcon_irq", .flags = GPIO_ASIS },
37 [DCON_LOAD] = { .name = "dcon_load", .flags = GPIO_ASIS },
38 [DCON_BLANK] = { .name = "dcon_blank", .flag = GPIO_ASIS },
39 };
40
41 struct gpio_desc *gpios[5];
42
43 static int dcon_init_xo_1(struct dcon_priv *dcon)
44 {
45 unsigned char lob;
46 int ret, i;
47 struct dcon_gpio *pin = &gpios_asis[0];
48
49 for (i = 0; i < ARRAY_SIZE(gpios_asis); i++) {
> 50 gpios[i] = devm_gpiod_get(&dcon->client->dev, pin[i]->name,
51 pin[i]->flags);
52 if (IS_ERR(gpios[i])) {
53 ret = PTR_ERR(gpios[i]);
> 54 pr_err("failed to request %s GPIO: %d\n", pin[i]->name,
55 ret);
56 return ret;
57 }
58 }
59
60 /* Turn off the event enable for GPIO7 just to be safe */
61 cs5535_gpio_clear(OLPC_GPIO_DCON_IRQ, GPIO_EVENTS_ENABLE);
62
63 /*
64 * Determine the current state by reading the GPIO bit; earlier
65 * stages of the boot process have established the state.
66 *
67 * Note that we read GPIO_OUTPUT_VAL rather than GPIO_READ_BACK here;
68 * this is because OFW will disable input for the pin and set a value..
69 * READ_BACK will only contain a valid value if input is enabled and
70 * then a value is set. So, future readings of the pin can use
71 * READ_BACK, but the first one cannot. Awesome, huh?
72 */
73 dcon->curr_src = cs5535_gpio_isset(OLPC_GPIO_DCON_LOAD, GPIO_OUTPUT_VAL)
74 ? DCON_SOURCE_CPU
75 : DCON_SOURCE_DCON;
76 dcon->pending_src = dcon->curr_src;
77
78 /* Set the directions for the GPIO pins */
79 gpiod_direction_input(gpios[DCON_STAT0]);
80 gpiod_direction_input(gpios[DCON_STAT1]);
81 gpiod_direction_input(gpios[DCON_IRQ]);
82 gpiod_direction_input(gpios[DCON_BLANK]);
83 gpiod_direction_output(gpios[DCON_LOAD],
84 dcon->curr_src == DCON_SOURCE_CPU);
85
86 /* Set up the interrupt mappings */
87
88 /* Set the IRQ to pair 2 */
89 cs5535_gpio_setup_event(OLPC_GPIO_DCON_IRQ, 2, 0);
90
91 /* Enable group 2 to trigger the DCON interrupt */
92 cs5535_gpio_set_irq(2, DCON_IRQ);
93
94 /* Select edge level for interrupt (in PIC) */
95 lob = inb(0x4d0);
96 lob &= ~(1 << DCON_IRQ);
97 outb(lob, 0x4d0);
98
99 /* Register the interrupt handler */
100 if (request_irq(DCON_IRQ, &dcon_interrupt, 0, "DCON", dcon)) {
101 pr_err("failed to request DCON's irq\n");
102 return -EIO;
103 }
104
105 /* Clear INV_EN for GPIO7 (DCONIRQ) */
106 cs5535_gpio_clear(OLPC_GPIO_DCON_IRQ, GPIO_INPUT_INVERT);
107
108 /* Enable filter for GPIO12 (DCONBLANK) */
109 cs5535_gpio_set(OLPC_GPIO_DCON_BLANK, GPIO_INPUT_FILTER);
110
111 /* Disable filter for GPIO7 */
112 cs5535_gpio_clear(OLPC_GPIO_DCON_IRQ, GPIO_INPUT_FILTER);
113
114 /* Disable event counter for GPIO7 (DCONIRQ) and GPIO12 (DCONBLANK) */
115 cs5535_gpio_clear(OLPC_GPIO_DCON_IRQ, GPIO_INPUT_EVENT_COUNT);
116 cs5535_gpio_clear(OLPC_GPIO_DCON_BLANK, GPIO_INPUT_EVENT_COUNT);
117
118 /* Add GPIO12 to the Filter Event Pair #7 */
119 cs5535_gpio_set(OLPC_GPIO_DCON_BLANK, GPIO_FE7_SEL);
120
121 /* Turn off negative Edge Enable for GPIO12 */
122 cs5535_gpio_clear(OLPC_GPIO_DCON_BLANK, GPIO_NEGATIVE_EDGE_EN);
123
124 /* Enable negative Edge Enable for GPIO7 */
125 cs5535_gpio_set(OLPC_GPIO_DCON_IRQ, GPIO_NEGATIVE_EDGE_EN);
126
127 /* Zero the filter amount for Filter Event Pair #7 */
128 cs5535_gpio_set(0, GPIO_FLTR7_AMOUNT);
129
130 /* Clear the negative edge status for GPIO7 and GPIO12 */
131 cs5535_gpio_set(OLPC_GPIO_DCON_IRQ, GPIO_NEGATIVE_EDGE_STS);
132 cs5535_gpio_set(OLPC_GPIO_DCON_BLANK, GPIO_NEGATIVE_EDGE_STS);
133
134 /* FIXME: Clear the positive status as well, just to be sure */
135 cs5535_gpio_set(OLPC_GPIO_DCON_IRQ, GPIO_POSITIVE_EDGE_STS);
136 cs5535_gpio_set(OLPC_GPIO_DCON_BLANK, GPIO_POSITIVE_EDGE_STS);
137
138 /* Enable events for GPIO7 (DCONIRQ) and GPIO12 (DCONBLANK) */
139 cs5535_gpio_set(OLPC_GPIO_DCON_IRQ, GPIO_EVENTS_ENABLE);
140 cs5535_gpio_set(OLPC_GPIO_DCON_BLANK, GPIO_EVENTS_ENABLE);
141
142 return 0;
143 }
144
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Attachment:
.config.gz
Description: application/gzip