Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp30341153rwd; Thu, 6 Jul 2023 04:40:55 -0700 (PDT) X-Google-Smtp-Source: APBJJlF1tj4fUmrPj55w7GDMuLSbKhNXL1ot4/iqZwbTIzha7kLNkIHC6VSnhDcTiUCgyqVG04MJ X-Received: by 2002:aca:2b04:0:b0:3a3:6ae3:bd9 with SMTP id i4-20020aca2b04000000b003a36ae30bd9mr1330763oik.55.1688643655426; Thu, 06 Jul 2023 04:40:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1688643655; cv=none; d=google.com; s=arc-20160816; b=PcCS/FDjy+yT6Ou7XLhnSppbeXcs2HX38I0Z5uLtOz1HgJkjLSAkC3IGlISQEG2tjC sTZ6/U0IBYoiqhuJh5i2n5wGeYW7Es7RIKDK4TdYJ2ahW3DXYmA7bz5bcVvnNym1amPz ZEOhVnUiGyhx1Hg99XaLmlQxZfU/K99NjsN8EeXBNno0lfJk/eQdfm+OuDbCwDr5r1DP 4zHltShDN/TZc62U6yl4gzrxvok81/rF0pC/3HK3dPsLyMxwYUqXnYOha4wJl4bj1Ql+ xnXGgXnR9bKJhtdXR3wjORiwzvcBczRVpRv7Wc33XQHC5I47MctMC6Ay2TD5ZeC5oPDd cQ+Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent :content-transfer-encoding:references:in-reply-to:date:cc:to:from :subject:message-id:dkim-signature; bh=l5+LL5sG+6oNk98vidsTUD1ZT1xF0tRZN4EIfi2Jzz8=; fh=HF2VCH2+THNERrhF5zKWYHSre7rwZF8l1VrurJgXhZk=; b=0y3rLn/MzZgKfhxE8MXHzeS2mc/22uLYyZr8mIZS29Y0oJajsS6ByiscQYi/lh+7Rg I5R0MvvYwDTjZL+DTj0tG2lsZEKsASRZR37Wy/QF3RxHvkx/Kiy7Qa3BQ2Vkgi5RRj7Y DAHU2JUHpzXwFmhiK/2vLWa9pGbl6RKwwQLz/IzNumin7oNpmh3PPT9iIbRKwahYNrus OyxGTjAFqJIbPzWqAS2opeApZmslBKTz7RCdwg3GVwFF8qWmtoKV5AGjGSj1sD1JUEjW 4bg75GYh/y8izE8ac5Z6h8AjqJs0ciTYUJHBSLGU/FgRnrYxkRSki3a8iJ/wVEEsXLXK 2SDQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=KInwh3I9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id l11-20020a63f30b000000b005573b447c40si1300500pgh.174.2023.07.06.04.40.41; Thu, 06 Jul 2023 04:40:55 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=KInwh3I9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232108AbjGFLdN (ORCPT + 99 others); Thu, 6 Jul 2023 07:33:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37132 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231833AbjGFLdL (ORCPT ); Thu, 6 Jul 2023 07:33:11 -0400 Received: from madras.collabora.co.uk (madras.collabora.co.uk [46.235.227.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 77A23AA; Thu, 6 Jul 2023 04:33:10 -0700 (PDT) Received: from [IPv6:2a00:23c8:b70a:ae01:35d0:8e34:613c:c44e] (unknown [IPv6:2a00:23c8:b70a:ae01:35d0:8e34:613c:c44e]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: obbardc) by madras.collabora.co.uk (Postfix) with ESMTPSA id B0B466602242; Thu, 6 Jul 2023 12:33:08 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1688643188; bh=l5+LL5sG+6oNk98vidsTUD1ZT1xF0tRZN4EIfi2Jzz8=; h=Subject:From:To:Cc:Date:In-Reply-To:References:From; b=KInwh3I9qZsDTaBRQW8F/t2QGDZ/ofjWuirTgSfnjvf1QBmRrbOhTjY3xbvIQSbpX Pit1IhOCJ2snhlQMmw0aDhv+7mrD/F8ov/0fLaHolXyNQmXdP1Oz7wjPeNxtDucp9A bssu7N7XnEgKC2b/y3ocQaS6HS8MwwCz5W4FdpPBBNo/KxmkXtq3B7u6EBtuDgaCGC B+ft4lFfJSpFsvqJkhNQ5BS0KMpucyYCvZjkvObuLrJcpWi1VI2c+QtHWVjutoWLHq uetic+AXTYY95fYRBXaW9giOQPHdJAf1A1tr/NgWWHrTnHbldG02pyk9wGhQ8lKBGe x/XNpt32UQfGg== Message-ID: <7fa80568e37cca3b8c4560e033cd2cfc18b2eb27.camel@collabora.com> Subject: Re: [PATCH v1 3/3] arm64: dts: rockchip: Add Radxa ROCK 4SE From: Christopher Obbard To: Krzysztof Kozlowski , linux-rockchip@lists.infradead.org Cc: kernel@collabora.com, Andy Yan , Chris Morgan , Conor Dooley , Heiko Stuebner , Jagan Teki , Krzysztof Kozlowski , Maya Matuszczyk , Rob Herring , Sebastian Reichel , Tianling Shen , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Date: Thu, 06 Jul 2023 12:33:06 +0100 In-Reply-To: References: <20230705141011.111568-1-chris.obbard@collabora.com> <20230705141011.111568-4-chris.obbard@collabora.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.48.4-1 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Krzysztof, On Wed, 2023-07-05 at 17:53 +0200, Krzysztof Kozlowski wrote: > On 05/07/2023 16:10, Christopher Obbard wrote: > > Add board-specific devicetree file for the RK3399T-based Radxa ROCK 4SE > > board. This board offers similar peripherals in a similar form-factor t= o > > the existing ROCK Pi 4B but uses the cost-optimised RK3399T processor > > (which has different OPP table than the RK3399) and other minimal hardw= are > > changes. > >=20 > > Signed-off-by: Christopher Obbard > > --- > >=20 >=20 >=20 > > + > > +&sdio0 { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0status =3D "okay"; > > + > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0brcmf: wifi@1 { > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0compatible =3D "brcm,bcm4329-fmac"; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0reg =3D <1>; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0interrupt-parent =3D <&gpio0>; > > +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0interrupts =3D ; >=20 > I just send a patch to fix this in some files, so let's try keep it > good. Use proper interrupt type, not none. Thank you for your review. I missed that patch, sent only a few hours befor= e mine. I will address this in v2. Cheers! Chris