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 2C4522F2378 for ; Fri, 4 Jul 2025 10:28:07 +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=1751624888; cv=none; b=D0AwRZ+95KlAtJ1T8ZGFsrEwsXHAKajCgMTzqEd4cY3+EhrFgCxk1NNNdT0cn1WGAKT6BgilBh1QzkUCvZRM/MiwqzAQU2y8uAqd6qLJY3yiS0oNu0ylplaqNDFmTL2KDhhqlAKe5EKuRHpqctt6XuepqmqiIGDbjSz4PUuXiBQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751624888; c=relaxed/simple; bh=G/e37K30BtcmlRmYfZxP0u0F075QyuQgrrhDDUK57+g=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=QFvyd52vLu/Tv+b7f9dsYibyccBAVJm3e1R0HUByYCg4jLjy61bmsbFPjtp6Gh2w4Dj04/6sZHlGrOZ1mqPoarz04I24Oqok9s8SIMMlFDHZsDcsrioiMkqPmIIXgfKBfH4M7igmY+1CUACfufedm/IYwtVhsJt7NfvyaT8YKs0= 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=jFDiprCT; 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="jFDiprCT" 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 5646gLNQ000415 for ; Fri, 4 Jul 2025 10:28:06 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= hcYhwg2H/XHGiaSGX5T/gqglxK5liyG/eBHo3NygJnQ=; b=jFDiprCTR4SolVmp XLAWECotUU907b9ehU9ztjT74CAmaigOUirsmPSc5Mr/rzEKDqiBJm9p1gGQBiOI /HoRnb/xjYDEmY3oJ/yT/fWjP3bsD65r7g3OQqcLaWRUYPz/0PNGYjKs/ciuX2vR gqaG6PMJdksxUk/yFFILycPuin3eu95MmVJcla7Yr6sT4thXJlub/peoLwNwAsJt jOHXSvRbHNq/dWZVpgvJdTNkWavM0UV6HcSQBmS/pwSab16F8V7y4EDrfREk9SVW YpdrmSTUULLyt7zcb3DOXQyUFTttL+HU+ArpJEHSxRYjAOzF+qzPQGU9WcCNS8RH K7wf8A== Received: from mail-qt1-f198.google.com (mail-qt1-f198.google.com [209.85.160.198]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 47j7qmkd8m-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Fri, 04 Jul 2025 10:28:06 +0000 (GMT) Received: by mail-qt1-f198.google.com with SMTP id d75a77b69052e-4a804126ed6so2427181cf.1 for ; Fri, 04 Jul 2025 03:28:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1751624885; x=1752229685; 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=hcYhwg2H/XHGiaSGX5T/gqglxK5liyG/eBHo3NygJnQ=; b=j1YknTWKhFx6XwaCBUPFQg1pX+ePXcCSS1gIt+yHHekoSEL2J26BzPT7uUyhYFSr+c 6u05b0IgCPET4Stm5xYCahkK+ItWcfxmspEfJjp4hrXwWuA2yo1YjY87TTQdeyjaSqwT 7OvHu8wb4CgzBpUOX/x+CYe6O/aBwh3VXA6lCBba2v0+NZjn4PusWYcTovpL12c8yH7q Mrf8pCdwI7sGawsB5vMRRB7zBl4yH/bwQUqE9fLloj98HMd/ObAK3eq6mvxp6akvwhGm DrKzxkP45Wl7SkWfy6qttQb6iIHy6RjSbsJTRAiuJlufLty4uH4cX8/LptDnsgqZcXR5 BV1w== X-Forwarded-Encrypted: i=1; AJvYcCW3KvR4yanUDpMTpm7Aj2lCMdM1VzWZ4RBVh5t7RhMx0ISp22NNz0gAAhIkMxuZIc1jsJrXI4G21aobMQ==@vger.kernel.org X-Gm-Message-State: AOJu0YxS7yRQ+eUXqPuYB47YB+gwLDqjpA8X4M1bnDzgot8bw23SZ/bJ fzPgAzcGtQR/kfz5RIan0EvJg1fFAb/PThie3jCqG/vhjZBGugPXpbO3DFie2MnVEA3e9nFSgDZ JftOtnJLraLAbNpUXcBIekmnQFRxjJVCNrwO6Oz+RlsE21Op+qDSeorO90PRdnD+pZA== X-Gm-Gg: ASbGncsQMpqXGXa0gkCcYSMNdLZEmJXODJbgDY/83DWLgMKYiRMvGgy3q+nkMV251ua Z+dtYnQqToWQeBFHburMiA62ebgOUGriVcrVVBBeE33PYTnkwKbX2j92TuCoNMsSg0Md81s89D1 ELayD7lJA1O7I7SdceGOkrblhtLJY2Ac1BnLXf01ygL/zWoJv4+OKZF12Ih2mPfoSOtYOzS2cUR m427FEA36hBQL0m08ulbASTdirhjFNWg3JhMr128hVvIwsVK7LoFJ4HxBk0z/zlP1MQtvwKPpXZ IpK6IFh0drUKKA6q14grrfG23J1DSYIGprRmVlpM+2s175DVLXK/1+8iifliLh8D+WHWysbf+L6 ZauLrvQ== X-Received: by 2002:a05:620a:4148:b0:7d5:c1d9:939a with SMTP id af79cd13be357-7d5ddc689bfmr82924585a.9.1751624885130; Fri, 04 Jul 2025 03:28:05 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEepMErcR3kSl7PpdDimjsfv0t/HHqo53Oh9ZaV5I2CnMfyJ6V0u/KZvLGfTyKT6SaxB8KzvA== X-Received: by 2002:a05:620a:4148:b0:7d5:c1d9:939a with SMTP id af79cd13be357-7d5ddc689bfmr82922585a.9.1751624884534; Fri, 04 Jul 2025 03:28:04 -0700 (PDT) Received: from [192.168.1.106] (83.9.29.45.neoplus.adsl.tpnet.pl. [83.9.29.45]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-60fca6676a2sm1134235a12.5.2025.07.04.03.28.02 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 04 Jul 2025 03:28:03 -0700 (PDT) Message-ID: <8b7e8174-38e7-45f0-ab34-c8b0517a9f59@oss.qualcomm.com> Date: Fri, 4 Jul 2025 12:28:01 +0200 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 0/5] Introduce "non-pixel" sub node within iris video node To: Bryan O'Donoghue , Dmitry Baryshkov Cc: Krzysztof Kozlowski , Vikash Garodia , Dikshita Agarwal , Abhinav Kumar , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <5ad418d9-8199-43c9-a477-1e3b939c054c@kernel.org> <7b6db4fa-2f73-376d-4eb3-64c1c7e6cda3@quicinc.com> <5qsgbqml367yq6g5vb4lotrzulojqhi5zlwwribze373a63qrn@rxi4kwyt66m2> <4f38058d-a2f1-4ac5-b234-228cfb2e85ff@kernel.org> <1ad2ca1e-1d57-4ad8-a057-ab0d804f1d49@oss.qualcomm.com> <7da769b4-88e9-401f-bb21-0ff123818b9c@kernel.org> <6840d462-8269-4359-a6e5-d154842b62db@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=C4TpyRP+ c=1 sm=1 tr=0 ts=6867acb6 cx=c_pps a=mPf7EqFMSY9/WdsSgAYMbA==:117 a=qmfFy4ndMtQ753Zl/n/b/A==:17 a=IkcTkHD0fZMA:10 a=Wb1JkmetP80A:10 a=DVKtXGnjxsSMy4abO9MA:9 a=QEXdDO2ut3YA:10 a=dawVfQjAaf238kedN5IG:22 X-Proofpoint-ORIG-GUID: BkmMpLulaA5drmtHvUOTwsdd79o_-NOM X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwNzA0MDA3OSBTYWx0ZWRfXxxK3Weu//Pij sj+Dc4jMcDy46vZuRsYU90FCfeqthYeGW/OcVat31z/20dPsmN7YvBI1LHjh5hyTMvCEkpAy/qY tJhgblUp6Mj4kFvdOyYpzcd/YBy1VYtVArs4Vbm0FjknpcErr1+wMzVxlnOecQK45M/32EQca7v xNRAGvRElcpgjVF2VgS56EtnRuHIoeO8anAAvlyCgZ28UKPCrUQIo+egu4bSu3qlDeFcDnOiBqd BpzUnDh0c0sXM+dWjROPtLQfNsYlO9IVBrIqTlc0VgIqoDXA52yQmmrQRmvRNBh5dADOgW1nz1J Z12txDpwDXHGtNAmVNlGpuoCluGy7Q5D6sNNU4vZSleaWeXo2gY8d0loKxmiv+wXjVVcD++mXNU inOOmdwZqjus0Bx1CT5blxe+k1sPMPR0HyxG34He+Bg+Gbg9vHLy40MKW/NxHtnX/u58NBmT X-Proofpoint-GUID: BkmMpLulaA5drmtHvUOTwsdd79o_-NOM X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1099,Hydra:6.1.7,FMLib:17.12.80.40 definitions=2025-07-04_04,2025-07-02_04,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 phishscore=0 suspectscore=0 bulkscore=0 lowpriorityscore=0 clxscore=1015 priorityscore=1501 spamscore=0 mlxscore=0 mlxlogscore=999 adultscore=0 malwarescore=0 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2505280000 definitions=main-2507040079 On 04-Jul-25 10:23, Bryan O'Donoghue wrote: > On 03/07/2025 22:23, Dmitry Baryshkov wrote: >>> I still give my RB for the series. >>> >>> To me the only question is should it be applied to sm8550 or to new SoCs >>> from now on, sa8775p, x1e and derivatives. >> I think we need to apply it to_all_ SoCs, maybe starting from MSM8x26 >> and MSM8x16. Likewise, we need to think bout secure buffers usecase. And >> once we do that, we will realize that it also changes the ABI for all >> SoCs that support either Venus or Iris. > > I think a dts change is a non-starter as its an ABI break. > > So if ABI break is out and reworking future dts to allow for a new device is out, then some API change is needed to allow the driver to stop the kernel automatically setting up the IOMMUs, create the new device as a platform device not dependent on DT change and have the probe() of the relevant drivers setup their own IOMMU extents based on - probably indexes we have in the driver configuration parameters. FWIW not even counting the address space limitations, no video hw binding today is ""complete"" (with all related SIDs bound, secure or nonsecure) Konrad