Hi, On 4/23/23 17:29, David Yang wrote: > Remove redundant underscore and fix some grammar in prepare_request doc. > > Reported-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> No, I didn't report this issue. You either found it on your own or someone/something else reported it. > Link: https://lore.kernel.org/r/b58dc77c-c975-46cf-581c-368d9a87ae64@xxxxxxxxxxxxx Also drop the Link: line. > Signed-off-by: David Yang <mmyangfl@xxxxxxxxx> > --- > include/crypto/engine.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/crypto/engine.h b/include/crypto/engine.h > index ae133e98d813..2038764b30c2 100644 > --- a/include/crypto/engine.h > +++ b/include/crypto/engine.h > @@ -78,7 +78,7 @@ struct crypto_engine { > > /* > * struct crypto_engine_op - crypto hardware engine operations > - * @prepare__request: do some prepare if need before handle the current request > + * @prepare_request: do some preparation if needed before handling the current request That part looks good. > * @unprepare_request: undo any work done by prepare_request() > * @do_one_request: do encryption for current request > */ -- ~Randy