lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7941dbfbaeaf5c23bb177e66165060d3@crapouillou.net>
Date:   Wed, 03 May 2017 11:12:34 +0200
From:   Paul Cercueil <paul@...pouillou.net>
To:     Linus Walleij <linus.walleij@...aro.org>,
        Alexandre Courbot <gnurou@...il.com>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Ralf Baechle <ralf@...ux-mips.org>
Cc:     Boris Brezillon <boris.brezillon@...e-electrons.com>,
        Thierry Reding <thierry.reding@...il.com>,
        Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
        Maarten ter Huurne <maarten@...ewalker.org>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Paul Burton <paul.burton@...tec.com>, james.hogan@...tec.com,
        linux-gpio@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-mips@...ux-mips.org,
        linux-mmc@...r.kernel.org, linux-mtd@...ts.infradead.org,
        linux-pwm@...r.kernel.org, linux-fbdev@...r.kernel.org
Subject: Re: [PATCH v5 03/14] pinctrl: add a pinctrl driver for the Ingenic
 jz47xx SoCs

The dependency on MFD is gone but now I notice I forgot to remove the 
'select MFD_CORE'
in the Kconfig. It'd be great if you can make a quick edit when merging 
this,
otherwise I'll send a v6.

- Paul

Le 2017-04-28 22:08, Paul Cercueil a écrit :
> This driver handles pin configuration and pin muxing for the
> JZ4740 and JZ4780 SoCs from Ingenic.
> 
> Signed-off-by: Paul Cercueil <paul@...pouillou.net>
> ---
>  drivers/pinctrl/Kconfig           |  10 +
>  drivers/pinctrl/Makefile          |   1 +
>  drivers/pinctrl/pinctrl-ingenic.c | 852 
> ++++++++++++++++++++++++++++++++++++++
>  3 files changed, 863 insertions(+)
>  create mode 100644 drivers/pinctrl/pinctrl-ingenic.c
> 
>  v2: Consider it's a new patch. Completely rewritten from v1.
>  v3: 'unsigned' -> 'unsigned int'
>  v4: Completely rewritten from v3.
>  v5: Probe child devices directly instead of using MFD framework
> 
> diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig
> index 8f8c2af45781..82ce72fcb8e0 100644
> --- a/drivers/pinctrl/Kconfig
> +++ b/drivers/pinctrl/Kconfig
> @@ -285,6 +285,16 @@ config PINCTRL_ZYNQ
>  	help
>  	  This selects the pinctrl driver for Xilinx Zynq.
> 
> +config PINCTRL_INGENIC
> +	bool "Pinctrl driver for the Ingenic JZ47xx SoCs"
> +	default y
> +	depends on MACH_INGENIC || COMPILE_TEST
> +	select GENERIC_PINCONF
> +	select GENERIC_PINCTRL_GROUPS
> +	select GENERIC_PINMUX_FUNCTIONS
> +	select REGMAP_MMIO
> +	select MFD_CORE
> +
>  source "drivers/pinctrl/aspeed/Kconfig"
>  source "drivers/pinctrl/bcm/Kconfig"
>  source "drivers/pinctrl/berlin/Kconfig"
> diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
> index a251f439626f..80f327239d4b 100644
> --- a/drivers/pinctrl/Makefile
> +++ b/drivers/pinctrl/Makefile
> @@ -38,6 +38,7 @@ obj-$(CONFIG_PINCTRL_LPC18XX)	+= pinctrl-lpc18xx.o
>  obj-$(CONFIG_PINCTRL_TB10X)	+= pinctrl-tb10x.o
>  obj-$(CONFIG_PINCTRL_ST) 	+= pinctrl-st.o
>  obj-$(CONFIG_PINCTRL_ZYNQ)	+= pinctrl-zynq.o
> +obj-$(CONFIG_PINCTRL_INGENIC)	+= pinctrl-ingenic.o
> 
>  obj-$(CONFIG_ARCH_ASPEED)	+= aspeed/
>  obj-y				+= bcm/
> diff --git a/drivers/pinctrl/pinctrl-ingenic.c
> b/drivers/pinctrl/pinctrl-ingenic.c
> new file mode 100644
> index 000000000000..d8473d929cb1
> --- /dev/null
> +++ b/drivers/pinctrl/pinctrl-ingenic.c
> @@ -0,0 +1,852 @@
> +/*
> + * Ingenic SoCs pinctrl driver
> + *
> + * Copyright (c) 2017 Paul Cercueil <paul@...pouillou.net>
> + *
> + * License terms: GNU General Public License (GPL) version 2
> + */
> +
> +#include <linux/compiler.h>
> +#include <linux/gpio.h>
> +#include <linux/interrupt.h>
> +#include <linux/io.h>
> +#include <linux/of_device.h>
> +#include <linux/of_platform.h>
> +#include <linux/pinctrl/pinctrl.h>
> +#include <linux/pinctrl/pinmux.h>
> +#include <linux/pinctrl/pinconf.h>
> +#include <linux/pinctrl/pinconf-generic.h>
> +#include <linux/platform_device.h>
> +#include <linux/regmap.h>
> +#include <linux/slab.h>
> +
> +#include "core.h"
> +#include "pinconf.h"
> +#include "pinmux.h"
> +
> +#define JZ4740_GPIO_DATA	0x10
> +#define JZ4740_GPIO_PULL_DIS	0x30
> +#define JZ4740_GPIO_FUNC	0x40
> +#define JZ4740_GPIO_SELECT	0x50
> +#define JZ4740_GPIO_DIR		0x60
> +#define JZ4740_GPIO_TRIG	0x70
> +#define JZ4740_GPIO_FLAG	0x80
> +
> +#define JZ4770_GPIO_INT		0x10
> +#define JZ4770_GPIO_MSK		0x20
> +#define JZ4770_GPIO_PAT1	0x30
> +#define JZ4770_GPIO_PAT0	0x40
> +#define JZ4770_GPIO_FLAG	0x50
> +#define JZ4770_GPIO_PEN		0x70
> +
> +#define REG_SET(x) ((x) + 0x4)
> +#define REG_CLEAR(x) ((x) + 0x8)
> +
> +#define PINS_PER_GPIO_CHIP 32
> +
> +enum jz_version {
> +	ID_JZ4740,
> +	ID_JZ4770,
> +	ID_JZ4780,
> +};
> +
> +struct ingenic_chip_info {
> +	unsigned int num_chips;
> +
> +	const struct group_desc *groups;
> +	unsigned int num_groups;
> +
> +	const struct function_desc *functions;
> +	unsigned int num_functions;
> +
> +	const u32 *pull_ups, *pull_downs;
> +};
> +
> +struct ingenic_pinctrl {
> +	struct device *dev;
> +	struct regmap *map;
> +	struct pinctrl_dev *pctl;
> +	struct pinctrl_pin_desc *pdesc;
> +	enum jz_version version;
> +
> +	const struct ingenic_chip_info *info;
> +};
> +
> +static const u32 jz4740_pull_ups[4] = {
> +	0xffffffff, 0xffffffff, 0xffffffff, 0xffffffff,
> +};
> +
> +static const u32 jz4740_pull_downs[4] = {
> +	0x00000000, 0x00000000, 0x00000000, 0x00000000,
> +};
> +
> +static int jz4740_mmc_1bit_pins[] = { 0x69, 0x68, 0x6a, };
> +static int jz4740_mmc_4bit_pins[] = { 0x6b, 0x6c, 0x6d, };
> +static int jz4740_uart0_data_pins[] = { 0x7a, 0x79, };
> +static int jz4740_uart0_hwflow_pins[] = { 0x7e, 0x7f, };
> +static int jz4740_uart1_data_pins[] = { 0x7e, 0x7f, };
> +static int jz4740_lcd_8bit_pins[] = {
> +	0x40, 0x41, 0x42, 0x43, 0x44, 0x45, 0x46, 0x47, 0x52, 0x53, 0x54,
> +};
> +static int jz4740_lcd_16bit_pins[] = {
> +	0x48, 0x49, 0x4a, 0x4b, 0x4c, 0x4d, 0x4e, 0x4f, 0x55,
> +};
> +static int jz4740_lcd_18bit_pins[] = { 0x50, 0x51, };
> +static int jz4740_lcd_18bit_tft_pins[] = { 0x56, 0x57, 0x31, 0x32, };
> +static int jz4740_nand_cs1_pins[] = { 0x39, };
> +static int jz4740_nand_cs2_pins[] = { 0x3a, };
> +static int jz4740_nand_cs3_pins[] = { 0x3b, };
> +static int jz4740_nand_cs4_pins[] = { 0x3c, };
> +static int jz4740_pwm_pwm0_pins[] = { 0x77, };
> +static int jz4740_pwm_pwm1_pins[] = { 0x78, };
> +static int jz4740_pwm_pwm2_pins[] = { 0x79, };
> +static int jz4740_pwm_pwm3_pins[] = { 0x7a, };
> +static int jz4740_pwm_pwm4_pins[] = { 0x7b, };
> +static int jz4740_pwm_pwm5_pins[] = { 0x7c, };
> +static int jz4740_pwm_pwm6_pins[] = { 0x7e, };
> +static int jz4740_pwm_pwm7_pins[] = { 0x7f, };
> +
> +static int jz4740_mmc_1bit_funcs[] = { 0, 0, 0, };
> +static int jz4740_mmc_4bit_funcs[] = { 0, 0, 0, };
> +static int jz4740_uart0_data_funcs[] = { 1, 1, };
> +static int jz4740_uart0_hwflow_funcs[] = { 1, 1, };
> +static int jz4740_uart1_data_funcs[] = { 2, 2, };
> +static int jz4740_lcd_8bit_funcs[] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
> 0, };
> +static int jz4740_lcd_16bit_funcs[] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, };
> +static int jz4740_lcd_18bit_funcs[] = { 0, 0, };
> +static int jz4740_lcd_18bit_tft_funcs[] = { 0, 0, 0, 0, };
> +static int jz4740_nand_cs1_funcs[] = { 0, };
> +static int jz4740_nand_cs2_funcs[] = { 0, };
> +static int jz4740_nand_cs3_funcs[] = { 0, };
> +static int jz4740_nand_cs4_funcs[] = { 0, };
> +static int jz4740_pwm_pwm0_funcs[] = { 0, };
> +static int jz4740_pwm_pwm1_funcs[] = { 0, };
> +static int jz4740_pwm_pwm2_funcs[] = { 0, };
> +static int jz4740_pwm_pwm3_funcs[] = { 0, };
> +static int jz4740_pwm_pwm4_funcs[] = { 0, };
> +static int jz4740_pwm_pwm5_funcs[] = { 0, };
> +static int jz4740_pwm_pwm6_funcs[] = { 0, };
> +static int jz4740_pwm_pwm7_funcs[] = { 0, };
> +
> +#define INGENIC_PIN_GROUP(name, id)			\
> +	{						\
> +		name,					\
> +		id##_pins,				\
> +		ARRAY_SIZE(id##_pins),			\
> +		id##_funcs,				\
> +	}
> +
> +static const struct group_desc jz4740_groups[] = {
> +	INGENIC_PIN_GROUP("mmc-1bit", jz4740_mmc_1bit),
> +	INGENIC_PIN_GROUP("mmc-4bit", jz4740_mmc_4bit),
> +	INGENIC_PIN_GROUP("uart0-data", jz4740_uart0_data),
> +	INGENIC_PIN_GROUP("uart0-hwflow", jz4740_uart0_hwflow),
> +	INGENIC_PIN_GROUP("uart1-data", jz4740_uart1_data),
> +	INGENIC_PIN_GROUP("lcd-8bit", jz4740_lcd_8bit),
> +	INGENIC_PIN_GROUP("lcd-16bit", jz4740_lcd_16bit),
> +	INGENIC_PIN_GROUP("lcd-18bit", jz4740_lcd_18bit),
> +	INGENIC_PIN_GROUP("lcd-18bit-tft", jz4740_lcd_18bit_tft),
> +	{ "lcd-no-pins", },
> +	INGENIC_PIN_GROUP("nand-cs1", jz4740_nand_cs1),
> +	INGENIC_PIN_GROUP("nand-cs2", jz4740_nand_cs2),
> +	INGENIC_PIN_GROUP("nand-cs3", jz4740_nand_cs3),
> +	INGENIC_PIN_GROUP("nand-cs4", jz4740_nand_cs4),
> +	INGENIC_PIN_GROUP("pwm0", jz4740_pwm_pwm0),
> +	INGENIC_PIN_GROUP("pwm1", jz4740_pwm_pwm1),
> +	INGENIC_PIN_GROUP("pwm2", jz4740_pwm_pwm2),
> +	INGENIC_PIN_GROUP("pwm3", jz4740_pwm_pwm3),
> +	INGENIC_PIN_GROUP("pwm4", jz4740_pwm_pwm4),
> +	INGENIC_PIN_GROUP("pwm5", jz4740_pwm_pwm5),
> +	INGENIC_PIN_GROUP("pwm6", jz4740_pwm_pwm6),
> +	INGENIC_PIN_GROUP("pwm7", jz4740_pwm_pwm7),
> +};
> +
> +static const char *jz4740_mmc_groups[] = { "mmc-1bit", "mmc-4bit", };
> +static const char *jz4740_uart0_groups[] = { "uart0-data", 
> "uart0-hwflow", };
> +static const char *jz4740_uart1_groups[] = { "uart1-data", };
> +static const char *jz4740_lcd_groups[] = {
> +	"lcd-8bit", "lcd-16bit", "lcd-18bit", "lcd-18bit-tft", "lcd-no-pins",
> +};
> +static const char *jz4740_nand_groups[] = {
> +	"nand-cs1", "nand-cs2", "nand-cs3", "nand-cs4",
> +};
> +static const char *jz4740_pwm0_groups[] = { "pwm0", };
> +static const char *jz4740_pwm1_groups[] = { "pwm1", };
> +static const char *jz4740_pwm2_groups[] = { "pwm2", };
> +static const char *jz4740_pwm3_groups[] = { "pwm3", };
> +static const char *jz4740_pwm4_groups[] = { "pwm4", };
> +static const char *jz4740_pwm5_groups[] = { "pwm5", };
> +static const char *jz4740_pwm6_groups[] = { "pwm6", };
> +static const char *jz4740_pwm7_groups[] = { "pwm7", };
> +
> +static const struct function_desc jz4740_functions[] = {
> +	{ "mmc", jz4740_mmc_groups, ARRAY_SIZE(jz4740_mmc_groups), },
> +	{ "uart0", jz4740_uart0_groups, ARRAY_SIZE(jz4740_uart0_groups), },
> +	{ "uart1", jz4740_uart1_groups, ARRAY_SIZE(jz4740_uart1_groups), },
> +	{ "lcd", jz4740_lcd_groups, ARRAY_SIZE(jz4740_lcd_groups), },
> +	{ "nand", jz4740_nand_groups, ARRAY_SIZE(jz4740_nand_groups), },
> +	{ "pwm0", jz4740_pwm0_groups, ARRAY_SIZE(jz4740_pwm0_groups), },
> +	{ "pwm1", jz4740_pwm1_groups, ARRAY_SIZE(jz4740_pwm1_groups), },
> +	{ "pwm2", jz4740_pwm2_groups, ARRAY_SIZE(jz4740_pwm2_groups), },
> +	{ "pwm3", jz4740_pwm3_groups, ARRAY_SIZE(jz4740_pwm3_groups), },
> +	{ "pwm4", jz4740_pwm4_groups, ARRAY_SIZE(jz4740_pwm4_groups), },
> +	{ "pwm5", jz4740_pwm5_groups, ARRAY_SIZE(jz4740_pwm5_groups), },
> +	{ "pwm6", jz4740_pwm6_groups, ARRAY_SIZE(jz4740_pwm6_groups), },
> +	{ "pwm7", jz4740_pwm7_groups, ARRAY_SIZE(jz4740_pwm7_groups), },
> +};
> +
> +static const struct ingenic_chip_info jz4740_chip_info = {
> +	.num_chips = 4,
> +	.groups = jz4740_groups,
> +	.num_groups = ARRAY_SIZE(jz4740_groups),
> +	.functions = jz4740_functions,
> +	.num_functions = ARRAY_SIZE(jz4740_functions),
> +	.pull_ups = jz4740_pull_ups,
> +	.pull_downs = jz4740_pull_downs,
> +};
> +
> +static const u32 jz4770_pull_ups[6] = {
> +	0x3fffffff, 0xfff0030c, 0xffffffff, 0xffff4fff, 0xfffffb7c, 
> 0xffa7f00f,
> +};
> +
> +static const u32 jz4770_pull_downs[6] = {
> +	0x00000000, 0x000f0c03, 0x00000000, 0x0000b000, 0x00000483, 
> 0x00580ff0,
> +};
> +
> +static int jz4770_uart0_data_pins[] = { 0xa0, 0xa3, };
> +static int jz4770_uart0_hwflow_pins[] = { 0xa1, 0xa2, };
> +static int jz4770_uart1_data_pins[] = { 0x7a, 0x7c, };
> +static int jz4770_uart1_hwflow_pins[] = { 0x7b, 0x7d, };
> +static int jz4770_uart2_data_pins[] = { 0x66, 0x67, };
> +static int jz4770_uart2_hwflow_pins[] = { 0x65, 0x64, };
> +static int jz4770_uart3_data_pins[] = { 0x6c, 0x85, };
> +static int jz4770_uart3_hwflow_pins[] = { 0x88, 0x89, };
> +static int jz4770_uart4_data_pins[] = { 0x54, 0x4a, };
> +static int jz4770_mmc0_8bit_a_pins[] = { 0x04, 0x05, 0x06, 0x07, 0x18, 
> };
> +static int jz4770_mmc0_4bit_a_pins[] = { 0x15, 0x16, 0x17, };
> +static int jz4770_mmc0_1bit_a_pins[] = { 0x12, 0x13, 0x14, };
> +static int jz4770_mmc0_4bit_e_pins[] = { 0x95, 0x96, 0x97, };
> +static int jz4770_mmc0_1bit_e_pins[] = { 0x9c, 0x9d, 0x94, };
> +static int jz4770_mmc1_4bit_d_pins[] = { 0x75, 0x76, 0x77, };
> +static int jz4770_mmc1_1bit_d_pins[] = { 0x78, 0x79, 0x74, };
> +static int jz4770_mmc1_4bit_e_pins[] = { 0x95, 0x96, 0x97, };
> +static int jz4770_mmc1_1bit_e_pins[] = { 0x9c, 0x9d, 0x94, };
> +static int jz4770_nemc_data_pins[] = {
> +	0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,
> +};
> +static int jz4770_nemc_cle_ale_pins[] = { 0x20, 0x21, };
> +static int jz4770_nemc_addr_pins[] = { 0x22, 0x23, 0x24, 0x25, };
> +static int jz4770_nemc_rd_we_pins[] = { 0x10, 0x11, };
> +static int jz4770_nemc_frd_fwe_pins[] = { 0x12, 0x13, };
> +static int jz4770_nemc_cs1_pins[] = { 0x15, };
> +static int jz4770_nemc_cs2_pins[] = { 0x16, };
> +static int jz4770_nemc_cs3_pins[] = { 0x17, };
> +static int jz4770_nemc_cs4_pins[] = { 0x18, };
> +static int jz4770_nemc_cs5_pins[] = { 0x19, };
> +static int jz4770_nemc_cs6_pins[] = { 0x1a, };
> +static int jz4770_i2c0_pins[] = { 0x6e, 0x6f, };
> +static int jz4770_i2c1_pins[] = { 0x8e, 0x8f, };
> +static int jz4770_i2c2_pins[] = { 0xb0, 0xb1, };
> +static int jz4770_i2c3_pins[] = { 0x6a, 0x6b, };
> +static int jz4770_i2c4_e_pins[] = { 0x8c, 0x8d, };
> +static int jz4770_i2c4_f_pins[] = { 0xb9, 0xb8, };
> +static int jz4770_cim_pins[] = {
> +	0x26, 0x27, 0x28, 0x29, 0x2a, 0x2b, 0x2c, 0x2d, 0x2e, 0x2f, 0x30, 
> 0x31,
> +};
> +static int jz4770_lcd_32bit_pins[] = {
> +	0x40, 0x41, 0x42, 0x43, 0x44, 0x45, 0x46, 0x47,
> +	0x48, 0x49, 0x4a, 0x4b, 0x4c, 0x4d, 0x4e, 0x4f,
> +	0x50, 0x51, 0x52, 0x53, 0x54, 0x55, 0x56, 0x57,
> +	0x58, 0x59, 0x51,
> +};
> +static int jz4770_pwm_pwm0_pins[] = { 0x80, };
> +static int jz4770_pwm_pwm1_pins[] = { 0x81, };
> +static int jz4770_pwm_pwm2_pins[] = { 0x82, };
> +static int jz4770_pwm_pwm3_pins[] = { 0x83, };
> +static int jz4770_pwm_pwm4_pins[] = { 0x84, };
> +static int jz4770_pwm_pwm5_pins[] = { 0x85, };
> +static int jz4770_pwm_pwm6_pins[] = { 0x6a, };
> +static int jz4770_pwm_pwm7_pins[] = { 0x6b, };
> +
> +static int jz4770_uart0_data_funcs[] = { 0, 0, };
> +static int jz4770_uart0_hwflow_funcs[] = { 0, 0, };
> +static int jz4770_uart1_data_funcs[] = { 0, 0, };
> +static int jz4770_uart1_hwflow_funcs[] = { 0, 0, };
> +static int jz4770_uart2_data_funcs[] = { 1, 1, };
> +static int jz4770_uart2_hwflow_funcs[] = { 1, 1, };
> +static int jz4770_uart3_data_funcs[] = { 0, 1, };
> +static int jz4770_uart3_hwflow_funcs[] = { 0, 0, };
> +static int jz4770_uart4_data_funcs[] = { 2, 2, };
> +static int jz4770_mmc0_8bit_a_funcs[] = { 1, 1, 1, 1, 1, };
> +static int jz4770_mmc0_4bit_a_funcs[] = { 1, 1, 1, };
> +static int jz4770_mmc0_1bit_a_funcs[] = { 1, 1, 0, };
> +static int jz4770_mmc0_4bit_e_funcs[] = { 0, 0, 0, };
> +static int jz4770_mmc0_1bit_e_funcs[] = { 0, 0, 0, };
> +static int jz4770_mmc1_4bit_d_funcs[] = { 0, 0, 0, };
> +static int jz4770_mmc1_1bit_d_funcs[] = { 0, 0, 0, };
> +static int jz4770_mmc1_4bit_e_funcs[] = { 1, 1, 1, };
> +static int jz4770_mmc1_1bit_e_funcs[] = { 1, 1, 1, };
> +static int jz4770_nemc_data_funcs[] = { 0, 0, 0, 0, 0, 0, 0, 0, };
> +static int jz4770_nemc_cle_ale_funcs[] = { 0, 0, };
> +static int jz4770_nemc_addr_funcs[] = { 0, 0, 0, 0, };
> +static int jz4770_nemc_rd_we_funcs[] = { 0, 0, };
> +static int jz4770_nemc_frd_fwe_funcs[] = { 0, 0, };
> +static int jz4770_nemc_cs1_funcs[] = { 0, };
> +static int jz4770_nemc_cs2_funcs[] = { 0, };
> +static int jz4770_nemc_cs3_funcs[] = { 0, };
> +static int jz4770_nemc_cs4_funcs[] = { 0, };
> +static int jz4770_nemc_cs5_funcs[] = { 0, };
> +static int jz4770_nemc_cs6_funcs[] = { 0, };
> +static int jz4770_i2c0_funcs[] = { 0, 0, };
> +static int jz4770_i2c1_funcs[] = { 0, 0, };
> +static int jz4770_i2c2_funcs[] = { 2, 2, };
> +static int jz4770_i2c3_funcs[] = { 1, 1, };
> +static int jz4770_i2c4_e_funcs[] = { 1, 1, };
> +static int jz4770_i2c4_f_funcs[] = { 1, 1, };
> +static int jz4770_cim_funcs[] = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
> };
> +static int jz4770_lcd_32bit_funcs[] = {
> +	0, 0, 0, 0, 0, 0, 0, 0,
> +	0, 0, 0, 0, 0, 0, 0, 0,
> +	0, 0, 0,
> +};
> +static int jz4770_pwm_pwm0_funcs[] = { 0, };
> +static int jz4770_pwm_pwm1_funcs[] = { 0, };
> +static int jz4770_pwm_pwm2_funcs[] = { 0, };
> +static int jz4770_pwm_pwm3_funcs[] = { 0, };
> +static int jz4770_pwm_pwm4_funcs[] = { 0, };
> +static int jz4770_pwm_pwm5_funcs[] = { 0, };
> +static int jz4770_pwm_pwm6_funcs[] = { 0, };
> +static int jz4770_pwm_pwm7_funcs[] = { 0, };
> +
> +static const struct group_desc jz4770_groups[] = {
> +	INGENIC_PIN_GROUP("uart0-data", jz4770_uart0_data),
> +	INGENIC_PIN_GROUP("uart0-hwflow", jz4770_uart0_hwflow),
> +	INGENIC_PIN_GROUP("uart1-data", jz4770_uart1_data),
> +	INGENIC_PIN_GROUP("uart1-hwflow", jz4770_uart1_hwflow),
> +	INGENIC_PIN_GROUP("uart2-data", jz4770_uart2_data),
> +	INGENIC_PIN_GROUP("uart2-hwflow", jz4770_uart2_hwflow),
> +	INGENIC_PIN_GROUP("uart3-data", jz4770_uart3_data),
> +	INGENIC_PIN_GROUP("uart3-hwflow", jz4770_uart3_hwflow),
> +	INGENIC_PIN_GROUP("uart4-data", jz4770_uart4_data),
> +	INGENIC_PIN_GROUP("mmc0-8bit-a", jz4770_mmc0_8bit_a),
> +	INGENIC_PIN_GROUP("mmc0-4bit-a", jz4770_mmc0_4bit_a),
> +	INGENIC_PIN_GROUP("mmc0-1bit-a", jz4770_mmc0_1bit_a),
> +	INGENIC_PIN_GROUP("mmc0-4bit-e", jz4770_mmc0_4bit_e),
> +	INGENIC_PIN_GROUP("mmc0-1bit-e", jz4770_mmc0_1bit_e),
> +	INGENIC_PIN_GROUP("mmc1-4bit-d", jz4770_mmc1_4bit_d),
> +	INGENIC_PIN_GROUP("mmc1-1bit-d", jz4770_mmc1_1bit_d),
> +	INGENIC_PIN_GROUP("mmc1-4bit-e", jz4770_mmc1_4bit_e),
> +	INGENIC_PIN_GROUP("mmc1-1bit-e", jz4770_mmc1_1bit_e),
> +	INGENIC_PIN_GROUP("nemc-data", jz4770_nemc_data),
> +	INGENIC_PIN_GROUP("nemc-cle-ale", jz4770_nemc_cle_ale),
> +	INGENIC_PIN_GROUP("nemc-addr", jz4770_nemc_addr),
> +	INGENIC_PIN_GROUP("nemc-rd-we", jz4770_nemc_rd_we),
> +	INGENIC_PIN_GROUP("nemc-frd-fwe", jz4770_nemc_frd_fwe),
> +	INGENIC_PIN_GROUP("nemc-cs1", jz4770_nemc_cs1),
> +	INGENIC_PIN_GROUP("nemc-cs2", jz4770_nemc_cs2),
> +	INGENIC_PIN_GROUP("nemc-cs3", jz4770_nemc_cs3),
> +	INGENIC_PIN_GROUP("nemc-cs4", jz4770_nemc_cs4),
> +	INGENIC_PIN_GROUP("nemc-cs5", jz4770_nemc_cs5),
> +	INGENIC_PIN_GROUP("nemc-cs6", jz4770_nemc_cs6),
> +	INGENIC_PIN_GROUP("i2c0-data", jz4770_i2c0),
> +	INGENIC_PIN_GROUP("i2c1-data", jz4770_i2c1),
> +	INGENIC_PIN_GROUP("i2c2-data", jz4770_i2c2),
> +	INGENIC_PIN_GROUP("i2c3-data", jz4770_i2c3),
> +	INGENIC_PIN_GROUP("i2c4-data-e", jz4770_i2c4_e),
> +	INGENIC_PIN_GROUP("i2c4-data-f", jz4770_i2c4_f),
> +	INGENIC_PIN_GROUP("cim-data", jz4770_cim),
> +	INGENIC_PIN_GROUP("lcd-32bit", jz4770_lcd_32bit),
> +	{ "lcd-no-pins", },
> +	INGENIC_PIN_GROUP("pwm0", jz4770_pwm_pwm0),
> +	INGENIC_PIN_GROUP("pwm1", jz4770_pwm_pwm1),
> +	INGENIC_PIN_GROUP("pwm2", jz4770_pwm_pwm2),
> +	INGENIC_PIN_GROUP("pwm3", jz4770_pwm_pwm3),
> +	INGENIC_PIN_GROUP("pwm4", jz4770_pwm_pwm4),
> +	INGENIC_PIN_GROUP("pwm5", jz4770_pwm_pwm5),
> +	INGENIC_PIN_GROUP("pwm6", jz4770_pwm_pwm6),
> +	INGENIC_PIN_GROUP("pwm7", jz4770_pwm_pwm7),
> +};
> +
> +static const char *jz4770_uart0_groups[] = { "uart0-data", 
> "uart0-hwflow", };
> +static const char *jz4770_uart1_groups[] = { "uart1-data", 
> "uart1-hwflow", };
> +static const char *jz4770_uart2_groups[] = { "uart2-data", 
> "uart2-hwflow", };
> +static const char *jz4770_uart3_groups[] = { "uart3-data", 
> "uart3-hwflow", };
> +static const char *jz4770_uart4_groups[] = { "uart4-data", };
> +static const char *jz4770_mmc0_groups[] = {
> +	"mmc0-8bit-a", "mmc0-4bit-a", "mmc0-1bit-a",
> +	"mmc0-1bit-e", "mmc0-4bit-e",
> +};
> +static const char *jz4770_mmc1_groups[] = {
> +	"mmc1-1bit-d", "mmc1-4bit-d", "mmc1-1bit-e", "mmc1-4bit-e",
> +};
> +static const char *jz4770_nemc_groups[] = {
> +	"nemc-data", "nemc-cle-ale", "nemc-addr", "nemc-rd-we", 
> "nemc-frd-fwe",
> +};
> +static const char *jz4770_cs1_groups[] = { "nemc-cs1", };
> +static const char *jz4770_cs6_groups[] = { "nemc-cs6", };
> +static const char *jz4770_i2c0_groups[] = { "i2c0-data", };
> +static const char *jz4770_i2c1_groups[] = { "i2c1-data", };
> +static const char *jz4770_i2c2_groups[] = { "i2c2-data", };
> +static const char *jz4770_i2c3_groups[] = { "i2c3-data", };
> +static const char *jz4770_i2c4_groups[] = { "i2c4-data-e", 
> "i2c4-data-f", };
> +static const char *jz4770_cim_groups[] = { "cim-data", };
> +static const char *jz4770_lcd_groups[] = { "lcd-32bit", "lcd-no-pins", 
> };
> +static const char *jz4770_pwm0_groups[] = { "pwm0", };
> +static const char *jz4770_pwm1_groups[] = { "pwm1", };
> +static const char *jz4770_pwm2_groups[] = { "pwm2", };
> +static const char *jz4770_pwm3_groups[] = { "pwm3", };
> +static const char *jz4770_pwm4_groups[] = { "pwm4", };
> +static const char *jz4770_pwm5_groups[] = { "pwm5", };
> +static const char *jz4770_pwm6_groups[] = { "pwm6", };
> +static const char *jz4770_pwm7_groups[] = { "pwm7", };
> +
> +static const struct function_desc jz4770_functions[] = {
> +	{ "uart0", jz4770_uart0_groups, ARRAY_SIZE(jz4770_uart0_groups), },
> +	{ "uart1", jz4770_uart1_groups, ARRAY_SIZE(jz4770_uart1_groups), },
> +	{ "uart2", jz4770_uart2_groups, ARRAY_SIZE(jz4770_uart2_groups), },
> +	{ "uart3", jz4770_uart3_groups, ARRAY_SIZE(jz4770_uart3_groups), },
> +	{ "uart4", jz4770_uart4_groups, ARRAY_SIZE(jz4770_uart4_groups), },
> +	{ "mmc0", jz4770_mmc0_groups, ARRAY_SIZE(jz4770_mmc0_groups), },
> +	{ "mmc1", jz4770_mmc1_groups, ARRAY_SIZE(jz4770_mmc1_groups), },
> +	{ "nemc", jz4770_nemc_groups, ARRAY_SIZE(jz4770_nemc_groups), },
> +	{ "nemc-cs1", jz4770_cs1_groups, ARRAY_SIZE(jz4770_cs1_groups), },
> +	{ "nemc-cs6", jz4770_cs6_groups, ARRAY_SIZE(jz4770_cs6_groups), },
> +	{ "i2c0", jz4770_i2c0_groups, ARRAY_SIZE(jz4770_i2c0_groups), },
> +	{ "i2c1", jz4770_i2c1_groups, ARRAY_SIZE(jz4770_i2c1_groups), },
> +	{ "i2c2", jz4770_i2c2_groups, ARRAY_SIZE(jz4770_i2c2_groups), },
> +	{ "i2c3", jz4770_i2c3_groups, ARRAY_SIZE(jz4770_i2c3_groups), },
> +	{ "i2c4", jz4770_i2c4_groups, ARRAY_SIZE(jz4770_i2c4_groups), },
> +	{ "cim", jz4770_cim_groups, ARRAY_SIZE(jz4770_cim_groups), },
> +	{ "lcd", jz4770_lcd_groups, ARRAY_SIZE(jz4770_lcd_groups), },
> +	{ "pwm0", jz4770_pwm0_groups, ARRAY_SIZE(jz4770_pwm0_groups), },
> +	{ "pwm1", jz4770_pwm1_groups, ARRAY_SIZE(jz4770_pwm1_groups), },
> +	{ "pwm2", jz4770_pwm2_groups, ARRAY_SIZE(jz4770_pwm2_groups), },
> +	{ "pwm3", jz4770_pwm3_groups, ARRAY_SIZE(jz4770_pwm3_groups), },
> +	{ "pwm4", jz4770_pwm4_groups, ARRAY_SIZE(jz4770_pwm4_groups), },
> +	{ "pwm5", jz4770_pwm5_groups, ARRAY_SIZE(jz4770_pwm5_groups), },
> +	{ "pwm6", jz4770_pwm6_groups, ARRAY_SIZE(jz4770_pwm6_groups), },
> +	{ "pwm7", jz4770_pwm7_groups, ARRAY_SIZE(jz4770_pwm7_groups), },
> +};
> +
> +static const struct ingenic_chip_info jz4770_chip_info = {
> +	.num_chips = 6,
> +	.groups = jz4770_groups,
> +	.num_groups = ARRAY_SIZE(jz4770_groups),
> +	.functions = jz4770_functions,
> +	.num_functions = ARRAY_SIZE(jz4770_functions),
> +	.pull_ups = jz4770_pull_ups,
> +	.pull_downs = jz4770_pull_downs,
> +};
> +
> +static inline void ingenic_config_pin(struct ingenic_pinctrl *jzpc,
> +		unsigned int pin, u8 reg, bool set)
> +{
> +	unsigned int idx = pin % PINS_PER_GPIO_CHIP;
> +	unsigned int offt = pin / PINS_PER_GPIO_CHIP;
> +
> +	regmap_write(jzpc->map, offt * 0x100 +
> +			(set ? REG_SET(reg) : REG_CLEAR(reg)), BIT(idx));
> +}
> +
> +static inline bool ingenic_get_pin_config(struct ingenic_pinctrl 
> *jzpc,
> +		unsigned int pin, u8 reg)
> +{
> +	unsigned int idx = pin % PINS_PER_GPIO_CHIP;
> +	unsigned int offt = pin / PINS_PER_GPIO_CHIP;
> +	unsigned int val;
> +
> +	regmap_read(jzpc->map, offt * 0x100 + reg, &val);
> +
> +	return val & BIT(idx);
> +}
> +
> +static struct pinctrl_ops ingenic_pctlops = {
> +	.get_groups_count = pinctrl_generic_get_group_count,
> +	.get_group_name = pinctrl_generic_get_group_name,
> +	.get_group_pins = pinctrl_generic_get_group_pins,
> +	.dt_node_to_map = pinconf_generic_dt_node_to_map_all,
> +	.dt_free_map = pinconf_generic_dt_free_map,
> +};
> +
> +static int ingenic_pinmux_set_pin_fn(struct ingenic_pinctrl *jzpc,
> +		int pin, int func)
> +{
> +	unsigned int idx = pin % PINS_PER_GPIO_CHIP;
> +	unsigned int offt = pin / PINS_PER_GPIO_CHIP;
> +
> +	dev_dbg(jzpc->dev, "set pin P%c%u to function %u\n",
> +			'A' + offt, idx, func);
> +
> +	if (jzpc->version >= ID_JZ4770) {
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_INT, false);
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_MSK, false);
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_PAT1, func & 0x2);
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_PAT0, func & 0x1);
> +	} else {
> +		ingenic_config_pin(jzpc, pin, JZ4740_GPIO_FUNC, true);
> +		ingenic_config_pin(jzpc, pin, JZ4740_GPIO_TRIG, func & 0x2);
> +		ingenic_config_pin(jzpc, pin, JZ4740_GPIO_SELECT, func > 0);
> +	}
> +
> +	return 0;
> +}
> +
> +static int ingenic_pinmux_set_mux(struct pinctrl_dev *pctldev,
> +		unsigned int selector, unsigned int group)
> +{
> +	struct ingenic_pinctrl *jzpc = pinctrl_dev_get_drvdata(pctldev);
> +	struct function_desc *func;
> +	struct group_desc *grp;
> +	unsigned int i;
> +
> +	func = pinmux_generic_get_function(pctldev, selector);
> +	if (!func)
> +		return -EINVAL;
> +
> +	grp = pinctrl_generic_get_group(pctldev, group);
> +	if (!grp)
> +		return -EINVAL;
> +
> +	dev_dbg(pctldev->dev, "enable function %s group %s\n",
> +		func->name, grp->name);
> +
> +	for (i = 0; i < grp->num_pins; i++) {
> +		int *pin_modes = grp->data;
> +
> +		ingenic_pinmux_set_pin_fn(jzpc, grp->pins[i], pin_modes[i]);
> +	}
> +
> +	return 0;
> +}
> +
> +static int ingenic_pinmux_gpio_set_direction(struct pinctrl_dev 
> *pctldev,
> +		struct pinctrl_gpio_range *range,
> +		unsigned int pin, bool input)
> +{
> +	struct ingenic_pinctrl *jzpc = pinctrl_dev_get_drvdata(pctldev);
> +	unsigned int idx = pin % PINS_PER_GPIO_CHIP;
> +	unsigned int offt = pin / PINS_PER_GPIO_CHIP;
> +
> +	dev_dbg(pctldev->dev, "set pin P%c%u to %sput\n",
> +			'A' + offt, idx, input ? "in" : "out");
> +
> +	if (jzpc->version >= ID_JZ4770) {
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_INT, false);
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_MSK, true);
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_PAT1, input);
> +	} else {
> +		ingenic_config_pin(jzpc, pin, JZ4740_GPIO_SELECT, false);
> +		ingenic_config_pin(jzpc, pin, JZ4740_GPIO_DIR, input);
> +		ingenic_config_pin(jzpc, pin, JZ4740_GPIO_FUNC, false);
> +	}
> +
> +	return 0;
> +}
> +
> +static struct pinmux_ops ingenic_pmxops = {
> +	.get_functions_count = pinmux_generic_get_function_count,
> +	.get_function_name = pinmux_generic_get_function_name,
> +	.get_function_groups = pinmux_generic_get_function_groups,
> +	.set_mux = ingenic_pinmux_set_mux,
> +	.gpio_set_direction = ingenic_pinmux_gpio_set_direction,
> +};
> +
> +static int ingenic_pinconf_get(struct pinctrl_dev *pctldev,
> +		unsigned int pin, unsigned long *config)
> +{
> +	struct ingenic_pinctrl *jzpc = pinctrl_dev_get_drvdata(pctldev);
> +	enum pin_config_param param = pinconf_to_config_param(*config);
> +	unsigned int idx = pin % PINS_PER_GPIO_CHIP;
> +	unsigned int offt = pin / PINS_PER_GPIO_CHIP;
> +	bool pull;
> +
> +	if (jzpc->version >= ID_JZ4770)
> +		pull = !ingenic_get_pin_config(jzpc, pin, JZ4770_GPIO_PEN);
> +	else
> +		pull = !ingenic_get_pin_config(jzpc, pin, JZ4740_GPIO_PULL_DIS);
> +
> +	switch (param) {
> +	case PIN_CONFIG_BIAS_DISABLE:
> +		if (pull)
> +			return -EINVAL;
> +		break;
> +
> +	case PIN_CONFIG_BIAS_PULL_UP:
> +		if (!pull || !(jzpc->info->pull_ups[offt] & BIT(idx)))
> +			return -EINVAL;
> +		break;
> +
> +	case PIN_CONFIG_BIAS_PULL_DOWN:
> +		if (!pull || !(jzpc->info->pull_downs[offt] & BIT(idx)))
> +			return -EINVAL;
> +		break;
> +
> +	default:
> +		return -ENOTSUPP;
> +	}
> +
> +	*config = pinconf_to_config_packed(param, 1);
> +	return 0;
> +}
> +
> +static void ingenic_set_bias(struct ingenic_pinctrl *jzpc,
> +		unsigned int pin, bool enabled)
> +{
> +	if (jzpc->version >= ID_JZ4770)
> +		ingenic_config_pin(jzpc, pin, JZ4770_GPIO_PEN, !enabled);
> +	else
> +		ingenic_config_pin(jzpc, pin, JZ4740_GPIO_PULL_DIS, !enabled);
> +}
> +
> +static int ingenic_pinconf_set(struct pinctrl_dev *pctldev, unsigned 
> int pin,
> +		unsigned long *configs, unsigned int num_configs)
> +{
> +	struct ingenic_pinctrl *jzpc = pinctrl_dev_get_drvdata(pctldev);
> +	unsigned int idx = pin % PINS_PER_GPIO_CHIP;
> +	unsigned int offt = pin / PINS_PER_GPIO_CHIP;
> +	unsigned int cfg;
> +
> +	for (cfg = 0; cfg < num_configs; cfg++) {
> +		switch (pinconf_to_config_param(configs[cfg])) {
> +		case PIN_CONFIG_BIAS_DISABLE:
> +		case PIN_CONFIG_BIAS_PULL_UP:
> +		case PIN_CONFIG_BIAS_PULL_DOWN:
> +			continue;
> +		default:
> +			return -ENOTSUPP;
> +		}
> +	}
> +
> +	for (cfg = 0; cfg < num_configs; cfg++) {
> +		switch (pinconf_to_config_param(configs[cfg])) {
> +		case PIN_CONFIG_BIAS_DISABLE:
> +			dev_dbg(jzpc->dev, "disable pull-over for pin P%c%u\n",
> +					'A' + offt, idx);
> +			ingenic_set_bias(jzpc, pin, false);
> +			break;
> +
> +		case PIN_CONFIG_BIAS_PULL_UP:
> +			if (!(jzpc->info->pull_ups[offt] & BIT(idx)))
> +				return -EINVAL;
> +			dev_dbg(jzpc->dev, "set pull-up for pin P%c%u\n",
> +					'A' + offt, idx);
> +			ingenic_set_bias(jzpc, pin, true);
> +			break;
> +
> +		case PIN_CONFIG_BIAS_PULL_DOWN:
> +			if (!(jzpc->info->pull_downs[offt] & BIT(idx)))
> +				return -EINVAL;
> +			dev_dbg(jzpc->dev, "set pull-down for pin P%c%u\n",
> +					'A' + offt, idx);
> +			ingenic_set_bias(jzpc, pin, true);
> +			break;
> +
> +		default:
> +			unreachable();
> +		}
> +	}
> +
> +	return 0;
> +}
> +
> +static int ingenic_pinconf_group_get(struct pinctrl_dev *pctldev,
> +		unsigned int group, unsigned long *config)
> +{
> +	const unsigned int *pins;
> +	unsigned int i, npins, old = 0;
> +	int ret;
> +
> +	ret = pinctrl_generic_get_group_pins(pctldev, group, &pins, &npins);
> +	if (ret)
> +		return ret;
> +
> +	for (i = 0; i < npins; i++) {
> +		if (ingenic_pinconf_get(pctldev, pins[i], config))
> +			return -ENOTSUPP;
> +
> +		/* configs do not match between two pins */
> +		if (i && (old != *config))
> +			return -ENOTSUPP;
> +
> +		old = *config;
> +	}
> +
> +	return 0;
> +}
> +
> +static int ingenic_pinconf_group_set(struct pinctrl_dev *pctldev,
> +		unsigned int group, unsigned long *configs,
> +		unsigned int num_configs)
> +{
> +	const unsigned int *pins;
> +	unsigned int i, npins;
> +	int ret;
> +
> +	ret = pinctrl_generic_get_group_pins(pctldev, group, &pins, &npins);
> +	if (ret)
> +		return ret;
> +
> +	for (i = 0; i < npins; i++) {
> +		ret = ingenic_pinconf_set(pctldev,
> +				pins[i], configs, num_configs);
> +		if (ret)
> +			return ret;
> +	}
> +
> +	return 0;
> +}
> +
> +static struct pinconf_ops ingenic_confops = {
> +	.is_generic = true,
> +	.pin_config_get = ingenic_pinconf_get,
> +	.pin_config_set = ingenic_pinconf_set,
> +	.pin_config_group_get = ingenic_pinconf_group_get,
> +	.pin_config_group_set = ingenic_pinconf_group_set,
> +};
> +
> +static const struct regmap_config ingenic_pinctrl_regmap_config = {
> +	.reg_bits = 32,
> +	.val_bits = 32,
> +	.reg_stride = 4,
> +};
> +
> +static const struct of_device_id ingenic_pinctrl_of_match[] = {
> +	{ .compatible = "ingenic,jz4740-pinctrl", .data = (void *) ID_JZ4740 
> },
> +	{ .compatible = "ingenic,jz4770-pinctrl", .data = (void *) ID_JZ4770 
> },
> +	{ .compatible = "ingenic,jz4780-pinctrl", .data = (void *) ID_JZ4780 
> },
> +	{},
> +};
> +
> +int ingenic_pinctrl_probe(struct platform_device *pdev)
> +{
> +	struct device *dev = &pdev->dev;
> +	struct ingenic_pinctrl *jzpc;
> +	struct pinctrl_desc *pctl_desc;
> +	void __iomem *base;
> +	const struct platform_device_id *id = platform_get_device_id(pdev);
> +	const struct of_device_id *of_id = of_match_device(
> +			ingenic_pinctrl_of_match, dev);
> +	const struct ingenic_chip_info *chip_info;
> +	unsigned int i;
> +	int err;
> +
> +	jzpc = devm_kzalloc(dev, sizeof(*jzpc), GFP_KERNEL);
> +	if (!jzpc)
> +		return -ENOMEM;
> +
> +	base = devm_ioremap_resource(dev,
> +			platform_get_resource(pdev, IORESOURCE_MEM, 0));
> +	if (IS_ERR(base)) {
> +		dev_err(dev, "Failed to ioremap registers\n");
> +		return PTR_ERR(base);
> +	}
> +
> +	jzpc->map = devm_regmap_init_mmio(dev, base,
> +			&ingenic_pinctrl_regmap_config);
> +	if (IS_ERR(jzpc->map)) {
> +		dev_err(dev, "Failed to create regmap\n");
> +		return PTR_ERR(jzpc->map);
> +	}
> +
> +	jzpc->dev = dev;
> +
> +	if (of_id)
> +		jzpc->version = (enum jz_version)of_id->data;
> +	else
> +		jzpc->version = (enum jz_version)id->driver_data;
> +
> +	if (jzpc->version >= ID_JZ4770)
> +		chip_info = &jz4770_chip_info;
> +	else
> +		chip_info = &jz4740_chip_info;
> +	jzpc->info = chip_info;
> +
> +	pctl_desc = devm_kzalloc(&pdev->dev, sizeof(*pctl_desc), GFP_KERNEL);
> +	if (!pctl_desc)
> +		return -ENOMEM;
> +
> +	/* fill in pinctrl_desc structure */
> +	pctl_desc->name = dev_name(dev);
> +	pctl_desc->owner = THIS_MODULE;
> +	pctl_desc->pctlops = &ingenic_pctlops;
> +	pctl_desc->pmxops = &ingenic_pmxops;
> +	pctl_desc->confops = &ingenic_confops;
> +	pctl_desc->npins = chip_info->num_chips * PINS_PER_GPIO_CHIP;
> +	pctl_desc->pins = jzpc->pdesc = devm_kzalloc(&pdev->dev,
> +			sizeof(*jzpc->pdesc) * pctl_desc->npins, GFP_KERNEL);
> +	if (!jzpc->pdesc)
> +		return -ENOMEM;
> +
> +	for (i = 0; i < pctl_desc->npins; i++) {
> +		jzpc->pdesc[i].number = i;
> +		jzpc->pdesc[i].name = kasprintf(GFP_KERNEL, "P%c%d",
> +						'A' + (i / PINS_PER_GPIO_CHIP),
> +						i % PINS_PER_GPIO_CHIP);
> +	}
> +
> +	jzpc->pctl = devm_pinctrl_register(dev, pctl_desc, jzpc);
> +	if (!jzpc->pctl) {
> +		dev_err(dev, "Failed to register pinctrl\n");
> +		return -EINVAL;
> +	}
> +
> +	for (i = 0; i < chip_info->num_groups; i++) {
> +		const struct group_desc *group = &chip_info->groups[i];
> +
> +		err = pinctrl_generic_add_group(jzpc->pctl, group->name,
> +				group->pins, group->num_pins, group->data);
> +		if (err) {
> +			dev_err(dev, "Failed to register group %s\n",
> +					group->name);
> +			return err;
> +		}
> +	}
> +
> +	for (i = 0; i < chip_info->num_functions; i++) {
> +		const struct function_desc *func = &chip_info->functions[i];
> +
> +		err = pinmux_generic_add_function(jzpc->pctl, func->name,
> +				func->group_names, func->num_group_names,
> +				func->data);
> +		if (err) {
> +			dev_err(dev, "Failed to register function %s\n",
> +					func->name);
> +			return err;
> +		}
> +	}
> +
> +	dev_set_drvdata(dev, jzpc->map);
> +
> +	if (dev->of_node) {
> +		err = of_platform_populate(dev->of_node, NULL, NULL, dev);
> +		if (err) {
> +			dev_err(dev, "Failed to probe GPIO devices\n");
> +			return err;
> +		}
> +	}
> +
> +	return 0;
> +}
> +
> +static const struct platform_device_id ingenic_pinctrl_ids[] = {
> +	{ "jz4740-pinctrl", ID_JZ4740 },
> +	{ "jz4770-pinctrl", ID_JZ4770 },
> +	{ "jz4780-pinctrl", ID_JZ4780 },
> +	{},
> +};
> +
> +static struct platform_driver ingenic_pinctrl_driver = {
> +	.driver = {
> +		.name = "pinctrl-ingenic",
> +		.of_match_table = of_match_ptr(ingenic_pinctrl_of_match),
> +		.suppress_bind_attrs = true,
> +	},
> +	.probe = ingenic_pinctrl_probe,
> +	.id_table = ingenic_pinctrl_ids,
> +};
> +
> +static int __init ingenic_pinctrl_drv_register(void)
> +{
> +	return platform_driver_register(&ingenic_pinctrl_driver);
> +}
> +postcore_initcall(ingenic_pinctrl_drv_register);

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