* [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
@ 2017-11-07 9:28 Giulio Benetti
0 siblings, 0 replies; 8+ messages in thread
From: Giulio Benetti @ 2017-11-07 9:28 UTC (permalink / raw)
To: robh+dt
Cc: mark.rutland, linux, maxime.ripard, wens, devicetree,
linux-arm-kernel, linux-kernel, Giulio Benetti
This patch gives possibility to set parallel lcd pins for RGB888.
To be used with tcon0 in your <target>.dts
Best regards
Giulio Benetti
Micronova srl
Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
arch/arm/boot/dts/sun7i-a20.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi
index 228c368..2e38dd3 100644
--- a/arch/arm/boot/dts/sun7i-a20.dtsi
+++ b/arch/arm/boot/dts/sun7i-a20.dtsi
@@ -921,6 +921,14 @@
pins = "PI20", "PI21";
function = "uart7";
};
+
+ lcd_rgb888_pins: lcd-rgb888 {
+ pins = "PD0", "PD1", "PD2", "PD3", "PD4", "PD5", "PD6", "PD7",
+ "PD8", "PD9", "PD10", "PD11", "PD12", "PD13", "PD14", "PD15",
+ "PD16", "PD17", "PD18", "PD19", "PD20", "PD21", "PD22", "PD23",
+ "PD24", "PD25", "PD26", "PD27";
+ function = "lcd0";
+ };
};
timer@1c20c00 {
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
@ 2017-11-07 10:32 Giulio Benetti
2017-11-07 11:01 ` Maxime Ripard
0 siblings, 1 reply; 8+ messages in thread
From: Giulio Benetti @ 2017-11-07 10:32 UTC (permalink / raw)
To: robh+dt
Cc: mark.rutland, linux, maxime.ripard, wens, devicetree,
linux-arm-kernel, linux-kernel, Giulio Benetti
This patch gives possibility to set parallel lcd pins for RGB888.
To be used with tcon0 in your <target>.dts
Best regards
Giulio Benetti
Micronova srl
Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
arch/arm/boot/dts/sun7i-a20.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi
index 228c368..b85a917 100644
--- a/arch/arm/boot/dts/sun7i-a20.dtsi
+++ b/arch/arm/boot/dts/sun7i-a20.dtsi
@@ -921,6 +921,14 @@
pins = "PI20", "PI21";
function = "uart7";
};
+
+ lcd_rgb888_pins: lcd-rgb888@0 {
+ pins = "PD0", "PD1", "PD2", "PD3", "PD4", "PD5", "PD6", "PD7",
+ "PD8", "PD9", "PD10", "PD11", "PD12", "PD13", "PD14", "PD15",
+ "PD16", "PD17", "PD18", "PD19", "PD20", "PD21", "PD22", "PD23",
+ "PD24", "PD25", "PD26", "PD27";
+ function = "lcd0";
+ };
};
timer@1c20c00 {
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
2017-11-07 10:32 Giulio Benetti
@ 2017-11-07 11:01 ` Maxime Ripard
0 siblings, 0 replies; 8+ messages in thread
From: Maxime Ripard @ 2017-11-07 11:01 UTC (permalink / raw)
To: Giulio Benetti
Cc: robh+dt, mark.rutland, linux, wens, devicetree, linux-arm-kernel,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 518 bytes --]
Hi,
On Tue, Nov 07, 2017 at 11:32:39AM +0100, Giulio Benetti wrote:
> This patch gives possibility to set parallel lcd pins for RGB888.
> To be used with tcon0 in your <target>.dts
>
> Best regards
> Giulio Benetti
> Micronova srl
This isn't a cover letter, but a commit log. You should document why
and how you did that change, nothing else.
On what board is this supposed to be used?
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
@ 2017-11-07 12:49 Giulio Benetti
2017-11-07 13:03 ` Maxime Ripard
0 siblings, 1 reply; 8+ messages in thread
From: Giulio Benetti @ 2017-11-07 12:49 UTC (permalink / raw)
To: robh+dt
Cc: mark.rutland, linux, maxime.ripard, wens, devicetree,
linux-arm-kernel, linux-kernel, linux-sunxi, Giulio Benetti
On sun7i-a20-* there's no way to enable PA00--PA27 as RGB888 LCD0 pins.
Add lcd_rgb888_pins in sun7i-a20.dtsi.
It can be used on sun7i-a20-olinuxino-lime.dts.
This patch gives possibility to set parallel lcd pins for RGB888.
To be used with tcon0 in your <target>.dts
Best regards
Giulio Benetti
Micronova srl
Signed-off-by: Giulio Benetti <giulio.benetti@micronovasrl.com>
---
arch/arm/boot/dts/sun7i-a20.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi
index 228c368..b85a917 100644
--- a/arch/arm/boot/dts/sun7i-a20.dtsi
+++ b/arch/arm/boot/dts/sun7i-a20.dtsi
@@ -921,6 +921,14 @@
pins = "PI20", "PI21";
function = "uart7";
};
+
+ lcd_rgb888_pins: lcd-rgb888 {
+ pins = "PD0", "PD1", "PD2", "PD3", "PD4", "PD5", "PD6", "PD7",
+ "PD8", "PD9", "PD10", "PD11", "PD12", "PD13", "PD14", "PD15",
+ "PD16", "PD17", "PD18", "PD19", "PD20", "PD21", "PD22", "PD23",
+ "PD24", "PD25", "PD26", "PD27";
+ function = "lcd0";
+ };
};
timer@1c20c00 {
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
2017-11-07 12:49 Giulio Benetti
@ 2017-11-07 13:03 ` Maxime Ripard
2017-11-07 16:38 ` Giulio Benetti
0 siblings, 1 reply; 8+ messages in thread
From: Maxime Ripard @ 2017-11-07 13:03 UTC (permalink / raw)
To: Giulio Benetti
Cc: robh+dt, mark.rutland, linux, wens, devicetree, linux-arm-kernel,
linux-kernel, linux-sunxi
[-- Attachment #1: Type: text/plain, Size: 655 bytes --]
On Tue, Nov 07, 2017 at 01:49:59PM +0100, Giulio Benetti wrote:
> On sun7i-a20-* there's no way to enable PA00--PA27 as RGB888 LCD0 pins.
>
> Add lcd_rgb888_pins in sun7i-a20.dtsi.
>
> It can be used on sun7i-a20-olinuxino-lime.dts.
> This patch gives possibility to set parallel lcd pins for RGB888.
> To be used with tcon0 in your <target>.dts
My question was on which board do you want to use it right now. If the
answer is none, then we won't merge that patch, we have too many
pinctrl already to not add the unused ones.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
2017-11-07 13:03 ` Maxime Ripard
@ 2017-11-07 16:38 ` Giulio Benetti
2017-11-08 10:56 ` Maxime Ripard
0 siblings, 1 reply; 8+ messages in thread
From: Giulio Benetti @ 2017-11-07 16:38 UTC (permalink / raw)
To: Maxime Ripard
Cc: robh+dt, mark.rutland, linux, wens, devicetree, linux-arm-kernel,
linux-kernel, linux-sunxi
Board could be any with A20,
for example Olinuxino A20.
Or our Q027, S027 boards, but final dts still are not complete.
--
Giulio Benetti
R&D Manager &
Advanced Research
MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale € 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642
Il 07/11/2017 14:03, Maxime Ripard ha scritto:
> On Tue, Nov 07, 2017 at 01:49:59PM +0100, Giulio Benetti wrote:
>> On sun7i-a20-* there's no way to enable PA00--PA27 as RGB888 LCD0 pins.
>>
>> Add lcd_rgb888_pins in sun7i-a20.dtsi.
>>
>> It can be used on sun7i-a20-olinuxino-lime.dts.
>> This patch gives possibility to set parallel lcd pins for RGB888.
>> To be used with tcon0 in your <target>.dts
>
> My question was on which board do you want to use it right now. If the
> answer is none, then we won't merge that patch, we have too many
> pinctrl already to not add the unused ones.
>
> Maxime
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
2017-11-07 16:38 ` Giulio Benetti
@ 2017-11-08 10:56 ` Maxime Ripard
2017-11-10 1:05 ` Icenowy Zheng
0 siblings, 1 reply; 8+ messages in thread
From: Maxime Ripard @ 2017-11-08 10:56 UTC (permalink / raw)
To: Giulio Benetti
Cc: robh+dt, mark.rutland, linux, wens, devicetree, linux-arm-kernel,
linux-kernel, linux-sunxi
[-- Attachment #1: Type: text/plain, Size: 418 bytes --]
Hi,
On Tue, Nov 07, 2017 at 05:38:55PM +0100, Giulio Benetti wrote:
> Board could be any with A20,
> for example Olinuxino A20.
> Or our Q027, S027 boards, but final dts still are not complete.
Therefore no upstream boards are using it right now, so we'll merge it
when that will be the case.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD
2017-11-08 10:56 ` Maxime Ripard
@ 2017-11-10 1:05 ` Icenowy Zheng
0 siblings, 0 replies; 8+ messages in thread
From: Icenowy Zheng @ 2017-11-10 1:05 UTC (permalink / raw)
To: Maxime Ripard
Cc: Giulio Benetti, mark.rutland, devicetree, linux-sunxi, linux,
linux-kernel, wens, robh+dt, linux-arm-kernel
在 2017-11-08 18:56，Maxime Ripard 写道：
> Hi,
>
> On Tue, Nov 07, 2017 at 05:38:55PM +0100, Giulio Benetti wrote:
>> Board could be any with A20,
>> for example Olinuxino A20.
>> Or our Q027, S027 boards, but final dts still are not complete.
>
> Therefore no upstream boards are using it right now, so we'll merge it
> when that will be the case.
I think the LCD connector on Banana Pi's uses parallel interface,
although the LCD should be an external DT overlay.
>
> Maxime
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-11-10 1:06 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-07 9:28 [PATCH 1/1] add rgb888 pins to a20 to enable parallel rgb LCD Giulio Benetti
2017-11-07 10:32 Giulio Benetti
2017-11-07 11:01 ` Maxime Ripard
2017-11-07 12:49 Giulio Benetti
2017-11-07 13:03 ` Maxime Ripard
2017-11-07 16:38 ` Giulio Benetti
2017-11-08 10:56 ` Maxime Ripard
2017-11-10 1:05 ` Icenowy Zheng
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).