<neggles>
mrkiko: it should be able to do DBDC, but based on some comments in upstream patch discussions from mediatek staff, it depends on the specific board implementation
<neggles>
it looks like the "non-dbdc" setup is intended for when you're only using one of the two PHY interfaces on the WMAC
<neggles>
or for when it's being used with the cheapo PHYs
<neggles>
ah, sorry, I misread - the pinctrl change for "DBDC mode" vs "2G+5G mode" is just different funcs on the same pins, so yeah, it should just be DBDC capable end of story
<neggles>
but it *is* configurable as to whether it actually runs in DBDC mode or not, so might need a DT tweak or caldata to work?
goliath has joined #openwrt-devel
<mrkiko>
neggles: thanks a lot! Isn't the *dual.bin or *dbdc.bin file usage for eeprom data usage an indication? And, uncorrelated - what are DBDC limitations on modern designs ?
<slh>
on the typical mt7621a+mt7915DBDC, the drawback is obvious, cutting the 4x4 radio into two 2x2 radios - but the gl-mt6000 retains 4x4 for both bands
<mrkiko>
slh: thanks. So the commit message wasn't misleading at all. So then the design becomes as good as fast you are in switching antennas or something like that; I know very little about electronic per se
<mrkiko>
slh: or maybe totally different methodology used for sharing
gladiac is now known as Guest11984
gladiac has joined #openwrt-devel
Guest11984 has quit [Ping timeout: 480 seconds]
torv has quit [Quit: torv]
torv has joined #openwrt-devel
<rmilecki>
[2023-12-22] [09:43:18 CET] <rmilecki> i can't get much help on this, I think I'll just push:
<mrkiko>
rmilecki: eh, I acknowledge it's really too late, but I would have NACK'ed this patch, for what it counts (and I acknowledge it counts 0 (.
<rmilecki>
mrkiko: i really tried to get some feedback ;)
<mrkiko>
rmilecki: I also didn't follow enough the process behind this patch so ... can't really say much
<mrkiko>
rmilecki: yeah, you're right
<rmilecki>
what's potentially wrong about it?
<mrkiko>
rmilecki: I think that postinst != post-boot, and also am worried that people performs write operations in post-inst because - well, it's postinst. I am not a fan of write operations after every boot and they already happen, so it would be great to not add more
<mrkiko>
rmilecki: I think we are sending the wrong message to packers
<rmilecki>
mrkiko: i'm not sure if I understand
<rmilecki>
mrkiko: postinst doesn't get executed on every boot
<rmilecki>
it wasn't before my change and it isn't now
<mrkiko>
rmilecki: also, mod-ubus installation after installing uhttpd seems more a corner case in itelf rather than people using postinst to modify the filesystem. So much that restarting uhttpd just in mod-ubus package would, in my 2 cents, be acceptable as hacky as it is. Given that it's good to be able to install luci and use it with no reboot
<mrkiko>
ah ok, so I am totally off
<mrkiko>
I misread the commit, lookign at the patch clarifies things little more
<rmilecki>
:)
<rmilecki>
yeah, i'm not a perfect changes description writer
<rmilecki>
(i'm really trying thought!)
<rmilecki>
*though
<mrkiko>
rmilecki: you're great, it's me doing wrong connection; "after executing uci-defaults" did trip me into thinking that after each boot, after executing uci-defaults the system would unconditionally execute packages postinst
<mrkiko>
rmilecki: which may make sense if you write postinst with that in mind
<rmilecki>
ahh
<rmilecki>
i see how it could be misread
<mrkiko>
rmilecki: but if I did look at the patch without bothering you first :) :) ... I would have seen you change the default_postinst function - which BTW I never looked at / never looked where it's called, so ... sorry for the noise
<rmilecki>
np
<rmilecki>
now let's see if that actuallt blows up some packages ;)
<rmilecki>
i already sent a follow-up patch for uhttpd:
<rmilecki>
[PATCH] uhttpd: handle reload after uhttpd-mod-ubus installation using postinst