From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.14]) (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 BB2943C1D4A; Tue, 29 Sep 2026 09:04:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.14 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790672658; cv=none; b=RQ2c7GEahbYRQUpiQfBdiTbieKq7eyOhnBvXr+aX0jftzSOXjOemELvvwtjRq3eteX9+JUwsCMVyYXnZ6tcesQdnYZrCWQ0MxiWfMJIl53EIx3XNDQizSUH1LHJOSFwY0OiZD8tpSMSrQ0MVLp2i/bO39iikHkkIgn04EfPCpEc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790672658; c=relaxed/simple; bh=fjTgkfgps1Gk7j6xMRcB1fFz/DjnVOKoczfByJwdpzQ=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=UqPwxmUE50GHn3a3aT7raF/4zVuUMMHC6k4nrZdpJ5cp9X1nbrSOthUa77fBzeuebuQYz1nw9tRaQxg6Yu1HAPr79eYqwTQHTAq3agB/gz5tpjVMzoWE0e6i6WHTx0CHQwBRK3TdWZZlGT7WnDSl9qJxJy0Flb8mpqjeT9unflk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=WKt/pw4M; arc=none smtp.client-ip=192.198.163.14 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="WKt/pw4M" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1790672649; x=1822208649; h=from:to:cc:subject:in-reply-to:references:date: message-id:mime-version; bh=fjTgkfgps1Gk7j6xMRcB1fFz/DjnVOKoczfByJwdpzQ=; b=WKt/pw4M9U5RBLaUy65VJzDu8PEnPHnWr2Hxa2k170+WBSryNlIgrO63 HgNuyREUkxycLndkvLJ23ElTJJszrgi40ATtOIVq2i/VJUOCu/B5PS99L kXhoIyCVQtPjZNYDIWD5rSK0PMV7RjyaP7lB0Ye5b4egMIlRzdnlnddij TSyj/4afNa2DUqDCEuu3Jg9osvL+c85Z5oz75ALZGaRZGtsIhmtxXhDFX GMnr3UPqI5jhFnF3okaLFRL4RP1dHOsIh+JQY5kAagn8JMjcLQbl325OG Qs61viXYtaYOSiGBmrGL8eLasxrCvR++9Ar2ALRDM5sLYcSYBVhhdI8Gy Q==; X-CSE-ConnectionGUID: ygR5LGxBTpqD5065wZVEIg== X-CSE-MsgGUID: gVJFI2GyQlq9AIIFBQPe0w== X-IronPort-AV: E=McAfee;i="6800,10657,11919"; a="91405278" X-IronPort-AV: E=Sophos;i="6.27,130,1787036400"; d="scan'208";a="91405278" Received: from fmviesa009.fm.intel.com ([10.60.135.149]) by fmvoesa108.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Sep 2026 02:04:03 -0700 X-CSE-ConnectionGUID: L4F6TTbPScOGpDGQ/vD63w== X-CSE-MsgGUID: px97l+LIQsul4lmsr50Xug== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.27,130,1787036400"; d="scan'208";a="271804384" Received: from kniemiec-mobl1.ger.corp.intel.com (HELO localhost) ([10.245.244.233]) by fmviesa009-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Sep 2026 02:03:55 -0700 From: Jani Nikula To: Maxime Ripard , Neil Armstrong , Jessica Zhang , David Airlie , Simona Vetter , Maarten Lankhorst , Thomas Zimmermann , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Nathan Chancellor , Nick Desaulniers , Bill Wendling , Justin Stitt , Florian Fainelli , Broadcom internal kernel review list Cc: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Luca Ceresoli , Albert Esteve , Dave Stevenson , Javier Martinez Canillas , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org, llvm@lists.linux.dev, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, Maxime Ripard , Benjamin Tissoires Subject: Re: [PATCH 0/6] drm/bridge: Add a BPF-based MIPI-DSI panel driver In-Reply-To: <20260928-drm-mipi-dsi-panel-ebpf-v1-0-5244926aace4@kernel.org> Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs Bertel Jungin Aukio 5, 02600 Espoo, Finland References: <20260928-drm-mipi-dsi-panel-ebpf-v1-0-5244926aace4@kernel.org> Date: Tue, 29 Sep 2026 12:03:52 +0300 Message-ID: <054ef52478fc93fc213a9dae23ea77f73094aaf2@intel.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain On Mon, 28 Sep 2026, Maxime Ripard wrote: > Panels in general, and MIPI-DSI panels in particular, are pretty > difficult to support and require pretty much a panel driver for each > panel produced. Most of them are pretty simple, and require an opaque > initialization sequence that is usually poorly documented. > > This creates a tension between OEMs and distros because OEMs will > typically get a new panel to react to a sourcing issue during > production, and thus need some swift turnaround between getting their > new panel and it being operational in the OS. Distributions on the other > hand can take years to ship a kernel with that new panel driver. > > To solve this, I followed the example of HID-BPF and wrote a panel > driver that will rely on BPF programs to perform the panel > initialization. That way, we can ship the programs separately from the > kernel, and with a different lifecycle. If this driver is accepted, the > plan is to have a userspace component started by udev to identify and > load the right BPF program for the panels found on the device. I'd think for most panels it would suffice to have a declarative language describing what to do at what points in time. For example, at poweron, enable this GPIO, wait a little, write this set of DSI commands, etc. You rarely need actual programming, or even ability to read-modify-write something. It could be, say, YAML converted to some binary representation. Maybe that could be in the DT, or maybe you could override it with the firmware loader. And obviously any vendor "blob" could be trivially converted back to YAML and upstreamed. Where that falls short, you could always have a dedicated driver, or extend the declarative stuff. Now, the question is, how much more BPF solves over that, without requiring a dedicated driver, and is the added complexity worth it? FWIW, on the Intel platforms that support DSI, all of the initialization, poweron/poweroff, backlight on/off etc. sequences like that are stored in the BIOS, defined by the OEM. A plethora of DSI panels, one driver. Don't look at it for examples how to implement it, but I think the basic idea is workable. BR, Jani. -- Jani Nikula, Intel