From: kgene.kim@samsung.com (Kukjin Kim)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 5/7] ARM: s3c64xx: Add board file for boot using Device Tree
Date: Wed, 09 Jan 2013 16:24:42 -0800 [thread overview]
Message-ID: <00aa01cdeec8$e48ba7c0$ada2f740$@samsung.com> (raw)
In-Reply-To: <1357774909-29938-6-git-send-email-tomasz.figa@gmail.com>
Tomasz Figa wrote:
>
> This patch adds board file that will be used to boot S3C64xx-based boards
> using Device Tree.
>
> Signed-off-by: Tomasz Figa <tomasz.figa@gmail.com>
> ---
> arch/arm/mach-s3c64xx/Kconfig | 13 +++++
> arch/arm/mach-s3c64xx/Makefile | 1 +
> arch/arm/mach-s3c64xx/mach-s3c64xx-dt.c | 93
> +++++++++++++++++++++++++++++++++
> 3 files changed, 107 insertions(+)
> create mode 100644 arch/arm/mach-s3c64xx/mach-s3c64xx-dt.c
>
> diff --git a/arch/arm/mach-s3c64xx/Kconfig b/arch/arm/mach-
> s3c64xx/Kconfig
> index 131c862..a43b3f7 100644
> --- a/arch/arm/mach-s3c64xx/Kconfig
> +++ b/arch/arm/mach-s3c64xx/Kconfig
> @@ -309,3 +309,16 @@ config MACH_WLF_CRAGG_6410
> select SAMSUNG_GPIO_EXTRA128
> help
> Machine support for the Wolfson Cragganmore S3C6410 variant.
> +
> +config MACH_S3C64XX_DT
> + bool "Samsung S3C6400/S3C6410 machine using Device Tree"
> + select CPU_S3C6400
> + select CPU_S3C6410
> + select USE_OF
> + help
> + Machine support for Samsung S3C6400/S3C6410 machines with
> Device Tree
> + enabled.
> + Select this if a fdt blob is available for your S3C64XX SoC based
> + board.
> + Note: This is under development and not all peripherals can be
> + supported with this machine file.
> diff --git a/arch/arm/mach-s3c64xx/Makefile b/arch/arm/mach-
> s3c64xx/Makefile
> index f9ce1dc..59b3d06 100644
> --- a/arch/arm/mach-s3c64xx/Makefile
> +++ b/arch/arm/mach-s3c64xx/Makefile
> @@ -58,3 +58,4 @@ obj-$(CONFIG_MACH_SMARTQ7) += mach-
> smartq7.o
> obj-$(CONFIG_MACH_SMDK6400) += mach-smdk6400.o
> obj-$(CONFIG_MACH_SMDK6410) += mach-smdk6410.o
> obj-$(CONFIG_MACH_WLF_CRAGG_6410) += mach-crag6410.o mach-
> crag6410-module.o
> +obj-$(CONFIG_MACH_S3C64XX_DT) += mach-s3c64xx-dt.o
> diff --git a/arch/arm/mach-s3c64xx/mach-s3c64xx-dt.c b/arch/arm/mach-
> s3c64xx/mach-s3c64xx-dt.c
> new file mode 100644
> index 0000000..f5725ce
> --- /dev/null
> +++ b/arch/arm/mach-s3c64xx/mach-s3c64xx-dt.c
> @@ -0,0 +1,93 @@
> +/*
> + * Samsung's S3C64XX flattened device tree enabled machine
> + *
> + * Copyright (c) 2012 Tomasz Figa <tomasz.figa@gmail.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> +*/
> +
> +#include <linux/of_platform.h>
> +#include <linux/serial_core.h>
> +
> +#include <asm/mach/arch.h>
> +#include <asm/hardware/vic.h>
> +#include <mach/map.h>
> +
> +#include <plat/cpu.h>
> +#include <plat/regs-serial.h>
> +#include <plat/s5p-time.h>
> +
> +#include "common.h"
> +
> +/*
> + * The following lookup table is used to override device names when
> devices
> + * are registered from device tree. This is temporarily added to enable
> + * device tree support addition for the Exynos4 architecture.
> + *
> + * For drivers that require platform data to be provided from the machine
> + * file, a platform data pointer can also be supplied along with the
> + * devices names. Usually, the platform data elements that cannot be
> parsed
> + * from the device tree by the drivers (example: function pointers) are
> + * supplied. But it should be noted that this is a temporary mechanism
and
> + * at some point, the drivers should be capable of parsing all the
platform
> + * data from the device tree.
> + */
> +static const struct of_dev_auxdata s3c64xx_auxdata_lookup[] __initconst =
> {
> + OF_DEV_AUXDATA("samsung,s3c6400-uart", S3C_PA_UART0,
> + "s3c6400-uart.0", NULL),
> + OF_DEV_AUXDATA("samsung,s3c6400-uart", S3C_PA_UART1,
> + "s3c6400-uart.1", NULL),
> + OF_DEV_AUXDATA("samsung,s3c6400-uart", S3C_PA_UART2,
> + "s3c6400-uart.2", NULL),
> + OF_DEV_AUXDATA("samsung,s3c6400-uart", S3C_PA_UART3,
> + "s3c6400-uart.3", NULL),
I'd prefer to use 0x7F005000 instead of S3C_PA_UART0 so that we can remove
the static definitions at the map.h
> + {},
> +};
> +
> +static void __init s3c64xx_dt_map_io(void)
> +{
> + s3c64xx_init_io(NULL, 0);
> + s3c24xx_init_clocks(12000000);
The value of xtal clock for s3c24xx_init_clocks() should be defined via dt
according to board condition _next_time_ because it can be different on
each board, it's same on current s3c64xx boards though.
> +}
> +
> +static void __init s3c64xx_dt_machine_init(void)
> +{
> + of_platform_populate(NULL, of_default_bus_match_table,
> + s3c64xx_auxdata_lookup, NULL);
> +}
> +
> +static char const *s3c6400_dt_compat[] __initdata = {
> + "samsung,s3c6400",
> + NULL
> +};
> +
> +DT_MACHINE_START(S3C6400_DT, "Samsung S3C6400 (Flattened Device
> Tree)")
> + /* Maintainer: Tomasz Figa <tomasz.figa@gmail.com> */
> + .init_irq = s3c6400_init_irq,
> + .map_io = s3c64xx_dt_map_io,
> + .handle_irq = vic_handle_irq,
> + .init_machine = s3c64xx_dt_machine_init,
> + .init_late = s3c64xx_init_late,
> + .timer = &s3c24xx_timer,
> + .dt_compat = s3c6400_dt_compat,
> + .restart = s3c64xx_restart,
> +MACHINE_END
> +
> +static char const *s3c6410_dt_compat[] __initdata = {
> + "samsung,s3c6410",
> + NULL
> +};
I think, we don't need separation.
+static char const *s3c64xx_dt_compat[] __initdata = {
+ "samsung,s3c6400",
+ "samsung,s3c6410",
+ NULL
+};
> +
> +DT_MACHINE_START(S3C6410_DT, "Samsung S3C6410 (Flattened Device
> Tree)")
> + /* Maintainer: Tomasz Figa <tomasz.figa@gmail.com> */
> + .init_irq = s3c6410_init_irq,
> + .map_io = s3c64xx_dt_map_io,
> + .handle_irq = vic_handle_irq,
> + .init_machine = s3c64xx_dt_machine_init,
> + .init_late = s3c64xx_init_late,
> + .timer = &s3c24xx_timer,
> + .dt_compat = s3c6410_dt_compat,
+ .dt_compat = s3c64xx_dt_compat,
> + .restart = s3c64xx_restart,
> +MACHINE_END
> --
> 1.8.1
next prev parent reply other threads:[~2013-01-10 0:24 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-09 23:41 [RFC PATCH 0/7] Initial Device Tree support for S3C64xx Tomasz Figa
2013-01-09 23:41 ` [RFC PATCH 1/7] ARM: common: vic: Parse interrupt and resume masks from device tree Tomasz Figa
2013-01-10 11:03 ` Mark Brown
2013-01-10 11:10 ` Tomasz Figa
2013-01-10 11:24 ` Mark Brown
2013-01-09 23:41 ` [RFC PATCH 2/7] ARM: common: vic: Fix invalid first IRQ number in OF-based registration Tomasz Figa
2013-01-09 23:41 ` [RFC PATCH 3/7] ARM: s3c64xx: Add support for OF-based VIC initialization Tomasz Figa
2013-01-10 11:10 ` Mark Brown
2013-01-09 23:41 ` [RFC PATCH 4/7] ARM: s3c64xx: Fix build error with CONFIG_S3C_DEV_FB disabled Tomasz Figa
2013-01-10 0:15 ` Kukjin Kim
2013-01-10 0:32 ` Tomasz Figa
2013-01-09 23:41 ` [RFC PATCH 5/7] ARM: s3c64xx: Add board file for boot using Device Tree Tomasz Figa
2013-01-10 0:24 ` Kukjin Kim [this message]
2013-01-10 0:38 ` Tomasz Figa
2013-01-10 10:52 ` Mark Brown
2013-01-10 10:55 ` Tomasz Figa
2013-01-09 23:41 ` [RFC PATCH 6/7] ARM: dts: Add basic dts include files for Samsung S3C64xx SoCs Tomasz Figa
2013-01-09 23:41 ` [RFC PATCH 7/7] ARM: dts: Add dts file for S3C6410-based Mini6410 board Tomasz Figa
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='00aa01cdeec8$e48ba7c0$ada2f740$@samsung.com' \
--to=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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;
as well as URLs for NNTP newsgroup(s).