On 6/4/24 11:09, Johan Hovold wrote: > On Tue, Jun 04, 2024 at 10:30:21AM +0200, Pierre-Louis Bossart wrote: >> On 6/4/24 02:52, Johan Hovold wrote: >>> Soundwire driver probe errors are currently being logged both by the bus >>> code and driver core: >>> >>> wsa884x-codec sdw:4:0:0217:0204:00:0: Probe of wsa884x-codec failed: -12 >>> wsa884x-codec sdw:4:0:0217:0204:00:0: probe with driver wsa884x-codec failed with error -12 >>> >>> Drop the redundant bus error message, which is also incorrectly being >>> logged on probe deferral: >> >> It's only redundant in the QCOM case... This would remove all error logs >> for other codecs, e.g. see >> >> rt711_sdca_sdw_probe >> cs35l56_sdw_probe >> wcd9390_probe >> >> Looks like the wsa884x-codec is the chatty driver, others are just fine >> with the existing code. > > I believe you misunderstood this patch. The error messages above are not > printed by the wsa884x-codec driver, but by the soundwire bus code and > driver core, so the redundant error message will be printed for all > codecs on probe failures. > > And specifically, driver core will still log probe failures after this > change. Ah yes, you're right I read 'driver core' sideways, my bad. That's fine then.