Re: [PATCH v2 0/18] pata_parport: protocol drivers fixes and cleanups

Damien Le Moal
2023-02-14T23:51:13Z

On 2/15/23 07:59, Ondrej Zary wrote:
> This patch series fixes two bugs and cleans up pata_parport protocol drivers,
> making the code simpler with no changes in behavior (except logged messages).
>
> Signed-off-by: Ondrej Zary <linux@zary.sk>

Sergey did send you some reviewed-by tags for some of the patches. But I
do not see any in this v2. Did you forget to add the tags to the reviewed
patches ? That is nice to do so that I do not have to keep track of
reviews across series versions...

> ---
> Changes in v2:
> - added two bugfixes (first two patches)
> - addressed Sergey's comments (mostly split patches)
>
> drivers/ata/pata_parport/aten.c | 45 ++++----------
> drivers/ata/pata_parport/bpck.c | 86 ++++++++------------------
> drivers/ata/pata_parport/bpck6.c | 107 ++++++++-------------------------
> drivers/ata/pata_parport/comm.c | 52 +++++-----------
> drivers/ata/pata_parport/dstr.c | 45 ++++----------
> drivers/ata/pata_parport/epat.c | 48 ++++++---------
> drivers/ata/pata_parport/epia.c | 55 +++++------------
> drivers/ata/pata_parport/fit2.c | 37 ++++--------
> drivers/ata/pata_parport/fit3.c | 39 ++++--------
> drivers/ata/pata_parport/friq.c | 56 ++++++-----------
> drivers/ata/pata_parport/frpw.c | 71 ++++++----------------
> drivers/ata/pata_parport/kbic.c | 66 +++++++++-----------
> drivers/ata/pata_parport/ktti.c | 38 ++++--------
> drivers/ata/pata_parport/on20.c | 45 ++++----------
> drivers/ata/pata_parport/on26.c | 52 ++++------------
> drivers/ata/pata_parport/pata_parport.c | 31 +++++-----
> {include/linux => drivers/ata/pata_parport}/pata_parport.h | 41 ++++---------
> 17 files changed, 271 insertions(+), 643 deletions(-)
>
>

--
Damien Le Moal
Western Digital Research