All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomasz Figa <tomasz.figa@gmail.com>
To: "Andreas Färber" <afaerber@suse.de>, linux-samsung-soc@vger.kernel.org
Cc: Mark Rutland <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, Kukjin Kim <kgene.kim@samsung.com>,
	Vincent Palatin <vpalatin@chromium.org>,
	Pawel Moll <pawel.moll@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	open list <linux-kernel@vger.kernel.org>,
	Tomasz Figa <t.figa@samsung.com>,
	Kumar Gala <galak@codeaurora.org>,
	Stephan van Schaik <stephan@synkhronix.com>,
	Doug Anderson <dianders@chromium.org>,
	Rob Herring <robh+dt@kernel.org>, Ben Dooks <ben-linux@fluff.org>,
	Russell King <linux@arm.linux.org.uk>,
	Javier Martinez Canillas <javier.martinez@collabora.co.uk>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v6 03/10] ARM: dts: Prepare node labels for exynos5250
Date: Sat, 02 Aug 2014 14:19:23 +0200	[thread overview]
Message-ID: <53DCD74B.8090400@gmail.com> (raw)
In-Reply-To: <1406940750-15880-4-git-send-email-afaerber@suse.de>

Andreas,

Ehh, one more thing. It looks like my memory is playing tricks on me.
Please see inline.

On 02.08.2014 02:52, Andreas Färber wrote:
> Allows them to be extended by reference.
> 
> Signed-off-by: Andreas Färber <afaerber@suse.de>
> ---
>  v6: Split off from Snow/SMDK cleanups (Doug Anderson)

[snip]

