<neocturne>
Ansuel: in case you want to test the qca8k bridge port isolation patches I've just submitted for net-next on OpenWrt, I've pushed a backport that also hooks it up for IPQ4019 to my staging: https://git.openwrt.org/?p=openwrt/staging/neocturne.git;a=shortlog;h=refs/heads/qca8k
<Ansuel>
i have to check it but at times I also tried with an implementation and ktest didn't made the thing work correctly...
<neocturne>
Oh, what tests did you run?
<robimarko>
neocturne: Thanks for fixing the port bitmap
<neocturne>
You're welcome ^^ I noticed because my port isolation patch didn't quite work as expected at first
<neocturne>
Next ipq40xx issue on OpenWrt main is that my Fritzbox 4040 has all random MAC addresses... any ideas what might be wrong? Will look into it myself this weekend if there isn't already someone investigating
<robimarko>
On a quick look I dont see MAC-s being set in any way on that device
<neocturne>
Hmm, I'm pretty sure it was working in the past, but I'll have to check when that was
<robimarko>
Well, we are not setting it in userspace or via NVMEM, so probably bootloader sets it someway
<neocturne>
But the Device Tree doesn't have the ethernet0/ethernet1 aliases, so U-Boot can't apply a fixup
<neocturne>
OpenWrt 23.05 did set the alias: https://git.openwrt.org/?p=openwrt/openwrt.git;a=blob;f=target/linux/ipq40xx/patches-5.15/703-arm-dts-ipq4019-add-ethernet-controller-DT-node.patch;h=68fb4eb4ceeb84dbffbd731f488941edb847855e;hb=refs/heads/openwrt-23.05
<neocturne>
main isn't: https://git.openwrt.org/?p=openwrt/openwrt.git;a=blob;f=target/linux/ipq40xx/patches-6.6/705-ARM-dts-qcom-ipq4019-Add-description-for-the-IPQESS-.patch;h=6b66393d827b3234031bc432593fcf9f3dc2a476;hb=HEAD
<neocturne>
So all devices that get the address from the bootloader should be affected
<robimarko>
Because having a generic alias does not make sense
<neocturne>
Correct
<neocturne>
This should be set in the individual trees
<robimarko>
Cause the old 2 fake GMAC style driver is gone
<robimarko>
Yes, so its weird nobody noticed that 4040 is impacted
<neocturne>
Okay, I'll check how to set the aliases so the addresses match what the vendor firmware does
<neocturne>
Would be good to know if any other ipq40xx devices depended on the generic alias...
<neocturne>
There's a number of trees that set ethernet1, but not ethernet0 now, seems suspicious
<robimarko>
Probably because ethernet0 used to be set in the DTSI
<robimarko>
But do note that not all boards have been converted to DSA
<Ansuel>
necturne the linux kernel provide ktests for dsa to test all kind of stuff
<neocturne>
Ansuel: good to know, will have a look
<Ansuel>
they require some package to correctly works but they are really handy...
<Ansuel>
i will have to put all the thing support for the openwrt-test project
minimal has quit [Quit: Leaving]
goliath has joined #openwrt-devel
mrnuke has quit [Remote host closed the connection]
nixuser has quit [Remote host closed the connection]
<Mangix>
hrm https://git.openwrt.org/?p=openwrt/staging/neocturne.git;a=commit;h=150d0ceab7380538798df76a5d7d3c12a3632050 is interesting