Skip to content

Add new panel driver Samsung S6E88A0-AMS427AP24

Jakob Hauser dri-devel linux-devicetree lkml 4 versions – Applied

Status from this message: Thanks, Applied to https://gitlab.freedesktop.org/drm/misc/kernel.git (drm-misc-next)

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest5 patches 4 repliesApplied
  2. v35 patches 8 repliesSuperseded
  3. v25 patches 6 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v14 patches6 repliesSuperseded

Patches in v4 Applied

#Subject and filesSizeRepliesApplied
1/5dt-bindings: display: panel: Move flip properties to panel-commonDocumentation/devicetree/bindings/display/panel/panel-common.yaml Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml+10 -80 replies
2/5dt-bindings: display: panel: Add Samsung S6E88A0-AMS427AP24Documentation/devicetree/bindings/display/panel/samsung,s6e88a0-ams427ap24.yaml+65 -00 replies
3/5drm/panel: samsung-s6e88a0-ams427ap24: Add initial driverdrivers/gpu/drm/panel/Kconfig drivers/gpu/drm/panel/Makefile drivers/gpu/drm/panel/panel-samsung-s6e88a0-ams427ap24.c+261 -01 reply
4/5drm/panel: samsung-s6e88a0-ams427ap24: Add brightness controldrivers/gpu/drm/panel/Kconfig drivers/gpu/drm/panel/panel-samsung-s6e88a0-ams427ap24.c+521 -121 reply
5/5drm/panel: samsung-s6e88a0-ams427ap24: Add flip optiondrivers/gpu/drm/panel/panel-samsung-s6e88a0-ams427ap24.c+7 -01 reply

Reviews trailers per patch

All 5 patches have a Reviewed-by or Acked-by.

PatchKrzysztof KozlowskiLinus WalleijNeil Armstrong
1/5R-b R-b
2/5R-b R-b
3/5R-b
4/5R-b R-b
5/5R-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v4 thread

Cover letter [PATCH v4 0/5] Add new panel driver Samsung S6E88A0-AMS427AP24

The patchset adds a new driver for Samsung AMS427AP24 panel with S6E88A0 controller. Patches are based on current branch drm-misc-next.

The discussion in v3 was not finished. I send v4 nonetheless as a new base to continue the discussion. I hope that's ok.

Changes in v4:

Patch 3
- Removed include <linux/of.h>, it's not needed.
- Added comments to the mipi_dsi_dcs_write_seq_multi() lines in function s6e88a0_ams427ap24_on().

Patch 4
- New defines for payload data length, thereof a fixed and a variable part.
- In the array s6e88a0_ams427ap24_br_to_cd[] made the comment about the columns more clear and added an additional column "brightness from".
- Array s6e88a0_ams427ap24_aid[] reduced from five to two columns and s6e88a0_ams427ap24_elvss[] from two to one column. Now they hold the variable data only. I kept s6e88a0_ams427ap24_elvss[] as a two-dimensional array to allow using the same memcopy procedure for all three buffers aid, elvss and gamma in function s6e88a0_ams427ap24_set_brightness().
- In function s6e88a0_ams427ap24_set_brightness() initialized prototype payload array for b2 and b6 commands. And in the memcpy commands below used the new defines for the fixed and variable data length.

Patch 5
- Added a comment to the mipi_dsi_dcs_write_seq_multi() line, according to the other lines before and after.

v1: https://lore.kernel.org/dri-devel/[email protected]/T/#t
v2: https://lore.kernel.org/dri-devel/[email protected]/T/#t
v3: https://lore.kernel.org/dri-devel/[email protected]/T/#t

Jakob Hauser (5):
dt-bindings: display: panel: Move flip properties to panel-common
dt-bindings: display: panel: Add Samsung S6E88A0-AMS427AP24 drm/panel: samsung-s6e88a0-ams427ap24: Add initial driver drm/panel: samsung-s6e88a0-ams427ap24: Add brightness control drm/panel: samsung-s6e88a0-ams427ap24: Add flip option

 .../bindings/display/panel/panel-common.yaml  |   8 +
 .../panel/samsung,s6e88a0-ams427ap24.yaml     |  65 ++
 .../display/panel/samsung,s6e8aa0.yaml        |  10 +-
 drivers/gpu/drm/panel/Kconfig                 |  10 +
 drivers/gpu/drm/panel/Makefile                |   1 +
 .../panel/panel-samsung-s6e88a0-ams427ap24.c  | 766 ++++++++++++++++++
 6 files changed, 852 insertions(+), 8 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/display/panel/samsung,s6e88a0-ams427ap24.yaml
 create mode 100644 drivers/gpu/drm/panel/panel-samsung-s6e88a0-ams427ap24.c
Signature · 1 line
2.39.5