<div dir="ltr">Hi Daniel, <div>Do you have some more comments for our patch?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Kevin Tang <<a href="mailto:kevin3.tang@gmail.com" target="_blank">kevin3.tang@gmail.com</a>> 于2021年1月5日周二 下午9:46写道:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">ChangeList:<br>
RFC v1:<br>
1. only upstream modeset and atomic at first commit. <br>
2. remove some unused code;<br>
3. use alpha and blend_mode properties;<br>
3. add yaml support;<br>
4. remove auto-adaptive panel driver;<br>
5. bugfix<br>
<br>
RFC v2:<br>
1. add sprd crtc and plane module for KMS, preparing for multi crtc&encoder<br>
2. remove gem drivers, use generic CMA handlers<br>
3. remove redundant "module_init", all the sub modules loading by KMS<br>
<br>
RFC v3:<br>
1. multi crtc&encoder design have problem, so rollback to v1<br>
<br>
RFC v4:<br>
1. update to gcc-linaro-7.5.0<br>
2. update to Linux 5.6-rc3<br>
3. remove pm_runtime support<br>
4. add COMPILE_TEST, remove unused kconfig<br>
5. "drm_dev_put" on drm_unbind<br>
6. fix some naming convention issue<br>
7. remove semaphore lock for crtc flip<br>
8. remove static variables<br>
<br>
RFC v5:<br>
1. optimize encoder and connector code implementation<br>
2. use "platform_get_irq" and "platform_get_resource"<br>
3. drop useless function return type, drop unless debug log<br>
4. custom properties should be separate, so drop it<br>
5. use DRM_XXX replase pr_xxx<br>
6. drop dsi&dphy hal callback ops<br>
7. drop unless callback ops checking<br>
8. add comments for sprd dpu structure<br>
<br>
RFC v6:<br>
1. Access registers via readl/writel<br>
2. Checking for unsupported KMS properties (format, rotation, blend_mode, etc) on plane_check ops<br>
3. Remove always true checks for dpu core ops<br>
<br>
RFC v7:<br>
1. Fix DTC unit name warnings<br>
2. Fix the problem of maintainers<br>
3. Call drmm_mode_config_init to mode config init<br>
4. Embed drm_device in sprd_drm and use devm_drm_dev_alloc<br>
5. Replace DRM_XXX with drm_xxx on KMS module, but not suitable for other subsystems<br>
6. Remove plane_update stuff, dpu handles all the HW update in crtc->atomic_flush<br>
7. Dsi&Dphy Code structure adjustment, all move to "sprd/"<br>
<br>
v0:<br>
1. Remove dpu_core_ops stuff layer for sprd drtc driver, but dpu_layer need to keeping.<br>
   Because all the HW update in crtc->atomic_flush, we need temporary storage all layers for<br>
   the dpu pageflip of atomic_flush.<br>
2. Add ports subnode with port@X.<br>
<br>
v1:<br>
1. Remove dphy and dsi graph binding, merge the dphy driver into the dsi.<br>
2. Add commit messages for Unisoc's virtual nodes.<br>
<br>
v2:<br>
1. Use drm_xxx to replace all DRM_XXX.<br>
2. Use kzalloc to replace devm_kzalloc for sprd_dsi/sprd_dpu structure init.<br>
3. Remove dpu_core_ops midlayer.<br>
<br>
v3:<br>
1. Remove dpu_layer midlayer and commit layers by aotmic_update<br>
<br>
Kevin Tang (6):<br>
  dt-bindings: display: add Unisoc's drm master bindings<br>
  drm/sprd: add Unisoc's drm kms master<br>
  dt-bindings: display: add Unisoc's dpu bindings<br>
  drm/sprd: add Unisoc's drm display controller driver<br>
  dt-bindings: display: add Unisoc's mipi dsi controller bindings<br>
  drm/sprd: add Unisoc's drm mipi dsi&dphy driver<br>
<br>
 .../display/sprd/sprd,display-subsystem.yaml       |   64 +<br>
 .../bindings/display/sprd/sprd,sharkl3-dpu.yaml    |   77 +<br>
 .../display/sprd/sprd,sharkl3-dsi-host.yaml        |  102 ++<br>
 drivers/gpu/drm/Kconfig                            |    2 +<br>
 drivers/gpu/drm/Makefile                           |    1 +<br>
 drivers/gpu/drm/sprd/Kconfig                       |   13 +<br>
 drivers/gpu/drm/sprd/Makefile                      |    8 +<br>
 drivers/gpu/drm/sprd/dw_dsi_ctrl.c                 |  794 +++++++++++<br>
 drivers/gpu/drm/sprd/dw_dsi_ctrl.h                 | 1475 ++++++++++++++++++++<br>
 drivers/gpu/drm/sprd/dw_dsi_ctrl_ppi.c             |  157 +++<br>
 drivers/gpu/drm/sprd/dw_dsi_ctrl_ppi.h             |   26 +<br>
 drivers/gpu/drm/sprd/megacores_pll.c               |  317 +++++<br>
 drivers/gpu/drm/sprd/megacores_pll.h               |  146 ++<br>
 drivers/gpu/drm/sprd/sprd_dpu.c                    |  985 +++++++++++++<br>
 drivers/gpu/drm/sprd/sprd_dpu.h                    |  120 ++<br>
 drivers/gpu/drm/sprd/sprd_drm.c                    |  224 +++<br>
 drivers/gpu/drm/sprd/sprd_drm.h                    |   19 +<br>
 drivers/gpu/drm/sprd/sprd_dsi.c                    | 1162 +++++++++++++++<br>
 drivers/gpu/drm/sprd/sprd_dsi.h                    |  107 ++<br>
 19 files changed, 5799 insertions(+)<br>
 create mode 100644 Documentation/devicetree/bindings/display/sprd/sprd,display-subsystem.yaml<br>
 create mode 100644 Documentation/devicetree/bindings/display/sprd/sprd,sharkl3-dpu.yaml<br>
 create mode 100644 Documentation/devicetree/bindings/display/sprd/sprd,sharkl3-dsi-host.yaml<br>
 create mode 100644 drivers/gpu/drm/sprd/Kconfig<br>
 create mode 100644 drivers/gpu/drm/sprd/Makefile<br>
 create mode 100644 drivers/gpu/drm/sprd/dw_dsi_ctrl.c<br>
 create mode 100644 drivers/gpu/drm/sprd/dw_dsi_ctrl.h<br>
 create mode 100644 drivers/gpu/drm/sprd/dw_dsi_ctrl_ppi.c<br>
 create mode 100644 drivers/gpu/drm/sprd/dw_dsi_ctrl_ppi.h<br>
 create mode 100644 drivers/gpu/drm/sprd/megacores_pll.c<br>
 create mode 100644 drivers/gpu/drm/sprd/megacores_pll.h<br>
 create mode 100644 drivers/gpu/drm/sprd/sprd_dpu.c<br>
 create mode 100644 drivers/gpu/drm/sprd/sprd_dpu.h<br>
 create mode 100644 drivers/gpu/drm/sprd/sprd_drm.c<br>
 create mode 100644 drivers/gpu/drm/sprd/sprd_drm.h<br>
 create mode 100644 drivers/gpu/drm/sprd/sprd_dsi.c<br>
 create mode 100644 drivers/gpu/drm/sprd/sprd_dsi.h<br>
<br>
-- <br>
2.7.4<br>
<br>
</blockquote></div></div>