From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f179.google.com (mail-qk1-f179.google.com [209.85.222.179]) (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 1E98930D3F2 for ; Fri, 7 Aug 2026 19:41:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.179 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786131666; cv=none; b=uGcx+DF9B3vY2TU/YJeqYyewDJ0oIJTmNEAJklv3IT5gJHuhgI2cVBq6VyrPcIMsY6+Ovv+hFZDDTdTmArDdP5uCHPlu7fgq/4zgzi6El78kSbhyJwprZ3iUhCRFpAT320SCM3g6CXHPOHxhKOJa2IoGUvY6EVPluRFbA6T9c9Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786131666; c=relaxed/simple; bh=pIxz8es0l7NHVt4rPf5JLNwenGKE6L8ikfko/REfnio=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=PRNb7H2aWpihuh4HtWIaeiNzi2xZsaMK2IWLh1LKZzKdtCVsXynt+bvIEpZLKaQ2tkYY8wvnPmJHN4MW82hxJkcHaEtqIKn+Ky7sbJZMT11aMQ0AsuzZ9ihj+12Mn/i6t6hIuTcCnWahan5Lb1nzFe0AzXHqZ9imjtc+2ggdfz4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=riscstar.com; spf=pass smtp.mailfrom=riscstar.com; dkim=pass (2048-bit key) header.d=riscstar-com.20251104.gappssmtp.com header.i=@riscstar-com.20251104.gappssmtp.com header.b=M8lVAZcD; arc=none smtp.client-ip=209.85.222.179 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=riscstar.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=riscstar.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=riscstar-com.20251104.gappssmtp.com header.i=@riscstar-com.20251104.gappssmtp.com header.b="M8lVAZcD" Received: by mail-qk1-f179.google.com with SMTP id af79cd13be357-92e6a434cabso181265785a.1 for ; Fri, 07 Aug 2026 12:41:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=riscstar-com.20251104.gappssmtp.com; s=20251104; t=1786131664; x=1786736464; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to:content-type; bh=aYmF0A8uqvDVQZglorDtEEMVhUonz1Kois8AmVRMQns=; b=M8lVAZcDS8Jdl8lTPm+3L+9GwgI94QLYOIrWCtHJo59erfRfZOQESsBwQ0oj2VqgyA NqRarJRQHmIoygGgscWH9XsFKfM2WHzWMGzMLPjelLWVSatxSJOWSoc/EKgUCcKwcevY ZY7kV9z4ZBU+IQ7VHrlMzXKq0BFYBSIGa4okaCzwind9l0eKEP9BsFt+NCDi/9CYhgkZ ONWFgqepQybUUgeIpWH5rWxkMuZaIb9B5TyL5QhAvKYq3f6spIA9tRYDomb6nZCDjHBf k1hMAD1Ak8itTVs7gHn4ggNNMtf2fqLRM04chlGfddBWI45xoG0ZjVHnfDPhIt396ecj DGpA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786131664; x=1786736464; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=aYmF0A8uqvDVQZglorDtEEMVhUonz1Kois8AmVRMQns=; b=UPMKuj4Awoexc7oe8ZlzO066XkQfC8B6f1Od1x3SjhWSKAG+MLuYLrmzDPZhFGGk0Q 4B/qDbRkCAy97NC1NvE/F56PNoLSaojAFPidDb1RhheDW5b81rn74nRqCGIyeDtT9//Y oP+7IaFS0NUlvWFPAgNQQ6T9ZfFTlBjCm5PrBnVIUAmXFboqRcpWC2VouTDKzHuFIoxM iaMfJjedNXrILtuvwBbPRMW9yCQ85qC11wsqM9ORRhuau+QMw4LVSIdV4FnLs0C0L4t4 fz/mKy1HAx3tK9xs1Hyb0d9Vx1MXUpJQhzNHTwHC0x8Nx8WdoKtDtPLthGwKVXIQucVj qYrw== X-Forwarded-Encrypted: i=1; AHgh+RqvEoV6cW9cOO6obcFEM+OeYrgJF9NDOm2h0GrJJdcoFkR+2Wo9pWYyNjsaW7wMKj4A0xMIPrk+k8bTEFE=@vger.kernel.org X-Gm-Message-State: AOJu0YyYLbhFwh+DYCVrtnv8A2GR2NS6iGxA/8PU41Rlz9y3KZELMTjI 3dbvtD/1HDAY2n4e0vlQvONno9xaPGst3k71Wwtz8ALUDj2KldI45WvXZJ1XwZrogdI= X-Gm-Gg: AR+sD11OBRrFlt7HbhMFwTnTP4a+R40rLaCSL0aPU+HdpC1e0JuAgG3jK8f223MLTCV kzoTnWG2GAoxVKG+18RCm5ccNI2kH8fy6cAKMDN2mxG1jz7bt/n3mf7yVGzEoKRyewmInHrvaHk 8GCQia0MuQUmuSmHManTpLSyxRgEXhQw69vxQ24YeOP7WHt9ERSaeFGNKQSVMVyL5oGFmloWlaU csqEJ/dJ/KrEzi3kkEgf3EhnHYVu4lFFsgRugzJvaMbQeYYbyXEVGpRlEvpgTjz5ajEJxb9jUua OQfNt05no37ZwNTXATcMWBcQji48khvWRe7jOit6e/E30GoP765HOVFrm30W4qTWoRmD/hrqGss o0vCC5wOcG95aB3Vo63QZNQT0YFbYgxZ/0nJdIP8DmOW/jldEsizc+YSedsbwjZssSWgs/AJc1m 1AR3uxcgqwsGD2lmXzZ7TioBKkUP6nOi5izCJ2VXjQeiwCv7+HCP7cYLz/lT4AlREf X-Received: by 2002:a05:620a:7001:b0:915:a111:86ae with SMTP id af79cd13be357-936798a4868mr319767585a.35.1786131663991; Fri, 07 Aug 2026 12:41:03 -0700 (PDT) Received: from zippy.localdomain ([73.62.185.64]) by smtp.gmail.com with ESMTPSA id af79cd13be357-9366e03aeadsm219786585a.3.2026.08.07.12.41.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 07 Aug 2026 12:41:03 -0700 (PDT) From: Alex Elder To: bhelgaas@google.com, robh@kernel.org Cc: daniel@riscstar.com, mohd.anwar@oss.qualcomm.com, lorenzo.bianconi@oss.qualcomm.com, linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 0/4] PCI: of: warn on bogus device_type property Date: Fri, 7 Aug 2026 14:40:54 -0500 Message-ID: <20260807194100.455599-1-elder@riscstar.com> X-Mailer: git-send-email 2.53.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The purpose of this series lies in its final patch, where a new check is added whenever a PCI devicetree node is found to already exist when one might otherwise be dynamically created. PCI has a well-defined bus and device discovery process. The PCI_DYNAMIC_OF_NODES Kconfig option allows PCI devices to *also* have a devicetree node. This enables certain things that are not possible with PCI enumeration alone. While working on a Qualcomm platform, I learned that some PCI endpoint nodes were defined with device_type = "pci" properties. Herve Codina pointed out that this was not correct. Rob Herring indicated that people seem to have trouble getting the PCI devicetree nodes right, and asked whether we could warn if this particular problem occurred. The last patch in this series implements that check and warning. The first three patches are fairly trivial changes to clean up some related code. -Alex Alex Elder (4): PCI: of: drop the reg_num argument to of_pci_set_address() PCI: of: don't zero flags in of_pci_get_addr_flags() PCI: of: make a flags argument optional PCI: of: introduce of_pci_verify_node() drivers/pci/bus.c | 1 + drivers/pci/of.c | 27 +++++++++++++++++++++++++++ drivers/pci/of_property.c | 20 ++++++++++---------- drivers/pci/pci.h | 2 ++ 4 files changed, 40 insertions(+), 10 deletions(-) base-commit: 075b74841bd0065a3bda3440873c747938e69b68 -- 2.53.0