> -	serial@12C00000 {
> +	uart0: serial@12C00000 {
>  		clocks = <&clock CLK_UART0>, <&clock CLK_SCLK_UART0>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};
>  
> -	serial@12C10000 {
> +	uart1: serial@12C10000 {
>  		clocks = <&clock CLK_UART1>, <&clock CLK_SCLK_UART1>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};
>  
> -	serial@12C20000 {
> +	uart2: serial@12C20000 {
>  		clocks = <&clock CLK_UART2>, <&clock CLK_SCLK_UART2>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};
>  
> -	serial@12C30000 {
> +	uart3: serial@12C30000 {
>  		clocks = <&clock CLK_UART3>, <&clock CLK_SCLK_UART3>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};

I forgot about a series [1] that went through tty tree, which is going
to conflict with this patch. It already adds labels to uart nodes in
serial_X format. I don't really remember why I went with this format for
Exynos SoCs and with uartX for older ones...

[1]
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/arch/arm/boot/dts?id=1e64f48ea7c59fb94acd7551dd9fa68b8a752efc

Best regards,
Tomasz

WARNING: multiple messages have this Message-ID (diff)
From: tomasz.figa@gmail.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 03/10] ARM: dts: Prepare node labels for exynos5250
Date: Sat, 02 Aug 2014 14:19:23 +0200	[thread overview]
Message-ID: <53DCD74B.8090400@gmail.com> (raw)
In-Reply-To: <1406940750-15880-4-git-send-email-afaerber@suse.de>

Andreas,

Ehh, one more thing. It looks like my memory is playing tricks on me.
Please see inline.

On 02.08.2014 02:52, Andreas F?rber wrote:
> Allows them to be extended by reference.
> 
> Signed-off-by: Andreas F?rber <afaerber@suse.de>
> ---
>  v6: Split off from Snow/SMDK cleanups (Doug Anderson)

[snip]

> -	serial at 12C00000 {
> +	uart0: serial at 12C00000 {
>  		clocks = <&clock CLK_UART0>, <&clock CLK_SCLK_UART0>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};
>  
> -	serial at 12C10000 {
> +	uart1: serial at 12C10000 {
>  		clocks = <&clock CLK_UART1>, <&clock CLK_SCLK_UART1>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};
>  
> -	serial at 12C20000 {
> +	uart2: serial at 12C20000 {
>  		clocks = <&clock CLK_UART2>, <&clock CLK_SCLK_UART2>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};
>  
> -	serial at 12C30000 {
> +	uart3: serial at 12C30000 {
>  		clocks = <&clock CLK_UART3>, <&clock CLK_SCLK_UART3>;
>  		clock-names = "uart", "clk_uart_baud0";
>  	};

I forgot about a series [1] that went through tty tree, which is going
to conflict with this patch. It already adds labels to uart nodes in
serial_X format. I don't really remember why I went with this format for
Exynos SoCs and with uartX for older ones...

[1]
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/arch/arm/boot/dts?id=1e64f48ea7c59fb94acd7551dd9fa68b8a752efc

Best regards,
Tomasz

  parent reply	other threads:[~2014-08-02 12:19 UTC|newest]

Thread overview: 93+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-02  0:52 [PATCH v6 00/10] ARM: dts: exynos: Prepare Spring Andreas Färber
2014-08-02  0:52 ` Andreas Färber
     [not found] ` <1406940750-15880-1-git-send-email-afaerber-l3A5Bk7waGM@public.gmane.org>
2014-08-02  0:52   ` [PATCH v6 01/10] ARM: dts: Fix MMC pinctrl for exynos5250-snow Andreas Färber
2014-08-02  0:52     ` Andreas Färber
2014-08-02  0:52     ` Andreas Färber
2014-08-02  0:52 ` [PATCH v6 02/10] ARM: dts: Fold exynos5250-cros-common into exynos5250-snow Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52 ` [PATCH v6 03/10] ARM: dts: Prepare node labels for exynos5250 Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  4:48   ` Doug Anderson
2014-08-02  4:48     ` Doug Anderson
2014-08-02 12:19   ` Tomasz Figa [this message]
2014-08-02 12:19     ` Tomasz Figa
     [not found]     ` <53DCD74B.8090400-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-08-02 13:00       ` Andreas Färber
2014-08-02 13:00         ` Andreas Färber
2014-08-02 13:00         ` Andreas Färber
2014-08-20 13:06         ` Andreas Färber
2014-08-20 13:06           ` Andreas Färber
2014-08-20 13:02     ` [PATCH v7] " Andreas Färber
2014-08-20 13:02       ` Andreas Färber
2014-08-20 13:02       ` Andreas Färber
2014-08-22 10:49       ` Kukjin Kim
2014-08-22 10:49         ` Kukjin Kim
2014-08-22 11:37         ` Andreas Färber
2014-08-22 11:37           ` Andreas Färber
2014-08-22 16:14         ` Doug Anderson
2014-08-22 16:14           ` Doug Anderson
2014-08-02  0:52 ` [PATCH v6 04/10] ARM: dts: Clean up exynos5250-snow Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  4:54   ` Doug Anderson
2014-08-02  4:54     ` Doug Anderson
2014-08-02 12:21   ` Tomasz Figa
2014-08-02 12:21     ` Tomasz Figa
2014-08-02 12:56     ` Tomasz Figa
2014-08-02 12:56       ` Tomasz Figa
2014-08-02 13:08       ` Andreas Färber
2014-08-02 13:08         ` Andreas Färber
2014-08-02 13:09         ` Tomasz Figa
2014-08-02 13:09           ` Tomasz Figa
2014-08-02  0:52 ` [PATCH v6 05/10] ARM: dts: Fill in bootargs for exynos5250-snow Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  2:17   ` Javier Martinez Canillas
2014-08-02  2:17     ` Javier Martinez Canillas
2014-08-02  0:52 ` [PATCH v6 06/10] ARM: dts: Clean up exynos5250-smdk5250 Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02 12:57   ` Tomasz Figa
2014-08-02 12:57     ` Tomasz Figa
2014-08-02  0:52 ` [PATCH v6 07/10] ARM: dts: Clean up exynos5250-arndale Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
     [not found]   ` <1406940750-15880-8-git-send-email-afaerber-l3A5Bk7waGM@public.gmane.org>
2014-08-02 13:02     ` Tomasz Figa
2014-08-02 13:02       ` Tomasz Figa
2014-08-02 13:02       ` Tomasz Figa
2014-08-02 13:10       ` Andreas Färber
2014-08-02 13:10         ` Andreas Färber
2014-08-02  0:52 ` [PATCH v6 08/10] ARM: dts: Fix apparent GPIO typo in exynos5250-arndale Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52 ` [PATCH v6 09/10] ARM: dts: Simplify USB3503 on exynos5250-arndale Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52 ` [PATCH v6 10/10] ARM: dts: Add exynos5250-spring device tree Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02  0:52   ` Andreas Färber
2014-08-02 13:18   ` Tomasz Figa
2014-08-02 13:18     ` Tomasz Figa
2014-08-04 15:22   ` Doug Anderson
2014-08-04 15:22     ` Doug Anderson
2014-08-04 17:31     ` Javier Martinez Canillas
2014-08-04 17:31       ` Javier Martinez Canillas
2014-08-20 12:50     ` Andreas Färber
2014-08-20 12:50       ` Andreas Färber
2014-08-02  2:34 ` [PATCH v6 00/10] ARM: dts: exynos: Prepare Spring Javier Martinez Canillas
2014-08-02  2:34   ` Javier Martinez Canillas
2014-08-02  4:57   ` Doug Anderson
2014-08-02  4:57     ` Doug Anderson
2014-08-02 10:25     ` Andreas Färber
2014-08-02 10:25       ` Andreas Färber
2014-08-02 13:13       ` Tomasz Figa
2014-08-02 13:13         ` Tomasz Figa
2014-08-02 16:05         ` Andreas Färber
2014-08-02 16:05           ` Andreas Färber
2014-08-04 15:42       ` Doug Anderson
2014-08-04 15:42         ` Doug Anderson
2014-08-04 17:43         ` Javier Martinez Canillas
2014-08-04 17:43           ` Javier Martinez Canillas
2014-08-07  8:40         ` Javier Martinez Canillas
2014-08-07  8:40           ` Javier Martinez Canillas

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=53DCD74B.8090400@gmail.com \
    --to=tomasz.figa@gmail.com \
    --cc=afaerber@suse.de \
    --cc=ben-linux@fluff.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dianders@chromium.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=javier.martinez@collabora.co.uk \
    --cc=kgene.kim@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=stephan@synkhronix.com \
    --cc=t.figa@samsung.com \
    --cc=vpalatin@chromium.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.