mbox series

[v2,0/2] drm: bridge: add ssd2825 RGB/DSI bridge support

Message ID 20250217140910.108175-1-clamor95@gmail.com (mailing list archive)
Headers show
Series drm: bridge: add ssd2825 RGB/DSI bridge support | expand

Message

Svyatoslav Ryhel Feb. 17, 2025, 2:09 p.m. UTC
Solomon SSD2825 is a RGB to MIPI DSI bridge used in LG Optimus 4D P880
and LG Optimus Vu P895

---
Changes on switching from v1 to v2:
- added description for clock
- removed clock-names
- added boundries for hs-zero-delay-ns and hs-prep-delay-ns
- added mutex lock for host transfers
- converted to atomic ops
- get drm_display_mode mode with atomic helpers
- parameterized INTERFACE_CTRL_REG_6 configuration
- added video mode validation and fixup
- removed clock name
- switched to devm_regulator_bulk_get_const
- added default timings
---

Svyatoslav Ryhel (2):
  dt-bindings: display: bridge: Document Solomon SSD2825
  drm: bridge: Add support for Solomon SSD2825 RGB/DSI bridge

 .../display/bridge/solomon,ssd2825.yaml       | 141 +++
 drivers/gpu/drm/bridge/Kconfig                |  14 +
 drivers/gpu/drm/bridge/Makefile               |   1 +
 drivers/gpu/drm/bridge/ssd2825.c              | 824 ++++++++++++++++++
 4 files changed, 980 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/bridge/solomon,ssd2825.yaml
 create mode 100644 drivers/gpu/drm/bridge/ssd2825.c

Comments

Thomas Zimmermann Feb. 18, 2025, 12:36 p.m. UTC | #1
(cc'ing Javier)

Hi

Am 17.02.25 um 15:09 schrieb Svyatoslav Ryhel:
> Solomon SSD2825 is a RGB to MIPI DSI bridge used in LG Optimus 4D P880
> and LG Optimus Vu P895

There's a driver for Solomon 13xx displays in drm/solomon. Did you check 
that this new driver isn't just an extension of the existing code?

Best regards
Thomas

>
> ---
> Changes on switching from v1 to v2:
> - added description for clock
> - removed clock-names
> - added boundries for hs-zero-delay-ns and hs-prep-delay-ns
> - added mutex lock for host transfers
> - converted to atomic ops
> - get drm_display_mode mode with atomic helpers
> - parameterized INTERFACE_CTRL_REG_6 configuration
> - added video mode validation and fixup
> - removed clock name
> - switched to devm_regulator_bulk_get_const
> - added default timings
> ---
>
> Svyatoslav Ryhel (2):
>    dt-bindings: display: bridge: Document Solomon SSD2825
>    drm: bridge: Add support for Solomon SSD2825 RGB/DSI bridge
>
>   .../display/bridge/solomon,ssd2825.yaml       | 141 +++
>   drivers/gpu/drm/bridge/Kconfig                |  14 +
>   drivers/gpu/drm/bridge/Makefile               |   1 +
>   drivers/gpu/drm/bridge/ssd2825.c              | 824 ++++++++++++++++++
>   4 files changed, 980 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/display/bridge/solomon,ssd2825.yaml
>   create mode 100644 drivers/gpu/drm/bridge/ssd2825.c
>
Svyatoslav Ryhel Feb. 18, 2025, 12:48 p.m. UTC | #2
вт, 18 лют. 2025 р. о 14:36 Thomas Zimmermann <tzimmermann@suse.de> пише:
>
> (cc'ing Javier)
>
> Hi
>
> Am 17.02.25 um 15:09 schrieb Svyatoslav Ryhel:
> > Solomon SSD2825 is a RGB to MIPI DSI bridge used in LG Optimus 4D P880
> > and LG Optimus Vu P895
>
> There's a driver for Solomon 13xx displays in drm/solomon. Did you check
> that this new driver isn't just an extension of the existing code?
>

Definitely no, ssd2825 is a RGB to DSI bridge in a pure form. While
13xx as you have said are display controllers family.

> Best regards
> Thomas
>
> >
> > ---
> > Changes on switching from v1 to v2:
> > - added description for clock
> > - removed clock-names
> > - added boundries for hs-zero-delay-ns and hs-prep-delay-ns
> > - added mutex lock for host transfers
> > - converted to atomic ops
> > - get drm_display_mode mode with atomic helpers
> > - parameterized INTERFACE_CTRL_REG_6 configuration
> > - added video mode validation and fixup
> > - removed clock name
> > - switched to devm_regulator_bulk_get_const
> > - added default timings
> > ---
> >
> > Svyatoslav Ryhel (2):
> >    dt-bindings: display: bridge: Document Solomon SSD2825
> >    drm: bridge: Add support for Solomon SSD2825 RGB/DSI bridge
> >
> >   .../display/bridge/solomon,ssd2825.yaml       | 141 +++
> >   drivers/gpu/drm/bridge/Kconfig                |  14 +
> >   drivers/gpu/drm/bridge/Makefile               |   1 +
> >   drivers/gpu/drm/bridge/ssd2825.c              | 824 ++++++++++++++++++
> >   4 files changed, 980 insertions(+)
> >   create mode 100644 Documentation/devicetree/bindings/display/bridge/solomon,ssd2825.yaml
> >   create mode 100644 drivers/gpu/drm/bridge/ssd2825.c
> >
>
> --
> --
> Thomas Zimmermann
> Graphics Driver Developer
> SUSE Software Solutions Germany GmbH
> Frankenstrasse 146, 90461 Nuernberg, Germany
> GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
> HRB 36809 (AG Nuernberg)
>
Javier Martinez Canillas Feb. 18, 2025, 1:09 p.m. UTC | #3
Svyatoslav Ryhel <clamor95@gmail.com> writes:

> вт, 18 лют. 2025 р. о 14:36 Thomas Zimmermann <tzimmermann@suse.de> пише:
>>
>> (cc'ing Javier)
>>
>> Hi
>>
>> Am 17.02.25 um 15:09 schrieb Svyatoslav Ryhel:
>> > Solomon SSD2825 is a RGB to MIPI DSI bridge used in LG Optimus 4D P880
>> > and LG Optimus Vu P895
>>
>> There's a driver for Solomon 13xx displays in drm/solomon. Did you check
>> that this new driver isn't just an extension of the existing code?
>>
>
> Definitely no, ssd2825 is a RGB to DSI bridge in a pure form. While
> 13xx as you have said are display controllers family.
>

Exactly. Both chips are from the same vendor (Solomon Systech [0]) and my
guess is that the SSD prefix means "Solomon Systech Device" or something
like that.

[0]: https://www.solomon-systech.com
Svyatoslav Ryhel Feb. 18, 2025, 1:11 p.m. UTC | #4
вт, 18 лют. 2025 р. о 15:09 Javier Martinez Canillas <javierm@redhat.com> пише:
>
> Svyatoslav Ryhel <clamor95@gmail.com> writes:
>
> > вт, 18 лют. 2025 р. о 14:36 Thomas Zimmermann <tzimmermann@suse.de> пише:
> >>
> >> (cc'ing Javier)
> >>
> >> Hi
> >>
> >> Am 17.02.25 um 15:09 schrieb Svyatoslav Ryhel:
> >> > Solomon SSD2825 is a RGB to MIPI DSI bridge used in LG Optimus 4D P880
> >> > and LG Optimus Vu P895
> >>
> >> There's a driver for Solomon 13xx displays in drm/solomon. Did you check
> >> that this new driver isn't just an extension of the existing code?
> >>
> >
> > Definitely no, ssd2825 is a RGB to DSI bridge in a pure form. While
> > 13xx as you have said are display controllers family.
> >
>
> Exactly. Both chips are from the same vendor (Solomon Systech [0]) and my
> guess is that the SSD prefix means "Solomon Systech Device" or something
> like that.
>
> [0]: https://www.solomon-systech.com
>

You are correct, vendor is the same, I have not denied that, but the
this device is not related to product drivers which already exist in
the Linux Kernel.

> --
> Best regards,
>
> Javier Martinez Canillas
> Core Platforms
> Red Hat
>
Javier Martinez Canillas Feb. 18, 2025, 1:21 p.m. UTC | #5
Svyatoslav Ryhel <clamor95@gmail.com> writes:

[...]

>> Exactly. Both chips are from the same vendor (Solomon Systech [0]) and my
>> guess is that the SSD prefix means "Solomon Systech Device" or something
>> like that.
>>
>> [0]: https://www.solomon-systech.com
>>
>
> You are correct, vendor is the same, I have not denied that, but the
> this device is not related to product drivers which already exist in
> the Linux Kernel.
>

Oh yes, I didn't mean that are related.

I was just explaining to Thomas why both devices were named "Solomon SSD*"
even when they are quite different and can't be supported by same driver.