mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Sascha Hauer <sha@pengutronix.de>
To: Alexander Shiyan <eagle.alexander923@gmail.com>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH v2] ARM: clps711x: Switch to devicetree usage
Date: Wed, 25 May 2022 10:50:52 +0200	[thread overview]
Message-ID: <20220525085052.GB1615@pengutronix.de> (raw)
In-Reply-To: <20220524060411.7754-1-eagle.alexander923@gmail.com>

On Tue, May 24, 2022 at 09:04:11AM +0300, Alexander Shiyan wrote:
> This is a complex patch that switches the ARM CLPS711X architecture
> to work with the device tree. Includes changes to board initialization
> and any architecture drivers used.
> 
> Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
> ---

Applied, thanks

Sascha

>  arch/arm/Kconfig                              |   4 +
>  arch/arm/boards/clep7212/Makefile             |   4 +-
>  arch/arm/boards/clep7212/board.c              |  15 ++
>  arch/arm/boards/clep7212/clep7212.c           |  57 -------
>  arch/arm/boards/clep7212/lowlevel.c           |  23 ++-
>  arch/arm/configs/clps711x_defconfig           |   5 +-
>  arch/arm/dts/Makefile                         |   1 +
>  arch/arm/dts/ep7212-clep7212.dts              |  64 ++++++++
>  arch/arm/mach-clps711x/Kconfig                |   9 --
>  arch/arm/mach-clps711x/Makefile               |   2 +-
>  arch/arm/mach-clps711x/clock.c                |  46 +++---
>  arch/arm/mach-clps711x/common.c               | 149 ++++++++++++++++++
>  .../arm/mach-clps711x/include/mach/clps711x.h |  28 ++--
>  .../arm/mach-clps711x/include/mach/debug_ll.h |  21 +++
>  arch/arm/mach-clps711x/include/mach/devices.h |   9 --
>  arch/arm/mach-clps711x/lowlevel.c             |  72 +++++----
>  arch/arm/mach-clps711x/reset.c                |  29 ----
>  drivers/clocksource/clps711x.c                |   7 +-
>  drivers/gpio/gpio-clps711x.c                  |   7 +-
>  drivers/serial/serial_clps711x.c              |  57 ++++---
>  images/Makefile                               |   1 +
>  images/Makefile.clps711x                      |   8 +
>  22 files changed, 395 insertions(+), 223 deletions(-)
>  create mode 100644 arch/arm/boards/clep7212/board.c
>  delete mode 100644 arch/arm/boards/clep7212/clep7212.c
>  create mode 100644 arch/arm/dts/ep7212-clep7212.dts
>  create mode 100644 arch/arm/mach-clps711x/common.c
>  create mode 100644 arch/arm/mach-clps711x/include/mach/debug_ll.h
>  delete mode 100644 arch/arm/mach-clps711x/include/mach/devices.h
>  delete mode 100644 arch/arm/mach-clps711x/reset.c
>  create mode 100644 images/Makefile.clps711x
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 299e0ab080..8a1d75c19b 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -66,9 +66,13 @@ config ARCH_CLPS711X
>  	select CLKDEV_LOOKUP
>  	select CLOCKSOURCE_CLPS711X
>  	select COMMON_CLK
> +	select COMMON_CLK_OF_PROVIDER
>  	select CPU_32v4T
>  	select GPIOLIB
> +	select HAS_DEBUG_LL
> +	select HAVE_PBL_MULTI_IMAGES
>  	select MFD_SYSCON
> +	select RELOCATABLE
>  
>  config ARCH_DAVINCI
>  	bool "TI Davinci"
> diff --git a/arch/arm/boards/clep7212/Makefile b/arch/arm/boards/clep7212/Makefile
> index 096120e567..85d92c8a7f 100644
> --- a/arch/arm/boards/clep7212/Makefile
> +++ b/arch/arm/boards/clep7212/Makefile
> @@ -1,5 +1,5 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  
> -obj-y += clep7212.o
> +obj-y += board.o
>  lwl-y += lowlevel.o
> -bbenv-y += defaultenv-clep7212
> +bbenv-$(CONFIG_DEFAULT_ENVIRONMENT) += defaultenv-clep7212
> diff --git a/arch/arm/boards/clep7212/board.c b/arch/arm/boards/clep7212/board.c
> new file mode 100644
> index 0000000000..b3983f2f49
> --- /dev/null
> +++ b/arch/arm/boards/clep7212/board.c
> @@ -0,0 +1,15 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +// SPDX-FileCopyrightText: Alexander Shiyan <shc_work@mail.ru>
> +
> +#include <envfs.h>
> +#include <init.h>
> +#include <of.h>
> +
> +static __init int clep7212_init(void)
> +{
> +	if (of_machine_is_compatible("cirrus,clep7212"))
> +		defaultenv_append_directory(defaultenv_clep7212);
> +
> +	return 0;
> +}
> +device_initcall(clep7212_init);
> diff --git a/arch/arm/boards/clep7212/clep7212.c b/arch/arm/boards/clep7212/clep7212.c
> deleted file mode 100644
> index 3b497a6bd2..0000000000
> --- a/arch/arm/boards/clep7212/clep7212.c
> +++ /dev/null
> @@ -1,57 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0-or-later
> -// SPDX-FileCopyrightText: 2012 Alexander Shiyan <shc_work@mail.ru>
> -
> -#include <common.h>
> -#include <driver.h>
> -#include <envfs.h>
> -#include <init.h>
> -#include <partition.h>
> -#include <io.h>
> -#include <linux/sizes.h>
> -#include <asm/armlinux.h>
> -#include <asm/mmu.h>
> -#include <generated/mach-types.h>
> -
> -#include <mach/clps711x.h>
> -#include <mach/devices.h>
> -
> -static int clps711x_devices_init(void)
> -{
> -	u32 serial_h = 0, serial_l = readl(UNIQID);
> -	void *cfi_io;
> -
> -	/* Setup Chipselects */
> -	clps711x_setup_memcfg(0, MEMCFG_WAITSTATE_6_1 | MEMCFG_BUS_WIDTH_16);
> -	clps711x_setup_memcfg(1, MEMCFG_WAITSTATE_6_1 | MEMCFG_BUS_WIDTH_8);
> -	clps711x_setup_memcfg(2, MEMCFG_WAITSTATE_8_3 | MEMCFG_BUS_WIDTH_16 |
> -			      MEMCFG_CLKENB);
> -	clps711x_setup_memcfg(3, MEMCFG_WAITSTATE_7_1 | MEMCFG_BUS_WIDTH_32);
> -
> -	cfi_io = map_io_sections(CS0_BASE, (void *)0x90000000, SZ_32M);
> -	add_cfi_flash_device(DEVICE_ID_DYNAMIC, (unsigned long)cfi_io, SZ_32M,
> -			     IORESOURCE_MEM);
> -
> -	devfs_add_partition("nor0", 0x00000, SZ_512K, DEVFS_PARTITION_FIXED,
> -			    "self0");
> -	devfs_add_partition("nor0", SZ_256K, SZ_256K, DEVFS_PARTITION_FIXED,
> -			    "env0");
> -
> -	armlinux_set_architecture(MACH_TYPE_CLEP7212);
> -	armlinux_set_serial(((u64)serial_h << 32) | serial_l);
> -
> -	defaultenv_append_directory(defaultenv_clep7212);
> -
> -	return 0;
> -}
> -device_initcall(clps711x_devices_init);
> -
> -static int clps711x_console_init(void)
> -{
> -	barebox_set_model("Cirrus Logic CLEP7212");
> -	barebox_set_hostname("clep7212");
> -
> -	clps711x_add_uart(0);
> -
> -	return 0;
> -}
> -console_initcall(clps711x_console_init);
> diff --git a/arch/arm/boards/clep7212/lowlevel.c b/arch/arm/boards/clep7212/lowlevel.c
> index 41827dfa16..09f2762fac 100644
> --- a/arch/arm/boards/clep7212/lowlevel.c
> +++ b/arch/arm/boards/clep7212/lowlevel.c
> @@ -1,22 +1,21 @@
>  // SPDX-License-Identifier: GPL-2.0-or-later
> -// SPDX-FileCopyrightText: 2012 Alexander Shiyan <shc_work@mail.ru>
> +// SPDX-FileCopyrightText: Alexander Shiyan <shc_work@mail.ru>
>  
>  #include <common.h>
> -#include <init.h>
> -
> -#include <asm/barebox-arm-head.h>
> -
> +#include <asm/barebox-arm.h>
> +#include <linux/sizes.h>
>  #include <mach/clps711x.h>
>  
> -#ifdef CONFIG_CLPS711X_RAISE_CPUFREQ
> -# define CLPS711X_CPU_PLL_MULT	50
> -#else
> -# define CLPS711X_CPU_PLL_MULT	40
> -#endif
> +extern char __dtb_ep7212_clep7212_start[];
>  
> -void __naked __bare_init barebox_arm_reset_vector(uint32_t r0, uint32_t r1, uint32_t r2)
> +ENTRY_FUNCTION_WITHSTACK(start_ep7212_clep7212,
> +			 CS6_BASE + 48 * SZ_1K, r0, r1, r2)
>  {
> +	void *fdt;
> +
>  	arm_cpu_lowlevel_init();
>  
> -	clps711x_barebox_entry(CLPS711X_CPU_PLL_MULT, NULL);
> +	fdt = __dtb_ep7212_clep7212_start;
> +
> +	clps711x_start(fdt + get_runtime_offset());
>  }
> diff --git a/arch/arm/configs/clps711x_defconfig b/arch/arm/configs/clps711x_defconfig
> index d9eab565b9..684ae79e22 100644
> --- a/arch/arm/configs/clps711x_defconfig
> +++ b/arch/arm/configs/clps711x_defconfig
> @@ -1,8 +1,8 @@
>  CONFIG_ARCH_CLPS711X=y
> +CONFIG_CLPS711X_RAISE_CPUFREQ=y
>  CONFIG_AEABI=y
>  CONFIG_ARM_OPTIMZED_STRING_FUNCTIONS=y
>  # CONFIG_MEMINFO is not set
> -CONFIG_PBL_IMAGE=y
>  CONFIG_MMU=y
>  CONFIG_EXPERIMENTAL=y
>  CONFIG_BAUDRATE=57600
> @@ -28,6 +28,8 @@ CONFIG_CMD_TIMEOUT=y
>  CONFIG_CMD_CRC=y
>  CONFIG_CMD_CRC_CMP=y
>  CONFIG_CMD_FLASH=y
> +CONFIG_OFDEVICE=y
> +CONFIG_OF_BAREBOX_DRIVERS=y
>  # CONFIG_SPI is not set
>  CONFIG_MTD=y
>  CONFIG_DRIVER_CFI=y
> @@ -36,6 +38,7 @@ CONFIG_DRIVER_CFI=y
>  CONFIG_DISK=y
>  CONFIG_DISK_WRITE=y
>  CONFIG_DISK_INTF_PLATFORM_IDE=y
> +# CONFIG_PINCTRL is not set
>  CONFIG_FS_CRAMFS=y
>  CONFIG_FS_FAT=y
>  CONFIG_FS_FAT_LFN=y
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 46e5e67672..d67d10bde8 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -8,6 +8,7 @@ lwl-$(CONFIG_MACH_ADVANTECH_ROM_742X) += imx6dl-advantech-rom-7421.dtb.o
>  lwl-$(CONFIG_MACH_AFI_GF) += am335x-afi-gf.dtb.o
>  lwl-$(CONFIG_MACH_BEAGLEBONE) += am335x-bone.dtb.o am335x-boneblack.dtb.o am335x-bone-common.dtb.o
>  lwl-$(CONFIG_MACH_CANON_A1100) += canon-a1100.dtb.o
> +lwl-$(CONFIG_MACH_CLEP7212) += ep7212-clep7212.dtb.o
>  lwl-$(CONFIG_MACH_CM_FX6) += imx6dl-cm-fx6.dtb.o imx6q-cm-fx6.dtb.o imx6q-utilite.dtb.o
>  lwl-$(CONFIG_MACH_DFI_FS700_M60) += imx6q-dfi-fs700-m60-6q.dtb.o imx6dl-dfi-fs700-m60-6s.dtb.o
>  lwl-$(CONFIG_MACH_DUCKBILL) += imx28-duckbill.dtb.o
> diff --git a/arch/arm/dts/ep7212-clep7212.dts b/arch/arm/dts/ep7212-clep7212.dts
> new file mode 100644
> index 0000000000..84c5e79548
> --- /dev/null
> +++ b/arch/arm/dts/ep7212-clep7212.dts
> @@ -0,0 +1,64 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/* Author: Alexander Shiyan <shc_work@mail.ru> */
> +
> +#include <arm/ep7211.dtsi>
> +
> +/ {
> +	model = "Cirrus Logic EP7212";
> +	compatible = "cirrus,clep7212", "cirrus,ep7212", "cirrus,ep7209";
> +
> +	memory@c0000000 {
> +		device_type = "memory";
> +		reg = <0xc0000000 0x02000000>;
> +	};
> +
> +	chosen {
> +		stdout-path = &uart1;
> +
> +		environment {
> +			compatible = "barebox,environment";
> +			device-path = &flash, "partname:env";
> +		};
> +	};
> +};
> +
> +&bus {
> +	/* Setup Memory Timings */
> +	/* CS0 = WAITSTATE_6_1 | BUS_WIDTH_16 */
> +	/* CS1 = WAITSTATE_6_1 | BUS_WIDTH_8 */
> +	/* CS2 = WAITSTATE_8_3 | BUS_WIDTH_16 | CLKENB */
> +	/* CS3 = WAITSTATE_7_1 | BUS_WIDTH_32 */
> +	barebox,ep7209-memcfg1 = <0x25802b28>;
> +
> +	flash: nor@0,0 {
> +		compatible = "cfi-flash";
> +		reg = <0 0x00000000 0x02000000>;
> +		bank-width = <2>;
> +
> +		partitions {
> +			compatible = "fixed-partitions";
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +
> +			partition@0 {
> +				label = "boot";
> +				reg = <0x00000 0x80000>;
> +			};
> +
> +			partition@80000 {
> +				label = "env";
> +				reg = <0x80000 0x40000>;
> +			};
> +
> +			partition@c0000 {
> +				label = "kernel";
> +				reg = <0xc0000 0x340000>;
> +			};
> +
> +			partition@400000 {
> +				label = "root";
> +				reg = <0x400000 0>;
> +			};
> +		};
> +	};
> +};
> diff --git a/arch/arm/mach-clps711x/Kconfig b/arch/arm/mach-clps711x/Kconfig
> index 52cf8468c0..6cf6bc37a2 100644
> --- a/arch/arm/mach-clps711x/Kconfig
> +++ b/arch/arm/mach-clps711x/Kconfig
> @@ -16,7 +16,6 @@ menu "CLPS711X specific settings"
>  
>  config CLPS711X_RAISE_CPUFREQ
>  	bool "Raise CPU frequency to 90 MHz"
> -	depends on MACH_CLEP7212
>  	help
>  	  Raise CPU frequency to 90 MHz. This operation can be performed
>  	  only for devices which allow to operate at 90 MHz.
> @@ -24,12 +23,4 @@ config CLPS711X_RAISE_CPUFREQ
>  
>  endmenu
>  
> -config ARCH_TEXT_BASE
> -	hex
> -	default 0xc0740000 if MACH_CLEP7212
> -
> -config BAREBOX_MAX_IMAGE_SIZE
> -	hex
> -	default 0x00080000 if MACH_CLEP7212
> -
>  endif
> diff --git a/arch/arm/mach-clps711x/Makefile b/arch/arm/mach-clps711x/Makefile
> index 4d5950d9b5..871c5f7e9c 100644
> --- a/arch/arm/mach-clps711x/Makefile
> +++ b/arch/arm/mach-clps711x/Makefile
> @@ -1,4 +1,4 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  
> -obj-y += clock.o devices.o reset.o
> +obj-y += clock.o common.o
>  lwl-y += lowlevel.o
> diff --git a/arch/arm/mach-clps711x/clock.c b/arch/arm/mach-clps711x/clock.c
> index 2c5137c582..8674b8c801 100644
> --- a/arch/arm/mach-clps711x/clock.c
> +++ b/arch/arm/mach-clps711x/clock.c
> @@ -1,11 +1,5 @@
> -/*
> - * Copyright (C) 2012-2016 Alexander Shiyan <shc_work@mail.ru>
> - *
> - * This program is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU General Public License as
> - * published by the Free Software Foundation; either version 2 of
> - * the License, or (at your option) any later version.
> - */
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +// SPDX-FileCopyrightText: Alexander Shiyan <shc_work@mail.ru>
>  
>  #include <common.h>
>  #include <init.h>
> @@ -21,13 +15,15 @@
>  #define CLPS711X_EXT_FREQ	13000000
>  
>  static struct clk *clks[CLPS711X_CLK_MAX];
> +static struct clk_onecell_data clk_data;
>  
> -static struct clk_div_table tdiv_tbl[] = {
> +static const struct clk_div_table tdiv_tbl[] = {
>  	{ .val = 0, .div = 256, },
>  	{ .val = 1, .div = 1, },
> +	{ }
>  };
>  
> -static __init int clps711x_clk_init(void)
> +static int clps711x_clk_probe(struct device_d *dev)
>  {
>  	unsigned int f_cpu, f_bus, f_uart, f_timer_ref, pll;
>  	u32 tmp;
> @@ -50,7 +46,7 @@ static __init int clps711x_clk_init(void)
>  			f_bus = 36864000 / 2;
>  	}
>  
> -	f_uart = f_bus / 10;
> +	f_uart = DIV_ROUND_CLOSEST(f_bus, 10);
>  
>  	if (tmp & SYSFLG2_CKMODE) {
>  		tmp = readw(SYSCON2);
> @@ -72,23 +68,25 @@ static __init int clps711x_clk_init(void)
>  	clks[CLPS711X_CLK_UART] = clk_fixed("uart", f_uart);
>  	clks[CLPS711X_CLK_TIMERREF] = clk_fixed("timer_ref", f_timer_ref);
>  	clks[CLPS711X_CLK_TIMER1] = clk_divider_table("timer1", "timer_ref", 0,
> -		IOMEM(SYSCON1), 5, 1, tdiv_tbl, ARRAY_SIZE(tdiv_tbl));
> +		IOMEM(SYSCON1), 5, 1, tdiv_tbl, 0);
>  	clks[CLPS711X_CLK_TIMER2] = clk_divider_table("timer2", "timer_ref", 0,
> -		IOMEM(SYSCON1), 7, 1, tdiv_tbl, ARRAY_SIZE(tdiv_tbl));
> +		IOMEM(SYSCON1), 7, 1, tdiv_tbl, 0);
>  
> -	clkdev_add_physbase(clks[CLPS711X_CLK_UART], UARTDR1, NULL);
> -	clkdev_add_physbase(clks[CLPS711X_CLK_UART], UARTDR2, NULL);
> -	clkdev_add_physbase(clks[CLPS711X_CLK_TIMER2], TC2D, NULL);
> +	clk_data.clks = clks;
> +	clk_data.clk_num = CLPS711X_CLK_MAX;
> +	of_clk_add_provider(dev->device_node, of_clk_src_onecell_get, &clk_data);
>  
>  	return 0;
>  }
> -postcore_initcall(clps711x_clk_init);
>  
> -static __init int clps711x_core_init(void)
> -{
> -	add_generic_device("clps711x-cs", DEVICE_ID_SINGLE, NULL,
> -			   TC2D, SZ_2, IORESOURCE_MEM, NULL);
> +static const struct of_device_id __maybe_unused clps711x_clk_dt_ids[] = {
> +	{ .compatible = "cirrus,ep7209-clk", },
> +	{ }
> +};
>  
> -	return 0;
> -}
> -coredevice_initcall(clps711x_core_init);
> +static struct driver_d clps711x_clk_driver = {
> +	.probe = clps711x_clk_probe,
> +	.name = "clps711x-clk",
> +	.of_compatible = DRV_OF_COMPAT(clps711x_clk_dt_ids),
> +};
> +postcore_platform_driver(clps711x_clk_driver);
> diff --git a/arch/arm/mach-clps711x/common.c b/arch/arm/mach-clps711x/common.c
> new file mode 100644
> index 0000000000..c0b817872d
> --- /dev/null
> +++ b/arch/arm/mach-clps711x/common.c
> @@ -0,0 +1,149 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +// SPDX-FileCopyrightText: Alexander Shiyan <shc_work@mail.ru>
> +
> +#include <common.h>
> +#include <driver.h>
> +#include <restart.h>
> +#include <asm/io.h>
> +#include <asm/mmu.h>
> +#include <mach/clps711x.h>
> +
> +#define CLPS711X_MAP_ADDR	0x90000000
> +
> +static u32 remap_size = 0;
> +
> +static void __noreturn clps711x_restart(struct restart_handler *rst)
> +{
> +	shutdown_barebox();
> +
> +	asm("mov pc, #0");
> +
> +	hang();
> +}
> +
> +static __init int is_clps711x_compatible(void)
> +{
> +	return of_machine_is_compatible("cirrus,ep7209");
> +}
> +
> +static __init int clps711x_init(void)
> +{
> +	char *serial;
> +
> +	if (!is_clps711x_compatible())
> +		return 0;
> +
> +	restart_handler_register_fn("vector", clps711x_restart);
> +
> +	serial = basprintf("%08x%08x", 0, readl(UNIQID));
> +
> +	barebox_set_serial_number(serial);
> +
> +	free(serial);
> +
> +	return 0;
> +}
> +postcore_initcall(clps711x_init);
> +
> +static int __init clps711x_bus_map(void)
> +{
> +	if (is_clps711x_compatible() && remap_size)
> +		map_io_sections(0, (void *)CLPS711X_MAP_ADDR, remap_size);
> +
> +	return 0;
> +}
> +postmmu_initcall(clps711x_bus_map);
> +
> +/* Scan for devices that start at zero address and maps them
> + * to a different unused address.
> + * To start the kernel, a fixup is used that rewrites the address
> + * of the patched device to its original state.
> + */
> +
> +static void clps711x_bus_patch(struct device_node *node,
> +			       u32 compare, u32 change)
> +{
> +	const __be32 *ranges;
> +	int rsize;
> +
> +	ranges = of_get_property(node, "ranges", &rsize);
> +
> +	if (ranges) {
> +		int banks = rsize / (sizeof(u32) * 4);
> +		__be32 *fixed, *fixedptr;
> +
> +		fixed = xmalloc(rsize);
> +		fixedptr = fixed;
> +
> +		while (banks--) {
> +			u32 bank, cell, addr, size;
> +
> +			bank = be32_to_cpu(*ranges++);
> +			cell = be32_to_cpu(*ranges++);
> +			addr = be32_to_cpu(*ranges++);
> +			size = be32_to_cpu(*ranges++);
> +
> +			if (addr == compare) {
> +				addr = change;
> +				remap_size = size;
> +			}
> +
> +			*fixedptr++ = cpu_to_be32(bank);
> +			*fixedptr++ = cpu_to_be32(cell);
> +			*fixedptr++ = cpu_to_be32(addr);
> +			*fixedptr++ = cpu_to_be32(size);
> +		}
> +
> +		of_set_property(node, "ranges", fixed, rsize, 0);
> +
> +		free(fixed);
> +	}
> +}
> +
> +static int clps711x_bus_fixup(struct device_node *root, void *context)
> +{
> +	struct device_node *node = context;
> +
> +	if (remap_size)
> +		clps711x_bus_patch(node, CLPS711X_MAP_ADDR, 0);
> +
> +	return 0;
> +}
> +
> +static int clps711x_bus_probe(struct device_d *dev)
> +{
> +	u32 mcfg;
> +
> +	/* Setup bus timings */
> +	if (!of_property_read_u32(dev->device_node,
> +				  "barebox,ep7209-memcfg1", &mcfg))
> +		writel(mcfg, MEMCFG1);
> +	if (!of_property_read_u32(dev->device_node,
> +				  "barebox,ep7209-memcfg2", &mcfg))
> +		writel(mcfg, MEMCFG2);
> +
> +	clps711x_bus_patch(dev->device_node, 0, CLPS711X_MAP_ADDR);
> +
> +	of_platform_populate(dev->device_node, NULL, dev);
> +
> +	of_register_fixup(clps711x_bus_fixup, dev->device_node);
> +
> +	return 0;
> +}
> +
> +static const struct of_device_id __maybe_unused clps711x_bus_dt_ids[] = {
> +	{ .compatible = "cirrus,ep7209-bus", },
> +	{ }
> +};
> +
> +static struct driver_d clps711x_bus_driver = {
> +	.name = "clps711x-bus",
> +	.probe = clps711x_bus_probe,
> +	.of_compatible = DRV_OF_COMPAT(clps711x_bus_dt_ids),
> +};
> +
> +static int __init clps711x_bus_init(void)
> +{
> +	return platform_driver_register(&clps711x_bus_driver);
> +}
> +core_initcall(clps711x_bus_init);
> diff --git a/arch/arm/mach-clps711x/include/mach/clps711x.h b/arch/arm/mach-clps711x/include/mach/clps711x.h
> index 957b2b8477..9aef7f3fd3 100644
> --- a/arch/arm/mach-clps711x/include/mach/clps711x.h
> +++ b/arch/arm/mach-clps711x/include/mach/clps711x.h
> @@ -1,13 +1,7 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
>  /*
> - *  Hardware definitions for Cirrus Logic CLPS711X
> - *
> - *  Copyright (C) 2000 Deep Blue Solutions Ltd.
> - *  Copyright (C) 2012 Alexander Shiyan <shc_work@mail.ru>
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> + * Copyright (C) 2000 Deep Blue Solutions Ltd.
> + * Copyright (C) 2012-2022 Alexander Shiyan <shc_work@mail.ru>
>   */
>  
>  #ifndef __MACH_CLPS711X_H
> @@ -159,6 +153,10 @@
>  #define SYSCON2_CLKENSL		(1 << 13)
>  #define SYSCON2_BUZFREQ		(1 << 14)
>  
> +#define SYSCON_UARTEN		(1 << 8)
> +#define SYSFLG_UBUSY		(1 << 11)
> +#define SYSFLG_UTXFF		(1 << 23)
> +
>  #define SYNCIO_FRMLEN(x)	(((x) & 0x1f) << 8)
>  #define SYNCIO_SMCKEN		(1 << 13)
>  #define SYNCIO_TXFRMEN		(1 << 14)
> @@ -247,6 +245,16 @@
>  #define MEMCFG_WAITSTATE_2_0	(14 << 2)
>  #define MEMCFG_WAITSTATE_1_0	(15 << 2)
>  
> -void clps711x_barebox_entry(u32, void *);
> +#define UBRLCR_BREAK		(1 << 12)
> +#define UBRLCR_PRTEN		(1 << 13)
> +#define UBRLCR_EVENPRT		(1 << 14)
> +#define UBRLCR_XSTOP		(1 << 15)
> +#define UBRLCR_FIFOEN		(1 << 16)
> +#define UBRLCR_WRDLEN5		(0 << 17)
> +#define UBRLCR_WRDLEN6		(1 << 17)
> +#define UBRLCR_WRDLEN7		(2 << 17)
> +#define UBRLCR_WRDLEN8		(3 << 17)
> +
> +void clps711x_start(void *);
>  
>  #endif
> diff --git a/arch/arm/mach-clps711x/include/mach/debug_ll.h b/arch/arm/mach-clps711x/include/mach/debug_ll.h
> new file mode 100644
> index 0000000000..342bb7628a
> --- /dev/null
> +++ b/arch/arm/mach-clps711x/include/mach/debug_ll.h
> @@ -0,0 +1,21 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +// SPDX-FileCopyrightText: Alexander Shiyan <shc_work@mail.ru>
> +
> +#ifndef __MACH_DEBUG_LL_H__
> +#define __MACH_DEBUG_LL_H__
> +
> +#include <asm/io.h>
> +#include <mach/clps711x.h>
> +
> +static inline void PUTC_LL(char c)
> +{
> +	do {
> +	} while (readl(SYSFLG1) & SYSFLG_UTXFF);
> +
> +	writew(c, UARTDR1);
> +
> +	do {
> +	} while (readl(SYSFLG1) & SYSFLG_UBUSY);
> +}
> +
> +#endif
> diff --git a/arch/arm/mach-clps711x/include/mach/devices.h b/arch/arm/mach-clps711x/include/mach/devices.h
> deleted file mode 100644
> index 77c43be25c..0000000000
> --- a/arch/arm/mach-clps711x/include/mach/devices.h
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -/* SPDX-License-Identifier: GPL-2.0-only */
> -
> -#ifndef __MACH_DEVICES_H
> -#define __MACH_DEVICES_H
> -
> -void clps711x_setup_memcfg(int bank, u32 val);
> -void clps711x_add_uart(unsigned int id);
> -
> -#endif
> diff --git a/arch/arm/mach-clps711x/lowlevel.c b/arch/arm/mach-clps711x/lowlevel.c
> index 35b8b35e87..608f0778d7 100644
> --- a/arch/arm/mach-clps711x/lowlevel.c
> +++ b/arch/arm/mach-clps711x/lowlevel.c
> @@ -1,25 +1,31 @@
> -/*
> - * Copyright (C) 2012 Alexander Shiyan <shc_work@mail.ru>
> - *
> - * This program is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU General Public License as
> - * published by the Free Software Foundation; either version 2 of
> - * the License, or (at your option) any later version.
> - */
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +// SPDX-FileCopyrightText: Alexander Shiyan <shc_work@mail.ru>
>  
> +#include <asm/io.h>
> +#include <asm/barebox-arm.h>
>  #include <common.h>
> -#include <init.h>
> +#include <debug_ll.h>
>  #include <linux/sizes.h>
> +#include <mach/clps711x.h>
>  
> -#include <asm/io.h>
> -#include <asm/barebox-arm.h>
> -#include <asm/barebox-arm-head.h>
> +#define DEBUG_LL_BAUDRATE	(57600)
>  
> -#include <mach/clps711x.h>
> +static inline void setup_uart(const u32 bus_speed)
> +{
> +	u32 baud_base = DIV_ROUND_CLOSEST(bus_speed, 10);
> +	u32 baud_divisor =
> +		DIV_ROUND_CLOSEST(baud_base, DEBUG_LL_BAUDRATE * 16) - 1;
> +
> +	writel(baud_divisor | UBRLCR_FIFOEN | UBRLCR_WRDLEN8, UBRLCR1);
> +	writel(0, STFCLR);
> +	writel(SYSCON_UARTEN, SYSCON1);
>  
> -void __naked __bare_init clps711x_barebox_entry(u32 pllmult, void *data)
> +	putc_ll('>');
> +}
> +
> +void clps711x_start(void *fdt)
>  {
> -	u32 cpu, bus;
> +	u32 bus, pll;
>  
>  	/* Check if we running from external 13 MHz clock */
>  	if (!(readl(SYSFLG2) & SYSFLG2_CKMODE)) {
> @@ -27,24 +33,17 @@ void __naked __bare_init clps711x_barebox_entry(u32 pllmult, void *data)
>  		writel(SYSCON3_CLKCTL0 | SYSCON3_CLKCTL1, SYSCON3);
>  		asm("nop");
>  
> -		/* Check valid multiplier, default to 74 MHz */
> -		if ((pllmult < 20) || (pllmult > 50))
> -			pllmult = 40;
> +		if (IS_ENABLED(CONFIG_CLPS711X_RAISE_CPUFREQ)) {
> +			/* Setup PLL to 92160000 Hz */
> +			writel(50 << 24, PLLW);
> +			asm("nop");
> +		}
>  
> -		/* Setup PLL */
> -		writel(pllmult << 24, PLLW);
> -		asm("nop");
> -
> -		/* Check for old CPUs without PLL */
> -		if ((readl(PLLR) >> 24) != pllmult)
> -			cpu = 73728000;
> -		else
> -			cpu = pllmult * 3686400;
> -
> -		if (cpu >= 36864000)
> -			bus = cpu / 2;
> +		pll = readl(PLLR) >> 24;
> +		if (pll)
> +			bus = (pll * 3686400) / 4;
>  		else
> -			bus = 36864000 / 2;
> +			bus = 73728000 / 4;
>  	} else {
>  		bus = 13000000;
>  		/* Setup bus wait state scaling factor to 1  */
> @@ -52,6 +51,13 @@ void __naked __bare_init clps711x_barebox_entry(u32 pllmult, void *data)
>  		asm("nop");
>  	}
>  
> +
> +	/* Disable UART, IrDa, LCD */
> +	writel(0, SYSCON1);
> +
> +	if (IS_ENABLED(CONFIG_DEBUG_LL))
> +		setup_uart(bus);
> +
>  	/* CLKEN select, SDRAM width=32 */
>  	writel(SYSCON2_CLKENSL, SYSCON2);
>  
> @@ -62,12 +68,10 @@ void __naked __bare_init clps711x_barebox_entry(u32 pllmult, void *data)
>  	/* Setup Refresh Rate (64ms 8K Blocks) */
>  	writel((64 * bus) / (8192 * 1000), SDRFPR);
>  
> -	/* Disable UART, IrDa, LCD */
> -	writel(0, SYSCON1);
>  	/* Disable PWM */
>  	writew(0, PMPCON);
>  	/* Disable LED flasher */
>  	writew(0, LEDFLSH);
>  
> -	barebox_arm_entry(SDRAM0_BASE, SZ_8M, data);
> +	barebox_arm_entry(SDRAM0_BASE, SZ_8M, fdt);
>  }
> diff --git a/arch/arm/mach-clps711x/reset.c b/arch/arm/mach-clps711x/reset.c
> deleted file mode 100644
> index 90ddb8f5d2..0000000000
> --- a/arch/arm/mach-clps711x/reset.c
> +++ /dev/null
> @@ -1,29 +0,0 @@
> -/*
> - * Copyright (C) 2012 Alexander Shiyan <shc_work@mail.ru>
> - *
> - * This program is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU General Public License as
> - * published by the Free Software Foundation; either version 2 of
> - * the License, or (at your option) any later version.
> - */
> -
> -#include <common.h>
> -#include <init.h>
> -#include <restart.h>
> -
> -static void __noreturn clps711x_restart_soc(struct restart_handler *rst)
> -{
> -	shutdown_barebox();
> -
> -	asm("mov pc, #0");
> -
> -	hang();
> -}
> -
> -static int restart_register_feature(void)
> -{
> -	restart_handler_register_fn("vector", clps711x_restart_soc);
> -
> -	return 0;
> -}
> -coredevice_initcall(restart_register_feature);
> diff --git a/drivers/clocksource/clps711x.c b/drivers/clocksource/clps711x.c
> index 1fe7f6c891..d6ab695afa 100644
> --- a/drivers/clocksource/clps711x.c
> +++ b/drivers/clocksource/clps711x.c
> @@ -27,6 +27,11 @@ static int clps711x_cs_probe(struct device_d *dev)
>  	struct resource *iores;
>  	u32 rate;
>  	struct clk *timer_clk;
> +	int id;
> +
> +	id = of_alias_get_id(dev->device_node, "timer");
> +	if (id != 1)
> +		return 0;
>  
>  	timer_clk = clk_get(dev, NULL);
>  	if (IS_ERR(timer_clk))
> @@ -46,7 +51,7 @@ static int clps711x_cs_probe(struct device_d *dev)
>  	return init_clock(&clps711x_cs);
>  }
>  
> -static __maybe_unused struct of_device_id clps711x_timer_dt_ids[] = {
> +static const struct of_device_id __maybe_unused clps711x_timer_dt_ids[] = {
>  	{ .compatible = "cirrus,ep7209-timer", },
>  	{ }
>  };
> diff --git a/drivers/gpio/gpio-clps711x.c b/drivers/gpio/gpio-clps711x.c
> index a1965b33e4..cb4bd4025c 100644
> --- a/drivers/gpio/gpio-clps711x.c
> +++ b/drivers/gpio/gpio-clps711x.c
> @@ -10,13 +10,10 @@
>  static int clps711x_gpio_probe(struct device_d *dev)
>  {
>  	struct resource *iores;
> -	int err, id = dev->id;
> +	int err, id = of_alias_get_id(dev->device_node, "gpio");
>  	void __iomem *dat, *dir = NULL, *dir_inv = NULL;
>  	struct bgpio_chip *bgc;
>  
> -	if (dev->device_node)
> -		id = of_alias_get_id(dev->device_node, "gpio");
> -
>  	if (id < 0 || id > 4)
>  		return -ENODEV;
>  
> @@ -62,7 +59,7 @@ out_err:
>  	return err;
>  }
>  
> -static struct of_device_id __maybe_unused clps711x_gpio_dt_ids[] = {
> +static const struct of_device_id __maybe_unused clps711x_gpio_dt_ids[] = {
>  	{ .compatible = "cirrus,ep7209-gpio", },
>  	{ /* sentinel */ }
>  };
> diff --git a/drivers/serial/serial_clps711x.c b/drivers/serial/serial_clps711x.c
> index 7aac0970bf..cb7043d8e0 100644
> --- a/drivers/serial/serial_clps711x.c
> +++ b/drivers/serial/serial_clps711x.c
> @@ -35,7 +35,7 @@
>  
>  struct clps711x_uart {
>  	void __iomem		*base;
> -	void __iomem		*syscon;
> +	struct regmap		*regmap;
>  	struct clk		*uart_clk;
>  	struct console_device	cdev;
>  };
> @@ -61,8 +61,7 @@ static void clps711x_init_port(struct console_device *cdev)
>  	u32 tmp;
>  
>  	/* Disable the UART */
> -	tmp = readl(s->syscon + SYSCON);
> -	writel(tmp & ~SYSCON_UARTEN, s->syscon + SYSCON);
> +	regmap_update_bits(s->regmap, SYSCON, SYSCON_UARTEN, 0);
>  
>  	/* Setup Line Control Register */
>  	tmp = readl(s->base + UBRLCR) & UBRLCR_BAUD_MASK;
> @@ -70,17 +69,19 @@ static void clps711x_init_port(struct console_device *cdev)
>  	writel(tmp, s->base + UBRLCR);
>  
>  	/* Enable the UART */
> -	tmp = readl(s->syscon + SYSCON);
> -	writel(tmp | SYSCON_UARTEN, s->syscon + SYSCON);
> +	regmap_update_bits(s->regmap, SYSCON, SYSCON_UARTEN, SYSCON_UARTEN);
>  }
>  
>  static void clps711x_putc(struct console_device *cdev, char c)
>  {
>  	struct clps711x_uart *s = cdev->dev->priv;
> +	u32 tmp;
>  
>  	/* Wait until there is space in the FIFO */
>  	do {
> -	} while (readl(s->syscon + SYSFLG) & SYSFLG_UTXFF);
> +		regmap_read(s->regmap, SYSFLG, &tmp);
> +		
> +	} while (tmp & SYSFLG_UTXFF);
>  
>  	/* Send the character */
>  	writew(c, s->base + UARTDR);
> @@ -90,10 +91,12 @@ static int clps711x_getc(struct console_device *cdev)
>  {
>  	struct clps711x_uart *s = cdev->dev->priv;
>  	u16 data;
> +	u32 tmp;
>  
>  	/* Wait until there is data in the FIFO */
>  	do {
> -	} while (readl(s->syscon + SYSFLG) & SYSFLG_URXFE);
> +		regmap_read(s->regmap, SYSFLG, &tmp);
> +	} while (tmp & SYSFLG_URXFE);
>  
>  	data = readw(s->base + UARTDR);
>  
> @@ -107,32 +110,32 @@ static int clps711x_getc(struct console_device *cdev)
>  static int clps711x_tstc(struct console_device *cdev)
>  {
>  	struct clps711x_uart *s = cdev->dev->priv;
> +	u32 tmp;
>  
> -	return !(readl(s->syscon + SYSFLG) & SYSFLG_URXFE);
> +	regmap_read(s->regmap, SYSFLG, &tmp);
> +
> +	return !(tmp & SYSFLG_URXFE);
>  }
>  
>  static void clps711x_flush(struct console_device *cdev)
>  {
>  	struct clps711x_uart *s = cdev->dev->priv;
> +	u32 tmp;
>  
>  	do {
> -	} while (readl(s->syscon + SYSFLG) & SYSFLG_UBUSY);
> +		regmap_read(s->regmap, SYSFLG, &tmp);
> +	} while (tmp & SYSFLG_UBUSY);
>  }
>  
>  static int clps711x_probe(struct device_d *dev)
>  {
> +	struct device_node *syscon;
>  	struct clps711x_uart *s;
> -	int err, id = dev->id;
> -	char syscon_dev[8];
>  	const char *devname;
> -
> -	if (dev->device_node)
> -		id = of_alias_get_id(dev->device_node, "serial");
> -
> -	if (id != 0 && id != 1)
> -		return -EINVAL;
> +	int err;
>  
>  	s = xzalloc(sizeof(struct clps711x_uart));
> +
>  	s->uart_clk = clk_get(dev, NULL);
>  	if (IS_ERR(s->uart_clk)) {
>  		err = PTR_ERR(s->uart_clk);
> @@ -140,19 +143,15 @@ static int clps711x_probe(struct device_d *dev)
>  	}
>  
>  	s->base = dev_get_mem_region(dev, 0);
> -	if (IS_ERR(s->base))
> -		return PTR_ERR(s->base);
> -
> -	if (!dev->device_node) {
> -		sprintf(syscon_dev, "syscon%i", id + 1);
> -		s->syscon = syscon_base_lookup_by_pdevname(syscon_dev);
> -	} else {
> -		s->syscon = syscon_base_lookup_by_phandle(dev->device_node,
> -							  "syscon");
> +	if (IS_ERR(s->base)) {
> +		err = PTR_ERR(s->base);
> +		goto out_err;
>  	}
>  
> -	if (IS_ERR(s->syscon)) {
> -		err = PTR_ERR(s->syscon);
> +	syscon = of_parse_phandle(dev->device_node, "syscon", 0);
> +	s->regmap = syscon_node_to_regmap(syscon);
> +	if (IS_ERR(s->regmap)) {
> +		err = PTR_ERR(s->regmap);
>  		goto out_err;
>  	}
>  
> @@ -182,7 +181,7 @@ out_err:
>  	return err;
>  }
>  
> -static struct of_device_id __maybe_unused clps711x_uart_dt_ids[] = {
> +static const struct of_device_id __maybe_unused clps711x_uart_dt_ids[] = {
>  	{ .compatible = "cirrus,ep7209-uart", },
>  	{ /* sentinel */ }
>  };
> diff --git a/images/Makefile b/images/Makefile
> index 5e2b9ecaaa..a148cf4176 100644
> --- a/images/Makefile
> +++ b/images/Makefile
> @@ -152,6 +152,7 @@ include $(srctree)/images/Makefile.ar231x
>  include $(srctree)/images/Makefile.ath79
>  include $(srctree)/images/Makefile.bcm283x
>  include $(srctree)/images/Makefile.bcm47xx
> +include $(srctree)/images/Makefile.clps711x
>  include $(srctree)/images/Makefile.imx
>  include $(srctree)/images/Makefile.loongson
>  include $(srctree)/images/Makefile.malta
> diff --git a/images/Makefile.clps711x b/images/Makefile.clps711x
> new file mode 100644
> index 0000000000..d76911d627
> --- /dev/null
> +++ b/images/Makefile.clps711x
> @@ -0,0 +1,8 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +#
> +# barebox image generation Makefile for CLPS711X images
> +#
> +
> +pblb-$(CONFIG_MACH_CLEP7212) += start_ep7212_clep7212
> +FILE_barebox-ep7212-clep7212.img = start_ep7212_clep7212.pblb
> +image-$(CONFIG_MACH_CLEP7212) += barebox-ep7212-clep7212.img
> -- 
> 2.32.0
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


      reply	other threads:[~2022-05-25  8:52 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-24  6:04 Alexander Shiyan
2022-05-25  8:50 ` Sascha Hauer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220525085052.GB1615@pengutronix.de \
    --to=sha@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=eagle.alexander923@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox