All of lore.kernel.org
 help / color / mirror / Atom feed
From: mark.rutland@arm.com (Mark Rutland)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 2/5] Documentation: DT: Add bindings for Spreadtrum Sharkl Platform
Date: Fri, 17 Oct 2014 13:50:34 +0100	[thread overview]
Message-ID: <20141017125033.GA5587@leverpostej> (raw)
In-Reply-To: <1413539665-11484-3-git-send-email-chunyan.zhang@spreadtrum.com>

On Fri, Oct 17, 2014 at 10:54:22AM +0100, Chunyan Zhang wrote:
> Adds Spreadtrum's prefix "sprd" to vendor-prefixes file.
> Adds the devicetree binding documentation for Spreadtrum's serial
> and Sharkl3 SoC based on the Sharkl Platform which is the 64-bit
> SoC Platform of Spreadtrum.
> 
> Signed-off-by: Chunyan Zhang <chunyan.zhang@spreadtrum.com>
> ---
>  Documentation/devicetree/bindings/arm/sprd.txt     |   11 +++++++++++
>  .../devicetree/bindings/serial/sprd-serial.txt     |    6 ++++++
>  .../devicetree/bindings/vendor-prefixes.txt        |    1 +
>  3 files changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/sprd.txt
>  create mode 100644 Documentation/devicetree/bindings/serial/sprd-serial.txt
> 
> diff --git a/Documentation/devicetree/bindings/arm/sprd.txt b/Documentation/devicetree/bindings/arm/sprd.txt
> new file mode 100644
> index 0000000..40b40d6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/sprd.txt
> @@ -0,0 +1,11 @@
> +Spreadtrum Sharkl Platforms Device Tree Bindings
> +----------------------------------------------------
> +
> +Sharkl is a Spreadtrum's SoC Platform which is based
> +on ARM 64-bit processor.
> +
> +Sharkl3 FPGA board with Sharkl3 SoC based on the
> +Sharkl Platform shall have the following properties.
> +
> +Required root node properties:
> +        - compatible = "sprd,sharkl3-fpga", "sprd,sharkl3";
> diff --git a/Documentation/devicetree/bindings/serial/sprd-serial.txt b/Documentation/devicetree/bindings/serial/sprd-serial.txt
> new file mode 100644
> index 0000000..c97b0a6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/serial/sprd-serial.txt
> @@ -0,0 +1,6 @@
> +* Spreadtrum serial UART
> +
> +Required properties:
> +- compatible: must be "sprd,serial"
> +- reg: offset and length of the register set for the device
> +- interrupts: exactly one interrupt specifier

I'm still a little wary of then name "sprd,serial". What was the first
device to feature this UART, if it doesn't have a specific name of its
own?

> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index ac7269f..5dbf7db 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -129,6 +129,7 @@ smsc	Standard Microsystems Corporation
>  snps 	Synopsys, Inc.
>  solidrun	SolidRun
>  spansion	Spansion Inc.
> +sprd	Spreadtrum Communications Inc.
>  st	STMicroelectronics
>  ste	ST-Ericsson
>  stericsson	ST-Ericsson

The vendor prefix addition looks fine to me.

Thanks,
Mark.

WARNING: multiple messages have this Message-ID (diff)
From: Mark Rutland <mark.rutland@arm.com>
To: Chunyan Zhang <chunyan.zhang@spreadtrum.com>
Cc: Catalin Marinas <Catalin.Marinas@arm.com>,
	"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
	"ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
	"jslaby@suse.cz" <jslaby@suse.cz>,
	"galak@codeaurora.org" <galak@codeaurora.org>,
	"broonie@linaro.org" <broonie@linaro.org>,
	"m-karicheri2@ti.com" <m-karicheri2@ti.com>,
	Pawel Moll <Pawel.Moll@arm.com>,
	"artagnon@gmail.com" <artagnon@gmail.com>,
	"rrichter@cavium.com" <rrichter@cavium.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	Will Deacon <Will.Deacon@arm.com>,
	"orsonzhai@gmail.com" <orsonzhai@gmail.com>,
	"geng.ren@spreadtrum.com" <geng.ren@spreadtrum.com>,
	"zhizhou.zhang@spreadtrum.com" <zhizhou.zhang@spreadtrum.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kern>
Subject: Re: [PATCH v2 2/5] Documentation: DT: Add bindings for Spreadtrum Sharkl Platform
Date: Fri, 17 Oct 2014 13:50:34 +0100	[thread overview]
Message-ID: <20141017125033.GA5587@leverpostej> (raw)
In-Reply-To: <1413539665-11484-3-git-send-email-chunyan.zhang@spreadtrum.com>

