On Fri, Jan 22, 2016 at 03:23:33PM +0800, Peter Hung wrote: > Add F81866 GPIO supports > > Fintek F81866 is a SuperIO. It contains HWMON/GPIO/Serial Ports. > and it has totally 72(9x8 sets) gpio pins. > > Here is the PDF spec: > http://www.alldatasheet.com/datasheet-pdf/pdf/459085/FINTEK/F81866AD-I.html > > The control method is the same with F7188x, but we should care the address > of GPIO8x. > > GPIO address is below: > GPIO0x based: 0xf0 > GPIO1x based: 0xe0 > GPIO2x based: 0xd0 > GPIO3x based: 0xc0 > GPIO4x based: 0xb0 > GPIO5x based: 0xa0 > GPIO6x based: 0x90 > GPIO7x based: 0x80 > GPIO8x based: 0x88 <-- not 0x70. > > Signed-off-by: Peter Hung <hpeter+linux_ker...@gmail.com> > --- > drivers/gpio/Kconfig | 4 ++-- > drivers/gpio/gpio-f7188x.c | 27 ++++++++++++++++++++++++--- > 2 files changed, 26 insertions(+), 5 deletions(-)
Hi Peter,
Acked-by: Simon Guinot <simon.gui...@sequanux.org>
Thanks,
Simon
>
> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
> index cb212eb..c1ad573 100644
> --- a/drivers/gpio/Kconfig
> +++ b/drivers/gpio/Kconfig
> @@ -513,10 +513,10 @@ config GPIO_104_IDI_48
> via the idi_48_irq module parameter.
>
> config GPIO_F7188X
> - tristate "F71869, F71869A, F71882FG and F71889F GPIO support"
> + tristate "F71869, F71869A, F71882FG, F71889F and F81866 GPIO support"
> help
> This option enables support for GPIOs found on Fintek Super-I/O
> - chips F71869, F71869A, F71882FG and F71889F.
> + chips F71869, F71869A, F71882FG, F71889F and F81866.
>
> To compile this driver as a module, choose M here: the module will
> be called f7188x-gpio.
> diff --git a/drivers/gpio/gpio-f7188x.c b/drivers/gpio/gpio-f7188x.c
> index d62fd6b..0417798 100644
> --- a/drivers/gpio/gpio-f7188x.c
> +++ b/drivers/gpio/gpio-f7188x.c
> @@ -1,5 +1,5 @@
> /*
> - * GPIO driver for Fintek Super-I/O F71869, F71869A, F71882 and F71889
> + * GPIO driver for Fintek Super-I/O F71869, F71869A, F71882, F71889 and
> F81866
> *
> * Copyright (C) 2010-2013 LaCie
> *
> @@ -36,14 +36,16 @@
> #define SIO_F71869A_ID 0x1007 /* F71869A chipset ID */
> #define SIO_F71882_ID 0x0541 /* F71882 chipset ID */
> #define SIO_F71889_ID 0x0909 /* F71889 chipset ID */
> +#define SIO_F81866_ID 0x1010 /* F81866 chipset ID */
>
> -enum chips { f71869, f71869a, f71882fg, f71889f };
> +enum chips { f71869, f71869a, f71882fg, f71889f, f81866 };
>
> static const char * const f7188x_names[] = {
> "f71869",
> "f71869a",
> "f71882fg",
> "f71889f",
> + "f81866",
> };
>
> struct f7188x_sio {
> @@ -190,6 +192,18 @@ static struct f7188x_gpio_bank f71889_gpio_bank[] = {
> F7188X_GPIO_BANK(70, 8, 0x80),
> };
>
> +static struct f7188x_gpio_bank f81866_gpio_bank[] = {
> + F7188X_GPIO_BANK(0, 8, 0xF0),
> + F7188X_GPIO_BANK(10, 8, 0xE0),
> + F7188X_GPIO_BANK(20, 8, 0xD0),
> + F7188X_GPIO_BANK(30, 8, 0xC0),
> + F7188X_GPIO_BANK(40, 8, 0xB0),
> + F7188X_GPIO_BANK(50, 8, 0xA0),
> + F7188X_GPIO_BANK(60, 8, 0x90),
> + F7188X_GPIO_BANK(70, 8, 0x80),
> + F7188X_GPIO_BANK(80, 8, 0x88),
> +};
> +
> static int f7188x_gpio_direction_in(struct gpio_chip *chip, unsigned offset)
> {
> int err;
> @@ -318,6 +332,10 @@ static int f7188x_gpio_probe(struct platform_device
> *pdev)
> data->nr_bank = ARRAY_SIZE(f71889_gpio_bank);
> data->bank = f71889_gpio_bank;
> break;
> + case f81866:
> + data->nr_bank = ARRAY_SIZE(f81866_gpio_bank);
> + data->bank = f81866_gpio_bank;
> + break;
> default:
> return -ENODEV;
> }
> @@ -395,6 +413,9 @@ static int __init f7188x_find(int addr, struct f7188x_sio
> *sio)
> case SIO_F71889_ID:
> sio->type = f71889f;
> break;
> + case SIO_F81866_ID:
> + sio->type = f81866;
> + break;
> default:
> pr_info(DRVNAME ": Unsupported Fintek device 0x%04x\n", devid);
> goto err;
> @@ -485,6 +506,6 @@ static void __exit f7188x_gpio_exit(void)
> }
> module_exit(f7188x_gpio_exit);
>
> -MODULE_DESCRIPTION("GPIO driver for Super-I/O chips F71869, F71869A,
> F71882FG and F71889F");
> +MODULE_DESCRIPTION("GPIO driver for Super-I/O chips F71869, F71869A,
> F71882FG, F71889F and F81866");
> MODULE_AUTHOR("Simon Guinot <simon.gui...@sequanux.org>");
> MODULE_LICENSE("GPL");
> --
> 1.9.1
signature.asc
Description: PGP signature
