On 11/03/2019 11:51, Maxime Jourdan wrote: > When the DRM driver for the meson platform was created, the bindings > required that the DMC register region was provided. > > Through those DMC registers, the display driver could configure an IP > called "canvas", a video lookup table used by the display IP. > > It was later discovered that "canvas" is actually an IP shared by other > components than display: video decoder, 2D engine.. and that it wasn't > possible to keep the canvas code in DRM. > > Over the past few months, incremental efforts have been deployed to > create a standalone meson-canvas driver [1], and the DRM driver was > patched to optionally use it if present [2]. > > This is the final step of those efforts where we simply remove any > control over DMC that the meson DRM driver has. > > Please note that this breaks compatibility with older DTs that only > provide the DMC register range but not the amlogic,canvas node. > > [1] https://patchwork.kernel.org/cover/10573771/ > [2] https://patchwork.freedesktop.org/series/52076/ > > Signed-off-by: Maxime Jourdan <mjourdan@xxxxxxxxxxxx> > --- > .../devicetree/bindings/display/amlogic,meson-vpu.txt | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/amlogic,meson-vpu.txt b/Documentation/devicetree/bindings/display/amlogic,meson-vpu.txt > index c65fd7a7467c..419444e2b4d3 100644 > --- a/Documentation/devicetree/bindings/display/amlogic,meson-vpu.txt > +++ b/Documentation/devicetree/bindings/display/amlogic,meson-vpu.txt > @@ -60,15 +60,14 @@ Required properties: > - reg: base address and size of he following memory-mapped regions : > - vpu > - hhi > - - dmc > - reg-names: should contain the names of the previous memory regions > - interrupts: should contain the VENC Vsync interrupt number > +- amlogic,canvas: phandle to canvas provider node as described in the file > + ../soc/amlogic/amlogic,canvas.txt > > Optional properties: > - power-domains: Optional phandle to associated power domain as described in > the file ../power/power_domain.txt > -- amlogic,canvas: phandle to canvas provider node as described in the file > - ../soc/amlogic/amlogic,canvas.txt > > Required nodes: > > Reviewed-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>
