From mboxrd@z Thu Jan 1 00:00:00 1970 From: timur@codeaurora.org (Timur Tabi) Date: Mon, 21 Mar 2016 21:06:04 -0500 Subject: [PATCH 2/2] serial: amba-pl011: complete support to ZTE uart In-Reply-To: References: <1457920282-14823-1-git-send-email-jun.nie@linaro.org> <1457920282-14823-2-git-send-email-jun.nie@linaro.org> <56F05C16.1030805@codeaurora.org> <56F0A628.2080607@codeaurora.org> Message-ID: <56F0A88C.9050209@codeaurora.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Jun Nie wrote: > That's a fair comment. I suppose all platform need to enable clock for > the device. For device tree case, we need these code for clock > enabling. But why do you need to enable the clock? The pl011 driver works fine for every platform without your patch. If the ZTE platform needs to call clk_get() in order to work, than that is something different about your platform. > For ACPI case, it may be handled automatically. If you agree > my opinion, I will move these lines to patch 1. Please correct me if I > am wrong. I think you need to create a separate patch that adds that call, and make sure it's tested by other platforms before it gets accepted. -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation.