* [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2
@ 2020-02-10 10:35 agriveaux
2020-02-11 13:15 ` Maxime Ripard
0 siblings, 1 reply; 6+ messages in thread
From: agriveaux @ 2020-02-10 10:35 UTC (permalink / raw)
To: robh+dt, mark.rutland, mripard, wens; +Cc: linux-kernel, devicetree, agriveaux
From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
Add Inet 86V Rev 2 support, based upon Inet 86VS.
Missing things:
- Accelerometer (MXC6225X)
- Touchpanel (Sitronix SL1536)
- Nand (29F32G08CBACA)
- Camera (HCWY0308)
Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
---
.../devicetree/bindings/arm/sunxi.yaml | 5 +++++
arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts | 17 +++++++++++++++++
2 files changed, 22 insertions(+)
create mode 100644 arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
index 327ce6730823..af0db1fe69c9 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.yaml
+++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
@@ -282,6 +282,11 @@ properties:
- const: primux,inet86dz
- const: allwinner,sun8i-a23
+ - description: iNet-86V Rev 02
+ items:
+ - const: primux,inet86v-rev2
+ - const: allwinner,sun5i-a13
+
- description: iNet-9F Rev 03
items:
- const: inet-tek,inet9f-rev03
diff --git a/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts b/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
new file mode 100644
index 000000000000..9b4c5349f048
--- /dev/null
+++ b/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
@@ -0,0 +1,17 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright 2020 Alexandre Griveaux <agriveaux@deutnet.info>
+ *
+ * Minimal dts file for the iNet 86V
+ */
+
+/dts-v1/;
+
+#include "sun5i-a13.dtsi"
+#include "sun5i-reference-design-tablet.dtsi"
+
+/ {
+ model = "iNET 86V Rev 02";
+ compatible = "primux,inet86v-rev2", "allwinner,sun5i-a13";
+
+};
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2
2020-02-10 10:35 [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2 agriveaux
@ 2020-02-11 13:15 ` Maxime Ripard
2020-02-12 9:45 ` Alexandre GRIVEAUX
0 siblings, 1 reply; 6+ messages in thread
From: Maxime Ripard @ 2020-02-11 13:15 UTC (permalink / raw)
To: agriveaux; +Cc: robh+dt, mark.rutland, wens, linux-kernel, devicetree
[-- Attachment #1: Type: text/plain, Size: 691 bytes --]
On Mon, Feb 10, 2020 at 11:35:52AM +0100, agriveaux@deutnet.info wrote:
> From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>
> Add Inet 86V Rev 2 support, based upon Inet 86VS.
>
> Missing things:
> - Accelerometer (MXC6225X)
> - Touchpanel (Sitronix SL1536)
> - Nand (29F32G08CBACA)
> - Camera (HCWY0308)
>
> Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
Please read the documentation I sent you yesterday. In particular,
when submitting multiple versions, you should remove have the version
number in the title and a changelog.
Also, please ask questions if you're unsure about something, or
discuss something you do not agree with instead of ignoring the
comment.
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2
2020-02-11 13:15 ` Maxime Ripard
@ 2020-02-12 9:45 ` Alexandre GRIVEAUX
2020-02-14 13:17 ` Maxime Ripard
0 siblings, 1 reply; 6+ messages in thread
From: Alexandre GRIVEAUX @ 2020-02-12 9:45 UTC (permalink / raw)
To: Maxime Ripard; +Cc: robh+dt, mark.rutland, wens, linux-kernel, devicetree
Le 11/02/2020 à 14:15, Maxime Ripard a écrit :
> On Mon, Feb 10, 2020 at 11:35:52AM +0100, agriveaux@deutnet.info wrote:
>> From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>>
>> Add Inet 86V Rev 2 support, based upon Inet 86VS.
>>
>> Missing things:
>> - Accelerometer (MXC6225X)
>> - Touchpanel (Sitronix SL1536)
>> - Nand (29F32G08CBACA)
>> - Camera (HCWY0308)
>>
>> Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
> Please read the documentation I sent you yesterday. In particular,
> when submitting multiple versions, you should remove have the version
> number in the title and a changelog.
>
> Also, please ask questions if you're unsure about something, or
> discuss something you do not agree with instead of ignoring the
> comment.
>
> Maxime
Hello,
Seem I misusing 'git send-email' because i already changed the subject
line (same apply to u-boot):
git send-email --from agriveaux@deutnet.info --to
robh+dt@kernel.org,mark.rutland@arm.com,mripard@kernel.org,wens@csie.org
--cc
linux-kernel@vger.kernel.org,devicetree@vger.kernel.org,agriveaux@deutnet.info
--subject '[v2] ARM: dts: sun5i: Add dts for inet86v_rev2' --compose
/tmp/linux/0001-ARM-dts-sun5i-Add-dts-for-inet86v_rev2.patch
It's the first time i use git send-email, previously i used mutt to send
patchsets (on mips mostly).
Effectively i've forgot the changelog...
At why do we need another device tree:
inet86vs use a GSL1680 touchpanel controller and 4GB nand [0]
inet86vsuse a Sitonix SL1536 touchpanel controller and 8GB nand (can
have 16GB nand)[1]
Seem i've forgot to use:
git format-patch --subject-prefix="PATCH v2"
Thanks.
[0]https://fccid.io/2AARJ-AP7S118/Schematics/Schematics-2042312.pdfinet86vs
schematic
[1]https://fccid.io/XHW026/Schematics/Schematics-1820577.pdfinet86v
schematic
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2
2020-02-12 9:45 ` Alexandre GRIVEAUX
@ 2020-02-14 13:17 ` Maxime Ripard
0 siblings, 0 replies; 6+ messages in thread
From: Maxime Ripard @ 2020-02-14 13:17 UTC (permalink / raw)
To: Alexandre GRIVEAUX; +Cc: robh+dt, mark.rutland, wens, linux-kernel, devicetree
[-- Attachment #1: Type: text/plain, Size: 2229 bytes --]
Hi,
On Wed, Feb 12, 2020 at 10:45:03AM +0100, Alexandre GRIVEAUX wrote:
> Le 11/02/2020 à 14:15, Maxime Ripard a écrit :
> > On Mon, Feb 10, 2020 at 11:35:52AM +0100, agriveaux@deutnet.info wrote:
> >> From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
> >>
> >> Add Inet 86V Rev 2 support, based upon Inet 86VS.
> >>
> >> Missing things:
> >> - Accelerometer (MXC6225X)
> >> - Touchpanel (Sitronix SL1536)
> >> - Nand (29F32G08CBACA)
> >> - Camera (HCWY0308)
> >>
> >> Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
> > Please read the documentation I sent you yesterday. In particular,
> > when submitting multiple versions, you should remove have the version
> > number in the title and a changelog.
> >
> > Also, please ask questions if you're unsure about something, or
> > discuss something you do not agree with instead of ignoring the
> > comment.
> >
> > Maxime
>
>
> Seem I misusing 'git send-email' because i already changed the subject
> line (same apply to u-boot):
>
> git send-email --from agriveaux@deutnet.info --to
> robh+dt@kernel.org,mark.rutland@arm.com,mripard@kernel.org,wens@csie.org
> --cc
> linux-kernel@vger.kernel.org,devicetree@vger.kernel.org,agriveaux@deutnet.info
> --subject '[v2] ARM: dts: sun5i: Add dts for inet86v_rev2' --compose
> /tmp/linux/0001-ARM-dts-sun5i-Add-dts-for-inet86v_rev2.patch
>
> It's the first time i use git send-email, previously i used mutt to send
> patchsets (on mips mostly).
>
>
> Effectively i've forgot the changelog...
A simpler way, especially for a single patch, would be to use
git send-email -v2 --to $recipients --annotate $COMMIT^..$COMMIT
You can even keep the change log in the commit log itself by using ---
as a separator, so something like:
ARM: dts: my super title
Some nice awesome change
---
Changes from v1:
- Whatever.
> At why do we need another device tree:
>
> inet86vs use a GSL1680 touchpanel controller and 4GB nand [0]
>
> inet86vsuse a Sitonix SL1536 touchpanel controller and 8GB nand (can
> have 16GB nand)[1]
NAND size will be discovered so it's not really a concern. For the
touchscreen, you should make that clear in your commit log.
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* ARM: dts: sun5i: Add dts for inet86v_rev2
@ 2020-02-10 9:27 agriveaux
2020-02-10 9:27 ` [PATCH] " agriveaux
0 siblings, 1 reply; 6+ messages in thread
From: agriveaux @ 2020-02-10 9:27 UTC (permalink / raw)
To: robh+dt, mark.rutland, mripard, wens; +Cc: linux-kernel, devicetree, agriveaux
ARM: dts: sun5i: Add dts for inet86v_rev2
Add Inet 86V Rev 2 support, based upon Inet 86VS.
Missing things:
- Accelerometer (MXC6225X)
- Touchpanel (Sitronix SL1536)
- Nand (29F32G08CBACA)
- Camera (HCWY0308)
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2
2020-02-10 9:27 agriveaux
@ 2020-02-10 9:27 ` agriveaux
2020-02-10 9:41 ` Maxime Ripard
0 siblings, 1 reply; 6+ messages in thread
From: agriveaux @ 2020-02-10 9:27 UTC (permalink / raw)
To: robh+dt, mark.rutland, mripard, wens; +Cc: linux-kernel, devicetree, agriveaux
From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
Add Inet 86V Rev 2 support, based upon Inet 86VS.
Missing things:
- Accelerometer (MXC6225X)
- Touchpanel (Sitronix SL1536)
- Nand (29F32G08CBACA)
- Camera (HCWY0308)
---
arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
create mode 100644 arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
diff --git a/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts b/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
new file mode 100644
index 000000000000..e73abb9a1e32
--- /dev/null
+++ b/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
@@ -0,0 +1,17 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright 2020 Alexandre Griveaux <agriveaux@deutnet.info>
+ *
+ * Minimal dts file for the iNet 86V
+ */
+
+/dts-v1/;
+
+#include "sun5i-a13.dtsi"
+#include "sun5i-reference-design-tablet.dtsi"
+
+/ {
+ model = "iNET 86V Rev 02";
+ compatible = "inet,86v-rev2", "allwinner,sun5i-a13";
+
+};
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2
2020-02-10 9:27 ` [PATCH] " agriveaux
@ 2020-02-10 9:41 ` Maxime Ripard
0 siblings, 0 replies; 6+ messages in thread
From: Maxime Ripard @ 2020-02-10 9:41 UTC (permalink / raw)
To: agriveaux; +Cc: robh+dt, mark.rutland, wens, linux-kernel, devicetree
[-- Attachment #1: Type: text/plain, Size: 1257 bytes --]
Hi,
On Mon, Feb 10, 2020 at 10:27:36AM +0100, agriveaux@deutnet.info wrote:
> From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>
> Add Inet 86V Rev 2 support, based upon Inet 86VS.
>
> Missing things:
> - Accelerometer (MXC6225X)
> - Touchpanel (Sitronix SL1536)
> - Nand (29F32G08CBACA)
> - Camera (HCWY0308)
Same thing than for U-Boot, you're missing your SoB.
> ---
> arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
> create mode 100644 arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
>
> diff --git a/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts b/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
> new file mode 100644
> index 000000000000..e73abb9a1e32
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun5i-a13-inet-86v-rev2.dts
> @@ -0,0 +1,17 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Copyright 2020 Alexandre Griveaux <agriveaux@deutnet.info>
> + *
> + * Minimal dts file for the iNet 86V
> + */
> +
> +/dts-v1/;
> +
> +#include "sun5i-a13.dtsi"
> +#include "sun5i-reference-design-tablet.dtsi"
> +
> +/ {
> + model = "iNET 86V Rev 02";
> + compatible = "inet,86v-rev2", "allwinner,sun5i-a13";
> +
> +};
If it's exactly the same device, why do we need another device tree?
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-02-14 13:17 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-10 10:35 [PATCH] ARM: dts: sun5i: Add dts for inet86v_rev2 agriveaux
2020-02-11 13:15 ` Maxime Ripard
2020-02-12 9:45 ` Alexandre GRIVEAUX
2020-02-14 13:17 ` Maxime Ripard
-- strict thread matches above, loose matches on Subject: below --
2020-02-10 9:27 agriveaux
2020-02-10 9:27 ` [PATCH] " agriveaux
2020-02-10 9:41 ` Maxime Ripard
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).