From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: Re: [PATCH v2 4/4] ARM: dts: keystone: Add "ti, da830-uart" compatible string Date: Wed, 4 Jan 2017 14:00:01 -0800 Message-ID: <93c4aed0-6faa-2c27-7931-454fc1f48e79@oracle.com> References: <1483561814-21953-1-git-send-email-david@lechnology.com> <1483561814-21953-5-git-send-email-david@lechnology.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1483561814-21953-5-git-send-email-david@lechnology.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: David Lechner , Greg Kroah-Hartman , Sekhar Nori , Santosh Shilimkar Cc: Mark Rutland , devicetree@vger.kernel.org, Axel Haslam , Kevin Hilman , linux-kernel@vger.kernel.org, Bartosz Golaszewski , Rob Herring , Alexandre Bailon , linux-serial@vger.kernel.org, Jiri Slaby , Franklin S Cooper Jr , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On 1/4/2017 12:30 PM, David Lechner wrote: > The TI Keystone SoCs have extra UART registers beyond the standard 8250 > registers, so we need a new compatible string to indicate this. Also, at > least one of these registers uses the full 32 bits, so we need to specify > reg-io-width in addition to reg-shift. > > "ns16550a" is left in the compatible specification since it does work as > long as the bootloader configures the SoC UART power management registers. > NAK!! We can't break the booting boards with existing boot loaders. I suggest you to first get the driver updated to take care of the UART PM register and then enable the support for it.