Re: [PATCH v2] N900: add device tree
Hi, On Sat, Jul 13, 2013 at 02:17:09PM +0200, Pavel Machek wrote: > > This adds device tree with neccessary support to boot with functional > video (on both emulator and real N900 device). > > Signed-off-by: Pavel Machek You can also add: Signed-off-by: Aaro Koskinen Thanks, A. > > --- > > From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that > can be enabled on 3.10, and tested it on that kernel. > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index f0895c5..1950aed 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ > omap3-devkit8000.dtb \ > omap3-beagle-xm.dtb \ > omap3-evm.dtb \ > + omap3-n900.dtb \ > omap3-tobi.dtb \ > omap3-igep0020.dtb \ > omap3-igep0030.dtb \ > diff --git a/arch/arm/boot/dts/omap3-n900.dts > b/arch/arm/boot/dts/omap3-n900.dts > new file mode 100644 > index 000..fb461bf > --- /dev/null > +++ b/arch/arm/boot/dts/omap3-n900.dts > @@ -0,0 +1,92 @@ > +/* > + * Copyright (C) 2013 Pavel Machek > + * Copyright 2013 Aaro Koskinen > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +/dts-v1/; > + > +/include/ "omap34xx.dtsi" > + > +/ { > + model = "Nokia N900"; > + compatible = "nokia,omap3-n900", "ti,omap3"; > + > + cpus { > + cpu@0 { > + cpu0-supply = <&vcc>; > + }; > + }; > + > + memory { > + device_type = "memory"; > + reg = <0x8000 0x1000>; /* 256 MB */ > + }; > + > +}; > + > +&i2c1 { > + clock-frequency = <220>; > + > + twl: twl@48 { > + reg = <0x48>; > + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ > + interrupt-parent = <&intc>; > + }; > +}; > + > +/include/ "twl4030.dtsi" > + > +&twl_gpio { > + ti,pullups = <0x0>; > + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ > +}; > + > +&i2c2 { > + clock-frequency = <40>; > +}; > + > +&i2c3 { > + clock-frequency = <10>; > +}; > + > +&mmc1 { > + status = "disabled"; > +}; > + > +&mmc2 { > + status = "disabled"; > +}; > + > +&mmc3 { > + status = "disabled"; > +}; > + > +&mcspi1 { > + // For some reason, touchscreen is neccessary for screen to work at > + // all on real hw. It works well without it on emulator. > + // > + // Also... order in the device tree actually matters here. > + tsc2005@0 { > + compatible = "tsc2005"; > + spi-max-frequency = <600>; > + reg = <0>; > + }; > + mipid@2 { > + compatible = "acx565akm"; > + spi-max-frequency = <600>; > + reg = <2>; > + // turbo_mode = 0, > + // cs_per_word = 0 > + }; > +}; > + > +&usb_otg_hs { > + interface-type = <0>; > + usb-phy = <&usb2_phy>; > + mode = <2>; > + power = <50>; > +}; > > -- > (english) http://www.livejournal.com/~pavelmachek > (cesky, pictures) > http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
On Mon 2013-07-15 01:28:19, Aaro Koskinen wrote: > Hi, > > On Sat, Jul 13, 2013 at 02:17:09PM +0200, Pavel Machek wrote: > > > > This adds device tree with neccessary support to boot with functional > > video (on both emulator and real N900 device). > > > > Signed-off-by: Pavel Machek > > You can also add: > > Signed-off-by: Aaro Koskinen Benoit, could you apply this? Tony said you are doing dts changes, and the patch seems to be ready. > > From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that > > can be enabled on 3.10, and tested it on that kernel. > > > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > > index f0895c5..1950aed 100644 > > --- a/arch/arm/boot/dts/Makefile > > +++ b/arch/arm/boot/dts/Makefile > > @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ > > omap3-devkit8000.dtb \ > > omap3-beagle-xm.dtb \ > > omap3-evm.dtb \ > > + omap3-n900.dtb \ > > omap3-tobi.dtb \ > > omap3-igep0020.dtb \ > > omap3-igep0030.dtb \ > > diff --git a/arch/arm/boot/dts/omap3-n900.dts > > b/arch/arm/boot/dts/omap3-n900.dts > > new file mode 100644 > > index 000..fb461bf > > --- /dev/null > > +++ b/arch/arm/boot/dts/omap3-n900.dts > > @@ -0,0 +1,92 @@ > > +/* > > + * Copyright (C) 2013 Pavel Machek > > + * Copyright 2013 Aaro Koskinen > > + * > > + * This program is free software; you can redistribute it and/or modify > > + * it under the terms of the GNU General Public License version 2 as > > + * published by the Free Software Foundation. > > + */ > > + > > +/dts-v1/; > > + > > +/include/ "omap34xx.dtsi" > > + > > +/ { > > + model = "Nokia N900"; > > + compatible = "nokia,omap3-n900", "ti,omap3"; > > + > > + cpus { > > + cpu@0 { > > + cpu0-supply = <&vcc>; > > + }; > > + }; > > + > > + memory { > > + device_type = "memory"; > > + reg = <0x8000 0x1000>; /* 256 MB */ > > + }; > > + > > +}; > > + > > +&i2c1 { > > + clock-frequency = <220>; > > + > > + twl: twl@48 { > > + reg = <0x48>; > > + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ > > + interrupt-parent = <&intc>; > > + }; > > +}; > > + > > +/include/ "twl4030.dtsi" > > + > > +&twl_gpio { > > + ti,pullups = <0x0>; > > + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ > > +}; > > + > > +&i2c2 { > > + clock-frequency = <40>; > > +}; > > + > > +&i2c3 { > > + clock-frequency = <10>; > > +}; > > + > > +&mmc1 { > > + status = "disabled"; > > +}; > > + > > +&mmc2 { > > + status = "disabled"; > > +}; > > + > > +&mmc3 { > > + status = "disabled"; > > +}; > > + > > +&mcspi1 { > > + // For some reason, touchscreen is neccessary for screen to work at > > + // all on real hw. It works well without it on emulator. > > + // > > + // Also... order in the device tree actually matters here. > > + tsc2005@0 { > > + compatible = "tsc2005"; > > + spi-max-frequency = <600>; > > + reg = <0>; > > + }; > > + mipid@2 { > > + compatible = "acx565akm"; > > + spi-max-frequency = <600>; > > + reg = <2>; > > + // turbo_mode = 0, > > + // cs_per_word = 0 > > + }; > > +}; > > + > > +&usb_otg_hs { > > + interface-type = <0>; > > + usb-phy = <&usb2_phy>; > > + mode = <2>; > > + power = <50>; > > +}; > > > > -- > > (english) http://www.livejournal.com/~pavelmachek > > (cesky, pictures) > > http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
On Sun 2013-07-28 15:44:09, Pavel Machek wrote: > On Mon 2013-07-15 01:28:19, Aaro Koskinen wrote: > > Hi, > > > > On Sat, Jul 13, 2013 at 02:17:09PM +0200, Pavel Machek wrote: > > > > > > This adds device tree with neccessary support to boot with functional > > > video (on both emulator and real N900 device). > > > > > > Signed-off-by: Pavel Machek > > > > You can also add: > > > > Signed-off-by: Aaro Koskinen > > Benoit, could you apply this? Tony said you are doing dts changes, and > the patch seems to be ready. Tony, Benoit was apparently abducted by aliens or something. Is there chance you could take the patch? Thanks, Pavel > > > From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that > > > can be enabled on 3.10, and tested it on that kernel. > > > > > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > > > index f0895c5..1950aed 100644 > > > --- a/arch/arm/boot/dts/Makefile > > > +++ b/arch/arm/boot/dts/Makefile > > > @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ > > > omap3-devkit8000.dtb \ > > > omap3-beagle-xm.dtb \ > > > omap3-evm.dtb \ > > > + omap3-n900.dtb \ > > > omap3-tobi.dtb \ > > > omap3-igep0020.dtb \ > > > omap3-igep0030.dtb \ > > > diff --git a/arch/arm/boot/dts/omap3-n900.dts > > > b/arch/arm/boot/dts/omap3-n900.dts > > > new file mode 100644 > > > index 000..fb461bf > > > --- /dev/null > > > +++ b/arch/arm/boot/dts/omap3-n900.dts > > > @@ -0,0 +1,92 @@ > > > +/* > > > + * Copyright (C) 2013 Pavel Machek > > > + * Copyright 2013 Aaro Koskinen > > > + * > > > + * This program is free software; you can redistribute it and/or modify > > > + * it under the terms of the GNU General Public License version 2 as > > > + * published by the Free Software Foundation. > > > + */ > > > + > > > +/dts-v1/; > > > + > > > +/include/ "omap34xx.dtsi" > > > + > > > +/ { > > > + model = "Nokia N900"; > > > + compatible = "nokia,omap3-n900", "ti,omap3"; > > > + > > > + cpus { > > > + cpu@0 { > > > + cpu0-supply = <&vcc>; > > > + }; > > > + }; > > > + > > > + memory { > > > + device_type = "memory"; > > > + reg = <0x8000 0x1000>; /* 256 MB */ > > > + }; > > > + > > > +}; > > > + > > > +&i2c1 { > > > + clock-frequency = <220>; > > > + > > > + twl: twl@48 { > > > + reg = <0x48>; > > > + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ > > > + interrupt-parent = <&intc>; > > > + }; > > > +}; > > > + > > > +/include/ "twl4030.dtsi" > > > + > > > +&twl_gpio { > > > + ti,pullups = <0x0>; > > > + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ > > > +}; > > > + > > > +&i2c2 { > > > + clock-frequency = <40>; > > > +}; > > > + > > > +&i2c3 { > > > + clock-frequency = <10>; > > > +}; > > > + > > > +&mmc1 { > > > + status = "disabled"; > > > +}; > > > + > > > +&mmc2 { > > > + status = "disabled"; > > > +}; > > > + > > > +&mmc3 { > > > + status = "disabled"; > > > +}; > > > + > > > +&mcspi1 { > > > + // For some reason, touchscreen is neccessary for screen to work at > > > + // all on real hw. It works well without it on emulator. > > > + // > > > + // Also... order in the device tree actually matters here. > > > + tsc2005@0 { > > > + compatible = "tsc2005"; > > > + spi-max-frequency = <600>; > > > + reg = <0>; > > > + }; > > > + mipid@2 { > > > + compatible = "acx565akm"; > > > + spi-max-frequency = <600>; > > > + reg = <2>; > > > + // turbo_mode = 0, > > > + // cs_per_word = 0 > > > + }; > > > +}; > > > + > > > +&usb_otg_hs { > > > + interface-type = <0>; > > > + usb-phy = <&usb2_phy>; > > > + mode = <2>; > > > + power = <50>; > > > +}; > > > > > > -- > > > (english) http://www.livejournal.com/~pavelmachek > > > (cesky, pictures) > > > http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html > -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
On Fri 2013-08-02 01:43:55, Pavel Machek wrote: > On Sun 2013-07-28 15:44:09, Pavel Machek wrote: > > On Mon 2013-07-15 01:28:19, Aaro Koskinen wrote: > > > Hi, > > > > > > On Sat, Jul 13, 2013 at 02:17:09PM +0200, Pavel Machek wrote: > > > > > > > > This adds device tree with neccessary support to boot with functional > > > > video (on both emulator and real N900 device). > > > > > > > > Signed-off-by: Pavel Machek > > > > > > You can also add: > > > > > > Signed-off-by: Aaro Koskinen > > > > Benoit, could you apply this? Tony said you are doing dts changes, and > > the patch seems to be ready. > > Tony, Benoit was apparently abducted by aliens or something. Is there > chance you could take the patch? Hello, is there anybody out there? There's nice and simple patch enabling device tree on n900, it is even reviewed by Aaro, and it is neither applied nor are negative comments coming? ...for a month...? Tony also seen the patch and comment was "Benoit takes these", so I assume it looks basically ok. Benoit, you are supposed to maintain device tree for omap2. [I wonder if this is clean-enough cause for "trivial in resubmit mode?] Pavel --- This adds device tree with neccessary support to boot with functional video (on both emulator and real N900 device). Signed-off-by: Pavel Machek Signed-off-by: Aaro Koskinen --- >From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that can be enabled on 3.10, and tested it on that kernel. diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index f0895c5..1950aed 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ omap3-devkit8000.dtb \ omap3-beagle-xm.dtb \ omap3-evm.dtb \ + omap3-n900.dtb \ omap3-tobi.dtb \ omap3-igep0020.dtb \ omap3-igep0030.dtb \ diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts new file mode 100644 index 000..fb461bf --- /dev/null +++ b/arch/arm/boot/dts/omap3-n900.dts @@ -0,0 +1,92 @@ +/* + * Copyright (C) 2013 Pavel Machek + * Copyright 2013 Aaro Koskinen + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + */ + +/dts-v1/; + +/include/ "omap34xx.dtsi" + +/ { + model = "Nokia N900"; + compatible = "nokia,omap3-n900", "ti,omap3"; + + cpus { + cpu@0 { + cpu0-supply = <&vcc>; + }; + }; + + memory { + device_type = "memory"; + reg = <0x8000 0x1000>; /* 256 MB */ + }; + +}; + +&i2c1 { + clock-frequency = <220>; + + twl: twl@48 { + reg = <0x48>; + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ + interrupt-parent = <&intc>; + }; +}; + +/include/ "twl4030.dtsi" + +&twl_gpio { + ti,pullups = <0x0>; + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ +}; + +&i2c2 { + clock-frequency = <40>; +}; + +&i2c3 { + clock-frequency = <10>; +}; + +&mmc1 { + status = "disabled"; +}; + +&mmc2 { + status = "disabled"; +}; + +&mmc3 { + status = "disabled"; +}; + +&mcspi1 { + // For some reason, touchscreen is neccessary for screen to work at + // all on real hw. It works well without it on emulator. + // + // Also... order in the device tree actually matters here. + tsc2005@0 { + compatible = "tsc2005"; + spi-max-frequency = <600>; + reg = <0>; + }; + mipid@2 { + compatible = "acx565akm"; + spi-max-frequency = <600>; + reg = <2>; + // turbo_mode = 0, + // cs_per_word = 0 + }; +}; + +&usb_otg_hs { + interface-type = <0>; + usb-phy = <&usb2_phy>; + mode = <2>; + power = <50>; +}; -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
Hi All, On Sat, Aug 10, 2013 at 1:09 PM, Pavel Machek wrote: > On Fri 2013-08-02 01:43:55, Pavel Machek wrote: >> On Sun 2013-07-28 15:44:09, Pavel Machek wrote: >> > On Mon 2013-07-15 01:28:19, Aaro Koskinen wrote: >> > > Hi, >> > > >> > > On Sat, Jul 13, 2013 at 02:17:09PM +0200, Pavel Machek wrote: >> > > > >> > > > This adds device tree with neccessary support to boot with functional >> > > > video (on both emulator and real N900 device). >> > > > >> > > > Signed-off-by: Pavel Machek >> > > >> > > You can also add: >> > > >> > > Signed-off-by: Aaro Koskinen >> > >> > Benoit, could you apply this? Tony said you are doing dts changes, and >> > the patch seems to be ready. >> >> Tony, Benoit was apparently abducted by aliens or something. Is there >> chance you could take the patch? > > Hello, is there anybody out there? > > There's nice and simple patch enabling device tree on n900, it is even > reviewed by Aaro, and it is neither applied nor are negative comments > coming? ...for a month...? Same for gta04 (omap3 based device) [1]. It was send upstream 2 times and second time there is no reply from 1.3.2013. > > Tony also seen the patch and comment was "Benoit takes these", so I > assume it looks basically ok. > > Benoit, you are supposed to maintain device tree for omap2. > > [I wonder if this is clean-enough cause for "trivial in resubmit > mode?] > Pavel > --- > > This adds device tree with neccessary support to boot with functional > video (on both emulator and real N900 device). > > Signed-off-by: Pavel Machek > Signed-off-by: Aaro Koskinen > > --- > > From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that > can be enabled on 3.10, and tested it on that kernel. > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index f0895c5..1950aed 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ > omap3-devkit8000.dtb \ > omap3-beagle-xm.dtb \ > omap3-evm.dtb \ > + omap3-n900.dtb \ > omap3-tobi.dtb \ > omap3-igep0020.dtb \ > omap3-igep0030.dtb \ > diff --git a/arch/arm/boot/dts/omap3-n900.dts > b/arch/arm/boot/dts/omap3-n900.dts > new file mode 100644 > index 000..fb461bf > --- /dev/null > +++ b/arch/arm/boot/dts/omap3-n900.dts > @@ -0,0 +1,92 @@ > +/* > + * Copyright (C) 2013 Pavel Machek > + * Copyright 2013 Aaro Koskinen > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +/dts-v1/; > + > +/include/ "omap34xx.dtsi" > + > +/ { > + model = "Nokia N900"; > + compatible = "nokia,omap3-n900", "ti,omap3"; > + > + cpus { > + cpu@0 { > + cpu0-supply = <&vcc>; > + }; > + }; > + > + memory { > + device_type = "memory"; > + reg = <0x8000 0x1000>; /* 256 MB */ > + }; > + > +}; > + > +&i2c1 { > + clock-frequency = <220>; > + > + twl: twl@48 { > + reg = <0x48>; > + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ > + interrupt-parent = <&intc>; > + }; > +}; > + > +/include/ "twl4030.dtsi" > + > +&twl_gpio { > + ti,pullups = <0x0>; > + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ > +}; > + > +&i2c2 { > + clock-frequency = <40>; > +}; > + > +&i2c3 { > + clock-frequency = <10>; > +}; > + > +&mmc1 { > + status = "disabled"; > +}; > + > +&mmc2 { > + status = "disabled"; > +}; > + > +&mmc3 { > + status = "disabled"; > +}; > + > +&mcspi1 { > + // For some reason, touchscreen is neccessary for screen to work at > + // all on real hw. It works well without it on emulator. > + // > + // Also... order in the device tree actually matters here. > + tsc2005@0 { > + compatible = "tsc2005"; > + spi-max-frequency = <600>; > + reg = <0>; > + }; > + mipid@2 { > + compatible = "acx565akm"; > + spi-max-frequency = <600>; > + reg = <2>; > + // turbo_mode = 0, > + // cs_per_word = 0 > + }; > +}; > + > +&usb_otg_hs { > + interface-type = <0>; > + usb-phy = <&usb2_phy>; > + mode = <2>; > + power = <50>; > +}; > > -- > (english) http://www.livejournal.com/~pavelmachek > (cesky, pictures) > http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html [1] - https://lkml.org/lkml/201
Re: [PATCH v2] N900: add device tree
On Sat, 10 Aug 2013, Pavel Machek wrote: > [I wonder if this is clean-enough cause for "trivial in resubmit > mode?] On Sat, 10 Aug 2013, Belisko Marek wrote: > Same for gta04 (omap3 based device) [1]. > It was send upstream 2 times and second time there is no reply from 1.3.2013. Hi guys, once I am able to match signoffs in the patches to MAINTAINERS, I can take those through trivial.git in "re-transmission mode" (or feeeding those thgough akpm si an option as well). Please just send the patches to me in a new thread properly with all the Acks/Signoffs, and mark those appropriately (i.e. maintainer that would normally push this upstream desurfaced). -- Jiri Kosina SUSE Labs -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
On Sat, Aug 10, 2013 at 10:29 PM, Jiri Kosina wrote: > On Sat, 10 Aug 2013, Pavel Machek wrote: > >> [I wonder if this is clean-enough cause for "trivial in resubmit >> mode?] > > On Sat, 10 Aug 2013, Belisko Marek wrote: > >> Same for gta04 (omap3 based device) [1]. >> It was send upstream 2 times and second time there is no reply from 1.3.2013. > > Hi guys, > > once I am able to match signoffs in the patches to MAINTAINERS, I can take > those through trivial.git in "re-transmission mode" (or feeeding those > thgough akpm si an option as well). > > Please just send the patches to me in a new thread properly with all the > Acks/Signoffs, and mark those appropriately (i.e. maintainer that would > normally push this upstream desurfaced). > Hi folks, I noticed that you are using Benoit older emails address (b-cous...@ti.com) which no longer exists instead of his new email address (benoit.cous...@linaro.org). So, it may be possible that Benoit have not even been getting your patches. I'll send a patch to MAINTAINERS updating Benoit's email address so get_maintainer.pl will stop lying. Jiri, is ok if I send this patch to you? I guess this falls into the trivial group. Thanks a lot and best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
Hi! > >> [I wonder if this is clean-enough cause for "trivial in resubmit > >> mode?] > > > > On Sat, 10 Aug 2013, Belisko Marek wrote: > > > >> Same for gta04 (omap3 based device) [1]. > >> It was send upstream 2 times and second time there is no reply from > >> 1.3.2013. > > > > Hi guys, > > > > once I am able to match signoffs in the patches to MAINTAINERS, I can take > > those through trivial.git in "re-transmission mode" (or feeeding those > > thgough akpm si an option as well). > > > > Please just send the patches to me in a new thread properly with all the > > Acks/Signoffs, and mark those appropriately (i.e. maintainer that would > > normally push this upstream desurfaced). > > I noticed that you are using Benoit older emails address > (b-cous...@ti.com) which no longer exists instead of his new email > address (benoit.cous...@linaro.org). So, it may be possible that > Benoit have not even been getting your patches. Thanks for update. > I'll send a patch to MAINTAINERS updating Benoit's email address so > get_maintainer.pl will stop lying. Thanks! Benoit, could you take a look at the patch from this thread? I can resend it, but it should be in pretty much _all_ the webmail archives now :-). Regards, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
Hi Pavel, some minor comments about your patch below On Sat, Jul 13, 2013 at 2:17 PM, Pavel Machek wrote: > > This adds device tree with neccessary support to boot with functional > video (on both emulator and real N900 device). > > Signed-off-by: Pavel Machek > > --- > > From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that > can be enabled on 3.10, and tested it on that kernel. > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index f0895c5..1950aed 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ > omap3-devkit8000.dtb \ > omap3-beagle-xm.dtb \ > omap3-evm.dtb \ > + omap3-n900.dtb \ > omap3-tobi.dtb \ > omap3-igep0020.dtb \ > omap3-igep0030.dtb \ > diff --git a/arch/arm/boot/dts/omap3-n900.dts > b/arch/arm/boot/dts/omap3-n900.dts > new file mode 100644 > index 000..fb461bf > --- /dev/null > +++ b/arch/arm/boot/dts/omap3-n900.dts > @@ -0,0 +1,92 @@ > +/* > + * Copyright (C) 2013 Pavel Machek > + * Copyright 2013 Aaro Koskinen > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +/dts-v1/; > + > +/include/ "omap34xx.dtsi" > + The current trend is to use #include "omap34xx.dtsi" instead /include/ in order to use the C pre-processor and the macros defined in include/dt-bindings. > +/ { > + model = "Nokia N900"; > + compatible = "nokia,omap3-n900", "ti,omap3"; > + > + cpus { > + cpu@0 { > + cpu0-supply = <&vcc>; > + }; > + }; > + > + memory { > + device_type = "memory"; > + reg = <0x8000 0x1000>; /* 256 MB */ > + }; > + > +}; > + > +&i2c1 { > + clock-frequency = <220>; > + > + twl: twl@48 { > + reg = <0x48>; > + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ > + interrupt-parent = <&intc>; > + }; > +}; > + > +/include/ "twl4030.dtsi" > + > +&twl_gpio { > + ti,pullups = <0x0>; > + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ > +}; > + > +&i2c2 { > + clock-frequency = <40>; > +}; > + > +&i2c3 { > + clock-frequency = <10>; > +}; > + > +&mmc1 { > + status = "disabled"; > +}; > + > +&mmc2 { > + status = "disabled"; > +}; > + > +&mmc3 { > + status = "disabled"; > +}; > + > +&mcspi1 { > + // For some reason, touchscreen is neccessary for screen to work at > + // all on real hw. It works well without it on emulator. > + // > + // Also... order in the device tree actually matters here. > + tsc2005@0 { > + compatible = "tsc2005"; > + spi-max-frequency = <600>; > + reg = <0>; > + }; > + mipid@2 { > + compatible = "acx565akm"; > + spi-max-frequency = <600>; > + reg = <2>; > + // turbo_mode = 0, > + // cs_per_word = 0 > + }; > +}; You should remove these properties if they are not being used instead of keeping them as commented. > + > +&usb_otg_hs { > + interface-type = <0>; > + usb-phy = <&usb2_phy>; > + mode = <2>; > + power = <50>; > +}; > > -- Thanks a lot and best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
* Jiri Kosina [130810 13:36]: > On Sat, 10 Aug 2013, Pavel Machek wrote: > > > [I wonder if this is clean-enough cause for "trivial in resubmit > > mode?] > > On Sat, 10 Aug 2013, Belisko Marek wrote: > > > Same for gta04 (omap3 based device) [1]. > > It was send upstream 2 times and second time there is no reply from > > 1.3.2013. > > Hi guys, > > once I am able to match signoffs in the patches to MAINTAINERS, I can take > those through trivial.git in "re-transmission mode" (or feeeding those > thgough akpm si an option as well). > > Please just send the patches to me in a new thread properly with all the > Acks/Signoffs, and mark those appropriately (i.e. maintainer that would > normally push this upstream desurfaced). Sorry for the delays, I think Benoit is still on vacation. Plus the non-functioning email address issue certainly does not help.. Anyways, let's just let Benoit queue these to avoid pointless merge conflicts with the .dts files as discussed several times earlier. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
* Pavel Machek [130811 06:54]: > Hi! > > > >> [I wonder if this is clean-enough cause for "trivial in resubmit > > >> mode?] > > > > > > On Sat, 10 Aug 2013, Belisko Marek wrote: > > > > > >> Same for gta04 (omap3 based device) [1]. > > >> It was send upstream 2 times and second time there is no reply from > > >> 1.3.2013. > > > > > > Hi guys, > > > > > > once I am able to match signoffs in the patches to MAINTAINERS, I can take > > > those through trivial.git in "re-transmission mode" (or feeeding those > > > thgough akpm si an option as well). > > > > > > Please just send the patches to me in a new thread properly with all the > > > Acks/Signoffs, and mark those appropriately (i.e. maintainer that would > > > normally push this upstream desurfaced). > > > > I noticed that you are using Benoit older emails address > > (b-cous...@ti.com) which no longer exists instead of his new email > > address (benoit.cous...@linaro.org). So, it may be possible that > > Benoit have not even been getting your patches. > > Thanks for update. > > > I'll send a patch to MAINTAINERS updating Benoit's email address so > > get_maintainer.pl will stop lying. > > Thanks! > > Benoit, could you take a look at the patch from this thread? I can > resend it, but it should be in pretty much _all_ the webmail archives > now :-). Hopefully Benoit will be back online after vacation soon :) Regaqrds, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] N900: add device tree
Hi Pavel, I finally got released by the aliens. It took longer than expected and beside a small scar on the back of my neck, I feel pretty OK. I just have few cosmetic comments on top of Javier's ones. On 11/08/2013 17:02, Javier Martinez Canillas wrote: Hi Pavel, some minor comments about your patch below On Sat, Jul 13, 2013 at 2:17 PM, Pavel Machek wrote: This adds device tree with neccessary support to boot with functional Typo^ video (on both emulator and real N900 device). Signed-off-by: Pavel Machek --- From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that can be enabled on 3.10, and tested it on that kernel. diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index f0895c5..1950aed 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ omap3-devkit8000.dtb \ omap3-beagle-xm.dtb \ omap3-evm.dtb \ + omap3-n900.dtb \ omap3-tobi.dtb \ omap3-igep0020.dtb \ omap3-igep0030.dtb \ diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts new file mode 100644 index 000..fb461bf --- /dev/null +++ b/arch/arm/boot/dts/omap3-n900.dts @@ -0,0 +1,92 @@ +/* + * Copyright (C) 2013 Pavel Machek + * Copyright 2013 Aaro Koskinen + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + */ + +/dts-v1/; + +/include/ "omap34xx.dtsi" + The current trend is to use #include "omap34xx.dtsi" instead /include/ in order to use the C pre-processor and the macros defined in include/dt-bindings. +/ { + model = "Nokia N900"; + compatible = "nokia,omap3-n900", "ti,omap3"; + + cpus { + cpu@0 { + cpu0-supply = <&vcc>; + }; + }; + + memory { + device_type = "memory"; + reg = <0x8000 0x1000>; /* 256 MB */ + }; + +}; + +&i2c1 { + clock-frequency = <220>; + + twl: twl@48 { + reg = <0x48>; + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ + interrupt-parent = <&intc>; + }; +}; + +/include/ "twl4030.dtsi" + +&twl_gpio { + ti,pullups = <0x0>; + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ +}; + +&i2c2 { + clock-frequency = <40>; +}; + +&i2c3 { + clock-frequency = <10>; +}; + +&mmc1 { + status = "disabled"; +}; + +&mmc2 { + status = "disabled"; +}; + +&mmc3 { + status = "disabled"; +}; + +&mcspi1 { + // For some reason, touchscreen is neccessary for screen to work at Same typo than before---^ + // all on real hw. It works well without it on emulator. + // + // Also... order in the device tree actually matters here. Nit: Please use the regular Linux style comment. The order should not matter at all in DT, it should be a static representation of the HW, so there is probably something wrong in the code that make the device creation order important. + tsc2005@0 { + compatible = "tsc2005"; + spi-max-frequency = <600>; + reg = <0>; + }; + mipid@2 { + compatible = "acx565akm"; + spi-max-frequency = <600>; + reg = <2>; + // turbo_mode = 0, + // cs_per_word = 0 + }; +}; You should remove these properties if they are not being used instead of keeping them as commented. + +&usb_otg_hs { + interface-type = <0>; + usb-phy = <&usb2_phy>; + mode = <2>; + power = <50>; +}; Beside my comments and the ones from Javier, it looks good. If you can repost ASAP, I'll take it right after. It will be the first one in the list. Thanks, Benoit -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
OMAP device tree maintainance (was Re: [PATCH v2] N900: add device tree)
Hi! > > [I wonder if this is clean-enough cause for "trivial in resubmit > > mode?] > > On Sat, 10 Aug 2013, Belisko Marek wrote: > > > Same for gta04 (omap3 based device) [1]. > > It was send upstream 2 times and second time there is no reply from > > 1.3.2013. > > Hi guys, > > once I am able to match signoffs in the patches to MAINTAINERS, I can take > those through trivial.git in "re-transmission mode" (or feeeding those > thgough akpm si an option as well). > > Please just send the patches to me in a new thread properly with all the > Acks/Signoffs, and mark those appropriately (i.e. maintainer that would > normally push this upstream desurfaced). Mail sent (just to you, to keep mail volume down). I noticed two non-delivery reports in my INBOX; seems Benoit disappeared rather permanently: ... If you do so, please include this problem report. You can delete your own text from the attached returned message. The mail system : host cluster5.us.messagelabs.com[216.82.250.99] said: 550 Invalid recipient (#5.1.1) (in reply to RCPT TO command) ...so maybe MAINTAINERS file will need fixing? OMAP DEVICE TREE SUPPORT M: BenoƮt Cousson M: Tony Lindgren L: linux-omap@vger.kernel.org L: devicet...@vger.kernel.org S: Maintained F: arch/arm/boot/dts/*omap* F: arch/arm/boot/dts/*am3* :-(. Thanks, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] ARM: dts: omap3 / n900 support, Benoit please take me (was Re: [PATCH v2] N900: add device tree)
On Sat 2013-08-10 13:09:55, Pavel Machek wrote: > On Fri 2013-08-02 01:43:55, Pavel Machek wrote: > > On Sun 2013-07-28 15:44:09, Pavel Machek wrote: > > > On Mon 2013-07-15 01:28:19, Aaro Koskinen wrote: > > > > Hi, > > > > > > > > On Sat, Jul 13, 2013 at 02:17:09PM +0200, Pavel Machek wrote: > > > > > > > > > > This adds device tree with neccessary support to boot with functional > > > > > video (on both emulator and real N900 device). > > > > > > > > > > Signed-off-by: Pavel Machek > > > > > > > > You can also add: > > > > > > > > Signed-off-by: Aaro Koskinen > > > > > > Benoit, could you apply this? Tony said you are doing dts changes, and > > > the patch seems to be ready. > > > > Tony, Benoit was apparently abducted by aliens or something. Is there > > chance you could take the patch? > > Hello, is there anybody out there? > > There's nice and simple patch enabling device tree on n900, it is even > reviewed by Aaro, and it is neither applied nor are negative comments > coming? ...for a month...? > > Tony also seen the patch and comment was "Benoit takes these", so I > assume it looks basically ok. > > Benoit, you are supposed to maintain device tree for omap2. >From archives I see Benoit is normally quite active... so I tried adding few more keywords to the subject line. Hopefully he's back from holidays and notices this...? > [I wonder if this is clean-enough cause for "trivial in resubmit > mode?] > Pavel > --- > > This adds device tree with neccessary support to boot with functional > video (on both emulator and real N900 device). > > Signed-off-by: Pavel Machek > Signed-off-by: Aaro Koskinen > > --- > > From v1: Aaro wants just GPLv2, so I did that. I re-enabled parts that > can be enabled on 3.10, and tested it on that kernel. > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index f0895c5..1950aed 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -141,6 +141,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \ > omap3-devkit8000.dtb \ > omap3-beagle-xm.dtb \ > omap3-evm.dtb \ > + omap3-n900.dtb \ > omap3-tobi.dtb \ > omap3-igep0020.dtb \ > omap3-igep0030.dtb \ > diff --git a/arch/arm/boot/dts/omap3-n900.dts > b/arch/arm/boot/dts/omap3-n900.dts > new file mode 100644 > index 000..fb461bf > --- /dev/null > +++ b/arch/arm/boot/dts/omap3-n900.dts > @@ -0,0 +1,92 @@ > +/* > + * Copyright (C) 2013 Pavel Machek > + * Copyright 2013 Aaro Koskinen > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +/dts-v1/; > + > +/include/ "omap34xx.dtsi" > + > +/ { > + model = "Nokia N900"; > + compatible = "nokia,omap3-n900", "ti,omap3"; > + > + cpus { > + cpu@0 { > + cpu0-supply = <&vcc>; > + }; > + }; > + > + memory { > + device_type = "memory"; > + reg = <0x8000 0x1000>; /* 256 MB */ > + }; > + > +}; > + > +&i2c1 { > + clock-frequency = <220>; > + > + twl: twl@48 { > + reg = <0x48>; > + interrupts = <7>; /* SYS_NIRQ cascaded to intc */ > + interrupt-parent = <&intc>; > + }; > +}; > + > +/include/ "twl4030.dtsi" > + > +&twl_gpio { > + ti,pullups = <0x0>; > + ti,pulldowns= <0x03ff3f>; /* BIT(0..5) | BIT(8..17) */ > +}; > + > +&i2c2 { > + clock-frequency = <40>; > +}; > + > +&i2c3 { > + clock-frequency = <10>; > +}; > + > +&mmc1 { > + status = "disabled"; > +}; > + > +&mmc2 { > + status = "disabled"; > +}; > + > +&mmc3 { > + status = "disabled"; > +}; > + > +&mcspi1 { > + // For some reason, touchscreen is neccessary for screen to work at > + // all on real hw. It works well without it on emulator. > + // > + // Also... order in the device tree actually matters here. > + tsc2005@0 { > + compatible = "tsc2005"; > + spi-max-frequency = <600>; > + reg = <0>; > + }; > + mipid@2 { > + compatible = "acx565akm"; > + spi-max-frequency = <600>; > + reg = <2>; > + // turbo_mode = 0, > + // cs_per_word = 0 > + }; > +}; > + > +&usb_otg_hs { > + interface-type = <0>; > + usb-phy = <&usb2_phy>; > + mode = <2>; > + power = <50>; > +}; > -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html