Re: Install LuCI for snapshots builds

2024-05-07 Thread Daniel Golle
On Tue, May 07, 2024 at 11:52:02PM +0200, Robert Marko wrote:
> On Tue, 7 May 2024 at 23:25, Paul Spooren  wrote:
> >
> > Hi all,
> >
> > For some reason (resource usage?) our snapshot builds do not include the 
> > LuCI web interface. I think it’s an advantage to have LuCI installed in 
> > snapshot images since a) it installed for all releases anyway and b) often 
> > it’s just nice to have the web interface directly available.
> >
> > Is anyone against having the interface installed by default? I remember 
> > from multiple (in-person) discussion with fellow developers, that they’d 
> > prefer it being installed.
> >
> > If it’s an oversight I’d like to see it added to the default packages (via 
> > the builedbot config), if there’s a reason to keep it from snapshots, I’d 
> > like to understand the details.
> 
> +1 for LuCI by default in snapshots as well from me.

I understand the usability we may gain from that but you should all
be aware that this basically means only having a single buildbot
phase with a very slow turnover time, and that is a HUGE disadvantage
for development and use of the buildbot as classic CI took.

Let me explain why:
Currently the snapshot builders are only building **target-specific**
packages as well as packages included in the image by default. (We call
that "phase1"). That means that a single build takes around 2~3 hours,
depending on the target and the machine carrying out the build. In this
way we manage to have every target build once approximately every 24
hours.

If we wanted to include LuCI, that would basically mean that we will not
only have to include all the LuCI modules and applications, but also
**all their dependencies** which is basically half of the packages feed.

A full build of the packages feed (called "phase2") takes around 4
additional hours (best-case) and up to 17h (worst-case). We also
don't build for each (sub-)targets (think: ramips/mt7621), but only for
each architecture (think: mips24kc), and many (sub-)targets share the
same architecture. This results in every **architecture** being re-built
approximately every two days.

If we would do this for all (sub-)targets, the number would obviously be
even worse, we'd probably only see fresh images once or twice a week,
which is too slow to catch problems and too long for users to test
changes in a timely manner. It would be a humongous slow down of
development and testing on generic and core parts.
For me it would mean that I would have to invest a mid four-digit $ amount
into hardware to still be able to do meaningful development, and probably
it would mean that for quite a few of us.

What I could imaging is to have an **additional** build stop on top of
that, lets call it "phase3". That could be triggered on completion of
phase2 and then assemble images for all (sub-)targets using that
architecture including LuCI **in addition** to the phase1 snapshot
builds.

From a usability point of view, maybe the answer can be much easier and
the solution is already there:
1. Go to https://firmware-selector.openwrt.org/
2. Select 'SNAPSHOT' on the right.
3. Enter the device you want to put OpenWrt on or update.
4. Click on "Customize installed packages and/or first boot script"
5. Add 'luci' (or 'luci-ssl' or 'luci-ssl-openssl', ...) to the list
   of to-be-installed packages.

Maybe we can have an easier way to do that in that web UI and then
everybody will be happy?

And yes, the images generated using the firmware-selector are being
cached there. In this way only the first user requesting an image
with additional packages (luci in this case) would have to wait
~ 1 minute for the image to be generated, every subsequent user
requesting the same image would be served instantly.

The advantage would also be that we don't generated huge amounts of
images for legacy devices without any users, nor for "science-fiction"
R platforms.


Just my 2 cents...

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: Issues w/ kexec-tools when building images

2024-05-07 Thread Philip Prindeville via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---
Actually, I *am* selecting kexec which is part of kexec-tools… though I forget 
why.

It might be a hangover from when I was bringing up various x86_64 platforms and 
I needed to be able to boot into a failsafe kernel if the kernel-under-test 
hung for whatever reason and the watchdog fired…

In any case, it should still build.

I’ll dig into it if I get a chance.  The suspect part is here:

x86_64-openwrt-linux-musl-gcc  -mcmodel=large -I./purgatory/include 
-I./purgatory/arch/x86_64/include -I./util_lib/include -I./include -Iinclude 
-I/home/philipp/lede/staging_dir/toolchain-x86_64_gcc-13.2.0_musl/lib/gcc/x86_64-openwrt-linux-musl/13.2.0/include
  -c -MD -o purgatory/arch/i386/entry32-16.o purgatory/arch/i386/entry32-16.S


And looking at the Makefile:

https://git.kernel.org/pub/scm/utils/kernel/kexec/kexec-tools.git/tree/purgatory/arch/x86_64/Makefile#n18

that is indeed what it wants.

The top-level Makefile says:

AS  = x86_64-openwrt-linux-musl-gcc -c -Os -pipe -g3 
-fno-caller-saves -fno-plt -fhonour-copts 
-fmacro-prefix-map=/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28=kexec-tools-2.0.28
 -ffunction-sections -fdata-sections -Wformat -Werror=format-security 
-fstack-protector -D_FORTIFY_SOURCE=1 -Wl,-z,now -Wl,-z,relro

but that’s not how it’s being built.

and no ARCH specific flags are defined:

ASFLAGS =  $($(ARCH)_ASFLAGS)

