| Message ID | 20200929083217.25406-1-jagan@amarulasolutions.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | arm64: dts: rockchip: Add Engicam PX30.Core | expand |
Hi Heiko, On Tue, Sep 29, 2020 at 2:02 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > PX30.Core is an EDIMM SOM based on Rockchip PX30 from Engicam. > > PX30.Core needs to mount on top of Engicam baseboards for creating > complete platform boards. > > Possible baseboards are, > - EDIMM2.2 Starter Kit > - C.TOUCH 2.0 Carrier Board > > Changes for v4: > - collect Rob A-b > Changes for v3: > - resolved Johan comments about sorting node properties > - add copyright to Amarula Solutions > - update px30 dtsi author > Changes for v2: > - include C.TOUCH 2.0 carrier board > - skip 10" OF LCD as it requires separate dts with panel support. > > Note: These baseboards can be used for i.MX8 SOM's as well. So having > baseboard on respective SoC seems to be easy rather than making it > common across all. > > Any inputs? > Jagan. > > Jagan Teki (6): > dt-bindings: arm: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > arm64: dts: rockchip: px30: Add Engicam EDIMM2.2 Starter Kit > arm64: dts: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > dt-bindings: arm: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > arm64: dts: rockchip: px30: Add Engicam C.TOUCH 2.0 > arm64: dts: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > Michael Trimarchi (1): > arm64: dts: rockchip: Add Engicam PX30.Core SOM > > .../devicetree/bindings/arm/rockchip.yaml | 12 + > arch/arm64/boot/dts/rockchip/Makefile | 2 + > .../dts/rockchip/px30-engicam-common.dtsi | 39 +++ > .../dts/rockchip/px30-engicam-ctouch2.dtsi | 8 + > .../dts/rockchip/px30-engicam-edimm2.2.dtsi | 7 + > .../dts/rockchip/px30-px30-core-ctouch2.dts | 22 ++ > .../dts/rockchip/px30-px30-core-edimm2.2.dts | 21 ++ > .../boot/dts/rockchip/px30-px30-core.dtsi | 232 ++++++++++++++++++ > 8 files changed, 343 insertions(+) > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-common.dtsi > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-ctouch2.dtsi > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-edimm2.2.dtsi > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core-ctouch2.dts > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core-edimm2.2.dts > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core.dtsi Any further comments? Jagan.
On Thu, Oct 22, 2020 at 12:27 AM Jagan Teki <jagan@amarulasolutions.com> wrote: > > Hi Heiko, > > On Tue, Sep 29, 2020 at 2:02 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > PX30.Core is an EDIMM SOM based on Rockchip PX30 from Engicam. > > > > PX30.Core needs to mount on top of Engicam baseboards for creating > > complete platform boards. > > > > Possible baseboards are, > > - EDIMM2.2 Starter Kit > > - C.TOUCH 2.0 Carrier Board > > > > Changes for v4: > > - collect Rob A-b > > Changes for v3: > > - resolved Johan comments about sorting node properties > > - add copyright to Amarula Solutions > > - update px30 dtsi author > > Changes for v2: > > - include C.TOUCH 2.0 carrier board > > - skip 10" OF LCD as it requires separate dts with panel support. > > > > Note: These baseboards can be used for i.MX8 SOM's as well. So having > > baseboard on respective SoC seems to be easy rather than making it > > common across all. > > > > Any inputs? > > Jagan. > > > > Jagan Teki (6): > > dt-bindings: arm: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > > arm64: dts: rockchip: px30: Add Engicam EDIMM2.2 Starter Kit > > arm64: dts: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > > dt-bindings: arm: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > arm64: dts: rockchip: px30: Add Engicam C.TOUCH 2.0 > > arm64: dts: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > > > Michael Trimarchi (1): > > arm64: dts: rockchip: Add Engicam PX30.Core SOM > > > > .../devicetree/bindings/arm/rockchip.yaml | 12 + > > arch/arm64/boot/dts/rockchip/Makefile | 2 + > > .../dts/rockchip/px30-engicam-common.dtsi | 39 +++ > > .../dts/rockchip/px30-engicam-ctouch2.dtsi | 8 + > > .../dts/rockchip/px30-engicam-edimm2.2.dtsi | 7 + > > .../dts/rockchip/px30-px30-core-ctouch2.dts | 22 ++ > > .../dts/rockchip/px30-px30-core-edimm2.2.dts | 21 ++ > > .../boot/dts/rockchip/px30-px30-core.dtsi | 232 ++++++++++++++++++ > > 8 files changed, 343 insertions(+) > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-common.dtsi > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-ctouch2.dtsi > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-edimm2.2.dtsi > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core-ctouch2.dts > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core-edimm2.2.dts > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core.dtsi > > Any further comments? Gentle ping.
Am Mittwoch, 4. November 2020, 20:54:40 CET schrieb Jagan Teki: > On Thu, Oct 22, 2020 at 12:27 AM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > Hi Heiko, > > > > On Tue, Sep 29, 2020 at 2:02 PM Jagan Teki <jagan@amarulasolutions.com> wrote: > > > > > > PX30.Core is an EDIMM SOM based on Rockchip PX30 from Engicam. > > > > > > PX30.Core needs to mount on top of Engicam baseboards for creating > > > complete platform boards. > > > > > > Possible baseboards are, > > > - EDIMM2.2 Starter Kit > > > - C.TOUCH 2.0 Carrier Board > > > > > > Changes for v4: > > > - collect Rob A-b > > > Changes for v3: > > > - resolved Johan comments about sorting node properties > > > - add copyright to Amarula Solutions > > > - update px30 dtsi author > > > Changes for v2: > > > - include C.TOUCH 2.0 carrier board > > > - skip 10" OF LCD as it requires separate dts with panel support. > > > > > > Note: These baseboards can be used for i.MX8 SOM's as well. So having > > > baseboard on respective SoC seems to be easy rather than making it > > > common across all. > > > > > > Any inputs? > > > Jagan. > > > > > > Jagan Teki (6): > > > dt-bindings: arm: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > > > arm64: dts: rockchip: px30: Add Engicam EDIMM2.2 Starter Kit > > > arm64: dts: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > > > dt-bindings: arm: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > > arm64: dts: rockchip: px30: Add Engicam C.TOUCH 2.0 > > > arm64: dts: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > > > > > Michael Trimarchi (1): > > > arm64: dts: rockchip: Add Engicam PX30.Core SOM > > > > > > .../devicetree/bindings/arm/rockchip.yaml | 12 + > > > arch/arm64/boot/dts/rockchip/Makefile | 2 + > > > .../dts/rockchip/px30-engicam-common.dtsi | 39 +++ > > > .../dts/rockchip/px30-engicam-ctouch2.dtsi | 8 + > > > .../dts/rockchip/px30-engicam-edimm2.2.dtsi | 7 + > > > .../dts/rockchip/px30-px30-core-ctouch2.dts | 22 ++ > > > .../dts/rockchip/px30-px30-core-edimm2.2.dts | 21 ++ > > > .../boot/dts/rockchip/px30-px30-core.dtsi | 232 ++++++++++++++++++ > > > 8 files changed, 343 insertions(+) > > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-common.dtsi > > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-ctouch2.dtsi > > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-engicam-edimm2.2.dtsi > > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core-ctouch2.dts > > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core-edimm2.2.dts > > > create mode 100644 arch/arm64/boot/dts/rockchip/px30-px30-core.dtsi > > > > Any further comments? > > Gentle ping. on my list to untangle :-) Heiko
Hi, Am Dienstag, 29. September 2020, 10:32:10 CET schrieb Jagan Teki: > PX30.Core is an EDIMM SOM based on Rockchip PX30 from Engicam. > > PX30.Core needs to mount on top of Engicam baseboards for creating > complete platform boards. > > Possible baseboards are, > - EDIMM2.2 Starter Kit > - C.TOUCH 2.0 Carrier Board > > Changes for v4: > - collect Rob A-b > Changes for v3: > - resolved Johan comments about sorting node properties > - add copyright to Amarula Solutions > - update px30 dtsi author > Changes for v2: > - include C.TOUCH 2.0 carrier board > - skip 10" OF LCD as it requires separate dts with panel support. > > Note: These baseboards can be used for i.MX8 SOM's as well. So having > baseboard on respective SoC seems to be easy rather than making it > common across all. > > Any inputs? > Jagan. > > Jagan Teki (6): > dt-bindings: arm: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > arm64: dts: rockchip: px30: Add Engicam EDIMM2.2 Starter Kit > arm64: dts: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > dt-bindings: arm: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > arm64: dts: rockchip: px30: Add Engicam C.TOUCH 2.0 > arm64: dts: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > Michael Trimarchi (1): > arm64: dts: rockchip: Add Engicam PX30.Core SOM I've applied the patches for 5.11 with some changes: (1) engicam,px30-px30-core became engicam,px30-core (2) px30-px30-core.dtsi became px30-engicam-px30-core.dtsi That double px30 is unnecessary and confusing in compatibles and the px30-core thingy needed something less generic, as that is not px30-specific but specific to the engicam boards. Heiko
On Mon, Nov 9, 2020 at 4:45 AM Heiko Stuebner <heiko@sntech.de> wrote: > > Hi, > > Am Dienstag, 29. September 2020, 10:32:10 CET schrieb Jagan Teki: > > PX30.Core is an EDIMM SOM based on Rockchip PX30 from Engicam. > > > > PX30.Core needs to mount on top of Engicam baseboards for creating > > complete platform boards. > > > > Possible baseboards are, > > - EDIMM2.2 Starter Kit > > - C.TOUCH 2.0 Carrier Board > > > > Changes for v4: > > - collect Rob A-b > > Changes for v3: > > - resolved Johan comments about sorting node properties > > - add copyright to Amarula Solutions > > - update px30 dtsi author > > Changes for v2: > > - include C.TOUCH 2.0 carrier board > > - skip 10" OF LCD as it requires separate dts with panel support. > > > > Note: These baseboards can be used for i.MX8 SOM's as well. So having > > baseboard on respective SoC seems to be easy rather than making it > > common across all. > > > > Any inputs? > > Jagan. > > > > Jagan Teki (6): > > dt-bindings: arm: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > > arm64: dts: rockchip: px30: Add Engicam EDIMM2.2 Starter Kit > > arm64: dts: rockchip: Add Engicam PX30.Core EDIMM2.2 Starter Kit > > dt-bindings: arm: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > arm64: dts: rockchip: px30: Add Engicam C.TOUCH 2.0 > > arm64: dts: rockchip: Add Engicam PX30.Core C.TOUCH 2.0 > > > > Michael Trimarchi (1): > > arm64: dts: rockchip: Add Engicam PX30.Core SOM > > I've applied the patches for 5.11 with some changes: > (1) engicam,px30-px30-core became engicam,px30-core > (2) px30-px30-core.dtsi became px30-engicam-px30-core.dtsi > > That double px30 is unnecessary and confusing in compatibles > and the px30-core thingy needed something less generic, as that is > not px30-specific but specific to the engicam boards. Agreed with the updates and thanks for picking up. Other than these few patches around the mailing list, let me know your comments, thanks! https://patchwork.kernel.org/project/linux-arm-kernel/patch/20200731160324.142097-1-jagan@amarulasolutions.com/ https://patchwork.kernel.org/project/linux-arm-kernel/patch/20201023181814.220974-1-jagan@amarulasolutions.com/ Jagan.