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

No way to modify existing configuration present in config.txt #1328

Open
adabool opened this issue May 23, 2024 · 4 comments
Open

No way to modify existing configuration present in config.txt #1328

adabool opened this issue May 23, 2024 · 4 comments

Comments

@adabool
Copy link

adabool commented May 23, 2024

Description
No way to modify existing configuration present in config.txt

Currently in order to add any extra configuration in config.txt one can use RPI_EXTRA_CONFIG.
But there is no existing way to modify or remove any existing configuration present in config.txt.
If unknowingly done by someone can cause serious undefined behavior (like no bootup) with the RPI device.
Steps to reproduce the issue:

  1. Add following in your local.conf or layer.conf file:

RPI_EXTRA_CONFIG = ' \n
dpms_timeout=0 \n
dtoverlay=vc4-fkms-v3d \n
hdmi_force_hotplug=1 \n
config_hdmi_boost=10 \n
hdmi_group=2 \n
hdmi_mode=87 \n
hdmi_cvt 1024 600 60 6 0 0 0 \n
'

Describe the results you received:
Device is not booting as there is already dtoverlay=vc4-kms-v3d defined.

Describe the results you expected:
Device should work properly

Additional information you deem important (e.g. issue happens only occasionally):

Additional details (revisions used, host distro, etc.):

@farias-automac
Copy link

Maybe you can overwrite rpi-config_git.bb file

@adabool
Copy link
Author

adabool commented May 30, 2024

@farias-automac thanks for this suggestion, do you mean by changing VC4DTBO ?= "vc4-kms-v3d" to VC4DTBO ?= "vc4-fkms-v3d" in rpi-config_git.bb file ?
Though this should work, isn't this a hacky way of doing this ?

@farias-automac
Copy link

Yes, that's what I was referring to. Otherwise I think you can create a bbappend file so you don't have to edit the original recipe directly. (although I'm not sure it works in this case, which is a do_deploy function)

Sorry my knowledge is limited, I'm a newbie. And this layer seems to not have much community support. Good luck !

@adabool
Copy link
Author

adabool commented May 30, 2024

I think we should have a support to remove configuration parameters defined in config.txt similar to what we have for adding (RPI_EXTRA_CONFIG).
That will be much better and generic approach to do so!

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

No branches or pull requests

2 participants