From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f50.google.com (mail-wm1-f50.google.com [209.85.128.50]) (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 332D41805E for ; Thu, 20 Nov 2025 15:31:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763652714; cv=none; b=BbAyI80l2W0JtWGGzmsdzzIL6BtENvCEWBdFdE7YPTRUlfts+EJ4GIXvwq3kFOZVsUAsLfV+Sb6obqhiHgNL5xGRdmX7yiXkzybIy2JcZ/RAelU0v3/L0Y7qCQ8EKtVDLHqQ/11+NYFncoMszThdtJu+DK3kck5AqBNzqv3hGzI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763652714; c=relaxed/simple; bh=dtzyF3iuy8kapZsEsjKfyLmM+ZoFSdqs8IgBpnNu5ng=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=oE5IVlOHNuoIUHVKwfQd3ZYCmXeOES7zpE9DVSGJjAlEt1Z7pnA8a//4IYAyn+SnJmU/yy6eh1Hk160of+/ssMURKU4RAvw8GfyqR7Iy6e7nrgnkSOR3z5E9mzQNBrcGxFeowVr28FWEiSC09Vi5aDGgqbvm+zosoTer8Q8RHRI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=UZI46eVg; arc=none smtp.client-ip=209.85.128.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="UZI46eVg" Received: by mail-wm1-f50.google.com with SMTP id 5b1f17b1804b1-477619f8ae5so6309175e9.3 for ; Thu, 20 Nov 2025 07:31:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1763652710; x=1764257510; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:from:to:cc:subject:date:message-id:reply-to; bh=vIq/O0/AL91lguWZ3GTEHU1lL1GNwH+lI8GY9Jzrdtg=; b=UZI46eVgy0HV3PnHrmfFZx0r4pmdBmkFbVnO0PHWgZBwzyUWJi59CQQTKOQDk1Qmry E+bz0+wePiwCdQPst/Ytmnb/UV3LRgkotaDbGleszgwH9ZB6hyJnTqZatRtvKtnLnFfN IEoPyUp0RUfZVBnN3Gx21CzIei2I/bVCOJEelNnjA8BOCUpH6sX28AbMsR5nSEEjjYZt qK7Ks3GefMyticqUxMFeImLIBqrdGRLqeNkNRQoD9kxFOSMwyFCs0YDLdv7oVWgh3W87 UoW5cxFBwy83V7Xtacyu3SYVVWVQgR7WRLf//oWnkzeVIUXv0Vze5iJH7Wuc3HbQyFPG /54Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763652710; x=1764257510; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=vIq/O0/AL91lguWZ3GTEHU1lL1GNwH+lI8GY9Jzrdtg=; b=BNPdQp2/oXiUWlEoc3pHUCaP3ZkCXp+DQSGBnKqTQ0wWIdILO6IG8/eqB5iCF0RpFd NHcZaTRxoUOWVzW9ZsiCVYn+zOVvAzH7jtvF4nFvcSuU2HqZ8Wfqgawbw6T9hC4vlki9 Ngy7te+Nb3sQRmiI+2atmOLc8HxO1PsPw4zIdKqPPSkEM4/XX3dqhopTImZXaKrkqWhG 4/deAI35HeYINCXbi/oQ8mLPgxDleLa/Sh3oYQPqqS3rCaZiR5bcz538TLShgl1FzaZD i88LfBLiq5R7qo4Qs/7gOUcksdI+ybhAvXnNcWvX/m6RtvG2CX/QyIQbCmOFNkFvP1NE n0Kg== X-Forwarded-Encrypted: i=1; AJvYcCUbcQ5cbTbBCDK80ej8UrQZqSz5uR1pqhGoCeBbkeQxyJPxJeExTupOrvQlxbPZFhOZxZvYDGXB7rIaj98=@vger.kernel.org X-Gm-Message-State: AOJu0Yxira1fhBKN2ZEKO4Yyd97wIwSdkwO5y9PnuSrmtFl/Dl1u71Rb +PV+23u8pioxKkAVM2WEbFkUgPgtgQ9a/ilUaypbwKBdY42lohiYsdn7loCBvg== X-Gm-Gg: ASbGncvlZZ3dI+SWLYJnWlbSMb8YMd1cYdk3JEdjaGV0AwxNA3PuoQVLnuLFUTRDN+P HDRWhqCUVm/AAPUhvBAjuA4eFEidgJseOJvC1JwjCojVTjKFCmgV5j/0MMlzPvsU1ZTQjcL6xcm DKEhtzcrd96005qpCkxiD2njfOW4WzPuzd78z2XJZywN9NZ7PZnwjTT9FMtngxJHxh9tEkfl5U/ A5VHNmT5nhIMf+kPdJXSs+1KtqF7AfeuidV3EkFwBxJLlZM8L02lD72Zg6Aba+6QMQGZG0WwEjl ulcMUVP+KCQrgg6/O6HtWJnbSSiJpN3Rd7nDEnRhyN4Ajc42mFOVLyux5O2UP3obzR/k3O1XYdm 0hT57+vM+vU36wQ1ZDo4dyZTObAxSCAudRgO4x4bEcRrMcSwoU0/wTVBC2AvUixjAfYChEwnVXz q6a97w7gP23EpP//nUKGMGO5QrtoMDhIM8q5jKnNI= X-Google-Smtp-Source: AGHT+IFVqJA7RS7HvRm6eGAX5ub1WJX8xe85GZesCDT1y7PS7OVTND/zM2W/thezDZ6HHob9eXthYg== X-Received: by 2002:a05:600c:4f47:b0:477:8ba7:fe0a with SMTP id 5b1f17b1804b1-477bac15384mr27816695e9.24.1763652709759; Thu, 20 Nov 2025 07:31:49 -0800 (PST) Received: from Ansuel-XPS. (93-34-90-37.ip49.fastwebnet.it. [93.34.90.37]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-477b10763a9sm113819275e9.12.2025.11.20.07.31.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 20 Nov 2025 07:31:49 -0800 (PST) Message-ID: <691f3465.050a0220.105096.7667@mx.google.com> X-Google-Original-Message-ID: Date: Thu, 20 Nov 2025 16:31:47 +0100 From: Christian Marangi To: Konrad Dybcio Cc: Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] dt-bindings: arm: qcom: add ipq8064 board variants References: <20251120135435.12824-1-ansuelsmth@gmail.com> <8e75be2b-643e-4380-a018-3cb718745101@oss.qualcomm.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=us-ascii Content-Disposition: inline In-Reply-To: <8e75be2b-643e-4380-a018-3cb718745101@oss.qualcomm.com> On Thu, Nov 20, 2025 at 04:25:37PM +0100, Konrad Dybcio wrote: > On 11/20/25 2:54 PM, Christian Marangi wrote: > > Document QCOM ipq8064 board variants ipq8062, ipq8065, ipq8066, > > ipq8068, ipq8069 now matched by the QCOM cpufreq nvmem driver if > > socinfo can't derive the variant from SMEM. > > > > Suggested-by: Rob Herring > > Signed-off-by: Christian Marangi > > --- > > .../devicetree/bindings/arm/qcom.yaml | 20 +++++++++++++++++++ > > 1 file changed, 20 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml > > index 18b5ed044f9f..0eb1619fede8 100644 > > --- a/Documentation/devicetree/bindings/arm/qcom.yaml > > +++ b/Documentation/devicetree/bindings/arm/qcom.yaml > > @@ -299,12 +299,32 @@ properties: > > - qcom,ipq5424-rdp466 > > - const: qcom,ipq5424 > > > > + - items: > > + - const: qcom,ipq8062 > > + - const: qcom,ipq8064 > > Since 'items' requires that all items are present (and in this order), > we would normally have a board name go first.. but I suppose this is > some sort of a fix to the issue that sparked this (posting the link > for others to have more context) > > But since these SoCs do exist, I wouldn't say this is necessarily > wrong.. > Well we can see this as a ""template"" for device that might be added using the ipq8062 or ipq8065 compatible. When device with that variant will be added we would have to just add an enum with the real device name on top of it (as first element). Honestly I should have added these compatible long time ago as on OpenWrt we have tons of device that are ipq8062 or ipq8065 with the compatible structure "device,name", "qcom,ipq8065", "qcom,ipq8064". I feel it might be wrong to simply add to the list "qcom,ipq8065" without the "qcom,ipq8064" part. > https://lore.kernel.org/linux-arm-msm/20251105112136.371376-2-ansuelsmth@gmail.com/ > Maybe I should add a Ref: tag referencing that? -- Ansuel