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 886A01ACECE 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-2163b0c09afso120521625ad.0 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=i5weFkwL1YVsxPv4p4RkjjuEgYrfTgbSwEoHYH5asDgNUfcuKm/U/ROEBD6v0vgjcx Gwwx8dmuBleJolBzvNInLt2lUs9XeRp4gRhOGa9QehejlVRj/xFsY2Cl9PxQ04PhPxez l3rfbgfJbMFvbXo7rt7RvgV5kcGUGaBFJE2c+VOPBcdxxp5epvnc2Aegfrn41gCKyquZ vnHUBXBW55MifsrTYIkFS61t7nEOAK7yvkd9b9kvppXHurRb/VuSKVWFMtEEJl+3JfMo LLtazETubArSat2OHeDQOD+0QW6ObWslHNqSloGajUNAL8DI0GxseHZvD/wJfIa+Ciq/ fq8Q== X-Forwarded-Encrypted: i=1; AJvYcCX+tDygL6gT+Wd/Rp9Met8FS5vDdRr4QemI9XeYzKPcCDVQ1lpXtTyPIErd1es4M2Hg91Kv4ohxqCDygcc=@vger.kernel.org X-Gm-Message-State: AOJu0Yw2amaxpUqdwfArAb4nDWBQJKkax0dc32SZNW7/lyFBR9vo2xd4 oyab9IdO6/H5ULmzMnBzt/nb4ZnT3J7oZXiVQU599QiI6HpGzsodXB0RjJYk7hb2RlzWsl78qL0 = X-Gm-Gg: ASbGncse30C0NgGJT6ZD3HXJtgCnttstloh7KLBFYCCB0rnoGLT6tPsW/1+9jAoBdy/ DZaWrUKE+yzqHJo38H7fZCRmjiIW/iGvUkJUOzA4CYEpxNTGKO1ku9riMi0Q5cuxvGHienz7ukQ RknqVkRPhAlavXaktkcS+snG6SRXSeqcIYK3TnoMrgvhunbe4+wXyjp+0pAz80c1Stxd1OvE5IT KhXPJ41Uz5ET6qJaF3nEd9DVRYQmtAKTEbD5v+hL+ZU1Ckls18fSgAGhXsL0MPyEg== 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-kernel@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 -- மணிவண்ணன் சதாசிவம்