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

[Colin King (gmail)]

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: OpenPGP_0x68C287DFC6A80226.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature