Skip to content

Add Jienuo JN-107AR-E-WIFI - #58

Merged
openipc-ai merged 1 commit into
OpenIPC:masterfrom
davidebeatrici:jienuo-jn-603-d-wifi
Sep 21, 2026
Merged

openipc-ai merged 1 commit into
OpenIPC:masterfrom
davidebeatrici:jienuo-jn-603-d-wifi

Conversation

@davidebeatrici

@davidebeatrici davidebeatrici commented Jun 8, 2025

Copy link
Copy Markdown
Contributor

SD card slot untested, everything else is confirmed working.

U-Boot password: HI2105CHIP

https://github.com/thegroove/openipc-wiki/blob/device-jienuo-jn-107ar-e-wifi/en%2Fdevice-jienuo-jn-107ar-e-wifi.md

Pictures




Possible successor (JN-603-D-WIFI)




@thegroove

@davidebeatrici
davidebeatrici force-pushed the jienuo-jn-603-d-wifi branch from 17c5111 to 9610ffe Compare June 8, 2025 22:25
@davidebeatrici davidebeatrici changed the title Add Jienuo JN-603-D-WIFI Add Jienuo JN-107AR-E-WIFI Jun 8, 2025
@davidebeatrici
davidebeatrici force-pushed the jienuo-jn-603-d-wifi branch from 9610ffe to 05647c1 Compare June 8, 2025 22:26
@davidebeatrici

Copy link
Copy Markdown
Contributor Author

Unfortunately there seems to be no way to control the IR LEDs, which are always on.

@viktorxda
viktorxda force-pushed the master branch 2 times, most recently from 81e55d6 to 3674581 Compare June 10, 2025 13:09
@viktorxda
viktorxda force-pushed the master branch 2 times, most recently from 44889b6 to c6db1cd Compare August 6, 2025 13:20

@flyrouter flyrouter left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good day
You have done a good job, but it is highly recommended to switch the firmware to Lite mode and not Ultimate using a script to exclude unnecessary components as in other cameras.

@openipc-ai

Copy link
Copy Markdown
Contributor

Thanks for this, and apologies for the long silence. Reviewed against current master — it is the tidiest of the device PRs currently open, and it should land.

Good news: you no longer need a master.yml matrix entry. Since #120 the build matrix is read off the tree, so a device is registered simply by having devices/<dir>/.../<dir>_defconfig. This repo also builds pull requests now, narrowed to the devices a diff can reach — so a rebase push here will build t31_ultimate_jienuo-jn-107ar-e-wifi for the first time, and only that device.

What checks out

  • fw_setenv wlandev rtl8733bu-t31-camhipro exists upstream in firmware's general/overlay/etc/wireless/usb.
  • Directory name, defconfig filename and the -nor.tgz in the upgrade URL all agree, which is what keeps self-update working.
  • No excludes list is fine here: it's a 16M target, and neither t31_ultimate_* device in the tree ships one.

Two things before merge

  1. The branch now conflicts with master — needs a rebase.
  2. While rebasing, drop BR2_PACKAGE_JSONFILTER=y. package: stop selecting jsonfilter, mirroring firmware #2304 #128 removed it from all 97 defconfigs after firmware/#2304; it drags in libubox for nothing on a device that enables neither uqmi-openipc nor wifibroadcast-ng, and it had three boards over their rootfs cap.

On the IR LEDs being stuck on — that doesn't block the merge, but would you mind reflecting it in the README row rather than done? Other rows use that form, e.g. video-yes, wifi-yes, motors-no. It saves the next person with this camera from assuming it's solved.

@openipc-ai openipc-ai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Marking this changes-requested so it shows up as such in the queue. Nothing new — the same two items from my review above are still outstanding:

  1. The branch conflicts with master and needs a rebase.
  2. While rebasing, drop BR2_PACKAGE_JSONFILTER=y (#128 removed it from all 97 defconfigs after firmware/#2304).

The README status wording for the always-on IR LEDs is still a nice-to-have, not a blocker.

T31X with an SC5235 and an RTL8733BU USB radio on 16 MiB NOR.

Rebuilt on current master: the original branch carried an unrelated 2025
commit touching ssc338q_apfpv, whose customizer no longer exists in the tree,
and its README hunk no longer applied against the device table. This is the
net device addition and nothing else.

BR2_PACKAGE_JSONFILTER is dropped — OpenIPC#128 removed it from every defconfig after
firmware/#2304, and it pulls in libubox for nothing unless the device enables
uqmi-openipc or wifibroadcast-ng.
@openipc-ai

Copy link
Copy Markdown
Contributor

Picking this back up rather than letting it age out. The device profile itself
was fine — only the rebase and the jsonfilter line were outstanding — so I
have done both and pushed to your branch.

A note on how, because the history changed more than a plain rebase would:

The branch carried a third commit, a44bc34 "Update ssc338q_apfpv_defconfig",
by another contributor, which edited
devices/ssc338q_apfpv/general/overlay/usr/share/openipc/customizer.sh. That
file no longer exists in master, so replaying it would only have produced a
conflict over an unrelated device. Your net change never touched apfpv anyway.
The README hunk had also gone stale against the current device table.

So rather than fight the replay I rebuilt the branch as your net change on top
of master: the two device files exactly as you wrote them, plus the README row
reinserted in its correct alphabetical slot, with BR2_PACKAGE_JSONFILTER=y
dropped. Authorship on the commit is still yours.

This was a force-push, so reset to the remote rather than merging if you have
the branch locally.

ci-matrix --self-test and the cli path lint pass, and the matrix narrows to
t31_ultimate_jienuo-jn-107ar-e-wifi. Build approved; if it is green I will
merge. Apologies this sat for so long.

@openipc-ai
openipc-ai merged commit 66933ef into OpenIPC:master Sep 21, 2026
6 checks passed
@davidebeatrici

Copy link
Copy Markdown
Contributor Author

Thank you!

@davidebeatrici
davidebeatrici deleted the jienuo-jn-603-d-wifi branch September 21, 2026 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants