From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7BB94438475 for ; Thu, 6 Aug 2026 10:07:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786010835; cv=none; b=oMqyG5a+Q0BmwiQnVpCrZyw2k7HIH4XTmBWT6RtlMaZcCOnargxcXbF4hLq5PU90/i4T6p6nN6A4d7QpJbRr8EwGtGp3/GpUa/mA1AcQZW9zfLOrKRMMFtwEVEVcWLnrn7n/e6bprrcsem1eIvRqRB0igcbU2W0JuDOstihFbUU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786010835; c=relaxed/simple; bh=vrlxjkRqJBr/96nA4lP0Rhr7mW/oLwAWnS7msdSZRyY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=KZ/ysmQICxFad1lpfzc4up5O+yc2PuwA8KpitRMGA+PGT7GHKq6UeHQ4CMN+abMrUGnbRN/KijmFCYSkOV1U+ktW0bjvVnYHpxZp4+MjCTzHu7lVEIU2dfR9yDwaoPUWe4gWxQdRyiwEliKFfRKoSruMZSaRXYqUr2J9glxUXCQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=dmja6bcW; arc=none smtp.client-ip=209.85.128.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="dmja6bcW" Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-4955aa106b1so18439005e9.0 for ; Thu, 06 Aug 2026 03:07:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1786010825; x=1786615625; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:content-language :from:references:cc:to:subject:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to:content-type; bh=EJsvtNqHDCxAmT1UUowASSNNS2LeJPAA9Hw6UmnAiyg=; b=dmja6bcWpZwgNlmaQrjxHCUzjudMSki9DdKWsIE2Udvo6EnzglJ4MgmSj/w9d6hhRJ WJyL9W0UVotddnwWYdh/2XiWxo+1fc79QNyaWZAX5ixIrGyIAw3UvxvBtrMORYKMzkgn 41j4CD5S84LTGvzvu+QEoyWcKQ7vhNl0XokgyLvIRj6qB/izajBNNOvVI4XGfHInPSd1 l93DA0wGQIEK8OfAS6chaGJluhytzNyZBhQh3v8/3IJVb6WVFuHtAaiWwHlEtPV/Pl0L pVbPgpG8fHpKfBgBi/NsL0TdLnJyj2akXtz44V8/65vGqIVZCp5ts0fmVnOaFlsrA+JL JZfQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786010825; x=1786615625; h=content-transfer-encoding:content-type:in-reply-to:content-language :from: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:content-type; bh=EJsvtNqHDCxAmT1UUowASSNNS2LeJPAA9Hw6UmnAiyg=; b=l/iPhGux9mT81KkpI0dueNoG+CaS5NzTa+4PIot3jPRbe2G/m9Yo2jxBz4TRO2Zq/Y VVJ4Ne0d8MGMnd7270TfWk3N4cOrjYtG9qTrSh+owrdcCULGOFniAA3no9RFWmIlI3im 4pXKFC3p0dBi6R/msWh0lS5yQmxNw9t+AY63ot0AafgrgzSXcbNH8jBpTQuW3UwpaOVl YLjYGKyBvVCWy7lGIwV/tH3YQggKO053pb/HGMk55ugqHtK0gnT/7A5vaTIRMBYPW/Iy WNyzIS1H1lUyEpH+tBS3rHF3hRAn8d9Jn/1FfcZ5oQAiPeGO+zYTYLvGOFt9H9NaXwx7 MjYA== X-Forwarded-Encrypted: i=1; AHgh+RrWdCdUsXvZr1HCAeoCs5yhGnWX5y/Bl+bNH3czeq30nyZQSnmY7k5CgcmyZvlzE+L89hE5mSoEs4y0@vger.kernel.org X-Gm-Message-State: AOJu0YwaWHHYYSaRxSuPQQpTydaDEz7kc0iZk3T2gbjpOrc4exbTsmei DgMfnnlKeDeeU4rXdGhstBzuV/UfkWszvtJ5GCS3ABYXfkJeQDS6Regklddx4BfD1l4= X-Gm-Gg: AR+sD12MkmO81Dd5/TfgcjhKwtrIgMX5aVQcG9JXoQqgv9TY29kKyLWfakWYlLIyphz STYaNGDyxljdMf45Lz/6QlUejHsRehIwigz/zy7fB5A6boX7BiVD7nlH3gtCxfLunh/Ma2ja0Xf +YzpP4nEw2Jcgy3w8G5Nzk1B8RXYm47IWXnUZfuGqRGqNMASgJTjrTtVNYa5NPnumGUEjhwJqy5 xYphRtZmTgBmqqP5ntXjnRm2bfHemFl0auZsLeDchOVLl8w3yi/CjrYn1XkNQX+yF23naj9M4D0 D1T4Zm5dr8CShgzBoZ2ASzQOM3RrGqiPRPLJVc1jOFXkqu1MoSYa2ugqDhTDHrZG+fQVT7IvOoL 7mb/sek/RkZuD68r0ReIgtqvfJ1QvzV2TESIL2iCZS5Vnvd4dn6PxDf5ipGenNi29EQYLf2R0li qDDsdarTH0D6BRFD3Iq5oR6O60KzCpN+Tp2XgEtRxXh0KmL8PHLnZGhRAmh9Hh4AtrP6sn6GrV6 NIEOGPi66oT X-Received: by 2002:a05:600c:1f96:b0:496:c249:dd9a with SMTP id 5b1f17b1804b1-4994e71364fmr167132715e9.3.1786010825341; Thu, 06 Aug 2026 03:07:05 -0700 (PDT) Received: from [192.168.0.101] ([109.76.115.165]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49954213479sm48108625e9.6.2026.08.06.03.07.03 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 06 Aug 2026 03:07:04 -0700 (PDT) Message-ID: Date: Thu, 6 Aug 2026 11:07:03 +0100 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 01/12] dt-bindings: media: qcom,venus: Add context bank subnodes to common schema To: Bryan O'Donoghue , Vikash Garodia , Dikshita Agarwal , Abhinav Kumar , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Konrad Dybcio , Stanimir Varbanov , Neil Armstrong , Dmitry Baryshkov , Stephan Gerhold Cc: linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Konrad Dybcio , stable@vger.kernel.org, Daniel J Blueman References: <20260731-vpu_iommu_iova_handling-v2-0-da52b5228dbd@oss.qualcomm.com> <20260731-vpu_iommu_iova_handling-v2-1-da52b5228dbd@oss.qualcomm.com> <689580d1-8169-41cf-a954-927e10135700@kernel.org> <8130900b-c9f2-4f5a-80f1-1e8567ee1b72@oss.qualcomm.com> From: Bryan O'Donoghue Content-Language: en-GB In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 06/08/2026 10:40, Bryan O'Donoghue wrote: > On 06/08/2026 08:57, Vikash Garodia wrote: >> On 8/6/2026 1:24 PM, Bryan O'Donoghue wrote: >>> On 06/08/2026 06:21, Vikash Garodia wrote: >>>> On 8/1/2026 3:36 PM, Bryan O'Donoghue wrote: >>>>> On 31/07/2026 19:22, Vikash Garodia wrote: >>>>>> Fixes: 41661853ae8e ("arm64: dts: qcom: sm8550: add iris DT node") >>>>>> Cc:stable@vger.kernel.org >>>>>> Tested-by: Daniel J Blueman >>>>>> Signed-off-by: Vikash Garodia >>>>> What about venus ? >>>> Plan is to get the reviews for IRIS3 family SOCs, once we have a go >>>> ahead from binding/DT maintainers, i can either bring in few series >>>> for remaining venus and iris SOC family OR do all of them in one >>>> single series. My preference would be to add in multiple series. >>> I still don't see why we can't just set >>> >>> minItems: 1 for the memory nodes - implement a fix for stable and then >>> iterate around this new way at leisure. >> you can follow the guidance from binding maintainer [1] and continue to >> discuss in the same thread if you have any question. >> >> [1] >> https://lore.kernel.org/linux-media/2b0ea2a2-c3ef-4147-801b- >> af41733fba7b@kernel.org/ > > LoL seriously - system resetting bug. > > How long exactly is this going to take - do you have buy in for sub- > nodes and when will that get merged. > > For reference I do not care what happens my concern is an unspecified > window where stuff remains broken and we take our leisurely time about > doing something about it. > > If we can't target something for -stable then this driver and venus > should be marked as BROKEN. > > --- > bod I'd just like to be clear. As far as I understand it a bug that can reset the system has been known about for months on the qcom side with little action to remediate in -stable. That's entirely unacceptable. Litigating DT bindings is _not_ addressing the fact of life, that a system resetting bug has been sat on for months, unaddressed. Daniel proposed a fix that will stop that reset which is being rejected? Arguments are being made to not mark this stuff as broken. And the debate about how a DT change should land is still ongoing in this thread. I tend to think - absent an obvious an immediately applicable fix these two drivers must be marked BROKEN, the alternative is to apply Daniel's fix and then live with the results for the bindings. Since the DT schema change is still under debate - what alternative is there ? I'm not far off sending that patch now myself because I'm not seeing urgency on fixing this and TBH that's pissing me off. As a user of a system, I expect resonsible and honest behaviour wrt to bugs, not architecture astronauty while my system can still reset despite perfectly functional fixes being available. Great you want to do sub-nodes in DT. Not great, that's not agreed yet, even more not great - the venus stuff isn't even on the list. Please indicate what should be done for -stable this week else both of these drivers need to be marked BROKEN. --- bod