From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f41.google.com (mail-ej1-f41.google.com [209.85.218.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E56BF219A93 for ; Thu, 9 Jan 2025 14:12:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736431977; cv=none; b=JKE+HbMtei6yYU/mBp6sCMJBdOvSOkw/e0X24j09yi1f1UhHux+9pd9QflKaaVS0N6bc8fkUAnaqGziqoLo34Y4LJUCoNUOz9l/VKbw2supiT8q379kIWcBen4DmVzpn+rvTmHF3DZhkfI6IwfjfeOaMSa73pSZmE3edoIr8leM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736431977; c=relaxed/simple; bh=2+kPAr0dvmHCcJWjxbEc5aC2Udav9z+QVmz7IcF6XTE=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=EWEYnyc+r1T0ED95dAjZb3ZvSVCNqyqy86UjBRaVL0HoeIh1eXKlPEFdaTjqAWF01fMvKBBXFMx8IIqMY2XjN+uMJb94UB0c7qfVsk25Wb0ATcqIE8lMY2bzrghj9m/sjHf5Dwbczvqw3Q/KKyCQ5i97kgbVrI9ORKdk+yZkIQA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=Yzv2soNm; arc=none smtp.client-ip=209.85.218.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="Yzv2soNm" Received: by mail-ej1-f41.google.com with SMTP id a640c23a62f3a-aa6c0dbce1fso138747166b.2 for ; Thu, 09 Jan 2025 06:12:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1736431974; x=1737036774; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:from:to:cc:subject:date:message-id:reply-to; bh=wlBu6Jl4eyQAszKaCyX1ruIoK5I/dtBYkp0OS6r+WLA=; b=Yzv2soNm0uBp2fUxOQWj9AGYLawzBswwdFnZVZg4SwuEFqtvM/USr9J/LJEg6EMwMC 2pFmK4gO5Gc9C9JgBdok3FICOHteEI+4lUwltS9wttnV+YW5vN5yMKQ6PHFy/FaRE2JN qfgVjHJ3mZp3BPlkiRiZwffeiY/vFDR1GS781O6u4tArk+cZ7pdc93YCx3KJpFxTVmY5 ABmlcLZcpfFqmt0x9Jr4L0kXVN5DmEHrHjOd/Uv3AXXWIHHI2xzZuHmuwyTHQjb9tfNG lv4olp0Lr7hwMQVdLgBL9CskOOqRyyPYo5i/Wkb21Ms8aAq8u3b7Qs5u7R/p6AEA9V0X bPyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736431974; x=1737036774; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=wlBu6Jl4eyQAszKaCyX1ruIoK5I/dtBYkp0OS6r+WLA=; b=nFnQxHRBo04jZJp6A78RCB3dhFCHrYPZtar87HwOFWMWiehzGW8dQko1ytAzNfgVJ1 Vx2EyQvIW/AvaEg6XQQqCpmnbblPvHOdFnNwLLMxiJ4Oyg9+K5holYc/KUj9/MEIhiCn FEhH0esMYxcq+wyF0ICAzTtR/URn/RmPV9dCdJw6B5LTGvpC5QaeqwWd/m1YswHoo4yz 4PgNUgm2qbb8dK4CXn6i26WDrgujDnIzEXD/exomr6wpXZ8K7fys1lOkWoHvKzFCAdU/ zToixd64TIo9c50LQIURJ5HjJ5KWE4b/9ACSypx4awMztQ+QnQSt/dmBuHa9r6M4RKwU X0mA== X-Forwarded-Encrypted: i=1; AJvYcCXg0qlnWNV23FNwmIrJq07MgMVw7fuas7ecVhKU7oAZONbzTMeLzGuka8MfdcG2PC/mrB+htqRxTR1q@vger.kernel.org X-Gm-Message-State: AOJu0YyfiYdOHHPIlrclxkFzzdpKNxv2yblhICKhSgNLG04F0tUlvNuV Fv7BJ3cy6DWvX1TeCuIEX5Y9S180ycKshrm+Dp1BbzVdLNOwptKO6k6zRkUb/qg= X-Gm-Gg: ASbGncv8cKvH/YAU6FTMDI0UTx3j4uAzUiOYUxeYfnlTZhjziGHHNRJsWLBq7zB6LG4 AMgiSHNTeAbB2dbLQAC7/Hpv/fs8I8Zq+SeMISqe5klmV6vQT4ym0ZWdNWbxBp6gqZf8Otbt20e eNMf93lSmkKMqcC9PO+Akn3UVaUP6+mPPS8FNZ4PH8CpCHccgr5vgmtJykwHyB09y5pfq+zzRXd cutj+ryMx5Ysyl8YwVwR9aMVJY0bUKai4K+Mrn+GKLRe97HNwno8PNROPOvwW5/2qW9Pia3I2oJ 9mKqJjAeDpR9llZjA8zuLHbMfsSO8g== X-Google-Smtp-Source: AGHT+IEOXSEbBA1axGYMcra+RQ2TbWnh4xp8zHitSs1AcfUx4imSb6e/9u6/B80hfC1lQxS7w4XDzw== X-Received: by 2002:a05:6402:530f:b0:5d1:2377:5af3 with SMTP id 4fb4d7f45d1cf-5d972e00027mr14520603a12.5.1736431974259; Thu, 09 Jan 2025 06:12:54 -0800 (PST) Received: from localhost (host-79-40-232-186.business.telecomitalia.it. [79.40.232.186]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-ab2c90e8888sm75783566b.80.2025.01.09.06.12.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jan 2025 06:12:53 -0800 (PST) From: Andrea della Porta X-Google-Original-From: Andrea della Porta Date: Thu, 9 Jan 2025 15:13:42 +0100 To: Andrea della Porta Cc: Rob Herring , Michael Turquette , Stephen Boyd , Krzysztof Kozlowski , Conor Dooley , Florian Fainelli , Broadcom internal kernel review list , Lorenzo Pieralisi , Krzysztof Wilczynski , Manivannan Sadhasivam , Bjorn Helgaas , Linus Walleij , Catalin Marinas , Will Deacon , Bartosz Golaszewski , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Greg Kroah-Hartman , Saravana Kannan , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linux-gpio@vger.kernel.org, Masahiro Yamada , Stefan Wahren , Herve Codina , Luca Ceresoli , Thomas Petazzoni , Andrew Lunn Subject: Re: [PATCH v5 08/10] misc: rp1: RaspberryPi RP1 misc driver Message-ID: References: <28fe72eec1c08781770cee65032bb10a6d5994a9.1733136811.git.andrea.porta@suse.com> <20241210224837.GA702616-robh@kernel.org> Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Hi Rob, On 15:08 Mon 16 Dec , Andrea della Porta wrote: > Hi Rob, > > On 16:48 Tue 10 Dec , Rob Herring wrote: > > On Mon, Dec 02, 2024 at 12:19:32PM +0100, Andrea della Porta wrote: > > > The RaspberryPi RP1 is a PCI multi function device containing > > > peripherals ranging from Ethernet to USB controller, I2C, SPI > > > and others. ... > > > +#define RP1_INT_ADC_FIFO 52 > > > +#define RP1_INT_PCIE_OUT 53 > > > +#define RP1_INT_SPI6 54 > > > +#define RP1_INT_SPI7 55 > > > +#define RP1_INT_SPI8 56 > > > +#define RP1_INT_SYSCFG 58 > > > +#define RP1_INT_CLOCKS_DEFAULT 59 > > > +#define RP1_INT_VBUSCTRL 60 > > > +#define RP1_INT_PROC_MISC 57 > > > > Why all these defines which will never be used because they come from > > DT? > > > > Right, those defines where originally designed to be included from dts, but > previous discussion deemed interrupt numbers to be hardcoded instead of being > specified as mnemonics. In the driver source code I just use RP1_INT_END as the > number of interrupts but I thought that the specific interrupt numbers should > be documented in some way or another. Since no one is currently referencing > those defines, would it be better to just turn those in a multiline comment > just to describe them in a more compact form? So, here's a couple of proposals about the interrupt defines: - since they were banned from devicetree, and are not used anywhere in the code, turn them into a (admittedly long) multiline comment, so they are still at least documented - since they were banned from devicetree, and are not use anywhere in the code, just drop them, we don't currently need them after all Not sure what's the best way here, anyone can advise? Many thanks, Andrea