From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f54.google.com (mail-wm1-f54.google.com [209.85.128.54]) (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 06C481F12F6 for ; Thu, 10 Oct 2024 18:34:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728585301; cv=none; b=e/PzvsGQcfEUt5Vn7mJWwwUVoYRRBuRBmphUWWXUQgqzlycHIoDqREp08FRnQ9ckX6aO8i03+IYOxaMIqDVZjdtSLs3UHxIMafXZSgznz/sWq1g5NRr7QsCrq91LJSj2rgYlN/OhscWUapTJ80DdB9tY2lwKHOWarPZCrzgvMGg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728585301; c=relaxed/simple; bh=5H0TXKb7vdMMBfioB1w565xsNdpUuNNsw5k1MJOU+qc=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Bmzxj0FXZ9V6mwh872FfukrNJJ90PMnCtcGkuUSip9VjgIefU9z384C+Msv1SCvetVwtlw4KofpLqz6KMNegQppPivz7Q/kfoiJyUUa+KLaY4sqcIv6/tUcGelaf9jnVVCoL1eUHNAfwGMu6gJl0x7hmzschlgE7BFANbxBDnzc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=g9aNNVCY; arc=none smtp.client-ip=209.85.128.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="g9aNNVCY" Received: by mail-wm1-f54.google.com with SMTP id 5b1f17b1804b1-430576ff251so10586995e9.0 for ; Thu, 10 Oct 2024 11:34:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1728585297; x=1729190097; darn=lists.linux.dev; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ytCziD1c5I5FhCXF5EK5g0FeePvbLPZ6YQt0NDF0j9M=; b=g9aNNVCYgsAGuJL8vbUB5CRJFbok9CBkiBca/vQ4leKACEMw9GRvlHORVVZknprJV6 5KzRZHbIGGqR2Pf0sgKrZQ1Jy0cHnh8/Pkrz5bZy2ZR6XG94bONFazjYbwnySnFKaoSX 5CxDGWxOOHCdMPfG56s1KzmeH/O10/5c1NqbqHmy7oDxqCwAkNnx0gChUy8GxBHrGGcE 6RJSdki3CQxi0gr4MnT6MjFZXPwdgH1WHsj5v6xCOUK4n1YctUcI5ueVzUBD7ObS+vRJ UxZ6XBQmijkI4a1uUY6Acl1PeE0/Lw+O0VoMJWhvrksJcZAANhjShSIWkbQLq+UA+kQ6 qK9A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1728585297; x=1729190097; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=ytCziD1c5I5FhCXF5EK5g0FeePvbLPZ6YQt0NDF0j9M=; b=NK4Wy+OtcMtJXfldda2+G0fFw7vbxzHx8/PkYABdV+Z68aNalYhMn14rvBlYnaqHLK yQ1NhAfSJZpVyF1fZZoB1zAVQNhLZbTAfl+YY7SNimxfIscDiF7XACILsGTjXaE4Ul1P MeoO/GRZU6u0GqcVs7oTZ0gWyhRvREVIQardTycxjp/XXJTkEdp1L7E67DQOMKMhfrUd KTu4S76fQBRwszM5lxYr+I1d2t3hkPll3CkFvm+Ou2Ew0rlynaFL1zzeiWMQf9iEEbMK yJQ2GO8uxaSqohHr4PRyuP8hlUD9r7SyfE2XayNTvnp64OUWS3sGy+oceH+2QmjI4k+u wteQ== X-Forwarded-Encrypted: i=1; AJvYcCWzBPBMZE9Mc6LjRAqffoaXDWbeWvGeJkJ4jUF6FJmCBrG+jJCp8L9fCGta2eGCREC1sijw2mNYXeLkB600@lists.linux.dev X-Gm-Message-State: AOJu0Yz4z7C01yNqUeDb4DJDQn9uzPtZw75yCj79jMDZ1AtY4EQhEpsn i9KZJlRrg6Y87xeoEIdcroXewK4TONnf6Dl60Jo5hbN2ShsCnQqRIjjxOd7/yhk= X-Google-Smtp-Source: AGHT+IGrQtiU9KkPJvVdlsqZUQMKOCUeX11EZAgMHCeWVATX5aFzmtkTu4Efnh91V5tJdIM1vfefqw== X-Received: by 2002:a05:600c:46c9:b0:42c:c401:6d86 with SMTP id 5b1f17b1804b1-431157e56e5mr37151355e9.27.1728585297410; Thu, 10 Oct 2024 11:34:57 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4311835696esm23005225e9.37.2024.10.10.11.34.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 10 Oct 2024 11:34:56 -0700 (PDT) Date: Thu, 10 Oct 2024 21:34:53 +0300 From: Dan Carpenter To: Alex Williamson Cc: Philipp Stanner , Damien Le Moal , Niklas Cassel , Sergey Shtylyov , Basavaraj Natikar , Jiri Kosina , Benjamin Tissoires , Arnd Bergmann , Greg Kroah-Hartman , Alex Dubov , Sudarsana Kalluru , Manish Chopra , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rasesh Mody , GR-Linux-NIC-Dev@marvell.com, Igor Mitsyanko , Sergey Matyukevich , Kalle Valo , Sanjay R Mehta , Shyam Sundar S K , Jon Mason , Dave Jiang , Allen Hubbe , Bjorn Helgaas , Juergen Gross , Stefano Stabellini , Oleksandr Tyshchenko , Jaroslav Kysela , Takashi Iwai , Mario Limonciello , Chen Ni , Ricky Wu , Al Viro , Breno Leitao , Kevin Tian , Thomas Gleixner , Ilpo =?iso-8859-1?Q?J=E4rvinen?= , Mostafa Saleh , Andy Shevchenko , Hannes Reinecke , John Garry , Soumya Negi , Jason Gunthorpe , Yi Liu , "Dr. David Alan Gilbert" , Christian Brauner , Ankit Agrawal , Reinette Chatre , Eric Auger , Ye Bin , Marek =?iso-8859-1?Q?Marczykowski-G=F3recki?= , Pierre-Louis Bossart , Maarten Lankhorst , Kai Vehmanen , Peter Ujfalusi , Rui Salvaterra , Marc Zyngier , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, linux-input@vger.kernel.org, netdev@vger.kernel.org, linux-wireless@vger.kernel.org, ntb@lists.linux.dev, linux-pci@vger.kernel.org, linux-staging@lists.linux.dev, kvm@vger.kernel.org, xen-devel@lists.xenproject.org, linux-sound@vger.kernel.org Subject: Re: [RFC PATCH 13/13] Remove devres from pci_intx() Message-ID: <0990d9f9-cab9-44c2-b2e3-bd8fa556cc02@stanley.mountain> References: <20241009083519.10088-1-pstanner@redhat.com> <20241009083519.10088-14-pstanner@redhat.com> <7f624c83-115b-4045-b068-0813a18c8200@stanley.mountain> <20241010114314.296db535.alex.williamson@redhat.com> Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241010114314.296db535.alex.williamson@redhat.com> On Thu, Oct 10, 2024 at 11:43:14AM -0600, Alex Williamson wrote: > FWIW, I think pcim_intx() and pci_intx() align better to our naming > convention for devres interfaces. Would it be sufficient if pci_intx() > triggered a WARN_ON if called for a pci_is_managed() device? Thanks, > To be honest, I also don't mind if you also just merge the patchset as-is. I was mostly just throwing out ideas. regards, dan carpenter