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

RFC: sound: Fixes for audioinjector-octo under 4.19 #2902

Merged
merged 1 commit into from
Apr 23, 2019

Conversation

pelwell
Copy link
Contributor

@pelwell pelwell commented Mar 21, 2019

  1. Move the DT alias declaration to the I2C shim in the cases
    where the shim is enabled. This works around a problem caused by a
    4.19 commit [1] that generates DT/OF uevents for I2C drivers.

  2. Fix the diagnostics in an error path of the soundcard driver to
    correctly identify the reason for the failure to load.

  3. Move the declaration of the clock node in the overlay outside
    the I2C node to avoid warnings.

  4. Sort the overlay nodes so that dependencies are only to earlier
    fragments, in an attempt to get runtime dtoverlay application to
    work (it still doesn't...)

See: Audio-Injector/Octo#14
Signed-off-by: Phil Elwell [email protected]

[1] af50371 ("i2c: core: report OF style module alias for devices registered via OF")

@pelwell
Copy link
Contributor Author

pelwell commented Apr 6, 2019

Do you have any feedback for this patch?

1. Move the DT alias declaration to the I2C shim in the cases
where the shim is enabled. This works around a problem caused by a
4.19 commit [1] that generates DT/OF uevents for I2C drivers.

2. Fix the diagnostics in an error path of the soundcard driver to
correctly identify the reason for the failure to load.

3. Move the declaration of the clock node in the overlay outside
the I2C node to avoid warnings.

4. Sort the overlay nodes so that dependencies are only to earlier
fragments, in an attempt to get runtime dtoverlay application to
work (it still doesn't...)

See: Audio-Injector/Octo#14
Signed-off-by: Phil Elwell <[email protected]>

[1] af50371 ("i2c: core: report OF style module alias for devices registered via OF")
@pelwell
Copy link
Contributor Author

pelwell commented Apr 22, 2019

Rebased.

@pelwell pelwell merged commit 070a8ae into raspberrypi:rpi-4.19.y Apr 23, 2019
popcornmix added a commit to raspberrypi/firmware that referenced this pull request Apr 23, 2019
kernel: audioinjector-octo: revert to dummy supplies
See: raspberrypi/linux#2834

kernel: sound: Fixes for audioinjector-octo under 4.19
See: raspberrypi/linux#2902

kernel: Maxim MAX98357A I2S DAC overlay
See: raspberrypi/linux#2935

kernel: Add DVBSky S960 patch to fix broken driver
See: raspberrypi/linux#2849
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request Apr 23, 2019
kernel: audioinjector-octo: revert to dummy supplies
See: raspberrypi/linux#2834

kernel: sound: Fixes for audioinjector-octo under 4.19
See: raspberrypi/linux#2902

kernel: Maxim MAX98357A I2S DAC overlay
See: raspberrypi/linux#2935

kernel: Add DVBSky S960 patch to fix broken driver
See: raspberrypi/linux#2849
@pelwell pelwell deleted the octofix branch September 15, 2021 07:56
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.

1 participant