Message ID | 1487756858-16730-12-git-send-email-ley.foon.tan@intel.com |
---|---|
State | Superseded |
Delegated to: | Marek Vasut |
Headers | show |
On 02/22/2017 10:47 AM, Ley Foon Tan wrote: > Add pinmux support for Arria 10. > > Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com> > Signed-off-by: Ley Foon Tan <ley.foon.tan@intel.com> > --- > arch/arm/mach-socfpga/Makefile | 1 + > arch/arm/mach-socfpga/include/mach/pinmux.h | 15 +++++ > arch/arm/mach-socfpga/pinmux_arria10.c | 98 +++++++++++++++++++++++++++++ > 3 files changed, 114 insertions(+) > create mode 100644 arch/arm/mach-socfpga/include/mach/pinmux.h > create mode 100644 arch/arm/mach-socfpga/pinmux_arria10.c > > diff --git a/arch/arm/mach-socfpga/Makefile b/arch/arm/mach-socfpga/Makefile > index 9c4617f..68d55e4 100644 > --- a/arch/arm/mach-socfpga/Makefile > +++ b/arch/arm/mach-socfpga/Makefile > @@ -12,6 +12,7 @@ obj-y += misc.o timer.o reset_manager.o clock_manager.o \ > > obj-$(CONFIG_TARGET_SOCFPGA_ARRIA10) += clock_manager_arria10.o \ > misc_arria10.o \ > + pinmux_arria10.o \ > reset_manager_arria10.o > > obj-$(CONFIG_SPL_BUILD) += spl.o freeze_controller.o > diff --git a/arch/arm/mach-socfpga/include/mach/pinmux.h b/arch/arm/mach-socfpga/include/mach/pinmux.h > new file mode 100644 > index 0000000..c5d5dd6 > --- /dev/null > +++ b/arch/arm/mach-socfpga/include/mach/pinmux.h > @@ -0,0 +1,15 @@ > +/* > + * Copyright (C) 2016-2017 Intel Corporation <www.intel.com> > + * > + * SPDX-License-Identifier: GPL-2.0 > + */ > + > +#ifndef _PINMUX_H_ > +#define _PINMUX_H_ Use [space] instead of [tab] after #ifdef/#define > +#ifndef __ASSEMBLY__ > +int config_dedicated_pins(const void *blob); > +int config_pins(const void *blob, const char *pin_grp); > +#endif > + > +#endif /* _PINMUX_H_ */ > diff --git a/arch/arm/mach-socfpga/pinmux_arria10.c b/arch/arm/mach-socfpga/pinmux_arria10.c > new file mode 100644 > index 0000000..04acaa2 > --- /dev/null > +++ b/arch/arm/mach-socfpga/pinmux_arria10.c > @@ -0,0 +1,98 @@ > +/* > + * Copyright (C) 2016-2017 Intel Corporation <www.intel.com> > + * > + * SPDX-License-Identifier: GPL-2.0 > + */ > + > +#include <common.h> > +#include <asm/io.h> > +#include <fdtdec.h> > +#include <asm/arch/pinmux.h> > + > +static int do_pinctr_pin(const void *blob, int child, const char *node_name) > +{ > + int len; > + fdt_addr_t base_addr; > + fdt_size_t size; > + const u32 *cell; > + u32 offset, value; > + > + base_addr = fdtdec_get_addr_size(blob, child, "reg", &size); > + if (base_addr != FDT_ADDR_T_NONE) { > + cell = fdt_getprop(blob, child, "pinctrl-single,pins", > + &len); > + if (cell != NULL) { > + debug("%p %d\n", cell, len); > + for (; len > 0; len -= (2*sizeof(u32))) { 2 * sizeof(u32) , spacing What about len == 0 , not used ? > + offset = fdt32_to_cpu(*cell++); > + value = fdt32_to_cpu(*cell++); > + debug("<0x%x 0x%x>\n", offset, value); > + writel(value, base_addr + offset); > + } > + return 0; > + } > + } > + return -EFAULT; > +} > + > +static int do_pinctrl_pins(const void *blob, int node, const char *child_name) > +{ > + int child, len; > + const char *node_name; > + > + child = fdt_first_subnode(blob, node); > + > + if (child < 0) > + return -EINVAL; > + > + node_name = fdt_get_name(blob, child, &len); > + > + while (node_name) { > + if (!strcmp(child_name, node_name)) > + return do_pinctr_pin(blob, child, node_name); > + > + child = fdt_next_subnode(blob, child); > + > + if (child < 0) > + break; > + > + node_name = fdt_get_name(blob, child, &len); > + } > + > + return -EFAULT; > +} > + > +int config_dedicated_pins(const void *blob) > +{ > + int node; > + > + node = fdtdec_next_compatible(blob, 0, > + COMPAT_ALTERA_SOCFPGA_PINCTRL_SINGLE); > + > + if (node < 0) > + return -EINVAL; > + > + if (do_pinctrl_pins(blob, node, "dedicated_cfg")) > + return -EFAULT; > + > + if (do_pinctrl_pins(blob, node, "dedicated")) > + return -EFAULT; > + > + return 0; > +} > + > +int config_pins(const void *blob, const char *pin_grp) > +{ > + int node; > + > + node = fdtdec_next_compatible(blob, 0, > + COMPAT_ALTERA_SOCFPGA_PINCTRL_SINGLE); > + > + if (node < 0) > + return -EINVAL; > + > + if (do_pinctrl_pins(blob, node, pin_grp)) > + return -EFAULT; > + > + return 0; > +} >
On Sab, 2017-02-25 at 22:41 +0100, Marek Vasut wrote: > On 02/22/2017 10:47 AM, Ley Foon Tan wrote: > > > > Add pinmux support for Arria 10. > > > > Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com> > > Signed-off-by: Ley Foon Tan <ley.foon.tan@intel.com> > > --- > > arch/arm/mach-socfpga/Makefile | 1 + > > arch/arm/mach-socfpga/include/mach/pinmux.h | 15 +++++ > > arch/arm/mach-socfpga/pinmux_arria10.c | 98 > > +++++++++++++++++++++++++++++ > > 3 files changed, 114 insertions(+) > > create mode 100644 arch/arm/mach-socfpga/include/mach/pinmux.h > > create mode 100644 arch/arm/mach-socfpga/pinmux_arria10.c > > > > diff --git a/arch/arm/mach-socfpga/Makefile b/arch/arm/mach- > > socfpga/Makefile > > index 9c4617f..68d55e4 100644 > > --- a/arch/arm/mach-socfpga/Makefile > > +++ b/arch/arm/mach-socfpga/Makefile > > @@ -12,6 +12,7 @@ obj-y += misc.o timer.o reset_manager.o > > clock_manager.o \ > > > > obj-$(CONFIG_TARGET_SOCFPGA_ARRIA10) += clock_manager_arria10.o \ > > misc_arria10.o > > \ > > + pinmux_arria10.o \ > > reset_manager_arria10.o > > > > obj-$(CONFIG_SPL_BUILD) += spl.o freeze_controller.o > > diff --git a/arch/arm/mach-socfpga/include/mach/pinmux.h > > b/arch/arm/mach-socfpga/include/mach/pinmux.h > > new file mode 100644 > > index 0000000..c5d5dd6 > > --- /dev/null > > +++ b/arch/arm/mach-socfpga/include/mach/pinmux.h > > @@ -0,0 +1,15 @@ > > +/* > > + * Copyright (C) 2016-2017 Intel Corporation <www.intel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0 > > + */ > > + > > +#ifndef _PINMUX_H_ > > +#define _PINMUX_H_ > Use [space] instead of [tab] after #ifdef/#define Okay. > > > > > +#ifndef __ASSEMBLY__ > > +int config_dedicated_pins(const void *blob); > > +int config_pins(const void *blob, const char *pin_grp); > > +#endif > > + > > +#endif /* _PINMUX_H_ */ > > diff --git a/arch/arm/mach-socfpga/pinmux_arria10.c > > b/arch/arm/mach-socfpga/pinmux_arria10.c > > new file mode 100644 > > index 0000000..04acaa2 > > --- /dev/null > > +++ b/arch/arm/mach-socfpga/pinmux_arria10.c > > @@ -0,0 +1,98 @@ > > +/* > > + * Copyright (C) 2016-2017 Intel Corporation <www.intel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0 > > + */ > > + > > +#include <common.h> > > +#include <asm/io.h> > > +#include <fdtdec.h> > > +#include <asm/arch/pinmux.h> > > + > > +static int do_pinctr_pin(const void *blob, int child, const char > > *node_name) > > +{ > > + int len; > > + fdt_addr_t base_addr; > > + fdt_size_t size; > > + const u32 *cell; > > + u32 offset, value; > > + > > + base_addr = fdtdec_get_addr_size(blob, child, "reg", > > &size); > > + if (base_addr != FDT_ADDR_T_NONE) { > > + cell = fdt_getprop(blob, child, "pinctrl- > > single,pins", > > + &len); > > + if (cell != NULL) { > > + debug("%p %d\n", cell, len); > > + for (; len > 0; len -= (2*sizeof(u32))) { > 2 * sizeof(u32) , spacing Okay. > > What about len == 0 , not used ? Will add the checking for len > 0. if (cell != NULL && len > 0) { } > > > > > + offset = fdt32_to_cpu(*cell++); > > + value = fdt32_to_cpu(*cell++); > > + debug("<0x%x 0x%x>\n", offset, > > value); > > + writel(value, base_addr + offset); > > + } > > + return 0; > > + } > > + } > > + return -EFAULT; > > +} [...] Regards Ley Foon
diff --git a/arch/arm/mach-socfpga/Makefile b/arch/arm/mach-socfpga/Makefile index 9c4617f..68d55e4 100644 --- a/arch/arm/mach-socfpga/Makefile +++ b/arch/arm/mach-socfpga/Makefile @@ -12,6 +12,7 @@ obj-y += misc.o timer.o reset_manager.o clock_manager.o \ obj-$(CONFIG_TARGET_SOCFPGA_ARRIA10) += clock_manager_arria10.o \ misc_arria10.o \ + pinmux_arria10.o \ reset_manager_arria10.o obj-$(CONFIG_SPL_BUILD) += spl.o freeze_controller.o diff --git a/arch/arm/mach-socfpga/include/mach/pinmux.h b/arch/arm/mach-socfpga/include/mach/pinmux.h new file mode 100644 index 0000000..c5d5dd6 --- /dev/null +++ b/arch/arm/mach-socfpga/include/mach/pinmux.h @@ -0,0 +1,15 @@ +/* + * Copyright (C) 2016-2017 Intel Corporation <www.intel.com> + * + * SPDX-License-Identifier: GPL-2.0 + */ + +#ifndef _PINMUX_H_ +#define _PINMUX_H_ + +#ifndef __ASSEMBLY__ +int config_dedicated_pins(const void *blob); +int config_pins(const void *blob, const char *pin_grp); +#endif + +#endif /* _PINMUX_H_ */ diff --git a/arch/arm/mach-socfpga/pinmux_arria10.c b/arch/arm/mach-socfpga/pinmux_arria10.c new file mode 100644 index 0000000..04acaa2 --- /dev/null +++ b/arch/arm/mach-socfpga/pinmux_arria10.c @@ -0,0 +1,98 @@ +/* + * Copyright (C) 2016-2017 Intel Corporation <www.intel.com> + * + * SPDX-License-Identifier: GPL-2.0 + */ + +#include <common.h> +#include <asm/io.h> +#include <fdtdec.h> +#include <asm/arch/pinmux.h> + +static int do_pinctr_pin(const void *blob, int child, const char *node_name) +{ + int len; + fdt_addr_t base_addr; + fdt_size_t size; + const u32 *cell; + u32 offset, value; + + base_addr = fdtdec_get_addr_size(blob, child, "reg", &size); + if (base_addr != FDT_ADDR_T_NONE) { + cell = fdt_getprop(blob, child, "pinctrl-single,pins", + &len); + if (cell != NULL) { + debug("%p %d\n", cell, len); + for (; len > 0; len -= (2*sizeof(u32))) { + offset = fdt32_to_cpu(*cell++); + value = fdt32_to_cpu(*cell++); + debug("<0x%x 0x%x>\n", offset, value); + writel(value, base_addr + offset); + } + return 0; + } + } + return -EFAULT; +} + +static int do_pinctrl_pins(const void *blob, int node, const char *child_name) +{ + int child, len; + const char *node_name; + + child = fdt_first_subnode(blob, node); + + if (child < 0) + return -EINVAL; + + node_name = fdt_get_name(blob, child, &len); + + while (node_name) { + if (!strcmp(child_name, node_name)) + return do_pinctr_pin(blob, child, node_name); + + child = fdt_next_subnode(blob, child); + + if (child < 0) + break; + + node_name = fdt_get_name(blob, child, &len); + } + + return -EFAULT; +} + +int config_dedicated_pins(const void *blob) +{ + int node; + + node = fdtdec_next_compatible(blob, 0, + COMPAT_ALTERA_SOCFPGA_PINCTRL_SINGLE); + + if (node < 0) + return -EINVAL; + + if (do_pinctrl_pins(blob, node, "dedicated_cfg")) + return -EFAULT; + + if (do_pinctrl_pins(blob, node, "dedicated")) + return -EFAULT; + + return 0; +} + +int config_pins(const void *blob, const char *pin_grp) +{ + int node; + + node = fdtdec_next_compatible(blob, 0, + COMPAT_ALTERA_SOCFPGA_PINCTRL_SINGLE); + + if (node < 0) + return -EINVAL; + + if (do_pinctrl_pins(blob, node, pin_grp)) + return -EFAULT; + + return 0; +}