From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4B54BD74970 for ; Fri, 19 Dec 2025 09:56:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=k2n+SE5qOjIgQy6DMMGVLWGLkjFAmY3VbtNAYsRbVQs=; b=B0Svd9ff3VgM7/GgjjTqxh4MWU Qlh8WSV7t4uKQy0l1IGZqgVS2I7WwDvDaLdPVx8VVTq8RrgvmGHd+sZumg8PO+QFkoCYA2M/3G+oF h/9KEBAu0OHSnFSdplGWnBdJ/YNsjqI0RFVj/utpdj0iKH/57JjH9oacjlAe5RXake8ogwN1liNxW /Bl92SU+eftOv708BqB+e8JOGyzdMpsT60fq5BG+dR+0KxIIKyykyDqYRH2tlx6756bTgNzmil6IN 4Spf5lEdE+gwai4OVW65WCyPG169q4eDinswu35I+jFJizrLx6MpmJI1ULTczxDEXZoXqBFERXUTG tZrGmuVQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vWXE9-00000009zsj-30Pb; Fri, 19 Dec 2025 09:56:49 +0000 Received: from mx0b-0031df01.pphosted.com ([205.220.180.131]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vWXE7-00000009zrX-0XiZ for linux-arm-kernel@lists.infradead.org; Fri, 19 Dec 2025 09:56:48 +0000 Received: from pps.filterd (m0279872.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5BJ4buSL3975405 for ; Fri, 19 Dec 2025 09:56:46 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= k2n+SE5qOjIgQy6DMMGVLWGLkjFAmY3VbtNAYsRbVQs=; b=mBvM2WnoXkzYN/kC SJGG65cEd1jX1y37nhMNwPXoYap3kWJQ87NpviuiG/9EpW6RY1R2+65CQo0TacQK sg5g261EgUkzpSZNRyIQ5Tbq94grrpgA54RJf6dw7RiPrt9VvtizQFAgaUOcEl8M GWgnCmPqSj0yr1tNRU87wP08XWkpyiLLJXDEohVRQEIe6jqZ3UEAy9xpKoaXIU/+ ForMEEIzdxlOZtKRPSjpS3PmCX2Ly5oARuhQGWCswtUnUjrxx7VjBJaXw4Eu3pbN YXSBkcSKxsz957JQmEVhedqMLHHF9ABw3MrRhNod1hp64Yv1Ut2wd6spRjrh8MHF YF2vfA== Received: from mail-qv1-f72.google.com (mail-qv1-f72.google.com [209.85.219.72]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4b4r2fj4ux-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 19 Dec 2025 09:56:46 +0000 (GMT) Received: by mail-qv1-f72.google.com with SMTP id 6a1803df08f44-88a316ede50so30272196d6.0 for ; Fri, 19 Dec 2025 01:56:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1766138205; x=1766743005; darn=lists.infradead.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=k2n+SE5qOjIgQy6DMMGVLWGLkjFAmY3VbtNAYsRbVQs=; b=cI/7wvEiPQgLwa/srrbZdMEVvfeuZXfPYsE5kUgGdbIHqVHuYn1IJ+vWHyk5YpqlWA ZXiEtvwgJ4n8qmrmdeHzhHd4OV1lDLH3o33yOofVihPooNg+nYZdriaxpydpfbouGYc6 z+oc8QoJY/7J1kBBSJNmFaxk2E/jL5BZzFknSnInWs1J9wGBMlV08iFetPnXmPL9+Yu3 gQe7rMCoNqKiYpbIIvrypGpmc3IIUxephVQpp8sDmuo5pexTRnFVC71Ex0Lp7bJECVH2 arRz9NGgfgiNtR1b/orCzdwYZJ4Iygl60OzH3agLXME61MdOq9e2Q21tZPrlepHjRbN0 1CeQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766138205; x=1766743005; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=k2n+SE5qOjIgQy6DMMGVLWGLkjFAmY3VbtNAYsRbVQs=; b=TOQuHu+gCstwku9BSvU/lWqiTiPLGjO2QyCR3Z1R+4YUB2O4daKy9lHlv7uEIOe8Xb r3gdYtjc6UJeJxkHhvAXdZuJoD+jvMSZeaR3rTvb4WkEliPYyAPduv6oLYwU81poAC93 0tzmbrGZat0LjGHZyM0WT4NYSlBouJWg5zYJ8Ml5CWjsxiojVImiNGa9EyxBwaPMAU/a n+7SMDLiUp7sh2+DtEDsUgOERAU8zz6Xj5S+QivyOqhJJrcaAs5JSYlLCNz8bEzadm51 V2C+w6eiP4YTY60phU3xrGiIn/JBL1/bi3cZdXO7xhbJz4NVodtXBQPkpCYuBnWAjvZK 0NFA== X-Forwarded-Encrypted: i=1; AJvYcCWOAwgF/M3vuJyDLmKW/6ycIUVRf68chv0iY1OcRyDRcmZ55jBrXUa2dsG5s4ovIyKdTZKU5jLjPMatuC0XlACt@lists.infradead.org X-Gm-Message-State: AOJu0YxJzgPvb7l8H2nYkqIlf5C2V22gDqU+gqsR4yM+xOnpIRoLd4BU PmPlEGu3hT4U8l6tzBx0dktPTICEbckwB0KaNDd/qbXteqsJui687xjdz0SwvCcpIjUj+zUetRS axQdhIX6Q1ByspwxniT5m3ipqwsYc/AyVKgRpNJHOHHnF18Lxdf19nrFdkOMedxytZFYMxzoDQP wZ/A== X-Gm-Gg: AY/fxX45+HUiEH9fuqf77364RCS33Ek//deWTyXDUpKD7Mkjtf+3AdtH5kEiqTFGxlE j2CeIdx359uduUCmfjucdohBk7NcQcQ/3ysYeb3yp3ZQPuikBuJW7zp7aIt5bS/kHO/hBFF0V8F WUdcD5vtZF2LCxEx7p48+gZxtAzdiHnuTogJ08vUaeA1ItwXARNrsgO0QQYDKXWJiy+Sj6nsijO RO6Nr7vKhzf69Kk5LuZtM/2OGhYwX47RCoeZpj/mHAwl6YkPDzUKV0kDirAyYD5gaznv+u7SbtQ lwv9KErURiTUk9YKQuQKmCsBmOEnp7aLPk45MhnJQD3O7G9Eq0jSl3eKHVv3fsW6V0mwYLhRuzm lIRRo7i0GYatosurKnd5CxRFKvqIqqWlEKKRmvtpGZovrzfUAeH0T6OKV2/VImdQd7yX0 X-Received: by 2002:a05:6214:1c4a:b0:880:4ed1:ce32 with SMTP id 6a1803df08f44-88d833b580bmr40114566d6.45.1766138205407; Fri, 19 Dec 2025 01:56:45 -0800 (PST) X-Google-Smtp-Source: AGHT+IEmL/9SGkWyvSpj/ayO4oxzYHHhXmjwsfj/l32rWMsHt4VSUg1NRyyPchBsCrruiDNCaZinHA== X-Received: by 2002:a05:6214:1c4a:b0:880:4ed1:ce32 with SMTP id 6a1803df08f44-88d833b580bmr40114456d6.45.1766138204950; Fri, 19 Dec 2025 01:56:44 -0800 (PST) Received: from [10.193.196.209] (82-128-196-119.bb.dnainternet.fi. [82.128.196.119]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-59a186f957asm542852e87.28.2025.12.19.01.56.42 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 19 Dec 2025 01:56:44 -0800 (PST) Message-ID: Date: Fri, 19 Dec 2025 11:56:41 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 00/11] drm/connector: hdmi: limit infoframes per driver capabilities To: Maxime Ripard Cc: Daniel Stone , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , Sandy Huang , =?UTF-8?Q?Heiko_St=C3=BCbner?= , Andy Yan , Chen-Yu Tsai , Samuel Holland , Dave Stevenson , =?UTF-8?Q?Ma=C3=ADra_Canal?= , Raspberry Pi Kernel Maintenance , Liu Ying , Rob Clark , Dmitry Baryshkov , Abhinav Kumar , Jessica Zhang , Sean Paul , Marijn Suijten , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org References: <20250929-gregarious-worm-of-memory-c5354d@houat> <20251003-uptight-echidna-of-stamina-815305@houat> <2a5fitdzr2bz235fj6rvqzxr6ckszkjbazjfszlvnizdh2cvbt@w3ypjo7vahhs> <20251201-enlightened-zebu-from-asgard-5a20be@houat> <5dyhjur3hkhvtlwrl4h2m342byor7f3ssvkunj4cggnhbhlmnb@l2mfz7ypjj37> <20251219-honest-slim-crab-02e932@houat> Content-Language: en-US From: Dmitry Baryshkov In-Reply-To: <20251219-honest-slim-crab-02e932@houat> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Authority-Analysis: v=2.4 cv=NODYOk6g c=1 sm=1 tr=0 ts=6945215e cx=c_pps a=7E5Bxpl4vBhpaufnMqZlrw==:117 a=UP4AmmvVc3sGQHJQO2VA4Q==:17 a=IkcTkHD0fZMA:10 a=wP3pNCr1ah4A:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=wWkPZpA8QBIs0yCpxSwA:9 a=QEXdDO2ut3YA:10 a=pJ04lnu7RYOZP9TFuWaZ:22 X-Proofpoint-ORIG-GUID: itUlQ4Wmfsm53Kl4LLnx4vvjnCa_c1Kk X-Proofpoint-GUID: itUlQ4Wmfsm53Kl4LLnx4vvjnCa_c1Kk X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMjE5MDA4MCBTYWx0ZWRfX6eH+W2k1Utp2 VX44hR9KqGk8d10yvXKCyXftUX6//81dps2/qkxPJt3JG2fpJKLoMR4MKM/mSaL36XEuHAPgibu PVesPI61pATeIzTkBnF4BLHrmh5JIKQLhYfEYC4VUYGFHtDCtvHEjVFpEKQ8XcGBtYbVPlFNLUq FnnNe1uJ6W+cufC+PhbYq9ltRViUNPmuES4pmpy0a7GTb/9INllGWmYZ33k6NEP2uYt+xSptA7m po//2CvfhhRA5jDJDAED1qamVmhuklwJBlMX++1kM4xX+O6KKA6J3XT77R92M6ia/vbHoaE5hDh 2QiOBQKiIdzpsKVfn5DmyG0V5yDaGqP14nB70CnVfheZh3CAvWXWqbkoDleMGmvOAeuJhGLbJ+e LBzyeaf/RvJ+dqtl/x9cU8G1rHdQA45uL//99zBk3yGmakoWdXxig2F5AsQoME3QLahNlFLbLRL 2as38P6UZSiI7Z3JewQ== X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-12-19_03,2025-12-17_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 phishscore=0 suspectscore=0 impostorscore=0 bulkscore=0 adultscore=0 malwarescore=0 priorityscore=1501 lowpriorityscore=0 spamscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2512120000 definitions=main-2512190080 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251219_015647_305997_FCA537E8 X-CRM114-Status: GOOD ( 27.74 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 19/12/2025 11:54, Maxime Ripard wrote: > On Sat, Dec 06, 2025 at 01:28:14PM +0200, Dmitry Baryshkov wrote: >> On Mon, Dec 01, 2025 at 06:01:56PM +0100, Maxime Ripard wrote: >>> On Fri, Nov 21, 2025 at 07:09:01PM +0200, Dmitry Baryshkov wrote: >>>>> So it's not really impossible, you just need some hardware and a day's >>>>> worth of work. >>>>> >>>>> There's no reason these should get a pass, it's breaking the spec for no >>>>> reason. >>>>> >>>>>>> For SPD, It's really not clear to me why atomic_check should do that in >>>>>>> the first place. Your initial concern was about exposing infoframes in >>>>>>> debugfs that wouldn't be used by the driver. >>>>>>> >>>>>>> If the driver doesn't register a debugfs file for SPD, and ignores >>>>>>> whatever is in the atomic state, what's should we force drivers to do >>>>>>> that? >>>>>> >>>>>> I really don't think that drivers should mess up with debugfs on their >>>>>> own. Making atomic_check() disable the unsupported InfoFrames makes the >>>>>> picture perfect: the DRM no longer tries to program them to the >>>>>> hardware, DebugFS files stay empty, so the whole state becomes >>>>>> consistent. >>>>> >>>>> In the "bridge has no access to infoframes" case, there's really no >>>>> infoframe. An empty file is "the infoframe can be there but isn't used", >>>>> not "we don't have access to it and can't report them". Only drivers >>>>> have those infos. >>>>> >>>>> If we do split up write_infoframe into multiple functions though, I >>>>> guess we could create the debugfs file only if the function pointer is >>>>> set, which removes drivers' involvement if you don't like that. >>>> >>>> I'm fine with not using HDMI connector framework for lt9611uxc. >>>> Likewise, I think, it's fine to have empty files for the infoframes >>>> which are not being sent over the wire for any reason (hw not supporting >>>> it is one of the reasons). >>> >>> I can't think of any other example in the kernel where an empty file >>> means that the driver doesn't support something. >> >> Okay. So we need to sort out implementing the split write_infoframes in >> drm_bridge_connector. Any suggestions there? I'm asking, because I don't >> want to end up exploding it. > > I guess it's only really a problem if we want to make it const, but we > don't have to? We could just as well allocate the structure directly at > probe with a drmm helper and fill it as we need to. Yes, I wanted to keep it const, as we usually do for all function tables. I will use drmm_alloc for it. -- With best wishes Dmitry