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.