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