From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2707DC433EF for ; Tue, 7 Dec 2021 14:33:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237888AbhLGOhS (ORCPT ); Tue, 7 Dec 2021 09:37:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45686 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231591AbhLGOhR (ORCPT ); Tue, 7 Dec 2021 09:37:17 -0500 Received: from mail-ot1-x32e.google.com (mail-ot1-x32e.google.com [IPv6:2607:f8b0:4864:20::32e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 98193C061574 for ; Tue, 7 Dec 2021 06:33:47 -0800 (PST) Received: by mail-ot1-x32e.google.com with SMTP id n17-20020a9d64d1000000b00579cf677301so18260244otl.8 for ; Tue, 07 Dec 2021 06:33:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=0m2GF26E40xs6mb1JEeofYn5H0g7vqPjjF1kUlGlw1g=; b=qTMDWbLjlnnwBWIxkq7P3nMajKwCc4EprCObUf3oK+rgEK99LWCDchKSgEBzp3taXi L9OdOkdcVYPJPOSSFjixzUJDR6Z6f5/5yiQI7AU50vOq7eDrvkkfGF09XoA7TRFDxZt7 6B7Mo1KflEauK7z7QPu2luy3bqlkZ0fFTTAicwhdgvGQ48fbrCGHyYKe2Hu720QR/85m PLsr02ynd/it9TtM/XFds17CdDdIPalHH/xZMFuX8cxcGFOErICua4RTr+oOrX5PwsOv D9vm6YT3Ufpfb2ZMp+P1Esa5Oar8v/aSvtil36hD1bfWnrX44y61rvvmgVjKOwknwH5N tdtA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=0m2GF26E40xs6mb1JEeofYn5H0g7vqPjjF1kUlGlw1g=; b=ineSf9B2W1HYe9KtbKHO3Q2Ir7d0Zds6el4cVOdNKRG5RU9otaFuPHDgRIAxUJEJ/T Ioa0zueXHYNh1F3PJliOxUT6hc9aGW3uoD4jMEHmeB/gLgZIs+7MQ1WkoTmS2E/UQI0e YivjTcrKtetoIjorAkOI9JLBfR1tvpkctfuYlB9zrc4OTcCMWgPV02SZQJWdQlLBOa9H yADqv8A+S/gL60AkPj6SfHC+4NlWj05qiHJjeRu5Nf6zIoDlRJGvZ31yJtxb340KfNdA hAOiuf5aB/HZPQujDcxuos10z9gEF+f45gaomvJ4I9lUqUqZrg/ZB+I6u4jRUxuDxPpS GXRg== X-Gm-Message-State: AOAM532C21TIoDdqiEsgUQ+IIK8bimHOb+v4ojALwHkbspQWemmtKZEk 3+bi+moIuEBNUaRGzf9kX22N5Q== X-Google-Smtp-Source: ABdhPJy9LoOphSM1nEVxuWvLYLMmN9Elgk4jmv3+PE9nQA06m25+qZiTiG26zXQGeLFhHPulg3eG2Q== X-Received: by 2002:a9d:62c5:: with SMTP id z5mr36478679otk.166.1638887626915; Tue, 07 Dec 2021 06:33:46 -0800 (PST) Received: from ripper (104-57-184-186.lightspeed.austtx.sbcglobal.net. [104.57.184.186]) by smtp.gmail.com with ESMTPSA id bh12sm3469459oib.25.2021.12.07.06.33.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Dec 2021 06:33:46 -0800 (PST) Date: Tue, 7 Dec 2021 06:35:11 -0800 From: Bjorn Andersson To: Vinod Koul Cc: Konrad Dybcio , linux-arm-msm@vger.kernel.org, Andy Gross , Rob Herring , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 01/15] arm64: dts: qcom: Add base SM8450 DTSI Message-ID: References: <20211201072915.3969178-1-vkoul@kernel.org> <20211201072915.3969178-2-vkoul@kernel.org> <3fe36e35-8c4b-f772-be43-8e8c49e97f63@somainline.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Sun 05 Dec 21:39 PST 2021, Vinod Koul wrote: > On 01-12-21, 16:03, Konrad Dybcio wrote: > > > > On 01.12.2021 08:29, Vinod Koul wrote: > > > This add based DTSI for SM8450 SoC and includes base description of > > > CPUs, GCC, RPMHCC, UART, interuupt-controller which helps to boot to > > > shell with console on boards with this SoC > > > > > > Signed-off-by: Vinod Koul > > > --- > > > arch/arm64/boot/dts/qcom/sm8450.dtsi | 476 +++++++++++++++++++++++++++ > > > 1 file changed, 476 insertions(+) > > > create mode 100644 arch/arm64/boot/dts/qcom/sm8450.dtsi > > > > > > diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi > > > new file mode 100644 > > > index 000000000000..d838283bde4b > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi > > > @@ -0,0 +1,476 @@ > > > +// SPDX-License-Identifier: BSD-3-Clause > > > +/* > > > + * Copyright (c) 2021, Linaro Limited > > > + */ > > > + > > > +#include > > > +#include > > > +#include > > > +#include > > > + > > > +/ { > > > + interrupt-parent = <&intc>; > > > + > > > + #address-cells = <2>; > > > + #size-cells = <2>; > > > + > > > + chosen { }; > > > + > > > + clocks { > > > + xo_board: xo-board { > > > + compatible = "fixed-clock"; > > > + #clock-cells = <0>; > > > + clock-frequency = <76800000>; > > > + clock-output-names = "xo_board"; > > > + }; > > > + > > > + sleep_clk: sleep-clk { > > > + compatible = "fixed-clock"; > > > + clock-frequency = <32000>; > > > + #clock-cells = <0>; > > > + }; > > > > No clock-output-names for this one? > > This should have one, will fix and update in v2 No need, the clock's name is not significant for anything beyond debugging. So if you're going to change something let it be dropping the clock-output-names from &xo_board. Thanks, Bjorn > > > > > > Other than that: > > > > Reviewed-by: Konrad Dybcio > > Thanks for the review! > > -- > ~Vinod