Re: [PATCH][next] ASoC: codec: sma1307: replace spelling mistake with new error message

["Colin King (gmail)" <colin.i.king@gmail.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 868 bytes --]

On 08/08/2025 12:30, Mark Brown wrote:
> On Fri, Aug 08, 2025 at 11:53:24AM +0100, Colin Ian King wrote:
> 
>> There is a spelling mistake in a failure message, replace the
>> message with something a little more meaningful.
> 
>>   	if ((sma1307->set.checksum >> 8) != SMA1307_SETTING_CHECKSUM) {
>> -		dev_err(sma1307->dev, "%s: failed by dismatch \"%s\"\n",
>> +		dev_err(sma1307->dev, "%s: checksum failed \"%s\"\n",
> 
> It's definitely an unusual term and the reword makes sense but that does
> make sense to me as a word, I don't think it's a spelling error as such.

"Dismatch" is not a standard English word. It's likely a misspelling of 
"mismatch" or a neologism formed by adding the prefix "dis-" to "match". 
I understand it a niche term, I just think it is preferable not to use 
it for folk who's first language isn't English.

Colin

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 4901 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]