On Fri, Oct 17, 2014 at 10:54:22AM +0100, Chunyan Zhang wrote:
> Adds Spreadtrum's prefix "sprd" to vendor-prefixes file.
> Adds the devicetree binding documentation for Spreadtrum's serial
> and Sharkl3 SoC based on the Sharkl Platform which is the 64-bit
> SoC Platform of Spreadtrum.
> 
> Signed-off-by: Chunyan Zhang <chunyan.zhang@spreadtrum.com>
> ---
>  Documentation/devicetree/bindings/arm/sprd.txt     |   11 +++++++++++
>  .../devicetree/bindings/serial/sprd-serial.txt     |    6 ++++++
>  .../devicetree/bindings/vendor-prefixes.txt        |    1 +
>  3 files changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/sprd.txt
>  create mode 100644 Documentation/devicetree/bindings/serial/sprd-serial.txt
> 
> diff --git a/Documentation/devicetree/bindings/arm/sprd.txt b/Documentation/devicetree/bindings/arm/sprd.txt
> new file mode 100644
> index 0000000..40b40d6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/sprd.txt
> @@ -0,0 +1,11 @@
> +Spreadtrum Sharkl Platforms Device Tree Bindings
> +----------------------------------------------------
> +
> +Sharkl is a Spreadtrum's SoC Platform which is based
> +on ARM 64-bit processor.
> +
> +Sharkl3 FPGA board with Sharkl3 SoC based on the
> +Sharkl Platform shall have the following properties.
> +
> +Required root node properties:
> +        - compatible = "sprd,sharkl3-fpga", "sprd,sharkl3";
> diff --git a/Documentation/devicetree/bindings/serial/sprd-serial.txt b/Documentation/devicetree/bindings/serial/sprd-serial.txt
> new file mode 100644
> index 0000000..c97b0a6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/serial/sprd-serial.txt
> @@ -0,0 +1,6 @@
> +* Spreadtrum serial UART
> +
> +Required properties:
> +- compatible: must be "sprd,serial"
> +- reg: offset and length of the register set for the device
> +- interrupts: exactly one interrupt specifier

I'm still a little wary of then name "sprd,serial". What was the first
device to feature this UART, if it doesn't have a specific name of its
own?

> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index ac7269f..5dbf7db 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -129,6 +129,7 @@ smsc	Standard Microsystems Corporation
>  snps 	Synopsys, Inc.
>  solidrun	SolidRun
>  spansion	Spansion Inc.
> +sprd	Spreadtrum Communications Inc.
>  st	STMicroelectronics
>  ste	ST-Ericsson
>  stericsson	ST-Ericsson

The vendor prefix addition looks fine to me.

Thanks,
Mark.

WARNING: multiple messages have this Message-ID (diff)
From: Mark Rutland <mark.rutland@arm.com>
To: Chunyan Zhang <chunyan.zhang@spreadtrum.com>
Cc: Catalin Marinas <Catalin.Marinas@arm.com>,
	"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
	"ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>,
	"jslaby@suse.cz" <jslaby@suse.cz>,
	"galak@codeaurora.org" <galak@codeaurora.org>,
	"broonie@linaro.org" <broonie@linaro.org>,
	"m-karicheri2@ti.com" <m-karicheri2@ti.com>,
	Pawel Moll <Pawel.Moll@arm.com>,
	"artagnon@gmail.com" <artagnon@gmail.com>,
	"rrichter@cavium.com" <rrichter@cavium.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	Will Deacon <Will.Deacon@arm.com>,
	"orsonzhai@gmail.com" <orsonzhai@gmail.com>,
	"geng.ren@spreadtrum.com" <geng.ren@spreadtrum.com>,
	"zhizhou.zhang@spreadtrum.com" <zhizhou.zhang@spreadtrum.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"sprdlinux@freelists.org" <sprdlinux@freelists.org>
Subject: Re: [PATCH v2 2/5] Documentation: DT: Add bindings for Spreadtrum Sharkl Platform
Date: Fri, 17 Oct 2014 13:50:34 +0100	[thread overview]
Message-ID: <20141017125033.GA5587@leverpostej> (raw)
In-Reply-To: <1413539665-11484-3-git-send-email-chunyan.zhang@spreadtrum.com>

On Fri, Oct 17, 2014 at 10:54:22AM +0100, Chunyan Zhang wrote:
> Adds Spreadtrum's prefix "sprd" to vendor-prefixes file.
> Adds the devicetree binding documentation for Spreadtrum's serial
> and Sharkl3 SoC based on the Sharkl Platform which is the 64-bit
> SoC Platform of Spreadtrum.
> 
> Signed-off-by: Chunyan Zhang <chunyan.zhang@spreadtrum.com>
> ---
>  Documentation/devicetree/bindings/arm/sprd.txt     |   11 +++++++++++
>  .../devicetree/bindings/serial/sprd-serial.txt     |    6 ++++++
>  .../devicetree/bindings/vendor-prefixes.txt        |    1 +
>  3 files changed, 18 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/sprd.txt
>  create mode 100644 Documentation/devicetree/bindings/serial/sprd-serial.txt
> 
> diff --git a/Documentation/devicetree/bindings/arm/sprd.txt b/Documentation/devicetree/bindings/arm/sprd.txt
> new file mode 100644
> index 0000000..40b40d6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/sprd.txt
> @@ -0,0 +1,11 @@
> +Spreadtrum Sharkl Platforms Device Tree Bindings
> +----------------------------------------------------
> +
> +Sharkl is a Spreadtrum's SoC Platform which is based
> +on ARM 64-bit processor.
> +
> +Sharkl3 FPGA board with Sharkl3 SoC based on the
> +Sharkl Platform shall have the following properties.
> +
> +Required root node properties:
> +        - compatible = "sprd,sharkl3-fpga", "sprd,sharkl3";
> diff --git a/Documentation/devicetree/bindings/serial/sprd-serial.txt b/Documentation/devicetree/bindings/serial/sprd-serial.txt
> new file mode 100644
> index 0000000..c97b0a6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/serial/sprd-serial.txt
> @@ -0,0 +1,6 @@
> +* Spreadtrum serial UART
> +
> +Required properties:
> +- compatible: must be "sprd,serial"
> +- reg: offset and length of the register set for the device
> +- interrupts: exactly one interrupt specifier

