Re: [PATCH v3 0/8] staging: vt6655: Fix line wrapping in `RFvWriteWakeProgSyn`

From: Dan Carpenter
Date: Mon Nov 15 2021 - 10:48:39 EST


On Mon, Nov 15, 2021 at 10:55:41AM +0000, Karolina Drobnik wrote:
> * merge two patches (using incrementation in `idx` and cleanup) into one,
> partially implementing Dan's suggestion

Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

I don't know why you didn't implement my suggestions exactly as I
described.... :/

If I were reading this patchset the first time through I would have
NAKed it because patch 2 is done in the wrong way. In a way, it's my
fault because I tried to explain exactly how the one thing per patch
rule worked and I obviously was not clear.

Ah well... Let's just merge it and move on.

regards,
dan carpenter