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 DA551275106 for ; Fri, 26 Sep 2025 11:47:47 +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=1758887269; cv=none; b=tFMXYHModemUUAv50huP9ZmkNpiSBqqNmD0QhbPRdt/0fRCpLmLtBqm9hx8BXhA+OwVKg9brXK+KUQHQpn/V0GJPHBMA25Atqq75UeS50MROagCN/E+1y/4yB3jDNmWIjIuKZ4QSIBdSp8MY4hs4I7zx47HSOhoTUVjKyFkrq74= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758887269; c=relaxed/simple; bh=OAgwbS/VknejtABxzuaNwsXHRFEJYCx7WnLwqMVsnJ4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=O+CHKe+OY5X5tvRt7wJqJ3yERNFHhhGYcRu8s+hT/tcFQkbsuKMjZbrJN42XUjtUkQOkjrBW6/fvm5fjvi4cGmSTUAcQgCZqbQ0GLuDtAGpuYoVpfbqPvLcAjD94QYuUPX2/Sqpscww+PKF0h2awOn8AndB3Y2AHd6jKaL0FWtA= 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=XBUlLIN6; 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="XBUlLIN6" Received: from pps.filterd (m0279870.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 58Q8vanG001506 for ; Fri, 26 Sep 2025 11:47:47 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= zeXt8RmZ2p5jBB+q4SRx697VU8GfH0Yutu8AdTfa1to=; b=XBUlLIN6zCNjcVfb t6IoF1uc9+Ascp3//GivOuaaeYuAa3oHjIpWzHBWLuZnIPmCADG4qL4L8QgZdjhF o2NLswvvz1ctSKn8J+0ulZebyL1VZP4rnBjh+UlFt72I+eIuS4xc6RoP6KN8FUmg uRmNLWBl2OLMVcKK2RjKJt/PtCqfUtBxIvnWsvnZJpwJNqKbi/BgqL5sfe9R9FxC xXgCFo4wtek8uL5eRO7iyISPK3mhnW+44WNDAu4Srxk45mqQak6nhxY49477wFcP WwF4oZVCcbp46Wnb33q1zyFsjqgBfybUq1+TVjPBZGErhPsR6VuohFtDwTHtqK9G KsGflQ== Received: from mail-qk1-f199.google.com (mail-qk1-f199.google.com [209.85.222.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 49db0ttqpj-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Fri, 26 Sep 2025 11:47:46 +0000 (GMT) Received: by mail-qk1-f199.google.com with SMTP id af79cd13be357-850dfe24644so87955385a.0 for ; Fri, 26 Sep 2025 04:47:46 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758887266; x=1759492066; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=zeXt8RmZ2p5jBB+q4SRx697VU8GfH0Yutu8AdTfa1to=; b=n2cm4B/J/yrrgBcos92xzMOFM1wTNfuB87HxNKoidoY8rkvfs/Z51B05jPbXTSEZ1e RjqigvFwgv07TioBjY5gl93VsN6oXqV5wtpgLhLyYEIW+AB2fPduh1CN+217fptlGXzg vQjqJK+J26mkGBCHtxZ3i87RJkpZYjihvnTqhd0pYwHmX4t7mIV8XhECROygRP7Bnk0W 6hRLdnxhTpXOen38azhEx4L/fiqIDL5Enni5ftqXCr4i6341Kw+EBqWEsSZmQUmSJlfE TQ7PBvQabdA6Ae8w84lC6G2WB02S+matwDBlkotgMESbzIX/dki6xErvkPeOdYg3jQvc Lruw== X-Forwarded-Encrypted: i=1; AJvYcCX3aU66fmOPPVNPMl7hnVbAcmjIvWuBbwnjdPxInvoRQ2gyTRErxGPgwNg27FOWNAy8Y/5OIXpoHaYsznw=@vger.kernel.org X-Gm-Message-State: AOJu0Yx5GaKBEniddikoGAy5qqapCWCAq7FaMWqWyyqIIyz4dLIvp1hF Et5kIx04JKiyqWRtFp+C2wvYmNt+Zs+rb1cs6vmHRJj0rFbn1IsNWk4gqDkZsBDyjf2SFu7b/yp 2u+9nHkWTSaIF8PIVOiDRy81BcBhf/C59TxBYa5GDTGs/EUNlrto8dY3DD3bShBQbFsQ= X-Gm-Gg: ASbGnctWGkhgTx95VVUD5GuHnw5S9cLgN8iIIbb7Og4EK2zJ8fLtzVfXkEHzurCLkzE lNJYJpTxS7FkEv5ws2uVShtxBvJxJZe7VOaZO7u91MpJDyo51dd+E10IYteiH0kelViN/is9v02 8MLoe32qTRCqBaEIj8R0L5q59JPyXWMqLTpJToCRiv1RMTw7D5gT97sSJF79xxrYeMvcFuZpGb1 4z7yn2ZiowxUn1ArKL1eyCWrVKj3VtUqIx11q4XaCk8fvWNRbQY+ADtFENktMGSroZ8JQFnD3pS r7eORQB6sgiU9HyL64nxs7Iej9kBndAWINJtx8JfBz11J3iyTdJ4Dj1aKq+oMXbLC6PIDvu7cAs NZizugxZaBpprWeP/wJd5kg== X-Received: by 2002:ad4:5dcb:0:b0:797:1974:b824 with SMTP id 6a1803df08f44-7fc2740a087mr61520926d6.2.1758887265851; Fri, 26 Sep 2025 04:47:45 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEr5bf6UETViENcg6o3qIp7ZoNsIv4FMqZ6Ft48XN9AJDhNwFvEybD2BRdLu3eBM3Rbt2dbWQ== X-Received: by 2002:ad4:5dcb:0:b0:797:1974:b824 with SMTP id 6a1803df08f44-7fc2740a087mr61520556d6.2.1758887265269; Fri, 26 Sep 2025 04:47:45 -0700 (PDT) Received: from [192.168.149.223] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-634a3ae31cfsm2693665a12.33.2025.09.26.04.47.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 26 Sep 2025 04:47:44 -0700 (PDT) Message-ID: <7c6ab647-0c54-4480-9eb2-5c2bbf5f857d@oss.qualcomm.com> Date: Fri, 26 Sep 2025 13:47:42 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/8] media: dt-bindings: qcom-kaanapali-iris: Add kaanapali video codec binding To: Dmitry Baryshkov , Vikash Garodia Cc: Dikshita Agarwal , Abhinav Kumar , Bryan O'Donoghue , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , linux-arm-msm@vger.kernel.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Vishnu Reddy References: <20250925-knp_video-v1-0-e323c0b3c0cd@oss.qualcomm.com> <20250925-knp_video-v1-1-e323c0b3c0cd@oss.qualcomm.com> <522d7244-0003-a42e-9be0-1d353df8d5bd@oss.qualcomm.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Authority-Analysis: v=2.4 cv=I9Vohdgg c=1 sm=1 tr=0 ts=68d67d62 cx=c_pps a=HLyN3IcIa5EE8TELMZ618Q==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=yJojWOMRYYMA:10 a=4CA5EJrnJ1NXJjxpXhEA:9 a=QEXdDO2ut3YA:10 a=bTQJ7kPSJx9SKPbeHEYW:22 X-Proofpoint-GUID: iH5KRfzlBHVnUeUtdNuY7IvgqIHrt_Nw X-Proofpoint-ORIG-GUID: iH5KRfzlBHVnUeUtdNuY7IvgqIHrt_Nw X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwOTI1MDE3MSBTYWx0ZWRfXyBg9fken50gA hHoX7Wa9ih/8jbeWDa1w5seuIPEwubf4fA6L8G+np4aNTU35w91HeaL2HlEO5FZZAw9JE2pZ6Wd vQ+kKwcVVywdZMwfYtB8nIlPPxk0kQXqu1JH8G2hnpJGHk0X19ZPVjuXZ8uq0KiyMIW9MfhiB9a KQRjXEyRFkc0w3Hqu56MQe/KfcuAvjDdAsfhF6/9Q0+MwvRHuhneYELorBQKnQF+p5hxFbehPdf nUqfAwNNqNC6QeLRr4+fV400ffXtjngzkOv5Ri9KMPKe+LiYMEp3ywjPsXNoHVQKHWMLxKvQHYO BwzfxQuMGPF+yYxK5th0U9FWmc/GWNAdcAdWaJDWnYMJOAYSUlm0+SsNF2RLz3WsO0lCRGKYtNK VBCvtTFKfmmQ1yJPR/Qank6/VdEO3A== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1117,Hydra:6.1.9,FMLib:17.12.80.40 definitions=2025-09-26_03,2025-09-26_01,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 spamscore=0 adultscore=0 suspectscore=0 lowpriorityscore=0 priorityscore=1501 malwarescore=0 impostorscore=0 phishscore=0 bulkscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2509150000 definitions=main-2509250171 On 9/25/25 9:38 PM, Dmitry Baryshkov wrote: > On Fri, Sep 26, 2025 at 01:01:29AM +0530, Vikash Garodia wrote: >> >> On 9/26/2025 12:55 AM, Dmitry Baryshkov wrote: >>> On Thu, Sep 25, 2025 at 04:44:39AM +0530, Vikash Garodia wrote: [...] >>>> + power-domains: >>>> + minItems: 5 >>>> + maxItems: 7 >>> >>> You are sending bindings for a single device on a single platform. How >>> comes that it has min != max? >> >> I was planning to reuse this binding for the variant SOCs of kaanapali/vpu4. If >> we do not have min interface, then for those variants, we have to either have >> separate bindings or add if/else conditions(?). Introducing min now can make it >> easily usable for upcoming vpu4 variants. > > No, it makes it harder to follow the changes. This platform has > this-and-that requirements. Then you add another platform and it's clear > that the changes are for that platform. Now you have mixed two different > patches into a single one. Vikash, preparing for future submissions is a very good thing, however "a binding" can be thought of as a tuple of (compatible, allowed_properties, required_properties) which needs(asterisk) to remain immutable You can make changes to this file later, when introducing said platforms and it will be fine, so long as you preserve the same allowed and required properties that you're trying to associate with Kanaapali here (i.e. YAML refactors are OK but the result must come out identical) Konrad