Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp4067863pxb; Tue, 26 Jan 2021 11:30:12 -0800 (PST) X-Google-Smtp-Source: ABdhPJx5q/9PbQ81N3X92e54M7JGjFJPd09QeF4cCqk10VQGT4TgHXYCzYKU4p9ucKjI9dgAbCMM X-Received: by 2002:a17:906:f988:: with SMTP id li8mr4565476ejb.84.1611689412661; Tue, 26 Jan 2021 11:30:12 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1611689412; cv=none; d=google.com; s=arc-20160816; b=n4GOpJ+08R7Sdou0jxgXHnQ/VYGo1jkFSBvLA+2WGt4qmzFy0Ib3s0BnwZY3HpVvab 2otcCCr4qjYrHgLn9fO082gbjJ5SWrgKDttpAdgoph+Y5Rn6rdR2ypA/Hc58+PnM9H6Y ksPl0XPrEzVFI38O/0qr1ceo01GwHohmKl6467+JhuhKbq667rkRtXuFIx+B0+ff31ij AWJGD02TZblU9d/gobsmh/b2UNvm8owk44mUq0AEn+hGxhHJ0WwLi5LT2/NRccMLhfOo yLL2hezWVhDGIDDA14l8rUnyTZce4ghZ/aam44u1XGtlUzq0lIUlrZNjmn5XpC6eVcPh g5aA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=hUvKQU0Snwdl605ZoDKkRxVXh6lpILjpBfiJhsZr2Qw=; b=cLV2tlnhF1RvrTmYDiqXOBcdY4hFTr8ho/rK8erf1oBHabfrxcPjPqYjoxlZa9Mfcz Azrkzsb7GxW3nGUVhxDhFQiVRbjfilWEqxjoL4ClzefDScNLc5BqE35caMm1ked3xhla xGtpNoS5ohAtUzG5p/nKEjoGCZAJMMOhfT3Fh9HVij4hIcz+f2Y2zE/hzq3+6oL+DVaP FHrGjNAXlaZPTfW7Rx2vLrhYOKjDk8q/U46NtiaxkdkEvMgLgeD6kVnbdrDMauOEgM31 LWmH28um28FKcdx7AHP7y1iOpLwo6QOuXckZjeRJD58UzN+ODfnrot9zUDrvI2KhxfxR GdwA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=Oj2q57S3; 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=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x21si5719039ejy.280.2021.01.26.11.29.47; Tue, 26 Jan 2021 11:30:12 -0800 (PST) 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; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=Oj2q57S3; 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=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2404115AbhAZP5G (ORCPT + 99 others); Tue, 26 Jan 2021 10:57:06 -0500 Received: from fllv0016.ext.ti.com ([198.47.19.142]:58642 "EHLO fllv0016.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404038AbhAZP4s (ORCPT ); Tue, 26 Jan 2021 10:56:48 -0500 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 10QFsqjg073040; Tue, 26 Jan 2021 09:54:52 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1611676492; bh=hUvKQU0Snwdl605ZoDKkRxVXh6lpILjpBfiJhsZr2Qw=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=Oj2q57S32ucPnNxs6Ysppalx9Zu9XTxUeJSAfqb6L+T12VaxJrw/50+VLjNuDOUTa 188yVzPdtOJ0M8AG3OzdqB6/LBKtB+ImenDgeKFvvcMr1PvAT8X43d8djLSvdkLc9o 52TGOt6vyX/WfhUbRA+G+tIhq6TjULuZx4RRyEVU= Received: from DFLE114.ent.ti.com (dfle114.ent.ti.com [10.64.6.35]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 10QFsqKd055804 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 26 Jan 2021 09:54:52 -0600 Received: from DFLE109.ent.ti.com (10.64.6.30) by DFLE114.ent.ti.com (10.64.6.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3; Tue, 26 Jan 2021 09:54:51 -0600 Received: from fllv0040.itg.ti.com (10.64.41.20) by DFLE109.ent.ti.com (10.64.6.30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3 via Frontend Transport; Tue, 26 Jan 2021 09:54:51 -0600 Received: from [10.250.35.71] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 10QFspg7120927; Tue, 26 Jan 2021 09:54:51 -0600 Subject: Re: [PATCH] dt-bindings: irqchip: Add #address-cells to PRUSS INTC To: Rob Herring CC: Marc Zyngier , Thomas Gleixner , Nishanth Menon , Lokesh Vutla , Sekhar Nori , Jan Kiszka , David Lechner , , linux-omap , linux-arm-kernel , "linux-kernel@vger.kernel.org" , Grzegorz Jaszczyk References: <20210115205819.19426-1-s-anna@ti.com> <20210126000443.GA1223706@robh.at.kernel.org> <8f4a47f8-18dc-cb73-10db-033e5e5adb25@ti.com> From: Suman Anna Message-ID: <23225695-57ea-f255-798b-17cf6962e543@ti.com> Date: Tue, 26 Jan 2021 09:54:46 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Rob, On 1/25/21 8:47 PM, Rob Herring wrote: > On Mon, Jan 25, 2021 at 6:16 PM Suman Anna wrote: >> >> Hi Rob, >> >> On 1/25/21 6:04 PM, Rob Herring wrote: >>> On Fri, Jan 15, 2021 at 02:58:19PM -0600, Suman Anna wrote: >>>> The '#address-cells' property looks to be a required property for >>>> interrupt controller nodes as indicated by a warning message seen >>>> when building dtbs with W=2. Adding the property to the PRUSS INTC >>>> dts nodes though fails the dtbs_check. Add this property to the >>>> PRUSS INTC binding to make it compliant with both dtbs_check and >>>> building dtbs. >>>> >>>> Signed-off-by: Suman Anna >>>> --- >>>> Hi Rob, >>>> >>>> This patch is also part of our effort to get rid of the warnings seen >>>> around interrupt providers on TI K3 dtbs [1]. I needed this in the PRUSS >>>> INTC bindings to not get a warning with dtbs_check while also ensuring >>>> no warnings while building dtbs with W=2. >>>> >>>> I would have expected the '#address-cells' requirement to be inherited >>>> automatically. And looking through the schema files, I actually do not >>>> see the interrupt-controller.yaml included automatically anywhere. You >>>> had asked us to drop the inclusion in this binding in our first version >>>> with YAML [3]. Am I missing something, and how do we ensure that this >>>> is enforced automatically for everyone? >>> >>> interrupt-controller.yaml is applied to any node named >>> 'interrupt-controller'. More generally, if 'compatible' is not present, >>> then we look at $nodename for the default 'select'. In your case, you >>> didn't name the node appropriately. >> >> Thanks for the clarification. Yeah, I didn't add anything specifically, since >> the expectation is interrupt-controller. Should I be adding that to this binding? > > No, either interrupt-controller.yaml needs to learn a new node name or > your node names need to be fixed. I prefer the latter, but if you have > more than 1 and don't have a unit-address (and in turn a 'reg' prop) > we'd have to do the former. How are the interrupts controllers > accessed if there's no way to address them? The PRUSS INTC will always have a unit-address, so we won't have the issues with having to maintain unique names. All my examples already have the nodes in the form 'interrupt-controller@'. Anyway, I will drop this patch, and post a new patch adding the $nodename to the binding. > >> >>> >>> We can't check this in interrupt-controller.yaml because #address-cells >>> is not always 0. GICv3 is one notable exception. >>> >>>> >>>> regards >>>> Suman >>>> >>>> [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210115083003.27387-1-lokeshvutla@ti.com/ >>> >>> I've commented on this thread now in regards to #address-cells. >> >> I suppose I still need this patch to be defined to unblock the ICSSG nodes >> getting accepted by our dts maintainer. Care to give your Reviewed-by for the >> change? Or I can spin a v2 with $nodename added as well if that's needed too. > > No, I don't think you have to add #address-cells. We need to fix the > warning in dtc. Thank you for clarifying this. regards Suman