Re: [PATCH v5 1/4] ASoC: Intel: bytcr_rt5640: Fix MCLK leak on platform_clock_control error
From: Mark Brown
Date: Wed Apr 01 2026 - 14:03:28 EST
On Tue, Mar 31, 2026 at 12:36:32PM -0400, aravindanilraj0702@xxxxxxxxx wrote:
> if (ret < 0) {
> - dev_err(card->dev, "could not configure MCLK state\n");
> + dev_err(card->dev, "could not configure MCLK state");
> return ret;
> }
To repeat what I have said multiple times:
| Please don't ignore review comments, people are generally making them
| for a reason and are likely to have the same concerns if issues remain
| unaddressed. Having to repeat the same comments can get repetitive and
| make people question the value of time spent reviewing. If you disagree
| with the review comments that's fine but you need to reply and discuss
| your concerns so that the reviewer can understand your decisions.
Please pay attention.
Attachment:
signature.asc
Description: PGP signature