Date: Thu, 22 Dec 2016 15:49:18 -0600
From: Reza Arbab <arbab@linux.vnet.ibm.com>
To: Heinrich Schuchardt <xypron.glpk@gmx.de>
Cc: Balbir Singh <bsingharora@gmail.com>, Rob Herring <robh+dt@kernel.org>,
        Frank Rowand <frowand.list@gmail.com>,
        Andrew Morton <akpm@linux-foundation.org>, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: Fwd: [PATCH 1/1] of/fdt: failed to mark hotplug range message
References: <20161222053405.27190-1-xypron.glpk@gmx.de>
 <5251e7ef-cadf-7833-9a7e-66ebf78e2e94@gmx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Disposition: inline
In-Reply-To: <5251e7ef-cadf-7833-9a7e-66ebf78e2e94@gmx.de>
Organization: IBM Linux Technology Center
User-Agent: NeoMutt/20161126 (1.7.1)
Message-Id: <20161222214917.7g6ldgkdvrlqsyq3@arbab-laptop.austin.ibm.com>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 668
Lines: 21

On Thu, Dec 22, 2016 at 06:52:45AM +0100, Heinrich Schuchardt wrote:
> diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c
> index c9b5cac03b36..fd129b6e5396 100644
> --- a/drivers/of/fdt.c
> +++ b/drivers/of/fdt.c
> @@ -1057,7 +1057,7 @@ int __init early_init_dt_scan_memory(unsigned long node, const char *uname,
>
>  		if (early_init_dt_mark_hotplug_memory_arch(base, size))
>  			pr_warn("failed to mark hotplug range 0x%llx - 0x%llx\n",
> -				base, base + size);
> +				base, base + size - 1);
>  	}
>
>  	return 0;

Isn't it implied that ranges printed this way are [start, end)?

If not, the entire file should be fixed, not just this occurrence.

-- 
Reza Arbab