Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
On Tue, 2015-08-04 at 18:42 +0100, Mark Brown wrote: > On Wed, Jul 29, 2015 at 07:04:32PM +0800, Leilk Liu wrote: > > Signed-off-by: Leilk Liu > > Please use subject lines reflecting the style for the subsystem so > people can spot if patches are relevant to them. > OK, I will change the title to "spi: Mediatek: Document devicetree bindings for spi bus". > > +- clocks: phandles to input clocks. > > + > > +- clock-names: tuple listing input clock names. > > + Required elements: "main" > > ...and there are no optional values? > clock tree provides some source clocks, CLK_PERI_SPI0 is the default one. I will support optional values on the next version. > > +- pad-select: should specify spi pad used, only required for MT8173. > > +This value should be 0~3. > > What do the values 0-3 mean? I guess it's the value for some register > or other part of the device, the binding should say so people can go and > check the datasheet, schematic or whatever to figure out what to set. > > This property should be optional, not required OK, I will explain what the values 0-3 mean. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
On Tue, 2015-08-04 at 18:42 +0100, Mark Brown wrote: On Wed, Jul 29, 2015 at 07:04:32PM +0800, Leilk Liu wrote: Signed-off-by: Leilk Liu leilk@mediatek.com Please use subject lines reflecting the style for the subsystem so people can spot if patches are relevant to them. OK, I will change the title to spi: Mediatek: Document devicetree bindings for spi bus. +- clocks: phandles to input clocks. + +- clock-names: tuple listing input clock names. + Required elements: main ...and there are no optional values? clock tree provides some source clocks, CLK_PERI_SPI0 is the default one. I will support optional values on the next version. +- pad-select: should specify spi pad used, only required for MT8173. +This value should be 0~3. What do the values 0-3 mean? I guess it's the value for some register or other part of the device, the binding should say so people can go and check the datasheet, schematic or whatever to figure out what to set. This property should be optional, not required OK, I will explain what the values 0-3 mean. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
On Wed, Jul 29, 2015 at 07:04:32PM +0800, Leilk Liu wrote: > Signed-off-by: Leilk Liu Please use subject lines reflecting the style for the subsystem so people can spot if patches are relevant to them. > +- clocks: phandles to input clocks. > + > +- clock-names: tuple listing input clock names. > + Required elements: "main" ...and there are no optional values? > +- pad-select: should specify spi pad used, only required for MT8173. > +This value should be 0~3. What do the values 0-3 mean? I guess it's the value for some register or other part of the device, the binding should say so people can go and check the datasheet, schematic or whatever to figure out what to set. This property should be optional, not required signature.asc Description: Digital signature
Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
On Wed, Jul 29, 2015 at 07:04:32PM +0800, Leilk Liu wrote: Signed-off-by: Leilk Liu leilk@mediatek.com Please use subject lines reflecting the style for the subsystem so people can spot if patches are relevant to them. +- clocks: phandles to input clocks. + +- clock-names: tuple listing input clock names. + Required elements: main ...and there are no optional values? +- pad-select: should specify spi pad used, only required for MT8173. +This value should be 0~3. What do the values 0-3 mean? I guess it's the value for some register or other part of the device, the binding should say so people can go and check the datasheet, schematic or whatever to figure out what to set. This property should be optional, not required signature.asc Description: Digital signature
Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
Hi Jonas,
On Thu, 2015-07-30 at 21:27 +0200, Jonas Gorski wrote:
> Hi,
>
> On Wed, Jul 29, 2015 at 1:04 PM, Leilk Liu wrote:
> > Signed-off-by: Leilk Liu
> > ---
> > .../devicetree/bindings/spi/spi-mt65xx.txt | 38
> > ++
> > 1 file changed, 38 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/spi/spi-mt65xx.txt
> >
> > diff --git a/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
> > b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
> > new file mode 100644
> > index 000..f8005d6
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
> > @@ -0,0 +1,38 @@
> > +MTK SPI device
>
> You are calling it "MediaTek SPI controller" in the Kconfig entry, so
> you should call it that here as well.
>
OK, I will fix it.
> > +
> > +Required properties:
> > +- compatible: should be one of the following.
> > +- mediatek,mt8173-spi: for mt8173 platforms
> > +- mediatek,mt8135-spi: for mt8135 platforms
> > +- mediatek,mt6589-spi: for mt6589 platforms
> > +
> > +- #address-cells: should be 1.
> > +
> > +- #size-cells: should be 0.
> > +
> > +- reg: Address and length of the register set for the device
> > +
> > +- interrupts: Should contain spi interrupt
> > +
> > +- clocks: phandles to input clocks.
> > +
> > +- clock-names: tuple listing input clock names.
> > + Required elements: "main"
> > +
> > +- pad-select: should specify spi pad used, only required for MT8173.
> > +This value should be 0~3.
>
> As already commented on the v3, this is a vendor property, and must
> have a vendor prefix, so it must be called "mediatek,pad-select".
>
OK, I will fix it on the next version.
> > +
> > +Example:
> > +
> > +- SoC Specific Portion:
> > +spi: spi@1100a000 {
> > + compatible = "mediatek,mt8173-spi";
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + reg = <0 0x1100a000 0 0x1000>;
> > + interrupts = ;
> > + clocks = < CLK_PERI_SPI0>;
> > + clock-names = "main";
> > + pad-select = <0>;
> > + status = "disabled";
> > +};
>
>
> Jonas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
Hi,
On Wed, Jul 29, 2015 at 1:04 PM, Leilk Liu wrote:
> Signed-off-by: Leilk Liu
> ---
> .../devicetree/bindings/spi/spi-mt65xx.txt | 38
> ++
> 1 file changed, 38 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/spi/spi-mt65xx.txt
>
> diff --git a/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
> b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
> new file mode 100644
> index 000..f8005d6
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
> @@ -0,0 +1,38 @@
> +MTK SPI device
You are calling it "MediaTek SPI controller" in the Kconfig entry, so
you should call it that here as well.
> +
> +Required properties:
> +- compatible: should be one of the following.
> +- mediatek,mt8173-spi: for mt8173 platforms
> +- mediatek,mt8135-spi: for mt8135 platforms
> +- mediatek,mt6589-spi: for mt6589 platforms
> +
> +- #address-cells: should be 1.
> +
> +- #size-cells: should be 0.
> +
> +- reg: Address and length of the register set for the device
> +
> +- interrupts: Should contain spi interrupt
> +
> +- clocks: phandles to input clocks.
> +
> +- clock-names: tuple listing input clock names.
> + Required elements: "main"
> +
> +- pad-select: should specify spi pad used, only required for MT8173.
> +This value should be 0~3.
As already commented on the v3, this is a vendor property, and must
have a vendor prefix, so it must be called "mediatek,pad-select".
> +
> +Example:
> +
> +- SoC Specific Portion:
> +spi: spi@1100a000 {
> + compatible = "mediatek,mt8173-spi";
> + #address-cells = <1>;
> + #size-cells = <0>;
> + reg = <0 0x1100a000 0 0x1000>;
> + interrupts = ;
> + clocks = < CLK_PERI_SPI0>;
> + clock-names = "main";
> + pad-select = <0>;
> + status = "disabled";
> +};
Jonas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
Hi,
On Wed, Jul 29, 2015 at 1:04 PM, Leilk Liu leilk@mediatek.com wrote:
Signed-off-by: Leilk Liu leilk@mediatek.com
---
.../devicetree/bindings/spi/spi-mt65xx.txt | 38
++
1 file changed, 38 insertions(+)
create mode 100644 Documentation/devicetree/bindings/spi/spi-mt65xx.txt
diff --git a/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
new file mode 100644
index 000..f8005d6
--- /dev/null
+++ b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
@@ -0,0 +1,38 @@
+MTK SPI device
You are calling it MediaTek SPI controller in the Kconfig entry, so
you should call it that here as well.
+
+Required properties:
+- compatible: should be one of the following.
+- mediatek,mt8173-spi: for mt8173 platforms
+- mediatek,mt8135-spi: for mt8135 platforms
+- mediatek,mt6589-spi: for mt6589 platforms
+
+- #address-cells: should be 1.
+
+- #size-cells: should be 0.
+
+- reg: Address and length of the register set for the device
+
+- interrupts: Should contain spi interrupt
+
+- clocks: phandles to input clocks.
+
+- clock-names: tuple listing input clock names.
+ Required elements: main
+
+- pad-select: should specify spi pad used, only required for MT8173.
+This value should be 0~3.
As already commented on the v3, this is a vendor property, and must
have a vendor prefix, so it must be called mediatek,pad-select.
+
+Example:
+
+- SoC Specific Portion:
+spi: spi@1100a000 {
+ compatible = mediatek,mt8173-spi;
+ #address-cells = 1;
+ #size-cells = 0;
+ reg = 0 0x1100a000 0 0x1000;
+ interrupts = GIC_SPI 110 IRQ_TYPE_LEVEL_LOW;
+ clocks = pericfg CLK_PERI_SPI0;
+ clock-names = main;
+ pad-select = 0;
+ status = disabled;
+};
Jonas
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
Hi Jonas,
On Thu, 2015-07-30 at 21:27 +0200, Jonas Gorski wrote:
Hi,
On Wed, Jul 29, 2015 at 1:04 PM, Leilk Liu leilk@mediatek.com wrote:
Signed-off-by: Leilk Liu leilk@mediatek.com
---
.../devicetree/bindings/spi/spi-mt65xx.txt | 38
++
1 file changed, 38 insertions(+)
create mode 100644 Documentation/devicetree/bindings/spi/spi-mt65xx.txt
diff --git a/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
new file mode 100644
index 000..f8005d6
--- /dev/null
+++ b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
@@ -0,0 +1,38 @@
+MTK SPI device
You are calling it MediaTek SPI controller in the Kconfig entry, so
you should call it that here as well.
OK, I will fix it.
+
+Required properties:
+- compatible: should be one of the following.
+- mediatek,mt8173-spi: for mt8173 platforms
+- mediatek,mt8135-spi: for mt8135 platforms
+- mediatek,mt6589-spi: for mt6589 platforms
+
+- #address-cells: should be 1.
+
+- #size-cells: should be 0.
+
+- reg: Address and length of the register set for the device
+
+- interrupts: Should contain spi interrupt
+
+- clocks: phandles to input clocks.
+
+- clock-names: tuple listing input clock names.
+ Required elements: main
+
+- pad-select: should specify spi pad used, only required for MT8173.
+This value should be 0~3.
As already commented on the v3, this is a vendor property, and must
have a vendor prefix, so it must be called mediatek,pad-select.
OK, I will fix it on the next version.
+
+Example:
+
+- SoC Specific Portion:
+spi: spi@1100a000 {
+ compatible = mediatek,mt8173-spi;
+ #address-cells = 1;
+ #size-cells = 0;
+ reg = 0 0x1100a000 0 0x1000;
+ interrupts = GIC_SPI 110 IRQ_TYPE_LEVEL_LOW;
+ clocks = pericfg CLK_PERI_SPI0;
+ clock-names = main;
+ pad-select = 0;
+ status = disabled;
+};
Jonas
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
Signed-off-by: Leilk Liu
---
.../devicetree/bindings/spi/spi-mt65xx.txt | 38 ++
1 file changed, 38 insertions(+)
create mode 100644 Documentation/devicetree/bindings/spi/spi-mt65xx.txt
diff --git a/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
new file mode 100644
index 000..f8005d6
--- /dev/null
+++ b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
@@ -0,0 +1,38 @@
+MTK SPI device
+
+Required properties:
+- compatible: should be one of the following.
+- mediatek,mt8173-spi: for mt8173 platforms
+- mediatek,mt8135-spi: for mt8135 platforms
+- mediatek,mt6589-spi: for mt6589 platforms
+
+- #address-cells: should be 1.
+
+- #size-cells: should be 0.
+
+- reg: Address and length of the register set for the device
+
+- interrupts: Should contain spi interrupt
+
+- clocks: phandles to input clocks.
+
+- clock-names: tuple listing input clock names.
+ Required elements: "main"
+
+- pad-select: should specify spi pad used, only required for MT8173.
+This value should be 0~3.
+
+Example:
+
+- SoC Specific Portion:
+spi: spi@1100a000 {
+ compatible = "mediatek,mt8173-spi";
+ #address-cells = <1>;
+ #size-cells = <0>;
+ reg = <0 0x1100a000 0 0x1000>;
+ interrupts = ;
+ clocks = < CLK_PERI_SPI0>;
+ clock-names = "main";
+ pad-select = <0>;
+ status = "disabled";
+};
--
1.8.1.1.dirty
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[PATCH v4 1/3] dt-binding: spi: Mediatek: Document devicetree bindings for spi bus
Signed-off-by: Leilk Liu leilk@mediatek.com
---
.../devicetree/bindings/spi/spi-mt65xx.txt | 38 ++
1 file changed, 38 insertions(+)
create mode 100644 Documentation/devicetree/bindings/spi/spi-mt65xx.txt
diff --git a/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
new file mode 100644
index 000..f8005d6
--- /dev/null
+++ b/Documentation/devicetree/bindings/spi/spi-mt65xx.txt
@@ -0,0 +1,38 @@
+MTK SPI device
+
+Required properties:
+- compatible: should be one of the following.
+- mediatek,mt8173-spi: for mt8173 platforms
+- mediatek,mt8135-spi: for mt8135 platforms
+- mediatek,mt6589-spi: for mt6589 platforms
+
+- #address-cells: should be 1.
+
+- #size-cells: should be 0.
+
+- reg: Address and length of the register set for the device
+
+- interrupts: Should contain spi interrupt
+
+- clocks: phandles to input clocks.
+
+- clock-names: tuple listing input clock names.
+ Required elements: main
+
+- pad-select: should specify spi pad used, only required for MT8173.
+This value should be 0~3.
+
+Example:
+
+- SoC Specific Portion:
+spi: spi@1100a000 {
+ compatible = mediatek,mt8173-spi;
+ #address-cells = 1;
+ #size-cells = 0;
+ reg = 0 0x1100a000 0 0x1000;
+ interrupts = GIC_SPI 110 IRQ_TYPE_LEVEL_LOW;
+ clocks = pericfg CLK_PERI_SPI0;
+ clock-names = main;
+ pad-select = 0;
+ status = disabled;
+};
--
1.8.1.1.dirty
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
