From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752169AbaE0MFr (ORCPT ); Tue, 27 May 2014 08:05:47 -0400 Received: from ns.mm-sol.com ([37.157.136.199]:33561 "EHLO extserv.mm-sol.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751927AbaE0MFo (ORCPT ); Tue, 27 May 2014 08:05:44 -0400 Message-ID: <53847FA1.60509@mm-sol.com> Date: Tue, 27 May 2014 15:05:53 +0300 From: Georgi Djakov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Matthias Brugger CC: Russell King , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , sboyd@codeaurora.org, rvaswani@codeaurora.org, davidb@codeaurora.org, "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , linux-arm-msm@vger.kernel.org Subject: Re: [PATCH v2 4/4] ARM: debug: qcom: add UART addresses to Kconfig help References: <1400857952-6963-1-git-send-email-gdjakov@mm-sol.com> <1400857952-6963-5-git-send-email-gdjakov@mm-sol.com> <53834578.9090305@mm-sol.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/26/2014 08:17 PM, Matthias Brugger wrote: > 2014-05-26 15:45 GMT+02:00 Georgi Djakov : >> On 23.05.14, 19:39, Matthias Brugger wrote: >>> >>> 2014-05-23 17:12 GMT+02:00 Georgi Djakov : >>>> >>>> Add information about the APQ8084 debug UART physical and virtual >>>> addresses in the DEBUG_QCOM_UARTDM Kconfig help section. >>>> Requires: https://lkml.org/lkml/2014/4/14/312 >>>> >>>> Signed-off-by: Georgi Djakov >>>> --- >>>> arch/arm/Kconfig.debug | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug >>>> index 40ee328..e71723d 100644 >>>> --- a/arch/arm/Kconfig.debug >>>> +++ b/arch/arm/Kconfig.debug >>>> @@ -376,6 +376,7 @@ choice >>>> their output to the serial port on Qualcomm devices. >>>> >>>> ARCH DEBUG_UART_PHYS DEBUG_UART_BASE >>>> + APQ8084 0xf995e000 0xfa75e000 >>>> MSM8X60 0x19c40000 0xf0040000 >>>> MSM8960 0x16440000 0xf0040000 >>>> MSM8974 0xf991e000 0xfa71e000 >>> >>> >>> You don't have any uart defined in your dts. You should add them >>> before any adding low level debug support. >>> Otherwise there might be problems in booting a multi-platform kernel. >>> >> >> Thanks for the comment! As the uart depends on clocks, i am adding it to the >> dts as part of my global clock controller patchset [1]. >> In this patch i am just adding some information to the help section. >> Do you see any problem with this? > > Are the patches for the clock controller applied already? > If not, I would consider to add this patch to clock controller series. > No, they are not applied yet. The uart dts definition depends on clocks, but the DEBUG_LL support does not. And the DEBUG_LL support is already there as it is common with other SoCs. I am sorry, but i don't see dependency between the dts definition and this patch. Could you explain please? Thanks, Georgi