<div dir="ltr"><div dir="ltr">Hi Sam,<div>Thanks for your feedback</div><div><br></div><div>In the future we will have dp(displayport), hdmi encoder and more, so all those put into sprd directory, maybe better?<br></div><div><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 22, 2020 at 5:17 AM Sam Ravnborg <<a href="mailto:sam@ravnborg.org">sam@ravnborg.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Kevin.<br>
<br>
On Fri, Feb 21, 2020 at 03:48:50PM +0800, Kevin Tang wrote:<br>
> ChangeList:<br>
> 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>
> 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>
> v3:<br>
> 1. multi crtc&encoder design have problem, so rollback to v1<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&dphy bindings<br>
>   drm/sprd: add Unisoc's drm mipi dsi&dphy driver<br>
> <br>
>  .../devicetree/bindings/display/sprd/dphy.yaml     |   78 ++<br>
>  .../devicetree/bindings/display/sprd/dpu.yaml      |   85 ++<br>
>  .../devicetree/bindings/display/sprd/drm.yaml      |   38 +<br>
>  .../devicetree/bindings/display/sprd/dsi.yaml      |  101 ++<br>
Good to see you are using DT Schema format.<br>
Consider to drop the sprd directory and then use<br>
filename with "sprd-" prefix.<br>
sprd.yaml<br>
sprd-dphy.yaml<br>
sprd-dpu.yaml<br>
sprd-dsi.yaml<br>
<br>
        Sam<br>
<br>
</blockquote></div></div>