-
Notifications
You must be signed in to change notification settings - Fork 615
vim4: upgrade kernel and gpu #13397
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
base: master
Are you sure you want to change the base?
vim4: upgrade kernel and gpu #13397
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
#!/bin/sh | ||
|
||
if test "$1" != "start" | ||
then | ||
exit 0 | ||
fi | ||
|
||
printf "Preload VIMs video firmware: " | ||
|
||
# Load video firmware | ||
/sbin/modprobe -q tee | ||
/sbin/modprobe -q amlogic-tee | ||
/sbin/modprobe -q optee | ||
/usr/bin/tee-supplicant & | ||
sleep 1 | ||
/usr/bin/tee_preload_fw /lib/firmware/video/video_ucode.bin | ||
|
||
echo "OK" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
options dhd firmware_path=../../../lib/firmware/brcm/ nvram_path=../../../lib/firmware/brcm/ |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is forcing to preload a lot of modules, why is this necessary? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The previous modules.conf was based on the board vendor's module.conf. This was to keep it consistent with that strategy. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. if you could, it seems excessive especially considering some should autoload when needed accordingly. it can be a separate PR. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,5 @@ | ||
LABEL Batocera.linux | ||
LINUX /boot/linux | ||
FDT /boot/kvim4.dtb | ||
FDTDIR /boot | ||
INITRD /boot/initrd.gz | ||
APPEND initrd=/boot/initrd.gz label=BATOCERA partition_type=generic rootwait quiet loglevel=0 console=tty3 console=ttyAML0,115200n8 console=ttyS0,921600n8 vt.global_cursor_default=0 fan=auto vout=1080p60hz,enable hdmimode=1080p60hz outputmode=1080p60hz ${bootargs} ${boot_user_args} | ||
|
||
APPEND initrd=/boot/initrd.gz label=BATOCERA partition_type=generic rootwait quiet loglevel=0 vt.global_cursor_default=0 fan=auto ${bootargs} ${boot_user_args} console=tty3 console=ttyAML0,115200n8 console=ttyS0,921600n8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this approach seems rather odd, can you explain why you have to do it this way?
does the kernel module not load these at boot? can we change the kernel configs for the drivers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The intent was to load the video ucode as early as possible to make the hardware decoders happy which get loaded later in the S06modprobe. Converting the inserted modules in the script to buitin forces other modules to be builtin as well. It seems some of the modules cannot be builtin because they need to be loaded in certain order.