"Aneesh Kumar K.V" <[email protected]> writes:

> use the local variable that already have the converted values.
>
> No functional change in this patch.
> Signed-off-by: Aneesh Kumar K.V <[email protected]>
> ---
>  drivers/nvdimm/pfn_devs.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/nvdimm/pfn_devs.c b/drivers/nvdimm/pfn_devs.c
> index 36b904a129b9..8b7342517895 100644
> --- a/drivers/nvdimm/pfn_devs.c
> +++ b/drivers/nvdimm/pfn_devs.c
> @@ -599,14 +599,12 @@ int nd_pfn_validate(struct nd_pfn *nd_pfn, const char 
> *sig)
>               return -EOPNOTSUPP;
>       }
>  
> -     if (!IS_ALIGNED(res->start + le32_to_cpu(pfn_sb->start_pad),
> -                             memremap_compat_align())) {
> +     if (!IS_ALIGNED(res->start + start_pad, memremap_compat_align())) {
>               dev_err(&nd_pfn->dev, "resource start misaligned\n");
>               return -EOPNOTSUPP;
>       }
>  
> -     if (!IS_ALIGNED(res->end + 1 - le32_to_cpu(pfn_sb->end_trunc),
> -                             memremap_compat_align())) {
> +     if (!IS_ALIGNED(res->end + 1 - end_trunc, memremap_compat_align())) {
>               dev_err(&nd_pfn->dev, "resource end misaligned\n");
>               return -EOPNOTSUPP;
>       }

Reviewed-by: Jeff Moyer <[email protected]>


Reply via email to