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

Conversation

jekkos
Copy link

@jekkos jekkos commented May 2, 2023

Description

Useful work! I compiled it to install drivers for dvb tuners on my synology, saved me quite some time.

Fixes #

  • The package did not install on DSM7 due to a wrong version string
  • Linuxtv media download was broken and it's using latest zip file with backports now

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

@jekkos jekkos changed the base branch from master to synokernel-linuxtv May 2, 2023 22:01
@th0ma7
Copy link
Owner

th0ma7 commented Jul 19, 2023

Wow! Sorty for the late response but I just noticed your PR :) (e.g. I'm not being notified on this repo).

Normally PR are being pushed against spksrc master repository but I get it that you've changed one of my development branch in here, nice.

Note that I have rather limited cycles. This really is on my todo list but linuxtv is a beast to get sorted it for automated builds over Synology. I intend to revisit this after summer vacation, further as DSM7.1 kernel sources are now available.

Thnx and I'll reach-out again in early fall or earlier.

Copy link
Owner

@th0ma7 th0ma7 left a comment

Choose a reason for hiding this comment

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

Also, FYI, I'm currently looking at including DSM-7.x kernel sources in this PR SynoCommunity#5790 but this is far from being ready yet.

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

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.

Comment on lines +2 to +6
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"
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

@jekkos
Copy link
Author

jekkos commented Jul 20, 2023

Wow! Sorty for the late response but I just noticed your PR :) (e.g. I'm not being notified on this repo).

Normally PR are being pushed against spksrc master repository but I get it that you've changed one of my development branch in here, nice.

Note that I have rather limited cycles. This really is on my todo list but linuxtv is a beast to get sorted it for automated builds over Synology. I intend to revisit this after summer vacation, further as DSM7.1 kernel sources are now available.

Thnx and I'll reach-out again in early fall or earlier.

The work was useful as I was able to install the modiules rather easily this wat, bumped into some issues thats why I opened the PR

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.

2 participants