I'm still a little wary of then name "sprd,serial". What was the first
device to feature this UART, if it doesn't have a specific name of its
own?

> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index ac7269f..5dbf7db 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -129,6 +129,7 @@ smsc	Standard Microsystems Corporation
>  snps 	Synopsys, Inc.
>  solidrun	SolidRun
>  spansion	Spansion Inc.
> +sprd	Spreadtrum Communications Inc.
>  st	STMicroelectronics
>  ste	ST-Ericsson
>  stericsson	ST-Ericsson

The vendor prefix addition looks fine to me.

Thanks,
Mark.

  reply	other threads:[~2014-10-17 12:50 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <sprd-1017-v2>
2014-10-17  9:54 ` [PATCH v2 0/5] Add Spreadtrum Sharkl Platform support Chunyan Zhang
2014-10-17  9:54   ` Chunyan Zhang
2014-10-17  9:54   ` Chunyan Zhang
2014-10-17  9:54   ` [PATCH v2 1/5] Documentation: DT: Renamed of-serial.txt to 8250.txt Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17 13:12     ` Mark Rutland
2014-10-17 13:12       ` Mark Rutland
2014-10-17 13:12       ` Mark Rutland
2014-10-20 13:47     ` Arnd Bergmann
2014-10-20 13:47       ` Arnd Bergmann
2014-10-17  9:54   ` [PATCH v2 2/5] Documentation: DT: Add bindings for Spreadtrum Sharkl Platform Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17 12:50     ` Mark Rutland [this message]
2014-10-17 12:50       ` Mark Rutland
2014-10-17 12:50       ` Mark Rutland
2014-10-20 13:48     ` Arnd Bergmann
2014-10-20 13:48       ` Arnd Bergmann
2014-10-20 13:48       ` Arnd Bergmann
2014-10-17  9:54   ` [PATCH v2 3/5] arm64: dts: Add support for Spreadtrum Sharkl3 SoC in dts and Makefile Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17 13:00     ` Mark Rutland
2014-10-17 13:00       ` Mark Rutland
2014-10-17 13:00       ` Mark Rutland
2014-10-17  9:54   ` [PATCH v2 4/5] arm64: Add support for Spreadtrum's Sharkl Platform in Kconfig and defconfig Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17  9:54   ` [PATCH v2 5/5] tty/serial: Add earlycon support for Spreadtrum serial driver Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17  9:54     ` Chunyan Zhang
2014-10-17 13:03     ` Mark Rutland
2014-10-17 13:03       ` Mark Rutland
2014-10-17 13:03       ` Mark Rutland
2014-10-20  7:27       ` Lyra Zhang
2014-10-20  7:27         ` Lyra Zhang
2014-10-20  7:27         ` Lyra Zhang
2014-10-20 10:32         ` Mark Rutland
2014-10-20 10:32           ` Mark Rutland
2014-10-20 10:32           ` Mark Rutland
2014-10-20 11:15           ` Lyra Zhang
2014-10-20 11:15             ` Lyra Zhang
2014-10-20 11:15             ` Lyra Zhang
2014-10-18 21:06     ` One Thousand Gnomes
2014-10-18 21:06       ` One Thousand Gnomes
2014-10-18 21:06       ` One Thousand Gnomes
2014-10-20 10:23       ` Orson Zhai
2014-10-20 10:23         ` Orson Zhai
2014-10-20 10:23         ` Orson Zhai
2014-10-21 12:26         ` One Thousand Gnomes
2014-10-21 12:26           ` One Thousand Gnomes
2014-10-21 12:26           ` One Thousand Gnomes
2014-10-21 13:34           ` Orson Zhai
2014-10-21 13:34             ` Orson Zhai
2014-10-21 13:34             ` Orson Zhai
2014-10-21 11:15   ` [PATCH v2 0/5] Add Spreadtrum Sharkl Platform support Pavel Machek
2014-10-21 11:15     ` Pavel Machek

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=20141017125033.GA5587@leverpostej \
    --to=mark.rutland@arm.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 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.