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 339FC19BBA for ; Thu, 5 Jun 2025 12:30:56 +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=1749126658; cv=none; b=d3fW1y93ZPPgib4LhQGIlOqzJVI5COtxRhrR8+/v7ocORIdAWRHRLjrNMcWessPQWm5wLbVhYoHXeW+g6fg+bCzofqyia7YQlAal9iXgAL6mcYA2wwVCu6tQ/ZVrFhDJxmEtjvxwIxWnPmFC+7ELbw3Qk2wWl2vOc2F4xVFr1Qw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749126658; c=relaxed/simple; bh=TZWnThcbnTjWRdFMqMCAUjB+YrFrDzfgiH5dW0zatfI=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=KIrYkAE5+pE4bnwezoXUuUslhasRHQUQKyTYTXOZDilLKxeCxS+cKrZMmtTyJXVAFC0hnBeCAJuA3/ZcfRtb2ROUFKGm0kPbyKNRuz9TP3XmBi7zCEUGe7AdKWeFythjcKqegSG2VCCVA7BN5scVL6zuqQvDu5GVl4nbhPC38v0= 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=LiDtUYWj; 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="LiDtUYWj" Received: from pps.filterd (m0279871.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 5558AcVo010418 for ; Thu, 5 Jun 2025 12:30:56 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=8zoylx1QuLi0pZwoA6HZK4hy +63LtfiIZwBR+DzOGNQ=; b=LiDtUYWj/wzhy50SS2P6OKcl/msD9X9Ubi3i+OQP AOxAHVNp1me+zbUvD3+LocuizFESHC7TqjZgfxHSEYPtfjv7KN3f/8TLBgdPevuB JGSmIshnDWUpc+yocLmAtFA4DuTkWIBtWce3FyAmuXjw0/Owz7vHqx7x3M10F8LR W7Vtg7Uxm2oVt+X+0pdAxD1RLvv9OUsWTmdCnBcl3NLcF9hs/3zBIKlePMNnAcIh pzXKYdmLzqtrNuOsBZdYZADhf/UVxSrtadTp8dkiIGT08VQ/iVbn+xpdPXl3ELbh iZgtNjQs7Z0sMvBBPe4BYorH3qaJsvC8+ZQzuXHbpFxuqQ== Received: from mail-pl1-f200.google.com (mail-pl1-f200.google.com [209.85.214.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 471g8ys9du-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Thu, 05 Jun 2025 12:30:55 +0000 (GMT) Received: by mail-pl1-f200.google.com with SMTP id d9443c01a7336-235eefe6a8fso6940295ad.1 for ; Thu, 05 Jun 2025 05:30:55 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1749126654; x=1749731454; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=8zoylx1QuLi0pZwoA6HZK4hy+63LtfiIZwBR+DzOGNQ=; b=PdXFQ5rBisDZZVUm8YYQF1MFOBDAR7zVPlN4s6Z8/fDNW1cysShr1gGtNS4baiOhd8 XSLrJ/aG5V9ZEYi5+36IpxC3DUpw/jCn1FofRVx9bVglPn8XSGjdmNpkJOcJzXms4uIE 8XAO5jWmqy2ngxapEuwBjlgFD6lEqKY1sU7Oxdiri3oeFLjJ9gd6ylQe8FMJLOw92LuE 1rvM1BmXiNjzBL3Lv4TfDa1NOL3uUZyjb+pJrxlhZGOPm8Fk7XhFxwa1XwDWQ+6liOS+ cgHWfI6SfRwsTBUmSYYGdwOtznRhZQ7kSdw8HOHD3ROQZ7ZevVFEa2xbJ8ZOm8ZCKuZf JTLg== X-Forwarded-Encrypted: i=1; AJvYcCWiGEnFfRatMgyrFhCx7CzF2FPDKImzYXuYuaUhH3GFJu6LM+MMZghirZO+U8um0fKo2p4LzwjXqN2R@vger.kernel.org X-Gm-Message-State: AOJu0YzqMRV+GKlpYioyyqlB1IVJ6ekIAf5N3yg+2NdJ6BizY5nBGAGM iX9Qo6N+H/cAAyVt/N/68UC5X6lrlADE3rH/IIgIKelALdoMni8XTaFMegxeHErST8eUyOb7/Gv XbgtMsczYonOg9N4xzzjuUat26/Yykty0yuEbw49KCkG+sV00znP9ZPka1Www1LF4V4TN1oGDvN PUO0pDvrEWW2zC1iPe3A5q66V5+LkpSgJsD6vHMOA= X-Gm-Gg: ASbGncsGASuEeLHaj7iI8RwFbDiVaGrp5/KoK6j53nW+1vPlZTGhJc+eOy+ueGq4jfL WuN0j3ufLsEH1DXgymPzlF7eqwEJ0bRxd8/NOpzwoZhAKQGSc9Aka+zDon4+TXaRNpAEOHpzAZw doccU= X-Received: by 2002:a17:903:2305:b0:235:91a:31 with SMTP id d9443c01a7336-235e110ddd2mr77356145ad.8.1749126654155; Thu, 05 Jun 2025 05:30:54 -0700 (PDT) X-Google-Smtp-Source: AGHT+IF/NaZFI6I8Q3iARrRK1yLX6Hwez8SAK/Pvn5ED6d5mV5zGH3OfKI9Oe4s7eSFk9px/R7JbYs2D+oCJQiXoxLU= X-Received: by 2002:a17:903:2305:b0:235:91a:31 with SMTP id d9443c01a7336-235e110ddd2mr77355695ad.8.1749126653761; Thu, 05 Jun 2025 05:30:53 -0700 (PDT) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20250530-add-venus-for-qcs615-v8-0-c0092ac616d0@quicinc.com> <285cae4a-219c-4514-818f-34c8225529de@quicinc.com> In-Reply-To: From: Dmitry Baryshkov Date: Thu, 5 Jun 2025 13:30:42 +0100 X-Gm-Features: AX0GCFvjBbRAMAXejejAvGq6nX_oZcZYy7MLtFcpvOHrihpW2Eqenm74lbaZ_iY Message-ID: Subject: Re: [PATCH v8 0/3] media: venus: enable venus on qcs615 To: Krzysztof Kozlowski Cc: Renjiang Han , Vikash Garodia , Dikshita Agarwal , "Bryan O'Donoghue" , Mauro Carvalho Chehab , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Nicolas Dufresne Content-Type: text/plain; charset="UTF-8" X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwNjA1MDEwNiBTYWx0ZWRfXxNet4Pn+ZGh/ HvBnTyixThhHaSBQYdBuhoWKuWx+u2xsbg1o6MNuzdWNokg8TJWvAIufTacPPhmevqau2h0jHr6 in5qeqC80BbS6L6nsqwuq5XofKZCBv19v+5AAb9YduFlqkAK6ZUTxRamHoQW1HJVcDy/2xPDYmy sJ51WAPjkbfHEfx/CbiLbPSmx+3uOitwb5iOEV/GpINCDYVFWvWUBkyzOjJNv6zL+uEgKkHxdjr jKldl4IJvd4BBFxMTokVLMRm380xTuUMSqK2LX4bhz0MxAt+dcOyX38rs5/XfrJueFblDjk6U6q CdlxxbB+IptokPR/GdAvi9fHK9Vp0TtYGBc8/eVDmuiJ+usrCgdpJM+aEgAwh/j68aKiIefWbO7 p907Qh+rCL+9ICW0wxqelXqg++fqA18StJqxISbje3PprtAv/6beX6XFp74uQ3n4QVajd4// X-Proofpoint-ORIG-GUID: n6lQ2l3ExeBbpODLbBJ6GvFdOME80Ucv X-Proofpoint-GUID: n6lQ2l3ExeBbpODLbBJ6GvFdOME80Ucv X-Authority-Analysis: v=2.4 cv=T/uMT+KQ c=1 sm=1 tr=0 ts=68418dff cx=c_pps a=IZJwPbhc+fLeJZngyXXI0A==:117 a=IkcTkHD0fZMA:10 a=6IFa9wvqVegA:10 a=VwQbUJbxAAAA:8 a=0G5chWBQ-Mji-mtY-ZkA:9 a=QEXdDO2ut3YA:10 a=uG9DUKGECoFWVXl0Dc02:22 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-05_02,2025-06-03_02,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxlogscore=441 suspectscore=0 impostorscore=0 mlxscore=0 bulkscore=0 lowpriorityscore=0 clxscore=1015 spamscore=0 malwarescore=0 phishscore=0 adultscore=0 priorityscore=1501 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2505280000 definitions=main-2506050106 On Thu, 5 Jun 2025 at 13:13, Krzysztof Kozlowski wrote: > > On 02/06/2025 08:16, Dmitry Baryshkov wrote: > > On Sat, May 31, 2025 at 08:05:24AM +0800, Renjiang Han wrote: > >> > >> On 5/31/2025 4:27 AM, Dmitry Baryshkov wrote: > >>> On Fri, May 30, 2025 at 09:32:12AM +0530, Renjiang Han wrote: > >>>> QCS615 uses the same video core as SC7180, so reuse the same resource > >>>> data of SC7180 for QCS615 to enable video functionality. > >>>> > >>>> There are no resources for the video-decoder and video-encoder nodes > >>>> in the device tree, so remove these two nodes from the device tree. In > >>>> addition, to ensure that the video codec functions properly, use [3] > >>>> to add encoder and decoder node entries in the venus driver. > >>>> > >>>> Validated this series on QCS615 and SC7180. > >>>> > >>>> Note: > >>>> This series consist of DT patches and a venus driver patch. The patch > >>>> 1/3, which is venus driver patch, can be picked independently without > >>>> having any functional dependency. But patch 2/3 & patch 3/3, which are > >>>> DT patches, still depend on [1]. > >>> I'd say 2/3 and 3/3 still depend on 1/3, otherwise we can get video core > >>> on QCS615 over(?)clocked. > >> Agree, so we need to make sure that the driver patch is not picked after the > >> DT patch. > > > > Worse: we need to make sure that the driver patch is present in the > > branch which picks up DT patches. Otherwise building & testing that > > > Well, that's a NAK then (although depends what you mean by DT). I mean qcs615.dtsi. I'd suggest an immutable branch for the driver patch. Or just merging the patches in two consequent releases. -- With best wishes Dmitry