On 28/07/2021 17:09, Mark Brown wrote:
On Thu, Jul 22, 2021 at 10:55:23AM +0100, Richard Fitzgerald wrote:
I don't mind if someone wants to change the core dapm functions if that is generally useful, providing that it also updates all callers of those functions to still work.
Changing the behaviour of core code to fix the Realtek driver without updating other callers of those functions is a problem.
The thing here is that nobody would have thought that that any caller would have been open coding this stuff like the component things were,
On the contrary, since that was the only way to use these functions with a prefixed component it's entirely possible that there is code already adding the prefix. Why would you expect nobody has ever written code that works?
it's simply the wrong abstraction level to be implementing something
Ok, but that doesn't mean that it could never have happened.
like this so people wouldn't think of auditing the callers to find uses
I don't think that it's either safe or desirable to skip checking how callers use functionality that you want to change. My understanding of Linux development protocol was that if you make a change that affects other code, you are responsible for updating that other code to match. Regardless of whether you agree with how that other code was implemented. It creates a lot of overhead for everyone if it's ok to make changes without trying to fix up other code that is affected by that change.