Skip to content
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

Synokernel linuxtv #9

Open
wants to merge 2 commits into
base: synokernel-linuxtv
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions driver/linuxtv/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,7 @@ else
# builds can be reproducible
PKG_VERS = 0fe857b
PKG_GIT_HASH = 0fe857b86addf382f6fd383948bd7736a3201403
MEDIA_VERSION = 2022-06-05-f2906aa86338
#MEDIA_VERSION = LATEST
MEDIA_VERSION = LATEST
Copy link
Owner

Choose a reason for hiding this comment

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

Issue in using this is that the builds become no longer reproducible. That's why specific build version is being used, in hope to update to newer builds once every little while.

Copy link
Author

Choose a reason for hiding this comment

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

I think the archive was removed and the download broken so I used latest instead

endif

.PHONY: linuxtv_pre_patch
Expand Down
2 changes: 1 addition & 1 deletion spk/synokernel-linuxtv/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ SPK_GIT_HASH = 70188fd
SPK_GIT_DATE = 20210323
endif

SPK_VERS = $(SPK_GIT_DATE)~$(SPK_GIT_HASH)
SPK_VERS = $(SPK_GIT_DATE)
Copy link
Owner

Choose a reason for hiding this comment

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

Same applies here as linked.

SPK_REV = 12
SPK_ICON = src/linuxtv.png
CHANGELOG = "1. First experimental linuxtv driver package set for apollolake<br/>2. Backport of MYGICA T230A driver for 4.4.59 kernel"
Expand Down
10 changes: 5 additions & 5 deletions spk/synokernel-linuxtv/src/synokernel-linuxtv.cfg
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# see also: /lib/udev/devicetable/usb.DVB.dep.table
default="regmap-i2c rc-core mc videodev tveeprom videobuf2-common videobuf2-v4l2 videobuf2-memops videobuf2-vmalloc dvb-core dvb-usb dvb_usb_v2 uvcvideo"
HAUPPAUGE_WINTV_DUALHD="rc-hauppauge si2157 lgdt3306a em28xx em28xx-dvb em28xx-rc"
PCTV461E="rc-pinnacle-pctv-hd ts2020 a8293 m88ds3103 em28xx em28xx-dvb em28xx-rc"
MYGICA_T230="rc-dvbsky si2157 si2168 m88ds3103 dvb-usb-dvbsky"
SMS_SIANO_MDTV="smsmdtv smsdvb smsusb"
default:"regmap-i2c rc-core mc videodev tveeprom videobuf2-common videobuf2-v4l2 videobuf2-memops videobuf2-vmalloc dvb-core dvb-usb dvb_usb_v2 uvcvideo dvb-usb-rtl28xxu"
#HAUPPAUGE_WINTV_DUALHD:"rc-hauppauge si2157 lgdt3306a em28xx em28xx-dvb em28xx-rc"
#PCTV461E:"rc-pinnacle-pctv-hd ts2020 a8293 m88ds3103 em28xx em28xx-dvb em28xx-rc"
#MYGICA_T230:"rc-dvbsky si2157 si2168 m88ds3103 dvb-usb-dvbsky"
#SMS_SIANO_MDTV:"smsmdtv smsdvb smsusb"
Comment on lines +2 to +6
Copy link
Owner

Choose a reason for hiding this comment

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

The default is always being included when loading any other lines. In your case here you should rather create a new line specifically for this adapter module loading.

Looking into the kernel the Makefile entry name is CONFIG_DVB_USB_RTL28XXU to which in this case I'd use:

RTL28XXU="rtl28xxu"

So when calling the loading script with RTL28XXU as config option to use, it first loads all modules from the default line then will load this specific mode as well.

Copy link
Author

Choose a reason for hiding this comment

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

Ok makes sense