and the top-level Makefile also has:

%.o: %.S
@$(MKDIR) -p $(@D)
$(COMPILE.S) -MD -o $@ $<

so the default definition of COMPILE.S seems to be lacking.  At which point I 
run out of -fu, because I can’t remember how to get “make” to dump its default 
definitions (i.e. the equivalent of "echo | gcc -dM -E -“ for make).





> On May 7, 2024, at 4:25 PM, Enéas Ulir de Queiroz  
> wrote:
> 
> I’m nowhere close to being able to even check this, but I will give you a 
> pointer. This usually happens when some Makefile defines multiple packages, 
> one of them depending on kexec-tools (or any package define in its Makefile), 
> and another one—which you are selecting—that doesn’t.  The build dependencies 
> are resolved at the Makefile level, so kexec is pulled if any package in that 
> Makefile is selected. 
> 
> To fix this, you can add a PACKAGE_foo: conditional to the kexec-tools 
> dependency. Then the build dependency will also have that condition, and 
> kexec-tools will not be built. Look at the OpenSSL Makefile for an example 
> (taking it from memory).
> 
> Cheers,
> 
> Eneas 
> 
>> Em 6 de mai. de 2024, à(s) 17:09, Philip Prindeville via openwrt-devel 
>>  escreveu:
>> 
>> The sender domain has a DMARC Reject/Quarantine policy which disallows
>> sending mailing list messages using the original "From" header.
>> 
>> To mitigate this problem, the original message has been wrapped
>> automatically by the mailing list software.
>> 
>> ___
>> openwrt-devel mailing list
>> openwrt-devel@lists.openwrt.org
>> https://lists.openwrt.org/mailman/listinfo/openwrt-devel


--- End Message ---
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: Issues w/ kexec-tools when building images

2024-05-07 Thread Enéas Ulir de Queiroz
I’m nowhere close to being able to even check this, but I will give you a 
pointer. This usually happens when some Makefile defines multiple packages, one 
of them depending on kexec-tools (or any package define in its Makefile), and 
another one—which you are selecting—that doesn’t.  The build dependencies are 
resolved at the Makefile level, so kexec is pulled if any package in that 
Makefile is selected. 

To fix this, you can add a PACKAGE_foo: conditional to the kexec-tools 
dependency. Then the build dependency will also have that condition, and 
kexec-tools will not be built. Look at the OpenSSL Makefile for an example 
(taking it from memory).

Cheers,

Eneas 

> Em 6 de mai. de 2024, à(s) 17:09, Philip Prindeville via openwrt-devel 
>  escreveu:
> 
> The sender domain has a DMARC Reject/Quarantine policy which disallows
> sending mailing list messages using the original "From" header.
> 
> To mitigate this problem, the original message has been wrapped
> automatically by the mailing list software.
> 
> ___
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: Issues w/ kexec-tools when building images (redux)

2024-05-07 Thread Christian Marangi (Ansuel)
Il giorno mer 8 mag 2024 alle ore 00:03 Philip Prindeville via
openwrt-devel  ha scritto:
>
> The sender domain has a DMARC Reject/Quarantine policy which disallows
> sending mailing list messages using the original "From" header.
>
> To mitigate this problem, the original message has been wrapped
> automatically by the mailing list software.
>
>
> -- Forwarded message --
> From: Philip Prindeville 
> To: openwrt-devel@lists.openwrt.org
> Cc:
> Bcc:
> Date: Tue, 7 May 2024 16:00:37 -0600
> Subject: Issues w/ kexec-tools when building images (redux)
> Hi,
>
> Sorry, had to add filters to turn DMARC off when sending mailing list mail….
>
> I fetched/rebased recently and now I’m not able to build images.  It fails in 
> kexec-tools, which is odd, because that’s not selected and nothing depends on 
> it.
>
> philipp@ubuntu22:~/lede$ grep kexec-tools .config
> # CONFIG_PACKAGE_kexec-tools is not set
> philipp@ubuntu22:~/lede$
>
> Here’s how the build reliably fails:
>
> make[3]: Entering directory '/home/philipp/lede/package/boot/kexec-tools'
> rm -f 
> /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built
> touch 
> /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built_check
> make -C /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28 
> DESTDIR="/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/ipkg-install"
>  all install
> make[4]: Entering directory 
> '/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28'
> x86_64-openwrt-linux-musl-gcc  -mcmodel=large -I./purgatory/include 
> -I./purgatory/arch/x86_64/include -I./util_lib/include -I./include -Iinclude 
> -I/home/philipp/lede/staging_dir/toolchain-x86_64_gcc-13.2.0_musl/lib/gcc/x86_64-openwrt-linux-musl/13.2.0/include
>   -c -MD -o purgatory/arch/i386/entry32-16.o purgatory/arch/i386/entry32-16.S
> purgatory/arch/i386/entry32-16.S: Assembler messages:
> purgatory/arch/i386/entry32-16.S:23: Error: 64bit mode not supported on 
> `i386'.
> make[4]: *** [Makefile:128: purgatory/arch/i386/entry32-16.o] Error 1
> make[4]: Leaving directory 
> '/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28'
> make[3]: *** [Makefile:133: 
> /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built] 
> Error 2
> make[3]: Leaving directory '/home/philipp/lede/package/boot/kexec-tools'
> time: package/boot/kexec-tools/compile#0.23#0.07#0.27
>  ERROR: package/boot/kexec-tools failed to build.
> make[2]: *** [package/Makefile:129: package/boot/kexec-tools/compile] Error 1
>
> I’m building for x86_64/generic on HEAD.
>
> Any ideas?  And who is the maintainer for kexec-tools?
>

I feel this might be a straight regression from a kernel bump.

I monitor the faillogs and kexec-tools appeared only recently...
Would be good to understand what commit caused this.

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Issues w/ kexec-tools when building images (redux)

2024-05-07 Thread Philip Prindeville via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---
Hi,

Sorry, had to add filters to turn DMARC off when sending mailing list mail….

I fetched/rebased recently and now I’m not able to build images.  It fails in 
kexec-tools, which is odd, because that’s not selected and nothing depends on 
it.

philipp@ubuntu22:~/lede$ grep kexec-tools .config
# CONFIG_PACKAGE_kexec-tools is not set
philipp@ubuntu22:~/lede$ 

Here’s how the build reliably fails:

make[3]: Entering directory '/home/philipp/lede/package/boot/kexec-tools'
rm -f /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built
touch 
/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built_check
make -C /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28 
DESTDIR="/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/ipkg-install"
 all install
make[4]: Entering directory 
'/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28'
x86_64-openwrt-linux-musl-gcc  -mcmodel=large -I./purgatory/include 
-I./purgatory/arch/x86_64/include -I./util_lib/include -I./include -Iinclude 
-I/home/philipp/lede/staging_dir/toolchain-x86_64_gcc-13.2.0_musl/lib/gcc/x86_64-openwrt-linux-musl/13.2.0/include
  -c -MD -o purgatory/arch/i386/entry32-16.o purgatory/arch/i386/entry32-16.S
purgatory/arch/i386/entry32-16.S: Assembler messages:
purgatory/arch/i386/entry32-16.S:23: Error: 64bit mode not supported on `i386'.
make[4]: *** [Makefile:128: purgatory/arch/i386/entry32-16.o] Error 1
make[4]: Leaving directory 
'/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28'
make[3]: *** [Makefile:133: 
/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built] 
Error 2
make[3]: Leaving directory '/home/philipp/lede/package/boot/kexec-tools'
time: package/boot/kexec-tools/compile#0.23#0.07#0.27
 ERROR: package/boot/kexec-tools failed to build.
make[2]: *** [package/Makefile:129: package/boot/kexec-tools/compile] Error 1

I’m building for x86_64/generic on HEAD.

Any ideas?  And who is the maintainer for kexec-tools?

Thanks,

-Philip


--- End Message ---
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: Install LuCI for snapshots builds

2024-05-07 Thread Robert Marko
On Tue, 7 May 2024 at 23:25, Paul Spooren  wrote:
>
> Hi all,
>
> For some reason (resource usage?) our snapshot builds do not include the LuCI 
> web interface. I think it’s an advantage to have LuCI installed in snapshot 
> images since a) it installed for all releases anyway and b) often it’s just 
> nice to have the web interface directly available.
>
> Is anyone against having the interface installed by default? I remember from 
> multiple (in-person) discussion with fellow developers, that they’d prefer it 
> being installed.
>
> If it’s an oversight I’d like to see it added to the default packages (via 
> the builedbot config), if there’s a reason to keep it from snapshots, I’d 
> like to understand the details.

