From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DFAE6283FC4 for ; Mon, 16 Jun 2025 16:18:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750090707; cv=none; b=au9JQmJJp4kmqRlVr3sjXrJPKyaBeRVjrk9pvJDPdKaCOtDkGjqVSKV4u/UklZy9hUDEbm1TBKN15eE4TyVemERakZKszajES0P79UULhgUk9YcgQ6sHRjsGfo8YJs9rPW84YsGbyI/qkz7AFqQK5uh0ijUCTDPPMUHDMBytoBU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1750090707; c=relaxed/simple; bh=5dxQ60As8s/8rmI2TuFgJPch2gZ+j1BB1ZNS7po3vgM=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Djl1jCCEF10S9cq9AkhExMRmtPd1XF5u6C+0mKXzYJqP+kjFyOhyDfLsiKNu1KgQVG7ksJBs+4r2hucwlXkMBnQNYF9E0HXYHwbAyK6Az1fvfh4zgzlJVmxSFRI8giQ4wyO7WfBg6HKC66gtVBQ6zRjRLT3rfijs0zBaqrnjpAo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=YQiRguyq; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="YQiRguyq" Received: from pps.filterd (m0279868.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 55G8lrEX005319 for ; Mon, 16 Jun 2025 16:18:25 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=qcppdkim1; bh=lenjTOuIiqQxj1snFC9JaLJ/ OZ0qD8QNh76r2FAzoEI=; b=YQiRguyq5DL80RlloScVAfpw9lTcjLnppkegCMZa XA3bRC9ogFQHNt0RhUOL4ceODzfvxjFD+qsYDyDR9S+0aXDgcR0xXljWFrqGLb4C XXJrG/LV3wDn2zt6EEFhv0eJrpT1T9+/S7oeaJ6BIMPsmW1YfIDF8FAyjXz8mtu/ cOQeBKOuYMJXOYM0Z9wCaK40w2Ueu2rS4YwrMBEFstW0GQSFzU9SjqfXigg1iUK3 QTZw4yT9ff3MdaU7YNg5I8GUPNgCKRHtCKHOMvzkgMUpEJqt6JLRvMg27m4uOycB NBZBWDuqz48C74/lMeLP2BSmHTR4cAYc+R4Jt8KR7dzDpw== Received: from mail-qk1-f197.google.com (mail-qk1-f197.google.com [209.85.222.197]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 47ag231b8b-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Mon, 16 Jun 2025 16:18:24 +0000 (GMT) Received: by mail-qk1-f197.google.com with SMTP id af79cd13be357-7c5cd0f8961so1056557885a.1 for ; Mon, 16 Jun 2025 09:18:24 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1750090704; x=1750695504; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:date:from:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=lenjTOuIiqQxj1snFC9JaLJ/OZ0qD8QNh76r2FAzoEI=; b=iVQcYq2GFqfySIfDhbRNJuIl5ENgIaUqHEqrJbSjdHyfnWP2OnC1VgctXG0t+1OVbm ba2xTksGnxVslJDoKwSKqnkQ1onnqJgTo7+V8p5aasX1H9TNReHq4gRiqYmYQvQ60uRF Gh+qR/8PKnUuZ8NFbtTs+1wH147Th6V3BggoxjQMagL7in8fSZpXuhLHZ3fHuH+Bgu6O euYLfDC8dnlfhfHyY9+L3Uq0s6DgFji6BCLHNYKeKxA1Z4M73RmElG41UJ2P1kur1w0+ 43cMA39p14EFPzON1B5MIFZDyof3tMEUcTDkvp8JrWZX5LjwT2ZkRi7FYRKjfmS0a8vE /+Mw== X-Forwarded-Encrypted: i=1; AJvYcCWRSrRJshDisOmlFS0RG9jr5UNifiokDU3FzeQOrJhK+aaL1vkve/3r4Bkjg9/Ajd8AKBlJrVPQ7/VVcls=@vger.kernel.org X-Gm-Message-State: AOJu0YzIiyRZJbIRHevv/KZAZ83eVTtUog+DOZkksXuCc2OjyAO1QwXQ zxfdZzxZvz71EP5YMPQGVfYq7INQbQL/hrT4J2ijaCLCuembae6EG4/MB8yngG5lN1trFUkAu1q TZez4pLHkugZBFAIxoYaXcFxkV/c1WSjyFSvtv6I3OcjoyFeAOkAYNHIXVw2MkHjtBfY= X-Gm-Gg: ASbGncswLz+F5ZoBc4tbzwqwNxheDgoISp3nEUvNQ7Fi5KWLZXjSwVvmTMxj4U51QLq AEyih2DsUFrwNSq4gx9syZnvGmpertcW34J4FpngA5mM2OILWM+jYxtDwT2jVNN42oDLhmg6Xjv v7WkSJp5i62za4mp96cJaTC1Osj+Q+BfkDLdFB6pG7jvkqFF7Y0oJx316/pOabgrGsp1C6an90h +Vl6ejGfLDlmx0ATVDKVbTOMm21psrBkvccGVkysjdAaa1BIMWfQbbaZJsLYMR/KvEzmQKk2RcI mZoJarpr6LFbholAPmzFJa2RCqRFfGMR495CadCZsIVdIHD/DF/Emz1Itg== X-Received: by 2002:a05:620a:450b:b0:7d3:b9b6:f1b9 with SMTP id af79cd13be357-7d3c6cd6c86mr1518729985a.26.1750090703916; Mon, 16 Jun 2025 09:18:23 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEYH0Bi1Qj/amsWCd72LYxFHGoIlfM8IzTVbUpM+all+BJ6C4oqXrBLJLX310yA58Ug9cHwug== X-Received: by 2002:a05:620a:450b:b0:7d3:b9b6:f1b9 with SMTP id af79cd13be357-7d3c6cd6c86mr1518724785a.26.1750090703338; Mon, 16 Jun 2025 09:18:23 -0700 (PDT) Received: from trex (132.red-79-144-190.dynamicip.rima-tde.net. [79.144.190.132]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4532e13d014sm149938815e9.24.2025.06.16.09.18.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Jun 2025 09:18:22 -0700 (PDT) From: Jorge Ramirez X-Google-Original-From: Jorge Ramirez Date: Mon, 16 Jun 2025 18:18:21 +0200 To: Krzysztof Kozlowski Cc: Jorge Ramirez , quic_vgarodia@quicinc.com, quic_dikshita@quicinc.com, bryan.odonoghue@linaro.org, mchehab@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, stanimir.varbanov@linaro.org, linux-arm-msm@vger.kernel.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/5] dt-bindings: media: venus: Add qcm2290 dt schema Message-ID: References: <20250613140402.3619465-1-jorge.ramirez@oss.qualcomm.com> <20250613140402.3619465-2-jorge.ramirez@oss.qualcomm.com> <6f4e715f-1c73-450e-b7eb-92781b7fa050@kernel.org> 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: X-Proofpoint-ORIG-GUID: I3ocZUJCWs8zTKCx2jJeXBfei8KS_YIA X-Authority-Analysis: v=2.4 cv=edY9f6EH c=1 sm=1 tr=0 ts=685043d0 cx=c_pps a=50t2pK5VMbmlHzFWWp8p/g==:117 a=wjE3nLva0YkvARyJ+Gfmxg==:17 a=kj9zAlcOel0A:10 a=6IFa9wvqVegA:10 a=4oi7HMmTljfVUEt6n0wA:9 a=CjuIK1q_8ugA:10 a=IoWCM6iH3mJn3m4BftBB:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwNjE2MDEwNiBTYWx0ZWRfX3gtJhEtDzoks lpZ8xQ/YD8w7I2TsigcSvtfAyYFvNzx7u76/wC9wgJ7d5bApfnW3hgqV4AgLEU3LvxTK6NBJuMm +iJY3NQ5sbMCsRZg06T/3CetF0H9r58wicQbEcKsr44J6uWRnBuMZdq69BwvT/AaP1jAgvjJ/XR 1pRwUO9/16kTQePtiBcLljzesBm77Ww8Pkivn3VR9XeJHBbWcsbiZWE0mLfRtq/CkwU1bl8MATT iQO9h0wHg+X0O5T1PvAJ8YfiQ5Xw9JhYhcaolRnJ1LvvRDCDgcSd2a6Ce65BzJ6gz22yWSMc5ai CENVqiSHUyQU002qfSrGiAxWLgd42wNHPGc7uu3ULK05rjcBVP7+xWzN2nnpNYy6QXZYlp/wSvk iNO8FHRb0cqRIV5ECbjpXHFq8G1XdLCiq9/pMif9sy0agfdlrdIOIF9vj6GFuySkWAoDeLNC X-Proofpoint-GUID: I3ocZUJCWs8zTKCx2jJeXBfei8KS_YIA X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1099,Hydra:6.0.736,FMLib:17.12.80.40 definitions=2025-06-16_08,2025-06-13_01,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 lowpriorityscore=0 priorityscore=1501 suspectscore=0 spamscore=0 bulkscore=0 impostorscore=0 mlxscore=0 clxscore=1015 mlxlogscore=999 malwarescore=0 phishscore=0 adultscore=0 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2505280000 definitions=main-2506160106 On 16/06/25 16:41:44, Krzysztof Kozlowski wrote: > On 16/06/2025 14:52, Jorge Ramirez wrote: > >> > >>> + The Venus AR50_LITE IP is a video encode and decode accelerator present > >>> + on Qualcomm platforms > >>> + > >>> +allOf: > >>> + - $ref: qcom,venus-common.yaml# > >>> + > >>> +properties: > >>> + compatible: > >>> + const: qcom,qcm2290-venus > >>> + > >>> + power-domains: > >>> + minItems: 2 > >>> + maxItems: 3 > >>> + > >>> + power-domain-names: > >>> + minItems: 2 > >> > >> Why is this flexible? Either you have two or three. Not mixed. > > > > please check 5b380f242f360256c96e96adabeb7ce9ec784306 > > This does not explain why this is optional HERE. You cannot use for a > new platform an argument that some existing platform was changed in > ABI-preserving way. thanks for quick the follow up. but bear with me please because I dont follow - why can the same logic be used - it being applicable - and therefore result in a definition similar to those other platforms? how would I capture such a requirement if not as done in this patch? > > BTW, also subject prefixes needs fixing. For DTS: it is never "arch". right, that is for patch 2 but ok, will fix there. > For this patch: wrong order (see DT submitting patches). yep, that is addressed in the next series TIA! > > Best regards, > Krzysztof