2020-09-02 09:37:51

by Yang Shen

[permalink] [raw]
Subject: Re: [PATCH RESEND 00/10] crypto: hisilicon/zip - misc clean up

any comment?

Thanks,
Yang

On 2020/8/24 11:11, Yang Shen wrote:
> This patchset make some clean up:
> patch 1:remove useless parameters
> patch 4:replace 'sprintf' with 'scnprintf'
> patch 7:fix static check warning
> and the rest patch fix some coding style
>
> Resend this patch series because it depends on
> https://patchwork.kernel.org/cover/11715927/
> (crypto: hisilicon/qm - misc fixes).
> Now the patch series(crypto: hisilicon/qm - misc fixes) has been applied.
> So this patch series will apply against cryptodev.
>
> Shukun Tan (1):
> crypto: hisilicon/zip - modify debugfs interface parameters
>
> Yang Shen (9):
> crypto: hisilicon/zip - remove some useless parameters
> crypto: hisilicon/zip - unify naming style for functions and macros
> crypto: hisilicon/zip - replace 'sprintf' with 'scnprintf'
> crypto: hisilicon/zip - use a enum parameter instead of some macros
> crypto: hisilicon/zip - add print for error branch
> crypto: hisilicon/zip - fix static check warning
> crypto: hisilicon/zip - move some private macros from 'zip.h' to
> 'zip_crypto.c'
> crypto: hisilicon/zip - supplement some comments
> crypto: hisilicon/zip - fix some coding styles
>
> drivers/crypto/hisilicon/zip/zip.h | 15 ----
> drivers/crypto/hisilicon/zip/zip_crypto.c | 126 ++++++++++++++++++++---------
> drivers/crypto/hisilicon/zip/zip_main.c | 130 ++++++++++++++----------------
> 3 files changed, 148 insertions(+), 123 deletions(-)
>
> --
> 2.7.4
>
>
> .
>