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 ADA53CCA47F for ; Mon, 20 Jun 2022 00:25:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235125AbiFTAZv (ORCPT ); Sun, 19 Jun 2022 20:25:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34400 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236633AbiFTAZu (ORCPT ); Sun, 19 Jun 2022 20:25:50 -0400 Received: from esa1.hgst.iphmx.com (esa1.hgst.iphmx.com [68.232.141.245]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1AF1CAE55 for ; Sun, 19 Jun 2022 17:25:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1655684749; x=1687220749; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=H0ulQrMBtA6pu622Zq7Ff/HeEgzOhbo0ZUSX4iWBZhA=; b=fccttzF94utVzcRKcT5TwY/aGdBAeDm0amO9LUrOFkZyJqpIG4OE/5hK nZeoO3n7jfMKY4Sro7IT8OUL/JN3vyRaYaAr7eHIwSFtVbqzKhtREP+7q WtVlAIwkWDdVVKmEKKFMYYnxda2VPGPlF2QD2yoA8QvauzJfZQB0ZZo0P JvQLAxxOYdQ+P2+8FIZHHZGqpydxvHeagbSENUpRiP4UQl/RPO69pAuUn i2z+u1okL4w9Crkp2jyVR4bytyQXtx41x3S86yyvWMuUY1cfvoAk9aJ2T +9ZdeJMHsRBf2uijz0bGAjIHuhQ0kk9g0TKk1zFSCcerFlhxu2c1OAfcT g==; X-IronPort-AV: E=Sophos;i="5.92,306,1650902400"; d="scan'208";a="315672135" Received: from h199-255-45-15.hgst.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 20 Jun 2022 08:25:47 +0800 IronPort-SDR: O/icw8ui8iIh/NosVlY0zDI1dyibWthagS7jbKXcDR1Za+JRJF0L+Cw3pkzK/wCrPuLa3XixP0 Zt+GAua4faIvuIb0xAC7u07EFa9zp+1cEUxf0y1vgvV5XAT+YB1tn7LXa0JLrnaSl1xUDhlN5p uXiNB1oAQCBJ7C/A6Cv2i0kZMK7e479Y4+5BnlqmaTB3kl9JgOhNkT7Ea9XzcR1GgRZ66l5tJr AbESj62lMSFXegTnkkyQU2NuSuJS/KhLd4hRTV15i/dRLEFlibxoZzEJm+LyK3TjcfXStOqsFM G/AlQjfU4aLDfAuCRdzWm6IX Received: from uls-op-cesaip02.wdc.com ([10.248.3.37]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 19 Jun 2022 16:43:56 -0700 IronPort-SDR: 4eQ5ZsmWaF8UKD8Lc8PEVEWQAajEdSv0OGvJVIm1LYbUQReqB+DT5EsbflFwZZr5Xx1V1PLAnj Mc7LC6hx1IE5tQH0isR2SGSdCISeKw7YLfFkc0Dt9DKsNc5nszpC6PtYbE7d7Qtjs67QJj+3KN eLZZZ9zNus/fDsPsgg1UA9GSZPcLc0zywGihhgIUyAW6J35WUDJw2ymJOIVX2EevxoNCWABgBy ECx/R32uq8EvNgs54KFv7hpJTrSUx9PSmy9DpjWjnHG9ykIks36VFnTg1QBwT5IRStNy/Fef7z zJo= WDCIronportException: Internal Received: from usg-ed-osssrv.wdc.com ([10.3.10.180]) by uls-op-cesaip02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 19 Jun 2022 17:25:47 -0700 Received: from usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTP id 4LR9RH0xvrz1SHwl for ; Sun, 19 Jun 2022 17:25:46 -0700 (PDT) Authentication-Results: usg-ed-osssrv.wdc.com (amavisd-new); dkim=pass reason="pass (just generated, assumed good)" header.d=opensource.wdc.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d= opensource.wdc.com; h=content-transfer-encoding:content-type :in-reply-to:organization:from:references:to:content-language :subject:user-agent:mime-version:date:message-id; s=dkim; t= 1655684746; x=1658276747; bh=H0ulQrMBtA6pu622Zq7Ff/HeEgzOhbo0ZUS X4iWBZhA=; b=C9w0xWLFB/KaDMQ+IzY1oZGaVKgyVSWgQTN/9sJJGaofpjPKWWt ZKKgIMwWS1Dy3N+AKNLls5E2/TsxAp6rP+S9l0Ik2SK7XIV5qtlSHKwhGAwp4LxJ 6gePz+4huJUkrzHySdcijhxdigID+rEmfTDa4Wt/eP1LPIzIGKKw5KxzF3Dtmtap p5Va13Ox1XAtZ0OpzHoWucY1cenLs5OjDloywiS4JT1zdmVBZAj8Mo2U+pX+WsHX Xq2w2dDe0rAcZyAfGZlDDozsM67SwB5DYyhQRX3WWnevylp14JZ8tNudXLaciW4o qUyYE7ltn31FuS+79D85xj771kaXmkpnCiQ== X-Virus-Scanned: amavisd-new at usg-ed-osssrv.wdc.com Received: from usg-ed-osssrv.wdc.com ([127.0.0.1]) by usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id zdU850oNSjxY for ; Sun, 19 Jun 2022 17:25:46 -0700 (PDT) Received: from [10.225.163.87] (unknown [10.225.163.87]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTPSA id 4LR9R75cZsz1Rvlc; Sun, 19 Jun 2022 17:25:39 -0700 (PDT) Message-ID: <891cf74c-ac0a-b380-1d5f-dd7ce5aeda9d@opensource.wdc.com> Date: Mon, 20 Jun 2022 09:25:38 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Subject: Re: [PATCH 07/14] riscv: dts: canaan: fix the k210's memory node Content-Language: en-US To: Conor.Dooley@microchip.com, mail@conchuod.ie, airlied@linux.ie, daniel@ffwll.ch, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, thierry.reding@gmail.com, sam@ravnborg.org, Eugeniy.Paltsev@synopsys.com, vkoul@kernel.org, lgirdwood@gmail.com, broonie@kernel.org, fancer.lancer@gmail.com, daniel.lezcano@linaro.org, palmer@dabbelt.com, palmer@rivosinc.com Cc: tglx@linutronix.de, paul.walmsley@sifive.com, aou@eecs.berkeley.edu, masahiroy@kernel.org, geert@linux-m68k.org, niklas.cassel@wdc.com, dillon.minfei@gmail.com, jee.heng.sia@intel.com, joabreu@synopsys.com, dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dmaengine@vger.kernel.org, alsa-devel@alsa-project.org, linux-spi@vger.kernel.org, linux-riscv@lists.infradead.org References: <20220618123035.563070-1-mail@conchuod.ie> <20220618123035.563070-8-mail@conchuod.ie> <9cd60b3b-44fe-62ac-9874-80ae2223d078@opensource.wdc.com> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-spi@vger.kernel.org On 6/20/22 08:54, Conor.Dooley@microchip.com wrote: > On 20/06/2022 00:38, Damien Le Moal wrote: >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe >> >> On 6/18/22 21:30, Conor Dooley wrote: >>> From: Conor Dooley >>> >>> The k210 memory node has a compatible string that does not match with >>> any driver or dt-binding & has several non standard properties. >>> Replace the reg names with a comment and delete the rest. >>> >>> Signed-off-by: Conor Dooley >>> --- >>> --- >>> arch/riscv/boot/dts/canaan/k210.dtsi | 6 ------ >>> 1 file changed, 6 deletions(-) >>> >>> diff --git a/arch/riscv/boot/dts/canaan/k210.dtsi b/arch/riscv/boot/dts/canaan/k210.dtsi >>> index 44d338514761..287ea6eebe47 100644 >>> --- a/arch/riscv/boot/dts/canaan/k210.dtsi >>> +++ b/arch/riscv/boot/dts/canaan/k210.dtsi >>> @@ -69,15 +69,9 @@ cpu1_intc: interrupt-controller { >>> >>> sram: memory@80000000 { >>> device_type = "memory"; >>> - compatible = "canaan,k210-sram"; >>> reg = <0x80000000 0x400000>, >>> <0x80400000 0x200000>, >>> <0x80600000 0x200000>; >>> - reg-names = "sram0", "sram1", "aisram"; >>> - clocks = <&sysclk K210_CLK_SRAM0>, >>> - <&sysclk K210_CLK_SRAM1>, >>> - <&sysclk K210_CLK_AI>; >>> - clock-names = "sram0", "sram1", "aisram"; >>> }; >> >> These are used by u-boot to setup the memory clocks and initialize the >> aisram. Sure the kernel actually does not use this, but to be in sync with >> u-boot DT, I would prefer keeping this as is. Right now, u-boot *and* the >> kernel work fine with both u-boot internal DT and the kernel DT. > > Right, but unfortunately that desire alone doesn't do anything about > the dtbs_check complaints. > > I guess the alternative approach of actually documenting the compatible > would be more palatable? Yes, I think so. That would allow keeping the fields without the DTB build warnings. > > Thanks, > Conor. > > > -- Damien Le Moal Western Digital Research