-
-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add target kirkwood for Linksys E4200 v2 #389
Conversation
31337d5
to
c578356
Compare
929e060
to
4e926e6
Compare
@grische The patch should apply cleanly after re-targeting v2023.1.2. Can you trigger the workflow please? |
We usually only add new targets to next. Any particular reason to target stable here? |
90dd268
to
37703d9
Compare
I noticed that I want changes from OpenWrt 23.05, so I target |
do we need to wait until: freifunk-gluon/gluon#3240 is merged? |
@T0biii not necessarily. If the device works and there's a need for it at FFMuc, we can add it before upstream has support for it. @darkdragon-001 you do plan to use it with FFMuc don't you? Btw: you need to push a commit for the ci to trigger after a branch change.
|
@darkdragon-001 maybe check this file if this needs modifications for the new target: https://github.com/freifunkMUC/site-ffm/blob/next/image-customization.lua |
37703d9
to
03d9c75
Compare
@grische Yes, I plan to use this device with FFMuc. Thanks for the hint, I pushed again after changing the branch. The error should be solved, at least in my fork it builds just fine. |
@darkdragon-001 can you check if the builds in the latest CI run work on your hardware? |
@grische Just installed it and so far everything seems to work fine. |
@darkdragon-001 great news! Can you give a list of things that are working for you? |
@grische basically everything I tested. Mostly SSH and AP to FF network. I cannot test 802.11s as there is no other FF router nearby but I remember having read that this is a hardware limitation anyways. |
If the device doesn't have 802.11s support, I think we should mark it as broken, similar to what you did here: Otherwise, I don't see a blocker for merging it. |
03d9c75
to
90e5518
Compare
@grische Done. |
From what I can tell you also need to adjust the targets.mk |
@grische Sorry, I can't follow. What do you want me to adjust where? |
90e5518
to
ead5b7e
Compare
No description provided.