Message ID | 20230814071053.5459-1-eugen.hristev@collabora.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b824:0:b0:3f2:4152:657d with SMTP id z4csp2591966vqi; Mon, 14 Aug 2023 01:09:40 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEQbPkxnIc0FNAUUg+4FQA7CA3ioTVxt6y+WURzveM6fHbgW62XjgnUdsMXmYoS5VJWuysd X-Received: by 2002:a17:902:e805:b0:1b6:8a99:4979 with SMTP id u5-20020a170902e80500b001b68a994979mr8344814plg.22.1692000580214; Mon, 14 Aug 2023 01:09:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1692000580; cv=none; d=google.com; s=arc-20160816; b=x8osI9GPEeSlBfFX1IomvXYd+qZZeITzO5KGsNKANMem0JBz2KAv0jwj0xEX8PZtG3 1VkZWHTQyjckgSoGabr2Hd/kGTU6BsQkfUzDpJwOEsLlx5d1kBzI7iban6JRN0dbR4PR Ya3HznDGk9KrCXYrkX+n18PQ3ItXJBDj4D0+/OsS8532N/wL3fpspXXfNm3XAOXDamLk n2HMmdZUB2RRB9jIV2SGHN/mkBtIp+JcMm6B926VXHMmeI25HLnBani+XWvQ4B2S3kov esa3GPnxz6k0rggKnbcVWVpvXTzshDO/LAaboq2+UErZAX9WYx48GqFJQygOUw/uMQmr Z6ow== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=ScMc7QngLstpgVAhhM6RUzyc9XkcPLvVT78mgVYvIBI=; fh=ssBu4pkfFqbjAqyhKUVXSFETZPXDQIWnujp78SL5tUk=; b=ZeGOsDHWBFf/n65TW3VmG1q5ucKZbMfqZKe/6b+UDZ3urVLlTzx/v3syDlhQ8iIHsg neSWjP/X3hZUvNqvTQmqQGpGpYOiFCxEH6CMbRjuFe14CC72aBBRLHI+W88eqWrmOxPL aZrpOZGsKTRDYjXIzjT6CKlU298chQQqEYDXgmOmj/9miS4gH5zGw/zrVXMPipd5So2Z f2wf+wHo3VlQUYVoJquGgNXv+UAdjzG4NisvtAawjmbWbW9yKDV+XB3jqfAn1aXN/tO3 yxqvAJGRkKyVS3Xs3gkaP8CsHa15c9tEe4DYcSQOFesVJUe9STwMR10lSDeKvfjLadxK BhoA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b="Lh/S4ftR"; 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: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c4-20020a170902f30400b001ab1d1a6802si7293194ple.1.2023.08.14.01.09.27; Mon, 14 Aug 2023 01:09:40 -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="Lh/S4ftR"; 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 S233869AbjHNHLc (ORCPT <rfc822;274620705z@gmail.com> + 99 others); Mon, 14 Aug 2023 03:11:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58724 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234057AbjHNHLI (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 14 Aug 2023 03:11:08 -0400 Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 42C59E7E; Mon, 14 Aug 2023 00:11:04 -0700 (PDT) Received: from eugen-station.. (unknown [82.77.136.132]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: ehristev) by madras.collabora.co.uk (Postfix) with ESMTPSA id A7CAE66071A3; Mon, 14 Aug 2023 08:11:01 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1691997063; bh=KaHF1lu9ki5nM7dq9OdNk5QTWGSREzV+dbzYiF0kfQs=; h=From:To:Cc:Subject:Date:From; b=Lh/S4ftR+nP+CCpsb+ioiENM+dlFGA4xZOaCjxyACXSiqhZLm7SR5lvdxMlh5i1v1 +Kp4mCia6yTn1WUwxDPzqbR5zhRSSZvJFa5ZgQhWvAYfUVv00BnVAo4eBzvrWZ0psB plQpCqi5TVe+rlVMjK8GfNiLfvULeZ3D3jVTba+OdwIa1pnYmjrkbfxuhkh/7NwT9p 5R4NJcc89sO9BLIYEYpqrzxHx29ZNSL++U2sB/7b6ZK+KSwyPWC2Yx1O85S9wGHpe1 4ceJwgPKveHAGxdxEZ25mVOZkGyAT+Yy/lNZu543VLRvwM+juL48l3FgbD1+4IXVe6 aC49Q6tqDWB5g== From: Eugen Hristev <eugen.hristev@collabora.com> To: linux-mediatek@lists.infradead.org Cc: krzysztof.kozlowski+dt@linaro.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, matthias.bgg@gmail.com, angelogioacchino.delregno@collabora.com, kernel@collabora.com, hsinyi@chromium.org, Eugen Hristev <eugen.hristev@collabora.com> Subject: [PATCH] arm64: dts: mediatek: mt8183-kukui-jacuzzi: fix dsi unnecessary cells properties Date: Mon, 14 Aug 2023 10:10:53 +0300 Message-Id: <20230814071053.5459-1-eugen.hristev@collabora.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED, SPF_HELO_NONE,SPF_PASS 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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1774191200476420284 X-GMAIL-MSGID: 1774191200476420284 |
Series |
arm64: dts: mediatek: mt8183-kukui-jacuzzi: fix dsi unnecessary cells properties
|
|
Commit Message
Eugen Hristev
Aug. 14, 2023, 7:10 a.m. UTC
dtbs_check throws a warning at the dsi node:
Warning (avoid_unnecessary_addr_size): /soc/dsi@14014000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
Other DTS have a panel child node with a reg, so the parent dtsi
must have the address-cells and size-cells, however this specific DT
has the panel removed, but not the cells, hence the warning above.
If panel is deleted then the cells must also be deleted since they are
tied together, as the child node in this DT does not have a reg.
Fixes: cabc71b08eb5 ("arm64: dts: mt8183: Add kukui-jacuzzi-damu board")
Signed-off-by: Eugen Hristev <eugen.hristev@collabora.com>
---
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi | 2 ++
1 file changed, 2 insertions(+)
Comments
Il 14/08/23 09:10, Eugen Hristev ha scritto: > dtbs_check throws a warning at the dsi node: > Warning (avoid_unnecessary_addr_size): /soc/dsi@14014000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property > > Other DTS have a panel child node with a reg, so the parent dtsi > must have the address-cells and size-cells, however this specific DT > has the panel removed, but not the cells, hence the warning above. > > If panel is deleted then the cells must also be deleted since they are > tied together, as the child node in this DT does not have a reg. > > Fixes: cabc71b08eb5 ("arm64: dts: mt8183: Add kukui-jacuzzi-damu board") > Signed-off-by: Eugen Hristev <eugen.hristev@collabora.com> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
On Mon, 14 Aug 2023 10:10:53 +0300, Eugen Hristev wrote: > dtbs_check throws a warning at the dsi node: > Warning (avoid_unnecessary_addr_size): /soc/dsi@14014000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property > > Other DTS have a panel child node with a reg, so the parent dtsi > must have the address-cells and size-cells, however this specific DT > has the panel removed, but not the cells, hence the warning above. > > [...] Applied, thanks! [1/1] arm64: dts: mediatek: mt8183-kukui-jacuzzi: fix dsi unnecessary cells properties commit: 4b66a34afe29d991155081b2f1e10482bd00cbaa Best regards,
On 29/11/2023 14:27, AngeloGioacchino Del Regno wrote: > > On Mon, 14 Aug 2023 10:10:53 +0300, Eugen Hristev wrote: >> dtbs_check throws a warning at the dsi node: >> Warning (avoid_unnecessary_addr_size): /soc/dsi@14014000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property >> >> Other DTS have a panel child node with a reg, so the parent dtsi >> must have the address-cells and size-cells, however this specific DT >> has the panel removed, but not the cells, hence the warning above. >> >> [...] > > Applied, thanks! > > [1/1] arm64: dts: mediatek: mt8183-kukui-jacuzzi: fix dsi unnecessary cells properties > commit: 4b66a34afe29d991155081b2f1e10482bd00cbaa > > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > You know, your SoB implies review. Unless you want to say you usually apply code without reviewing it... Best regards, Krzysztof
Il 30/11/23 10:01, Krzysztof Kozlowski ha scritto: > On 29/11/2023 14:27, AngeloGioacchino Del Regno wrote: >> >> On Mon, 14 Aug 2023 10:10:53 +0300, Eugen Hristev wrote: >>> dtbs_check throws a warning at the dsi node: >>> Warning (avoid_unnecessary_addr_size): /soc/dsi@14014000: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property >>> >>> Other DTS have a panel child node with a reg, so the parent dtsi >>> must have the address-cells and size-cells, however this specific DT >>> has the panel removed, but not the cells, hence the warning above. >>> >>> [...] >> >> Applied, thanks! >> >> [1/1] arm64: dts: mediatek: mt8183-kukui-jacuzzi: fix dsi unnecessary cells properties >> commit: 4b66a34afe29d991155081b2f1e10482bd00cbaa >> > >> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> >> > > You know, your SoB implies review. Unless you want to say you > usually apply code without reviewing it... > Yeah, of course it does imply review ... but if there's no actual problem with me also sending a R-b tag, it's more practical for me, because: 1. I have mail filters to show me what I reviewed, and 2. Matthias also can see what I reviewed.... ...but I get it that in the specific case of *this* patch, I've sent a R-b just before applying, so I agree in that this was actually really really useless to do. so.. whoops :-) Thanks, btw! Angelo
diff --git a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi index bf97b60ae4d1..649477af2f41 100644 --- a/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi.dtsi @@ -91,6 +91,8 @@ cros_ec_pwm: pwm { &dsi0 { status = "okay"; + /delete-property/#size-cells; + /delete-property/#address-cells; /delete-node/panel@0; ports { port {