Skip to content

drm/tilcdc: LCDC Revision 1 related fixes

Jyri Sarha dri-devel 4 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest10 patches 1 replyIn review
  2. v311 patches 18 repliesSuperseded
  3. v27 patches 11 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v17 patches2 repliesSuperseded

Changes in v4 from the cover letter

- "drm/tilcdc: Enable sync lost error and recovery handling for rev 1 LCDC"
- disable sync-lost irq also for rev1 LCDC
- LCDC_V1_SYNC_LOST_ENA to LCDC_V1_SYNC_LOST_INT_ENA
- "drm/tilcdc: Enable palette loading for revision 2 LCDC too"
- disable palette loaded interrupt after receiving it
- "drm/tilcdc: Enable frame done irq and functionality for LCDC rev 1"
- disable FRAME_DONE interrupt for rev 1 lcdc in interrupt routine
- LCDC_V1_FRAME_DONE_ENA to LCDC_V1_FRAME_DONE_INT_ENA
- Add: "drm/tilcdc: Configure video mode to HW in enable() not in mode_set_nofb()"
- Drop "drm/tilcdc: Use complete_all() to indicate completed palette loading"
- Drop "drm/tilcdc: Call reset() before loading the palette"
- "drm/tilcdc: Load palette at the end of mode_set_nofb()"
- Just load the palette every time, no runtime_resume hook

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10drm/tilcdc: Enable sync lost error and recovery handling for rev 1 LCDCdrivers/gpu/drm/tilcdc/tilcdc_crtc.c drivers/gpu/drm/tilcdc/tilcdc_regs.h+21 -150 replies
2/10drm/tilcdc: implement palette loading for rev1drivers/gpu/drm/tilcdc/tilcdc_crtc.c+87 -10 replies
3/10drm/tilcdc: Fix tilcdc_crtc_create() return value handlingdrivers/gpu/drm/tilcdc/tilcdc_crtc.c drivers/gpu/drm/tilcdc/tilcdc_drv.c drivers/gpu/drm/tilcdc/tilcdc_drv.h+12 -130 replies
4/10drm/tilcdc: Add tilcdc_write_mask() to tilcdc_regs.hdrivers/gpu/drm/tilcdc/tilcdc_regs.h+13 -00 replies
5/10drm/tilcdc: Fix load mode bit-field setting in tilcdc_crtc_enable()drivers/gpu/drm/tilcdc/tilcdc_crtc.c+3 -10 replies
6/10drm/tilcdc: Enable palette loading for revision 2 LCDC toodrivers/gpu/drm/tilcdc/tilcdc_crtc.c+47 -430 replies
7/10drm/tilcdc: Add timeout wait for palette loading to completedrivers/gpu/drm/tilcdc/tilcdc_crtc.c+5 -10 replies
8/10drm/tilcdc: Load palette at the end of mode_set_nofb()drivers/gpu/drm/tilcdc/tilcdc_crtc.c+5 -200 replies
9/10drm/tilcdc: Configure video mode to HW in enable() not in mode_set_nofb()drivers/gpu/drm/tilcdc/tilcdc_crtc.c drivers/gpu/drm/tilcdc/tilcdc_drv.c+212 -2200 replies
10/10drm/tilcdc: Enable frame done irq and functionality for LCDC rev 1drivers/gpu/drm/tilcdc/tilcdc_crtc.c drivers/gpu/drm/tilcdc/tilcdc_regs.h+12 -70 replies

Reviews trailers per patch

None of the 10 patches has a Reviewed-by or Acked-by.

PatchBartosz Golaszewski
SeriesT-b
1/10
2/10
3/10
4/10
5/10
6/10
7/10
8/10
9/10
10/10

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v4 thread

Cover letter [PATCH v4 00/10] drm/tilcdc: LCDC Revision 1 related fixes

The git branch bellow is updated.

Changes since v3:
- "drm/tilcdc: Enable sync lost error and recovery handling for rev 1 LCDC"
- disable sync-lost irq also for rev1 LCDC
- LCDC_V1_SYNC_LOST_ENA to LCDC_V1_SYNC_LOST_INT_ENA
- "drm/tilcdc: Enable palette loading for revision 2 LCDC too"
- disable palette loaded interrupt after receiving it
- "drm/tilcdc: Enable frame done irq and functionality for LCDC rev 1"
- disable FRAME_DONE interrupt for rev 1 lcdc in interrupt routine
- LCDC_V1_FRAME_DONE_ENA to LCDC_V1_FRAME_DONE_INT_ENA
- Add: "drm/tilcdc: Configure video mode to HW in enable() not in mode_set_nofb()"
- Drop "drm/tilcdc: Use complete_all() to indicate completed palette loading"
- Drop "drm/tilcdc: Call reset() before loading the palette"
- "drm/tilcdc: Load palette at the end of mode_set_nofb()"
- Just load the palette every time, no runtime_resume hook

Changes since v2:
- Add: "drm/tilcdc: Fix load mode bit-field setting in tilcdc_crtc_enable()"
- Drop: "drm/tilcdc: Free palette dma memory in tilcdc_crtc_destroy()"
- Add: "drm/tilcdc: Add timeout wait for palette loading to complete"
- Add: "drm/tilcdc: Call reset() before loading the palette"
- Add: "drm/tilcdc: Use complete_all() to indicate completed palette loading"
- Add "drm/tilcdc: Enable frame done irq and functionality for LCDC rev 1"
- Bartosz: Please test if this works! The symptom for not working is "timeout waiting for framedone" message when screen is blanked.

Changes since first version of the series:

- Move tilcdc_regs.h changes from "drm/tilcdc: Enable palette loading for revision 2 LCDC too" to "drm/tilcdc: Add tilcdc_write_mask() to tilcdc_regs.h"

These patches are inspired by this series form Bartosz Golaszewski:
https://www.spinics.net/lists/arm-kernel/msg539629.html

The patches are based on drm-next plus the earlier patches that I plan to send in a pull request for 4.10. The base + these patches are pushed here:

https://github.com/jsarha/linux drm-next-tilcdc-for-4.10-wip

Bartosz, please test if this branch works for rev1 LCDC, with your dts file!

Bartosz Golaszewski (1):
drm/tilcdc: implement palette loading for rev1

Jyri Sarha (9):
drm/tilcdc: Enable sync lost error and recovery handling for rev 1
LCDC
drm/tilcdc: Fix tilcdc_crtc_create() return value handling drm/tilcdc: Add tilcdc_write_mask() to tilcdc_regs.h drm/tilcdc: Fix load mode bit-field setting in tilcdc_crtc_enable() drm/tilcdc: Enable palette loading for revision 2 LCDC too drm/tilcdc: Add timeout wait for palette loading to complete drm/tilcdc: Load palette at the end of mode_set_nofb() drm/tilcdc: Configure video mode to HW in enable() not in

    mode_set_nofb()

drm/tilcdc: Enable frame done irq and functionality for LCDC rev 1

 drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 558 ++++++++++++++++++++---------------
 drivers/gpu/drm/tilcdc/tilcdc_drv.c  |  17 +-
 drivers/gpu/drm/tilcdc/tilcdc_drv.h  |   2 +-
 drivers/gpu/drm/tilcdc/tilcdc_regs.h |  15 +
 4 files changed, 344 insertions(+), 248 deletions(-)
Signature · 6 lines
1.9.1

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel