Re: [PATCH v2] staging: olpc_dcon: olpc_dcon_xo_1.c: Switch to the gpio descriptor interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux