Re: [PATCH] arm64: dts: marvell: espressobin: Add support for LED2

2020-10-21 Thread Andre Heider
On 19/10/2020 10:04, Pali Rohár wrote: Hello! Gregory, I would like to remind you following patch. Andre, if you have a time, could you test it too? I would, but I desoldered all LEDs a long time ago because they're way too bright... Regards, Andre

Re: [PATCH] arm64: dts: marvell: add DT for ESPRESSObin-Ultra

2020-10-12 Thread Andre Heider
On 12/10/2020 09:08, Andre Heider wrote: Hi Vladimir, a few comments, all by just comparing this device tree to the other espressobin files. On 09/10/2020 17:27, Vladimir Vid wrote: This adds support for ESPRESSObin-Ultra from Globalscale. Specifications are similar to the base ESPRESSObin

Re: [PATCH] arm64: dts: marvell: add DT for ESPRESSObin-Ultra

2020-10-12 Thread Andre Heider
Hi Vladimir, a few comments, all by just comparing this device tree to the other espressobin files. On 09/10/2020 17:27, Vladimir Vid wrote: This adds support for ESPRESSObin-Ultra from Globalscale. Specifications are similar to the base ESPRESSObin board, with main difference being being WA

Re: [PATCH] arm64: dts: marvell: espressobin: De-duplicate eMMC definitions

2020-09-26 Thread Andre Heider
spressobin. Signed-off-by: Pali Rohár Reviewed-by: Andre Heider Tested-by: Andre Heider --- Compiled DTB files armada-3720-espressobin-emmc.dtb and armada-3720-espressobin-v7-emmc.dtb are identical as without applying this patch. Files armada-3720-espressobin.dtb and armada-3720-espressobin-

[PATCH v3] arm64: dts: marvell: espressobin: Get rid of duplicate serial aliases

2020-09-21 Thread Andre Heider
The included armada-37xx.dtsi already defines these two aliases. Signed-off-by: Andre Heider Reviewed-by: Pali Rohár --- v3: really fix filename, sorry for the spam... too early, not enough coffee v2: fix filename in commit message This goes on top of Pali's patch: "arm64: dt

[PATCH v2] arm64: dts: marvell: espressobin: Get rid of duplicate serial aliases

2020-09-21 Thread Andre Heider
The included armada-3xxx.dtsi already defines these two aliases. Signed-off-by: Andre Heider Reviewed-by: Pali Rohár --- v2: fix filename in commit message This goes on top of Pali's patch: "arm64: dts: marvell: espressobin: Add ethernet switch aliases" The resulting .dtb fil

Re: [PATCH] arm64: dts: marvell: espressobin: Get rid of duplicate serial aliases

2020-09-21 Thread Andre Heider
On 21/09/2020 18:53, Pali Rohár wrote: On Monday 21 September 2020 18:48:30 Andre Heider wrote: The included armada-372x.dtsi already defines these two aliases. Where? I do not see it in that file: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts

[PATCH] arm64: dts: marvell: espressobin: Get rid of duplicate serial aliases

2020-09-21 Thread Andre Heider
The included armada-372x.dtsi already defines these two aliases. Signed-off-by: Andre Heider --- This goes on top of Pali's patch: "arm64: dts: marvell: espressobin: Add ethernet switch aliases" The resulting .dtb files are the same. arch/arm64/boot/dts/marvell/armada-3720-e

[PATCH] arm64: dts: marvell: espressobin: Simplify v7 ethernet port labeling

2020-09-08 Thread Andre Heider
Now that the switch ports have a label in the .dtsi, simplify the whole "switch0" block for the v7 dts files. Signed-off-by: Andre Heider --- This goes on top of Pali's patch: "arm64: dts: marvell: espressobin: Add ethernet switch aliases" The resulting .dtb files a

Re: [PATCH] arm64: dts: marvell: espressobin: Add ethernet switch aliases

2020-09-07 Thread Andre Heider
On 07/09/2020 19:47, Pali Rohár wrote: On Monday 07 September 2020 19:43:08 Andre Heider wrote: On 07/09/2020 19:35, Pali Rohár wrote: On Monday 07 September 2020 19:23:03 Andrew Lunn wrote: My dts-foo is a little rusty, but now that you labeled the ports in the .dtsi, can this whole "sw

Re: [PATCH] arm64: dts: marvell: espressobin: Add ethernet switch aliases

2020-09-07 Thread Andre Heider
On 07/09/2020 19:42, Pali Rohár wrote: On Monday 07 September 2020 19:13:41 Andre Heider wrote: @@ -120,7 +124,7 @@ #address-cells = <1>; #size-cells = <0>; - port@0 { + switch0port0: por

Re: [PATCH] arm64: dts: marvell: espressobin: Add ethernet switch aliases

2020-09-07 Thread Andre Heider
On 07/09/2020 19:35, Pali Rohár wrote: On Monday 07 September 2020 19:23:03 Andrew Lunn wrote: My dts-foo is a little rusty, but now that you labeled the ports in the .dtsi, can this whole "switch0" block reduced to something like: &switch0port1 { label = "lan1"; }; &switch0port3 {

Re: [PATCH] arm64: dts: marvell: espressobin: Add ethernet switch aliases

2020-09-07 Thread Andre Heider
Hi Pali, On 07/09/2020 13:27, Pali Rohár wrote: Espressobin boards have 3 ethernet ports and some of them got assigned more then one MAC address. MAC addresses are stored in U-Boot environment. Since commit a2c7023f7075c ("net: dsa: read mac address from DT for slave device") kernel can use MAC

Re: [PATCH] of: build dbts with symbols when CONFIG_OF_OVERLAY is set

2017-12-20 Thread Andre Heider
On 18/12/17 23:10, Frank Rowand wrote: On 12/16/17 09:25, Andre Heider wrote: Hi Frank, On 15/12/17 22:06, Frank Rowand wrote: On 12/14/17 07:12, Andre Heider wrote: The overlay feature requires the base dtb to be built with symbols, so lets build the dtbs with symbols when overlay support

Re: [PATCH] of: build dbts with symbols when CONFIG_OF_OVERLAY is set

2017-12-16 Thread Andre Heider
Hi Frank, On 15/12/17 22:06, Frank Rowand wrote: On 12/14/17 07:12, Andre Heider wrote: The overlay feature requires the base dtb to be built with symbols, so lets build the dtbs with symbols when overlay support was explicitly enabled. With CONFIG_OF_ALL_DTBS on ARCH=arm the 989 dtb files

[PATCH] of: build dbts with symbols when CONFIG_OF_OVERLAY is set

2017-12-14 Thread Andre Heider
-beagle-x15-revb1.dtb 134545 arch/arm/boot/dts/dra7-evm.dtb 35464440total Signed-off-by: Andre Heider --- Hi, while playing around with overlays I noticed that I needed to rebuilt my distro's device trees because they didn't come with symbols. Is that for a reason,

Re: [ 000/100] 3.8.3-stable review

2013-03-13 Thread Andre Heider
On Wed, Mar 13, 2013 at 9:33 AM, David Miller wrote: > From: Andre Heider > Date: Wed, 13 Mar 2013 09:02:46 +0100 > >> On Tue, Mar 12, 2013 at 11:30 PM, Greg Kroah-Hartman >> wrote: >>> This is the start of the stable review cycle for the 3.8.3 release. >>&g

Re: [ 000/100] 3.8.3-stable review

2013-03-13 Thread Andre Heider
On Tue, Mar 12, 2013 at 11:30 PM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.8.3 release. > There are 100 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. Hi,