Re: [PATCH v3] w1: w1_therm: use swap() to make code cleaner

From: Greg KH
Date: Tue Dec 21 2021 - 03:32:25 EST


On Tue, Dec 21, 2021 at 08:15:28AM +0800, davidcomponentone@xxxxxxxxx wrote:
> From: Yang Guang <yang.guang5@xxxxxxxxxx>
>
> Use the macro 'swap()' defined in 'include/linux/minmax.h' to avoid
> opencoding it.
>
> ---
> Changes from v1->v2:
> - Delete the curly {} braces.
>
> Changes from v2->v3::
> - Add --- line before version info.
>
> Reported-by: Zeal Robot <zealci@xxxxxxxxxx>
> Signed-off-by: David Yang <davidcomponentone@xxxxxxxxx>
> Signed-off-by: Yang Guang <yang.guang5@xxxxxxxxxx>

Why the signed-off-by below the --- line? That means it will be cut off
and not present in the patch at all :(