From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Simek <michal.simek@xilinx.com>
Date: Mon, 22 Feb 2016 08:40:04 +0100
Subject: [U-Boot] [PATCH] dm: ns16550: Add support for reg-offset
	property
In-Reply-To: <CAPnjgZ37X-f6a-5yMJzi-BqWsigVCmApR=N6bZfogU+8LdqfgA@mail.gmail.com>
References: <70bfc564698cfd32c932a382a98321be169ccd87.1455635867.git.michal.simek@xilinx.com>
 <CAPnjgZ37X-f6a-5yMJzi-BqWsigVCmApR=N6bZfogU+8LdqfgA@mail.gmail.com>
Message-ID: <56CABB54.8070901@xilinx.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 19.2.2016 21:55, Simon Glass wrote:
> Hi Michal,
> 
> On 16 February 2016 at 08:17, Michal Simek <michal.simek@xilinx.com> wrote:
>> reg-offset is the part of standard 8250 binding in the kernel.
>> It is shifting start of address space by reg-offset.
>> On Xilinx platform this offset is typically 0x1000.
>>
>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>> ---
>>
>>  drivers/serial/ns16550.c | 6 ++++--
>>  include/ns16550.h        | 1 +
>>  2 files changed, 5 insertions(+), 2 deletions(-)
> 
> Reviewed-by: Simon Glass <sjg@chromium.org>
> 
> Do you support the debug UART feature on your boards?

yes. I do support it but there you can put just address plus offset and
there is no reason to add one more option to Kconfig.
But let me know if you think that this is incorrect flow.

Thanks,
Michal