From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) (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 886471ACEC4 for ; Mon, 30 Dec 2024 18:22:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.173 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1735582932; cv=none; b=D0vld61dVl9E7ieWY9I8IzmDAkTtyPBEWRC8bKhXhBJWhfGn6sDqgZeohC/vl6LDmzK2MRNMsFfRDS5HbcWhyl6pp9AkWG8uRGu9TBFOjRSmNUK+NxKyfV1ThoXlLkhnYzx7tLRqXDntbFb1pfgq5LBfXnA3CY0UphN+W/D2tBo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1735582932; c=relaxed/simple; bh=ohF53bAlZOK876b57Tr3oe/521IXGf1Fjs/51DFutQE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=S2w3cxrSwWKmBZljOHNWPLeaNikWymLxznJRWR7ghvhpdnBhPKVuR10oF1T+ItxF8ArsJMBMWU6aUeoVqN6n8V0O0ZDVBHNSUTpO9iONN10/fLSYZBvdTPq9tXjyvW2sPWt0gZqln1SZEx4wZ69IcG1DerRqwoC18R9628a6dRw= 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=o3JIZJ5t; arc=none smtp.client-ip=209.85.214.173 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="o3JIZJ5t" Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-2156e078563so107646905ad.2 for ; Mon, 30 Dec 2024 10:22:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1735582930; x=1736187730; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=5whvZzYfxvB02wpstYeTdMUU94BDcz8zNaK1N90VIoo=; b=o3JIZJ5tj4Q4rthqLRFbX03w3iVVH4nzv+rERY0ULIXZ4gG/jiXrIRTeL1FsLOEoGu 7VXZD4wPN2NNCFHmz0OkqifX7GjIMwtCgvn2cTdM7N1QmXHNd5/l+p0IqE5JT2CVMHFg ZviJEQIXeLeBj9eNzvf0O/K+K54rbjAM+Ob0TtBG75xEu8D8kwDIpC8i1etDULudTomU V93U01gOB43JylOdEXcr6T0O/7cu8ZvURVyooLJn4VheinRfiVMfN8BpNMqN7CcVI36v VGqbg3GkGt5xMkwKZ7ZeqpK0z8xTa8Lo2/RSnx7KICqi0qeTtqQ/jiDIk7A8ZQ/A3F5S OJsw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1735582930; x=1736187730; h=in-reply-to:content-transfer-encoding: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=5whvZzYfxvB02wpstYeTdMUU94BDcz8zNaK1N90VIoo=; b=sNjAq+mgsGWxe0aFqyunUnjrXcpN4RXHG1AmEmecPpEBF5KKZFN1vIYTj+pT97XNq4 QivLlcoCiizRct6Z4TsN19c/QLMrkzoGV1FQ48+8LGa7w9hmTC+q5xqDWK29nazicTkg 5f62QKXVc2sDdhyuiqY45mqpha6eXjQLgZYdTixUIBdYK5KhToC7b+GHXtJXeaWk2hzN XJoYCfFyfLsMJ0wDR+rLnBUY55px++ez+kOSu6mPZBg8sdmFG9P+8Jof/ZGECRUOhY/8 D+QxgWwDIHLdYM6cDQuDLM2oW77rMBYsEgpvAqSTsO3OpGqrMo7K/+HbcFM9Pq8Z1hD/ xRnA== X-Forwarded-Encrypted: i=1; AJvYcCWUH2+dMR+OyMaln2YPqI+SBkghiPQ6YNDTMbor2u+/fRo0N870jEER/3sKJ2F+VIPuP2qqLzq6wunHyyQh@vger.kernel.org X-Gm-Message-State: AOJu0Yyx5WcQlDRwT5j8wHhbM+mpGX149WamcOQo24YmzM7WXHvA6bbF eAS6lNUhJFtPW+jjGkO4wzlXZHNopdd/QzZ/lAgsDre1N+IEJrWYcuRlkKUNfg== X-Gm-Gg: ASbGncvrkC+LsvW/AubNWvQKGYJnPzn25b8U838XXG1FcV8UUfo9kj41GmvdNl6dzmI OKW/a4VVrBWqbM+rsDOC0fizmQuEBSibFRty25lgovqqdWqTFWcklG0Qgm0tVLMSej13ODw7FKU Qc9W5PfiNU1/fC75PMf8Fxd/mD+heXXRXu1KmU+Lv20aTJpXcIJqsdxIngrdbhCdiL0S+aQLKia WHWhNfaFNZEnpY04in4/eCNVEeLURcuzdM3zj4bSIykEFl4LWEfWIjmHgKYrMt0Uw== X-Google-Smtp-Source: AGHT+IGmAZQbul0x33kNY3z5xhEmDXlGSFKtrk3n8yKah2UX0yu94Zn1yQJJuIjjti5kcSEoiVFc5A== X-Received: by 2002:a17:902:d48b:b0:216:7d22:f69 with SMTP id d9443c01a7336-219e6f262c4mr415053615ad.50.1735582929838; Mon, 30 Dec 2024 10:22:09 -0800 (PST) Received: from thinkpad ([120.60.54.70]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-842dc7edbadsm17723516a12.65.2024.12.30.10.22.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Dec 2024 10:22:09 -0800 (PST) Date: Mon, 30 Dec 2024 23:52:01 +0530 From: Manivannan Sadhasivam To: Dmitry Baryshkov Cc: Krishna Chaitanya Chundru , Rob Herring , andersson@kernel.org, Bjorn Helgaas , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Krzysztof Kozlowski , Conor Dooley , Konrad Dybcio , cros-qcom-dts-watchers@chromium.org, Jingoo Han , Bartosz Golaszewski , quic_vbadigan@quicinc.com, linux-arm-msm@vger.kernel.org, linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 1/6] dt-bindings: PCI: Add binding for qps615 Message-ID: <20241230182201.4nem2dvg4lg5vdjv@thinkpad> References: <20241112-qps615_pwr-v3-0-29a1e98aa2b0@quicinc.com> <20241112-qps615_pwr-v3-1-29a1e98aa2b0@quicinc.com> <20241115161848.GA2961450-robh@kernel.org> <74eaef67-18f2-c2a1-1b9c-ac97cefecc54@quicinc.com> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Mon, Dec 23, 2024 at 08:57:37PM +0200, Dmitry Baryshkov wrote: [...] > > This switch allows us to configure both upstream, downstream ports and > > also embedded Ethernet port which is internal to the switch. These > > properties are applicable for all of those. > > > > + > > > > + allOf: > > > > + - $ref: /schemas/pci/pci-bus.yaml# > > > > > > pci-pci-bridge.yaml is more specific and closer to what this device is. > > > > > I tried this now, I was getting warning saying the compatible > > /local/mnt/workspace/skales/kobj/Documentation/devicetree/bindings/pci/qcom,qps615.example.dtb: > > pcie@0,0: compatible: ['pci1179,0623'] does not contain items matching the > > given schema > > from schema $id: http://devicetree.org/schemas/pci/qcom,qps615.yaml# > > /local/mnt/workspace/skales/kobj/Documentation/devicetree/bindings/pci/qcom,qps615.example.dtb: > > pcie@0,0: Unevaluated properties are not allowed ('#address-cells', > > '#size-cells', 'bus-range', 'device_type', 'ranges' were unexpected) > > > > I think pci-pci-bridge is expecting the compatible string in this format > > only "pciclass,0604". > > I think the pci-pci-bridge schema requires to have "pciclass,0604" among > other compatibles. So you should be able to do something like: > > compatible = "pci1179,0623", "pciclass,0604"; > Even though a PCIe switch is supposed to be a network of PCI bridges, using PCI bridge fallback for this switch is not technically correct IMO. Mostly because, this switch requires other configurations which are not applicable to PCI bridges. So the drivers matching against the bridge compatible won't be able to use this switch. - Mani -- மணிவண்ணன் சதாசிவம்