+1 for LuCI by default in snapshots as well from me.

Regards,
Robert
>
> Thanks,
> Paul
> ___
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: Install LuCI for snapshots builds

2024-05-07 Thread Rich Brown
+1 - I endorse installing LuCI in snapshot builds.

- A significant fraction of people will wind up installing LuCI anyway.
- It's a FAQ on the forum. "I just installed a snapshot build, but there's no 
web GUI." (This even bites me from time to time...)
- LuCI poses less of a space concern (in the days of 4MByte Flash, it might 
have made a difference - no more.)
- People who're intentionally working on small Flash systems are rolling their 
own builds anyway.
- It's one fewer configuration option in our build process

Thanks for asking about this, Paul.

Rich

> On May 7, 2024, at 5:24 PM, Paul Spooren  wrote:
> 
> Hi all,
> 
> For some reason (resource usage?) our snapshot builds do not include the LuCI 
> web interface. I think it’s an advantage to have LuCI installed in snapshot 
> images since a) it installed for all releases anyway and b) often it’s just 
> nice to have the web interface directly available.
> 
> Is anyone against having the interface installed by default? I remember from 
> multiple (in-person) discussion with fellow developers, that they’d prefer it 
> being installed.
> 
> If it’s an oversight I’d like to see it added to the default packages (via 
> the builedbot config), if there’s a reason to keep it from snapshots, I’d 
> like to understand the details.
> 
> Thanks,
> Paul
> ___
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel


___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Issues w/ kexec-tools when building images

