| Message ID | 20240624-oneplus8-v1-0-388eecf2dff7@postmarketos.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | qcom: initial support for the OnePlus 8T | expand |
On Mon, Jun 24, 2024 at 03:30:24AM GMT, Caleb Connolly wrote: > Add bindings for the SM8250 OnePlus devices, a common devicetree, > touchscreen and display drivers, and a dts for the OnePlus 8T (kebab). > > The OnePlus 8 series is made up of 3 flagship smartphones from 2019, > featuring the Qualcomm X55 5G PCIe modem. > > This series introduces initial support for the 8T, adding drivers for > the 1080x2400 120Hz DSC panel and the Synaptics TCM Oncell touchscreen. > > The panel driver suffers from similar limitations to the LG SW43408 > panel found on the Pixel 3, namely that after toggling the reset GPIO it > is not possible to get the panel into a working state. Just to point it out: this is no longer true for SW43408. The panel wakes up and works after toggling the reset. It seems, there is an issue with one of the regulators, but not with the reset and/or panel startup. > Given the apparent prevelance of this issue, particularly with DSC > panels, I believe this is a bug in the core DSI code, and not a device > or panel specific issue. I think it is still useful to accept these > panel drivers into upstream since, from a users perspective, the panel > is fully functional just by leaving the reset GPIO alone and keeping the > regulator on. The only (theoretical) downside is worse battery life, > which is a small price to pay for a working display. >
On 24/06/2024 07:18, Dmitry Baryshkov wrote: > On Mon, Jun 24, 2024 at 03:30:24AM GMT, Caleb Connolly wrote: >> Add bindings for the SM8250 OnePlus devices, a common devicetree, >> touchscreen and display drivers, and a dts for the OnePlus 8T (kebab). >> >> The OnePlus 8 series is made up of 3 flagship smartphones from 2019, >> featuring the Qualcomm X55 5G PCIe modem. >> >> This series introduces initial support for the 8T, adding drivers for >> the 1080x2400 120Hz DSC panel and the Synaptics TCM Oncell touchscreen. >> >> The panel driver suffers from similar limitations to the LG SW43408 >> panel found on the Pixel 3, namely that after toggling the reset GPIO it >> is not possible to get the panel into a working state. > > Just to point it out: this is no longer true for SW43408. The panel > wakes up and works after toggling the reset. It seems, there is an issue > with one of the regulators, but not with the reset and/or panel startup. Hmm ok, I've heard mixed reports then, I should try it out myself again. I'll update the cover letter to reflect this. Thanks. > >> Given the apparent prevelance of this issue, particularly with DSC >> panels, I believe this is a bug in the core DSI code, and not a device >> or panel specific issue. I think it is still useful to accept these >> panel drivers into upstream since, from a users perspective, the panel >> is fully functional just by leaving the reset GPIO alone and keeping the >> regulator on. The only (theoretical) downside is worse battery life, >> which is a small price to pay for a working display. >> >
On Mon, 24 Jun 2024 at 14:33, Caleb Connolly <caleb@postmarketos.org> wrote: > > > > On 24/06/2024 07:18, Dmitry Baryshkov wrote: > > On Mon, Jun 24, 2024 at 03:30:24AM GMT, Caleb Connolly wrote: > >> Add bindings for the SM8250 OnePlus devices, a common devicetree, > >> touchscreen and display drivers, and a dts for the OnePlus 8T (kebab). > >> > >> The OnePlus 8 series is made up of 3 flagship smartphones from 2019, > >> featuring the Qualcomm X55 5G PCIe modem. > >> > >> This series introduces initial support for the 8T, adding drivers for > >> the 1080x2400 120Hz DSC panel and the Synaptics TCM Oncell touchscreen. > >> > >> The panel driver suffers from similar limitations to the LG SW43408 > >> panel found on the Pixel 3, namely that after toggling the reset GPIO it > >> is not possible to get the panel into a working state. > > > > Just to point it out: this is no longer true for SW43408. The panel > > wakes up and works after toggling the reset. It seems, there is an issue > > with one of the regulators, but not with the reset and/or panel startup. > > Hmm ok, I've heard mixed reports then, I should try it out myself again. During MAD24 we have seen an issue with the regulators, when the panel doesn't wake up. But resetting the panel works. > > I'll update the cover letter to reflect this. Thanks. > > > >> Given the apparent prevelance of this issue, particularly with DSC > >> panels, I believe this is a bug in the core DSI code, and not a device > >> or panel specific issue. I think it is still useful to accept these > >> panel drivers into upstream since, from a users perspective, the panel > >> is fully functional just by leaving the reset GPIO alone and keeping the > >> regulator on. The only (theoretical) downside is worse battery life, > >> which is a small price to pay for a working display. > >> > >
Add bindings for the SM8250 OnePlus devices, a common devicetree, touchscreen and display drivers, and a dts for the OnePlus 8T (kebab). The OnePlus 8 series is made up of 3 flagship smartphones from 2019, featuring the Qualcomm X55 5G PCIe modem. This series introduces initial support for the 8T, adding drivers for the 1080x2400 120Hz DSC panel and the Synaptics TCM Oncell touchscreen. The panel driver suffers from similar limitations to the LG SW43408 panel found on the Pixel 3, namely that after toggling the reset GPIO it is not possible to get the panel into a working state. Given the apparent prevelance of this issue, particularly with DSC panels, I believe this is a bug in the core DSI code, and not a device or panel specific issue. I think it is still useful to accept these panel drivers into upstream since, from a users perspective, the panel is fully functional just by leaving the reset GPIO alone and keeping the regulator on. The only (theoretical) downside is worse battery life, which is a small price to pay for a working display. The Synaptics TCM Oncell touchscreens are a new generation of Synaptics controllers with a totally incompatible firmware compared to the older rmi4 touchscreens. A new driver is written which currently only supports the S3908 controller found on the OnePlus 8T. Downstream vendor drivers suggest that the controller supports custom touch report configuration, one can define the exact bit packing of the touch reports, however the combination of controller and firmware available on this device does not allow for programming in cusotm configs, so for simplicity this initial driver uses a hardcoded bit packing to decode the touch reports. With this series, the OnePlus 8T can boot up to GNOME shell, connect to a wifi network and browse the web with GPU acceleration. The touchscreen driver included here is loosely based on a previous attempt by Frieder Hannenheim which can be found below. Link: https://lore.kernel.org/lkml/20240327214643.7055-1-friederhannenheim@riseup.net/ --- Caleb Connolly (7): dt-bindings: panel: document Samsung AMB655X dt-bindings: input: touchscreen: document synaptics TCM oncell dt-bindings: arm: qcom: add OnePlus 8 series drm: mipi: add mipi_dsi_generic_write_multi_type() drm/panel: add driver for samsung amb655x Input: touchscreen: add Synaptics TCM oncell S3908 arm64: dts: qcom: add OnePlus 8T (kebab) Documentation/devicetree/bindings/arm/qcom.yaml | 3 + .../bindings/display/panel/samsung,amb655x.yaml | 59 ++ .../input/touchscreen/syna,tcm-oncell.yaml | 66 ++ MAINTAINERS | 14 + arch/arm64/boot/dts/qcom/Makefile | 1 + .../arm64/boot/dts/qcom/sm8250-oneplus-common.dtsi | 866 +++++++++++++++++++++ arch/arm64/boot/dts/qcom/sm8250-oneplus-kebab.dts | 36 + drivers/gpu/drm/drm_mipi_dsi.c | 40 + drivers/gpu/drm/panel/Kconfig | 9 + drivers/gpu/drm/panel/Makefile | 1 + drivers/gpu/drm/panel/panel-samsung-amb655x.c | 420 ++++++++++ drivers/input/touchscreen/Kconfig | 11 + drivers/input/touchscreen/Makefile | 1 + drivers/input/touchscreen/synaptics_tcm_oncell.c | 617 +++++++++++++++ include/drm/drm_mipi_dsi.h | 16 + 15 files changed, 2160 insertions(+) --- change-id: 20240622-oneplus8-788005cc74ae base-commit: f76698bd9a8ca01d3581236082d786e9a6b72bb7 // Caleb (they/them)