On Mon, Jul 8, 2019 at 3:37 PM <yongqiang.niu@xxxxxxxxxxxx> wrote: > > From: Yongqiang Niu <yongqiang.niu@xxxxxxxxxxxx> > > This patch add mutex description for mt8183 display > > Signed-off-by: Yongqiang Niu <yongqiang.niu@xxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > index bb9274a..4a22d49 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > @@ -53,6 +53,7 @@ Required properties (all function blocks): > For most function blocks this is just a single clock input. Only the DSI and > DPI controller nodes have multiple clock inputs. These are documented in > mediatek,dsi.txt and mediatek,dpi.txt, respectively. > + for MT8183 mutex, this hardware is always free run, has no clocks control This should have capitalization, punctuation, and drop the trailing space. It also reads a little strangely. You may want something like "An exception is that the mt8183 mutex is always free running with no clocks property." > > Required properties (DMA function blocks): > - compatible: Should be one of > -- > 1.8.1.1.dirty >