2024-05-07 Thread Philip Prindeville via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---
Hi,

I fetched/rebased recently and now I’m not able to build images.  It fails in 
kexec-tools, which is odd, because that’s not selected and nothing depends on 
it.

philipp@ubuntu22:~/lede$ grep kexec-tools .config
# CONFIG_PACKAGE_kexec-tools is not set
philipp@ubuntu22:~/lede$ 

Here’s how the build reliably fails:

make[3]: Entering directory '/home/philipp/lede/package/boot/kexec-tools'
rm -f /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built
touch 
/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built_check
make -C /home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28 
DESTDIR="/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/ipkg-install"
 all install
make[4]: Entering directory 
'/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28'
x86_64-openwrt-linux-musl-gcc  -mcmodel=large -I./purgatory/include 
-I./purgatory/arch/x86_64/include -I./util_lib/include -I./include -Iinclude 
-I/home/philipp/lede/staging_dir/toolchain-x86_64_gcc-13.2.0_musl/lib/gcc/x86_64-openwrt-linux-musl/13.2.0/include
  -c -MD -o purgatory/arch/i386/entry32-16.o purgatory/arch/i386/entry32-16.S
purgatory/arch/i386/entry32-16.S: Assembler messages:
purgatory/arch/i386/entry32-16.S:23: Error: 64bit mode not supported on `i386'.
make[4]: *** [Makefile:128: purgatory/arch/i386/entry32-16.o] Error 1
make[4]: Leaving directory 
'/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28'
make[3]: *** [Makefile:133: 
/home/philipp/lede/build_dir/target-x86_64_musl/kexec-tools-2.0.28/.built] 
Error 2
make[3]: Leaving directory '/home/philipp/lede/package/boot/kexec-tools'
time: package/boot/kexec-tools/compile#0.23#0.07#0.27
   ERROR: package/boot/kexec-tools failed to build.
make[2]: *** [package/Makefile:129: package/boot/kexec-tools/compile] Error 1

I’m building for x86_64/generic on HEAD.

Any ideas?  And who is the maintainer for kexec-tools?

Thanks,

-Philip


--- End Message ---
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: Install LuCI for snapshots builds

2024-05-07 Thread Christian Marangi
On Tue, May 07, 2024 at 11:24:32PM +0200, Paul Spooren wrote:
> Hi all,
> 
> For some reason (resource usage?) our snapshot builds do not include the LuCI 
> web interface. I think it’s an advantage to have LuCI installed in snapshot 
> images since a) it installed for all releases anyway and b) often it’s just 
> nice to have the web interface directly available.
> 
> Is anyone against having the interface installed by default? I remember from 
> multiple (in-person) discussion with fellow developers, that they’d prefer it 
> being installed.
> 
> If it’s an oversight I’d like to see it added to the default packages (via 
> the builedbot config), if there’s a reason to keep it from snapshots, I’d 
> like to understand the details.
>

Reason was probably when we supported very low memory device of 16/32
where it was a miracle the thing booted and could handle some traffic.
uhttpd + luci was enough to make them run OOM.

I didn't check how LOW SPACE device are handled and maybe this is
already done, but maybe those device should not have the web ui
preinstalled?

I think another reason was to discourage people on using SNAPSHOT
instead of STABLE release (since no luci and they had to use SSH from
the start)

Aside from the LOW SPACE thing to check, totally OK for me to add webui
in SNAPSHOT image.

-- 
Ansuel

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Install LuCI for snapshots builds

2024-05-07 Thread Paul Spooren
Hi all,

For some reason (resource usage?) our snapshot builds do not include the LuCI 
web interface. I think it’s an advantage to have LuCI installed in snapshot 
images since a) it installed for all releases anyway and b) often it’s just 
nice to have the web interface directly available.

Is anyone against having the interface installed by default? I remember from 
multiple (in-person) discussion with fellow developers, that they’d prefer it 
being installed.

If it’s an oversight I’d like to see it added to the default packages (via the 
builedbot config), if there’s a reason to keep it from snapshots, I’d like to 
understand the details.

Thanks,
Paul
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] ipq40xx: add PCIe magic hack to improve VRX518 compatibility

2024-05-07 Thread Christian Marangi (Ansuel)
Il giorno mar 7 mag 2024 alle ore 18:53 Enrico Mioso
 ha scritto:
>
> Hello all!!
>
> is there any chance we can merge any form of this patch?
> The device it is related seems pretty popular and one of the rare devices 
> supporting VDSL, 35B profile and with nice specs.
> Even tough I can understand it is not desirable to maintain this patch 
> indefinitely should that be the case, I don't think leaving it out of OpenWrt 
> is the right answer.
> Also due to the fact this seems, in my opinion, one of these things that can 
> act as barrier to entry for a typical setup (at least when VDSL is involved).
>
> Thanks a lot to all of you for your great work.
>

For everyone involved, I would really love if we can move here
https://github.com/openwrt/openwrt/pull/15421 and test the alternative
patch. It does have a different and more restricted approach.

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: Re: [PATCH] ramips: mt7621: add support for Keenetic Viva (KN-1910)

2024-05-07 Thread wornandrew via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---
> I count 15 partitions in your dts not 17.
There are 18 partitions in total (0 to 17).
15 in nand and 3 in virtual_flash.

> What is /dev/mtd17 and how does the original partition table look like?
This is from a running OpenWrt:

root@OpenWrt:~# cat /proc/mtd 
dev:size   erasesize  name
mtd0: 0008 0002 "U-Boot"
mtd1: 0008 0002 "U-Config"
mtd2: 0008 0002 "RF-EEPROM"
mtd3: 01bc 0002 "Firmware_1"
mtd4: 0008 0002 "Config_1"
mtd5: 0020 0002 "Storage_Legacy"
mtd6: 0004 0002 "Dump"
mtd7: 01fc 0002 "Storage_A"
mtd8: 0008 0002 "U-State"
mtd9: 0008 0002 "U-Config_res"
mtd10: 0008 0002 "RF-EEPROM_res"
mtd11: 01bc 0002 "Firmware_2"
mtd12: 0008 0002 "Config_2"
mtd13: 0220 0002 "Storage_B"
mtd14: 0008 0002 "unused"
mtd15: 0040 0002 "kernel"
mtd16: 0338 0002 "ubi"
mtd17: 041c 0002 "storage"

And the following is an excerpt from vendor's firmware log.

mtd: creating 17 MTD partitions on "MT7621-NAND":
0x-0x0008 : "U-Boot"
0x0008-0x0010 : "U-Config"
0x0010-0x0018 : "RF-EEPROM"
0x0018-0x0036 : "Kernel_1"
0x0036-0x01d4 : "RootFS_1"
mtd: device 4 (RootFS_1) set to be root filesystem
0x0018-0x01d4 : "Firmware_1"
0x01d4-0x01dc : "Config_1"
0x01dc-0x01fc : "Storage_Legacy"
0x01fc-0x0200 : "Dump"
0x0200-0x03fc : "Storage_A"
0x03fc-0x0404 : "U-State"
0x0404-0x040c : "U-Config_res"
0x040c-0x0414 : "RF-EEPROM_res"
0x0414-0x05d0 : "Firmware_2"
0x05d0-0x05d8 : "Config_2"
0x05d8-0x07f8 : "Storage_B"
0x-0x07f8 : "Full"
...
Searching for suitable storage partitions...
Found 1st storage partition of size 33292288 bytes
Found 2nd storage partition of size 35651584 bytes
Registering UBI data partitions parser
Concatenating MTD devices:
(0): "Storage_A"
(1): "Storage_B"
into device "NDM combined UBI partition"
1 ndmubipart partitions found on MTD device NDM combined UBI partition
mtd: creating 1 MTD partitions on "NDM combined UBI partition":
0x-0x041c : "Storage"
Merging storage partitions OK

I have omitted overlapping partitions from dts.
Otherwise the layout is identical.
/dev/mtd17 "storage" in OpenWrt is the same as vendor's "Storage".


--- End Message ---
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] ipq40xx: add PCIe magic hack to improve VRX518 compatibility

2024-05-07 Thread Enrico Mioso
Hello all!!

is there any chance we can merge any form of this patch?
The device it is related seems pretty popular and one of the rare devices 
supporting VDSL, 35B profile and with nice specs.
Even tough I can understand it is not desirable to maintain this patch 
indefinitely should that be the case, I don't think leaving it out of OpenWrt 
is the right answer.
Also due to the fact this seems, in my opinion, one of these things that can 
act as barrier to entry for a typical setup (at least when VDSL is involved).

Thanks a lot to all of you for your great work.

Enrico

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH 2/2] realtek: add RTL821X_CHIP_ID

2024-05-07 Thread Jonas Gorski
On Tue, 7 May 2024 at 12:16, Bjørn Mork  wrote:
>
> Stijn Tintel  writes:
>
> > On 27/04/2024 11:16, Bjørn Mork wrote:
> >> st...@linux-ipv6.be writes:
> >>
> >>> phy_write_paged(phydev, 31, 27, 0x0002);
> >>> val = phy_read_paged(phydev, 31, 28);
> >> ..
> >>> phy_write_paged(phydev, 0x1f, 0x1b, 0x0002);
> >>> val = phy_read_paged(phydev, 0x1f, 0x1c);
> >>
> >> While you're doing spring cleaning  That piece of cut-n-paste code
> >> looks very funny.
> >>
> > I'd gladly do some more spring cleaning, but what's your actual
> > suggestion here? Use hex everywhere instead of mixed hex/base 10?
>
> I would have used hex since that 0x1f looks like a mask.

We can find out if we look at the function signature:

int phy_read_paged(struct phy_device *phydev, int page, u32 regnum);

so that's a page, not a mask. Upstream usage is mostly hex, with the
icplus phy driver being the sole exception. And unsurprisingly,
realtek drivers always use magic numbers, never defines. Can't have
other people understand how the hardware works.

Best Regards,
Jonas

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH] ramips: mt7621: add support for Keenetic Viva (KN-1910)

2024-05-07 Thread Felix Baumann via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---
I count 15 partitions in your dts not 17.
What is /dev/mtd17 and how does the original partition table look like?


Regards
Felix Baumann

--- End Message ---
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[PATCH] ramips: mt7621: add support for Keenetic Viva (KN-1910)

2024-05-07 Thread wornandrew via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---
From: Andrew Worn 

SoC   : MediaTek MT7621A
RAM   : 128M DDR3
Flash : 128M NAND
Switch: MT7530, 5 ports 1Gbps
WiFi  : MT7615DN, 2.4GHz n and 5GHz ac
USB   : 2 ports USB 2.0
GPIO  : 4 buttons, 4 LEDs, USB port power controls

LAN   : RF-EEPROM + 0x04
WAN   : RF-EEPROM + 0x28
2.4GHz: RF-EEPROM + 0x04
5GHz  : 2.4GHz + 82:00:00:00:00:00

1. Connect to a LAN port, configure interface to use 192.168.1.2/30
2. Serve the factory image renamed to KN-1910_recovery.bin via TFTP
3. Power up the router while pressing Restart button on the back
4. Release Restart button when Power LED starts blinking

Keenetic's bootloader supports booting a LZMA compressed kernel but
seems to fail if the uncompressed data is larger than a fixed buffer
therefore it is safer to use a uimage-lzma-loader

Vendor firmware uses around half of flash for a writable UBIFS partition
that is preserved here. It may be accessed with the following commands:
 ubiattach -p /dev/mtd17
 mount -t ubifs /dev/ubi1_0 /mnt/

Signed-off-by: Andrew Worn 
---
diff --git a/target/linux/ramips/dts/mt7621_keenetic_kn-1910.dts 
b/target/linux/ramips/dts/mt7621_keenetic_kn-1910.dts
new file mode 100644
index 00..1717d274a8
--- /dev/null
+++ b/target/linux/ramips/dts/mt7621_keenetic_kn-1910.dts
@@ -0,0 +1,271 @@
+// SPDX-License-Identifier: GPL-2.0-or-later OR MIT
+#include "mt7621.dtsi"
+
+#include 
+#include 
+
+/ {
+   compatible = "keenetic,kn-1910", "mediatek,mt7621-soc";
+   model = "Keenetic KN-1910";
+
+   aliases {
+   led-boot = _power;
+   led-failsafe = _power;
+   led-running = _power;
+   led-upgrade = _power;
+   label-mac-device = 
+   };
+
+   gpio_export {
+   compatible = "gpio-export";
+   #size-cells = <0>;
+
+   left_usb_power {
+   gpio-export,name = "left-usb:power";
+   gpio-export,output = <1>;
+   gpios = < 5 GPIO_ACTIVE_HIGH>;
+   };
+
+   right_usb_power {
+   gpio-export,name = "right-usb:power";
+   gpio-export,output = <1>;
+   gpios = < 11 GPIO_ACTIVE_HIGH>;
+   };
+   };
+
+   keys {
+   compatible = "gpio-keys";
+
+   restart {
+   label = "restart";
+   gpios = < 6 GPIO_ACTIVE_LOW>;
+   linux,code = ;
+   };
+
+   wps {
+   label = "wps";
+   gpios = < 12 GPIO_ACTIVE_LOW>;
+   linux,code = ;
+   };
+
+   fn1 {
+   label = "fn1";
+   gpios = < 10 GPIO_ACTIVE_LOW>;
+   linux,code = ;
+   };
+
+   fn2 {
+   label = "fn2";
+   gpios = < 7 GPIO_ACTIVE_LOW>;
+   linux,code = ;
+   };
+   };
+
+   leds {
+   compatible = "gpio-leds";
+
+   led_power: power {
+   label = "green:power";
+   gpios = < 9 GPIO_ACTIVE_HIGH>;
+   };
+
+   internet {
+   label = "green:internet";
+   gpios = < 17 GPIO_ACTIVE_HIGH>;
+   };
+
+   fn {
+   label = "green:fn";
+   gpios = < 13 GPIO_ACTIVE_HIGH>;
+   };
+
+   wifi {
+   label = "green:wifi";
+   gpios = < 15 GPIO_ACTIVE_HIGH>;
+   };
+   };
+
+   virtual_flash {
+   compatible = "mtd-concat";
+   devices = <_1>, <_2>, <_a>, 
<_b>;
+
+   partitions {
+   compatible = "fixed-partitions";
+   #address-cells = <1>;
+   #size-cells = <1>;
+
+   partition@0 {
+   label = "kernel";
+   reg = <0x0 0x40>;
+   };
+
+   partition@40 {
+   label = "ubi";
+   reg = <0x40 0x338>;
+   };
+
+   partition@378 {
+   label = "storage";
+   reg = <0x378 0x41c>;
+   };
+   };
+   };
+};
+
+ {
+   status = "okay";
+
+   partitions {
+   compatible = "fixed-partitions";
+   #address-cells = <1>;
+ 

Re: [PATCH 1/2] realtek/rtl839x: respect phy-is-integrated property

2024-05-07 Thread Andreas Böhler via openwrt-devel
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.--- Begin Message ---

Hi,

On 27/04/2024 00:40, st...@linux-ipv6.be wrote:

Respect the phy-is-integrated property on ethernet-phy nodes.

There are RTL8393M switches where the PHYs at address 48 and 49 are
provided by an external RTL8214FC. Hardcoding them to use the internal
SerDes makes it impossible to use the ports connected to such an
external PHY. Respect the phy-is-integrated property on ethernet-phy
nodes as a first step to support such ports.

The potential impact for this should be limited to RTL8393 based
switches, and looking at the commit messages and device tree files of
the supported switches based on this SoC, the SFP and/or combo ports are
either not working (D-Link DGS-1210-52, Netgear GS750E, TP-Link
SG2452P/T1600G-52PS), use PHYs at a different address (Panasonic
SwitchM48EG PN28480K), or already have the phy-is-integrated property
set on the PHYs at address 48 and 49.


Some time ago, Jan Hoffmann has been working on support for the HPE 
1920-48G switch. He came up with a very similar patch and also with 
patches for the RTL8214FC. I'm using his version for quite some time now 
on my HPE switch.


Here is his commit: 
https://github.com/janh/openwrt/commit/ee5d8a17b540a3a9aa7b666a679b479903799d10


Unfortunately, there are so many different patches floating around, 
scattered across different forks, it's hard to know who has done what.


Andreas


--- End Message ---
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH 2/2] realtek: add RTL821X_CHIP_ID

2024-05-07 Thread Bjørn Mork
Stijn Tintel  writes:

> On 27/04/2024 11:16, Bjørn Mork wrote:
>> st...@linux-ipv6.be writes:
>>
>>> phy_write_paged(phydev, 31, 27, 0x0002);
>>> val = phy_read_paged(phydev, 31, 28);
>> ..
>>> phy_write_paged(phydev, 0x1f, 0x1b, 0x0002);
>>> val = phy_read_paged(phydev, 0x1f, 0x1c);
>>
>> While you're doing spring cleaning  That piece of cut-n-paste code
>> looks very funny.
>>
> I'd gladly do some more spring cleaning, but what's your actual
> suggestion here? Use hex everywhere instead of mixed hex/base 10?

I would have used hex since that 0x1f looks like a mask.

But it should at least be consistent so that it is obvious that this
code is a target for de-duplication.


Bjørn

___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


Re: [PATCH 2/2] realtek: add RTL821X_CHIP_ID

2024-05-07 Thread Stijn Tintel

On 27/04/2024 11:16, Bjørn Mork wrote:

st...@linux-ipv6.be writes:


phy_write_paged(phydev, 31, 27, 0x0002);
val = phy_read_paged(phydev, 31, 28);

..

phy_write_paged(phydev, 0x1f, 0x1b, 0x0002);
val = phy_read_paged(phydev, 0x1f, 0x1c);


While you're doing spring cleaning  That piece of cut-n-paste code
looks very funny.

I'd gladly do some more spring cleaning, but what's your actual 
suggestion here? Use hex everywhere instead of mixed hex/base 10?


Thanks,
Stijn



___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel


[PATCH qca-wireless] ipq6018: add BDF for Edgecore EAP101

2024-05-07 Thread stijn
Taken from TIP OpenWiFi:
https://github.com/Telecominfraproject/wlan-ap/raw/88d6633c85acd4143cfcb1f0a4fdcfdc88f35f3e/feeds/ipq807x_v5.4/ath11k-wifi/board-edgecore-eap101.bin.IPQ6018

Signed-off-by: Stijn Tintel 
---
 board-edgecore_eap101.ipq6018 | Bin 0 -> 65644 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 board-edgecore_eap101.ipq6018

diff --git a/board-edgecore_eap101.ipq6018 b/board-edgecore_eap101.ipq6018
new file mode 100644
index 
..1c8cfc15ba8d3164a843b242a842caabb9bb075f
GIT binary patch
literal 65644
zcmeHQ3wTu3o(5fB*SF#AcQ1@BtXD`yx)0|ggi|qA!Tlt)1#FI~1
zA|fhQY=Qdd@?rI(ZkOGy)~>s4t<`O9wcCDe)fU^@$9}HzZS8*6?C*E)+?ktXCYb~h
z1D-R#xpRK!od5s)&+nXj=gv*$U2R)2CNWEv_qSD3z^Sw+d3dXnc{yc&^#C
zW6|o3YYJ}Ox@qd#jhou1Zd$jf&|qt}t=_(lbjGY%1-n;o-?aLsor~75+pvD^w(aYu
zu3z0gy>R;0t)?dp9-U6ZseDkWY2vueJ+s-^t}wPO#StAzsr0y2eRgmS
zhzfSDIigMSFkj{bD^%{2t$;2*V{J9Ldg|a1N6*4AC3>*@l%)JM<*z}0S#xJ>9no=r
zVq|-p`6>C!2Y>m1g@ceB5@+jb5IP9sa^Lrtz9O$a|JuHvE{b-y#mCLmOAA#s
z5Cxo)?tj1SZjE=}K9i<>Abj?gGUPU<-kJ>ix=6IpwZGy#qxa*B=PJ;m`(#zu`
zTDP&|$g$%mPCok8vrj*B{@HK;$BQp_Yp6xV#KgoJh>mtfMUMcDiAG_!b_1Yo1F%sh
z5h>u3=Z*IZ4Qm^F0Uv=Bsw_~cGD?AAqCk|id@ScY!({j>j6qajgq$_KRQ0YqWF}(n0&4L~_
zMZiW9$$njA{I^8LJ}Q!YK*W86NPLk<^k{rG`9)`+eP+ETlJTra+E+xpuup`2+$`89
z!Tv(u_Us_*M~LCu{uu6nj~4hqw9yIh@yot_WMK?}7-(S+D!ye+vmq}B?7Po*VD_%;^~o!OWk>G*)ZPVK^FHmV43^z-Y{eaOc6V)e;>g;4
zOWL#Byj8(=hmT{kR_xJN|%DExAHdyw+y{q>x+L61)W3#>MaP!^;
z?YXN`t_!w%_~^RZm+Z`6Yj1bYk=A{*O>GRe`||zudy97GulLjh%T(^t6lgzmI{3!(
z?xp!s)>n@L)eS9cHg+66ar*4J^UuEUvUOq}c+JD*wCu-mUWfCTv>S1bj(yUD^Xmkh
z3)WD!c0yq92)bNlgMz`5;a5grdbI3G*ExlsIwP!>JQ^8OR&=s2g$_!}9p
zb5DkkvG5VsvwioHydRh`WZ---Z5`}!obp@?dz_!gM|c@9_j_StNQaM6-FxiGd$U
z7;sM5uNYoL3^>2fIEZm<^6M}KJ|d0b75`k~b5h!R+g#$@IzF~%%%Muc>d>FLz`1<}
zW8hp+2d<}n%tb!Of%P!{5Y`mdLrNWd6v7ABlq*!!UCZ(?m)R#_4;>AAyq9^dfxR2{
zj;?LPF~bMuV*Fk3fw@S*nlxg-=gxTBADYA2a|{#kS#Lbn^wS`Q!CzmulI
z2adG~(GlptxxjuAm<#L|##~^(N{I0I(04C`-{O%kuRUD2JNKp(Tobd>xkq>1H+N6Y
zX0PqJ2EXH+ZoI
ztn~2l)dv^elDi?r7T=j;n-9$TOxE>D%dJ)-#~!@po_V_`-k97REPL$e#{JiB%53#i
z1k1j5qW$h!JF?d%Vy#-~{=09vbKaJTD^qYhv(lpvZ~DTu+w(WuHGSsT$yu3X
zYj^7ShC?&9kMpSA2b)5PmjCMUcdC=?it_R2
zd!(F+ITrAZIPKjwv^T%o5uf1h9(vBi+`Rlrlcy9+jc5!G8G{i_mYK3Zils_?vQpN`
z7TF-}D9_1vapcUc3-j@>
z6i5Y=C+Fvw$r!_oz!>~Zue_Hl`tOJtLA~SR;(}y&^S+%i+PuruC#Hhc$ET50j0pztiVY5rZK!3oxMMNt2
z`t-9e_TFknXQ+1g3i6;y7$+z2pX?$lTUvZbc;$=Fhu3t?x+MMxwWBqZCjICCIEkAAA5+wVUe901zr~B3@UXql=SNqm17sSR}KO;~r*b{PyQ4pi}V+_M>3|CbWH*@$L#vOqcN%GV>`JZw`Y*uN|KY4QDSZr6BCg)
zx80fr=(usr+P8tKkQoSq`_Ez-hS9_UO~0Zs7ip@{>UIG~9mj3#>y-Dmv4XU}cs
zTG-=X68J2f%V#eBCn;2595J69ZKx4*rmbfv`>r#0`B|tw2U2$VBkRA%fy6zSHvJ#L
z<+m8j@eRcHYs%kd36VFh8ZGNcbpGVDq>zWF;_;od?f28EuWw7%@i(g)%W)5
z`hsy^V^pe3f4_sRWez!?mwKJ1w)v;_WntuN
z4}UjA**AKWWd|mw?x|F_#Qj;sOuS?N(
zC~dDqZHF@Ep`43L>1!x8xf1)j6m6BQQee0#5HA;uy9IQ<66bN~>+qfW(}I5{kOv<;
zb&9c`$H(t*dHYx_`<9N^24??soTulRx4j3IR|+TvlmbeDfhaIqE}FL$iom$t{bAhS
zVh*~W8ufDQLH8Tjz8CxZ5%nWGqwmD?Pp)L+?kjyey4^i?RATJI?pH>=h3!Y!CXW1Y
z#I})RVjja|+KnZZR2mY?>@d#>3uvFYem3SC$x$^?&>NM_>P_wKMp5vXc*c
z`$+Pp57YJuX~e_qJrY~hX80;loVYsfR|)syIcL=F*zd-^8{0EhbMB?bjGOCB#9U)N
z?O0;2yI`HUc7nXO+*08~C8dB;Kq;UUPzopolmbctrGQdEDWDWk3Md7X0-rnu62uw7