2020-12-10 08:41:57

by Icenowy Zheng

[permalink] [raw]
Subject: [PATCH 0/3] PineTab with new panel DT

As discussed on the mailing list, here introduces a new DT for new
PineTabs.

Icenowy Zheng (3):
dt-bindings: arm: sunxi: add PineTab new panel DT binding
arm64: allwinner: dts: a64: add DT for PineTab with new LCD panel
dt-bindings: arm: sunxi: note that old PineTab compatible has old
panel

.../devicetree/bindings/arm/sunxi.yaml | 7 ++++-
arch/arm64/boot/dts/allwinner/Makefile | 1 +
.../sun50i-a64-pinetab-new-panel.dts | 26 +++++++++++++++++++
3 files changed, 33 insertions(+), 1 deletion(-)
create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-a64-pinetab-new-panel.dts

--
2.28.0


2020-12-10 11:15:43

by Icenowy Zheng

[permalink] [raw]
Subject: [PATCH 1/3] dt-bindings: arm: sunxi: add PineTab new panel DT binding

Early adopters' PineTabs (and all further releases) will have a new LCD
panel different with the one that is used when in development (because
the old panel's supply discontinued).

Add a new DT compatible for it.

Signed-off-by: Icenowy Zheng <[email protected]>
---
Documentation/devicetree/bindings/arm/sunxi.yaml | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
index 6db32fbf813f..73a6c8421172 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.yaml
+++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
@@ -700,6 +700,11 @@ properties:
- const: pine64,pinetab
- const: allwinner,sun50i-a64

+ - description: Pine64 PineTab with new LCD panel
+ items:
+ - const: pine64,pinetab-new-panel
+ - const: allwinner,sun50i-a64
+
- description: Pine64 SoPine Baseboard
items:
- const: pine64,sopine-baseboard
--
2.28.0

2020-12-10 14:09:53

by Icenowy Zheng

[permalink] [raw]
Subject: [PATCH 3/3] dt-bindings: arm: sunxi: note that old PineTab compatible has old panel

As the old LCD panel used by PineTab developer samples are discontinued,
there won't be furtherly any more units of the sample, and this should
be noted in the document.

Signed-off-by: Icenowy Zheng <[email protected]>
---
Documentation/devicetree/bindings/arm/sunxi.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
index 73a6c8421172..9f29b5811aa1 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.yaml
+++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
@@ -695,7 +695,7 @@ properties:
- const: pine64,pinephone-1.2
- const: allwinner,sun50i-a64

- - description: Pine64 PineTab
+ - description: Pine64 PineTab (developers' sample with old discontinued LCD panel, discontinued)
items:
- const: pine64,pinetab
- const: allwinner,sun50i-a64
--
2.28.0

2020-12-11 11:43:07

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: arm: sunxi: add PineTab new panel DT binding

On Thu, 10 Dec 2020 16:42:32 +0800, Icenowy Zheng wrote:
> Early adopters' PineTabs (and all further releases) will have a new LCD
> panel different with the one that is used when in development (because
> the old panel's supply discontinued).
>
> Add a new DT compatible for it.
>
> Signed-off-by: Icenowy Zheng <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/sunxi.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>

Reviewed-by: Rob Herring <[email protected]>

2020-12-11 16:28:10

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 3/3] dt-bindings: arm: sunxi: note that old PineTab compatible has old panel

On Thu, 10 Dec 2020 16:45:58 +0800, Icenowy Zheng wrote:
> As the old LCD panel used by PineTab developer samples are discontinued,
> there won't be furtherly any more units of the sample, and this should
> be noted in the document.
>
> Signed-off-by: Icenowy Zheng <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/sunxi.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>

2020-12-14 11:06:55

by Maxime Ripard

[permalink] [raw]
Subject: Re: [PATCH 3/3] dt-bindings: arm: sunxi: note that old PineTab compatible has old panel

On Thu, Dec 10, 2020 at 04:45:58PM +0800, Icenowy Zheng wrote:
> As the old LCD panel used by PineTab developer samples are discontinued,
> there won't be furtherly any more units of the sample, and this should
> be noted in the document.
>
> Signed-off-by: Icenowy Zheng <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/sunxi.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
> index 73a6c8421172..9f29b5811aa1 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi.yaml
> +++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
> @@ -695,7 +695,7 @@ properties:
> - const: pine64,pinephone-1.2
> - const: allwinner,sun50i-a64
>
> - - description: Pine64 PineTab
> + - description: Pine64 PineTab (developers' sample with old discontinued LCD panel, discontinued)

I'm not sure it's worth mentioning it's discontinued, making it clear
it's for the developers sample is enough

Maxime


Attachments:
(No filename) (1.07 kB)
signature.asc (235.00 B)
Download all attachments

2020-12-14 11:07:06

by Maxime Ripard

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: arm: sunxi: add PineTab new panel DT binding

On Thu, Dec 10, 2020 at 04:42:32PM +0800, Icenowy Zheng wrote:
> Early adopters' PineTabs (and all further releases) will have a new LCD
> panel different with the one that is used when in development (because
> the old panel's supply discontinued).
>
> Add a new DT compatible for it.
>
> Signed-off-by: Icenowy Zheng <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/sunxi.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/sunxi.yaml b/Documentation/devicetree/bindings/arm/sunxi.yaml
> index 6db32fbf813f..73a6c8421172 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi.yaml
> +++ b/Documentation/devicetree/bindings/arm/sunxi.yaml
> @@ -700,6 +700,11 @@ properties:
> - const: pine64,pinetab
> - const: allwinner,sun50i-a64
>
> + - description: Pine64 PineTab with new LCD panel
> + items:
> + - const: pine64,pinetab-new-panel
> + - const: allwinner,sun50i-a64
> +

We're on the right track, but new panel seems a bit too vague. What is
going to happen when they will change the panel again?

pinetab-early-adopter seems more robust there

Maxime


Attachments:
(No filename) (1.19 kB)
signature.asc (235.00 B)
Download all attachments