Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp3490289pxk; Mon, 7 Sep 2020 15:00:35 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxYF9yfCspr87L5kYnZfMEO+Q6G3+rs+AhPzoK7g+D2+0Pz4L9pLQ05tjKnRMMMvTJCoQtg X-Received: by 2002:a17:906:c289:: with SMTP id r9mr19132432ejz.402.1599516035476; Mon, 07 Sep 2020 15:00:35 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1599516035; cv=pass; d=google.com; s=arc-20160816; b=LsQm8wGBsj1Z8YlhaN0sZLKkokZtM8/Xtoxar8xfDOgPjlLtAswVfJ7NGNqhmG/2IY R0AFWuXyQYN/TyaP4BX9GKN/sTONaneNbJXQ+tmfEDxTHdhkfD/KP7S6D+In48h18fjc 9T0Z5cHfA+34eea6QpWuw41N5Wc/ePfmnd3zUuAGvt3bTw/KNP377xnJ9bSbV0hj5D6D ma554h3xj/3bHW4Z3VRQQV5wPeMcwgxubBp71negJfxk0jvr9N9KLIU/2Nmu66cIrvO5 yEs8vWYGqIyLgBYH84lT3vLiw4+Mqpwa6aO2/Oi0G70PHvqnMuxLGQAYap5rVwswKW2S SOaQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :content-language:in-reply-to:user-agent:date:message-id:from :references:to:subject; bh=3mMyIFLn7x13+26wfsZzZmU9kKND5Fz5URdzquQjUlg=; b=ChLc93j7aulCS71QVxeG4sF2qfXedZlHmvw8FJd7ai0Z+LEzNS6lVSRdJmkBpFhBiO Wj+wXvM3iul1kF6y5m5wjJN0s20rlGCKYYS3UEiyMG9nWekE74kKJJqJ/HUzINJnois7 owN+iP43zbS9VuLZRLGeC7wL0Va0qngWP4PodEAGikfcv/+vGDeZOx0aiTENyaeJ68vz bvgjEUdTmr5w/y4uh85L6wXpgwkn67JzNdmnaxMyhQyUFCTXWitBcfq1ok669cmER3lT U1dJL277omt9tt/r71RnNUyEq2ZlpeI0iaO4gLLnuH/jqheyDkbEBS+Ybply5W9E+t7M vxnQ== ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=live.ca Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id dn20si13629047ejc.377.2020.09.07.15.00.12; Mon, 07 Sep 2020 15:00:35 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=live.ca Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727769AbgIGV7h (ORCPT + 99 others); Mon, 7 Sep 2020 17:59:37 -0400 Received: from mail-mw2nam10olkn2076.outbound.protection.outlook.com ([40.92.42.76]:24161 "EHLO NAM10-MW2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726941AbgIGV7g (ORCPT ); Mon, 7 Sep 2020 17:59:36 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Ofs4DqrcGV6LCNF7NeTtTyP4rAl1klnY5W/TXYLaebEXlT/ozrwIThFxq5IGidFNGo1rU59B58ZLqCqi4Yx+2fHAbbUx6VQo0TvhFkbHc3HBlTXtuUu6H5bkmoPYi7qRDhY5RFYPlhOBnVgjmcnSdvEWYLGQ0rHn7C8FzcdeSsmV2wUexMqEpUgt7/LsaOpu//mOB5BUzvgUXakomhTB18yhaQc5R/N4ojJXLZToCOxxawfXARiBrmzn+KJVfs/ERmprlpWew9DHz+Itu4XtDPbmz23/iYp/4ZECctuTM4OacIlcWFeys9vzSPmiOWfWT33O9rD4kFVD/DivqBMPIw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=3mMyIFLn7x13+26wfsZzZmU9kKND5Fz5URdzquQjUlg=; b=oA05g+7mEorQARYbynDEh+OaJriBeYHKiYz/ScXnPzVTLPW+MtvEcSlPypTEP+cp1y8P05tbNBweLmzofItugpF6qToH9n1Cqc39REHDvpT/QZ4Jb56PsmTobzCH/UYliS1lCH9E2YRMMjM5cBWjX/6q7aUd9KmSvvtyuhZAf+mNOJG+9NwgqG9Rmv3j9q9oi+OKV3Xf5oghbrc0fiRXx/ntaIatyMpkL3FB1MlfszudH+1UGRafbcxuB4riTnwQlcdsXwzu9p4vg0dnnQTSg7wh4QkFnaxWW/VOoTcpP6REqZtkc6q0+LUV8NNuOPQbUTxUOXVcwOrG3qp4cxCCQw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none Received: from BN7NAM10FT016.eop-nam10.prod.protection.outlook.com (2a01:111:e400:7e8f::4c) by BN7NAM10HT177.eop-nam10.prod.protection.outlook.com (2a01:111:e400:7e8f::295) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3348.16; Mon, 7 Sep 2020 21:59:34 +0000 Received: from BN6PR04MB0660.namprd04.prod.outlook.com (2a01:111:e400:7e8f::4e) by BN7NAM10FT016.mail.protection.outlook.com (2a01:111:e400:7e8f::226) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3326.23 via Frontend Transport; Mon, 7 Sep 2020 21:59:34 +0000 X-IncomingTopHeaderMarker: OriginalChecksum:1AC2766FC08D98C8B0B437D50376D26C203F8DD870C1FDE5CE551708501B3778;UpperCasedChecksum:57B2A49983E1ABEDDB9A01E793BFF078E699D11AE9577201387F12BCC9FAFC0C;SizeAsReceived:9290;Count:48 Received: from BN6PR04MB0660.namprd04.prod.outlook.com ([fe80::303b:a75b:d03e:bd04]) by BN6PR04MB0660.namprd04.prod.outlook.com ([fe80::303b:a75b:d03e:bd04%3]) with mapi id 15.20.3348.019; Mon, 7 Sep 2020 21:59:34 +0000 Subject: Re: [RFT 06/25] ARM: dts: s5pv210: move fixed clocks under root node To: Krzysztof Kozlowski , Kukjin Kim , Rob Herring , Jonathan Cameron , Lars-Peter Clausen , Peter Meerwald-Stadler , =?UTF-8?Q?Pawe=c5=82_Chmiel?= , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org References: <20200907161141.31034-1-krzk@kernel.org> <20200907161141.31034-7-krzk@kernel.org> From: Jonathan Bakker Message-ID: Date: Mon, 7 Sep 2020 14:59:29 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 In-Reply-To: <20200907161141.31034-7-krzk@kernel.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-ClientProxiedBy: CO2PR18CA0056.namprd18.prod.outlook.com (2603:10b6:104:2::24) To BN6PR04MB0660.namprd04.prod.outlook.com (2603:10b6:404:d9::21) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [IPv6:2001:569:fb68:9c00:8067:f823:1e15:7520] (2001:569:fb68:9c00:8067:f823:1e15:7520) by CO2PR18CA0056.namprd18.prod.outlook.com (2603:10b6:104:2::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3348.16 via Frontend Transport; Mon, 7 Sep 2020 21:59:31 +0000 X-Microsoft-Original-Message-ID: X-TMN: [G6aTOyxuKOyt7jjK6HKCYUMiKhFeVfuFcb+vC0+Z1TT0ipj2LYdB/aXmC66kDrSu] X-MS-PublicTrafficType: Email X-IncomingHeaderCount: 48 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-Correlation-Id: c03078b4-7411-41d1-3092-08d853794df6 X-MS-TrafficTypeDiagnostic: BN7NAM10HT177: X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ftYHjl3RitJDUNld+16ZH5kVvuAyRhChR3oPbMONp6+A9wzbNLCE78J8+jxz2FvjB74XxSfvl6Ga7NmhQM8wBNsjDKJ/bemTdeTapMwlV7dKR0F9mhE91Upn0X5AzLud+w+HSxp93EsYQW9NGkjaNNrCbM5gfXJ+zBwR7QOO/OzdTqEPhUX7T57ziZ3oDehNPp3eXvFobGLY8W32QcwB1A== X-MS-Exchange-AntiSpam-MessageData: uKWby0FG/2bXOfx9WCUJnonk84yAgpFz03LYGbSFUMJ4ZCpoJLOGDbIo+VhV3AzLChgV6CVnKLnr3kLqQB6tFB1b5C0Z7PqF78JBPryR5oUz2dQMwhLeL9260/sxtJDtpWMi25avyUlieZxEtZgi8gZ1AL5Wi1enpjAKK9ozpVwrD7BFNY30ZLA56qAqv4gcsg3CFk9GWYxEKOWsFCimWQ== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: c03078b4-7411-41d1-3092-08d853794df6 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Sep 2020 21:59:34.0332 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-AuthSource: BN7NAM10FT016.eop-nam10.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: Internet X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN7NAM10HT177 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Works for me on the Galaxy S. Tested-by: Jonathan Bakker Thanks, Jonathan On 2020-09-07 9:11 a.m., Krzysztof Kozlowski wrote: > The fixed clocks are kept under dedicated 'external-clocks' node, thus a > fake 'reg' was added. This is not correct with dtschema as fixed-clock > binding does not have a 'reg' property. Moving fixed clocks out of > 'soc' to root node fixes multiple dtbs_check warnings: > > external-clocks: $nodename:0: 'external-clocks' does not match '^([a-z][a-z0-9\\-]+-bus|bus|soc|axi|ahb|apb)(@[0-9a-f]+)?$' > external-clocks: #size-cells:0:0: 0 is not one of [1, 2] > external-clocks: oscillator@0:reg:0: [0] is too short > external-clocks: oscillator@1:reg:0: [1] is too short > external-clocks: 'ranges' is a required property > oscillator@0: 'reg' does not match any of the regexes: 'pinctrl-[0-9]+' > > Signed-off-by: Krzysztof Kozlowski > --- > arch/arm/boot/dts/s5pv210.dtsi | 36 +++++++++++++--------------------- > 1 file changed, 14 insertions(+), 22 deletions(-) > > diff --git a/arch/arm/boot/dts/s5pv210.dtsi b/arch/arm/boot/dts/s5pv210.dtsi > index 84e4447931de..5c760a6d7955 100644 > --- a/arch/arm/boot/dts/s5pv210.dtsi > +++ b/arch/arm/boot/dts/s5pv210.dtsi > @@ -52,34 +52,26 @@ > }; > }; > > + xxti: oscillator-0 { > + compatible = "fixed-clock"; > + clock-frequency = <0>; > + clock-output-names = "xxti"; > + #clock-cells = <0>; > + }; > + > + xusbxti: oscillator-1 { > + compatible = "fixed-clock"; > + clock-frequency = <0>; > + clock-output-names = "xusbxti"; > + #clock-cells = <0>; > + }; > + > soc { > compatible = "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > ranges; > > - external-clocks { > - compatible = "simple-bus"; > - #address-cells = <1>; > - #size-cells = <0>; > - > - xxti: oscillator@0 { > - compatible = "fixed-clock"; > - reg = <0>; > - clock-frequency = <0>; > - clock-output-names = "xxti"; > - #clock-cells = <0>; > - }; > - > - xusbxti: oscillator@1 { > - compatible = "fixed-clock"; > - reg = <1>; > - clock-frequency = <0>; > - clock-output-names = "xusbxti"; > - #clock-cells = <0>; > - }; > - }; > - > onenand: onenand@b0600000 { > compatible = "samsung,s5pv210-onenand"; > reg = <0xb0600000 0x2000>, >