>>> It's generally nicer to replace embedded function names
>>> with "%s: ", __func__
>>>
>>>                     pr_warn("%s: cipher_encrypt failed\n", __func__);
>>
>> Do you want that I send a third patch series for the fine-tuning of these 
>> parameters?
> 
> If you want.

Would "a committer" fix such a small source code adjustment also without a 
resend of
a patch series?


> I just wanted you to be aware of it for future patches.

Thanks for your tip.

Does it make sense to express such implementation details in the Linux coding
style documentation more explicitly (besides the fact that this update 
suggestion
was also triggered by a warning from the script "checkpatch.pl").

Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to