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

component: Silence bind error on -EPROBE_DEFER #3539

Merged
merged 1 commit into from
Apr 11, 2020

Conversation

jameshilliard
Copy link
Contributor

If a component fails to bind due to -EPROBE_DEFER we should not log an error as this is not a real failure.

Fixes:

vc4-drm soc:gpu: failed to bind 3f902000.hdmi (ops vc4_hdmi_ops): -517
vc4-drm soc:gpu: master bind failed: -517

Partial fix for #3513.

@6by9
Copy link
Contributor

6by9 commented Apr 11, 2020

Acked by me.

drivers/base/component.c Outdated Show resolved Hide resolved
@jameshilliard
Copy link
Contributor Author

Upstream submission for reference https://patchwork.kernel.org/patch/11483959/.

If a component fails to bind due to -EPROBE_DEFER we should not log an
error as this is not a real failure.

Fixes:
vc4-drm soc:gpu: failed to bind 3f902000.hdmi (ops vc4_hdmi_ops): -517
vc4-drm soc:gpu: master bind failed: -517

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
@pelwell pelwell merged commit 2e79fd0 into raspberrypi:rpi-4.19.y Apr 11, 2020
@pelwell
Copy link
Contributor

pelwell commented Apr 11, 2020

Thanks.

@pelwell
Copy link
Contributor

pelwell commented Apr 11, 2020

C doesn't distinguish between spaces, TABs and newlines, so splitting a statement across multiple lines makes no difference. Braces would be needed if there were two statements within the body of the if.

@jameshilliard jameshilliard deleted the component-defer branch April 11, 2020 18:57
@jameshilliard
Copy link
Contributor Author

Ah, probably mixed that up since I mostly write Python code day to day.

@jameshilliard
Copy link
Contributor Author

Upstream v2 patch with braces removed https://patchwork.kernel.org/patch/11484225/.

popcornmix added a commit to raspberrypi/firmware that referenced this pull request Apr 17, 2020
kernel: configs: Add AUDIOINJECTOR_ISOLATED for Pi3 64-bit

kernel: overlays: Fix dtc warnings in i2c-gpio

kernel: media: rc: IR signal for Panasonic air conditioner too long

kernel: sound:soc:codecs:ma120x0p.c update and added to defconfigs
See: raspberrypi/linux#3534

kernel: drm/vc4: hdmi: Silence pixel clock error on -EPROBE_DEFER
See: raspberrypi/linux#3538

kernel: component: Silence bind error on -EPROBE_DEFER
See: raspberrypi/linux#3539

kernel: Fixes a problem with clock settings of HiFiBerry DAC+ADC PRO
See: raspberrypi/linux#3545

kernel: ARM: dts: Extend SCB bus address range
kernel: dts: bcm2711: Move emmc2 to its own bus
kernel: dts: bcm2711: Restore emmc2 regulator, brokencd
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request Apr 17, 2020
kernel: configs: Add AUDIOINJECTOR_ISOLATED for Pi3 64-bit

kernel: overlays: Fix dtc warnings in i2c-gpio

kernel: media: rc: IR signal for Panasonic air conditioner too long

kernel: sound:soc:codecs:ma120x0p.c update and added to defconfigs
See: raspberrypi/linux#3534

kernel: drm/vc4: hdmi: Silence pixel clock error on -EPROBE_DEFER
See: raspberrypi/linux#3538

kernel: component: Silence bind error on -EPROBE_DEFER
See: raspberrypi/linux#3539

kernel: Fixes a problem with clock settings of HiFiBerry DAC+ADC PRO
See: raspberrypi/linux#3545

kernel: ARM: dts: Extend SCB bus address range
kernel: dts: bcm2711: Move emmc2 to its own bus
kernel: dts: bcm2711: Restore emmc2 regulator, brokencd
@jameshilliard
Copy link
Contributor Author

This has now landed upstream FYI.

@pelwell
Copy link
Contributor

pelwell commented May 23, 2020

We noticed, thanks. rpi-5.7.y doesn't include your downstream patch.

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.

3 participants