From: "Heiko Stübner" <heiko@sntech.de>
To: Chris Zhong <zyw@rock-chips.com>
Cc: linux-rockchip@lists.infradead.org,
Mark Yao <mark.yao@rock-chips.com>,
David Airlie <airlied@linux.ie>, Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/3] dt-bindings: add the grf clock for dw-mipi-dsi
Date: Wed, 15 Mar 2017 11:10:54 +0100 [thread overview]
Message-ID: <2439695.Eb3n5AH02E@diego> (raw)
In-Reply-To: <58C90F8B.7070102@rock-chips.com>
Am Mittwoch, 15. März 2017, 17:55:23 CET schrieb Chris Zhong:
> Hi Heiko
>
> On 03/15/2017 05:03 PM, Heiko Stübner wrote:
> > Am Mittwoch, 15. März 2017, 16:42:30 CET schrieb Chris Zhong:
> >> For RK3399, the grf clock should be controlled by dw-mipi-dsi driver,
> >> add the description for this clock.
> >>
> >> Signed-off-by: Chris Zhong <zyw@rock-chips.com>
> >> ---
> >>
> >> .../devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip.txt
> >> | 2
> >>
> >> +- 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt
> >> b/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt index 188f6f7..7e17a60 100644
> >> ---
> >> a/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt +++
> >> b/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >>
> >> xt @@ -10,7 +10,7 @@ Required properties:
> >> - interrupts: Represent the controller's interrupt to the CPU(s).
> >> - clocks, clock-names: Phandles to the controller's pll reference
> >>
> >> clock(ref) and APB clock(pclk). For RK3399, a phy config clock
> >>
> >> - (phy_cfg) is additional required. As described in [1].
> >> + (phy_cfg) and a grf clock(grf) are additional required. As described
> >> in
> >> [1].
> >
> > your "grf" clock is optional, as it is not present on all socs (like the
> > rk3288) so should probably move to a separate section and not be in the
> > required properties
>
> For RK3399, the grf clock is required, according to the advice provided
> by rob[0],
> put it into "required properties " is better.
> [0] https://patchwork.kernel.org/patch/9220187/
ok, but it should probably be made clear, that this is rk3399-specific
WARNING: multiple messages have this Message-ID (diff)
From: "Heiko Stübner" <heiko@sntech.de>
To: Chris Zhong <zyw@rock-chips.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
dri-devel@lists.freedesktop.org,
linux-rockchip@lists.infradead.org,
Rob Herring <robh+dt@kernel.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/3] dt-bindings: add the grf clock for dw-mipi-dsi
Date: Wed, 15 Mar 2017 11:10:54 +0100 [thread overview]
Message-ID: <2439695.Eb3n5AH02E@diego> (raw)
In-Reply-To: <58C90F8B.7070102@rock-chips.com>
Am Mittwoch, 15. März 2017, 17:55:23 CET schrieb Chris Zhong:
> Hi Heiko
>
> On 03/15/2017 05:03 PM, Heiko Stübner wrote:
> > Am Mittwoch, 15. März 2017, 16:42:30 CET schrieb Chris Zhong:
> >> For RK3399, the grf clock should be controlled by dw-mipi-dsi driver,
> >> add the description for this clock.
> >>
> >> Signed-off-by: Chris Zhong <zyw@rock-chips.com>
> >> ---
> >>
> >> .../devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip.txt
> >> | 2
> >>
> >> +- 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt
> >> b/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt index 188f6f7..7e17a60 100644
> >> ---
> >> a/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt +++
> >> b/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >>
> >> xt @@ -10,7 +10,7 @@ Required properties:
> >> - interrupts: Represent the controller's interrupt to the CPU(s).
> >> - clocks, clock-names: Phandles to the controller's pll reference
> >>
> >> clock(ref) and APB clock(pclk). For RK3399, a phy config clock
> >>
> >> - (phy_cfg) is additional required. As described in [1].
> >> + (phy_cfg) and a grf clock(grf) are additional required. As described
> >> in
> >> [1].
> >
> > your "grf" clock is optional, as it is not present on all socs (like the
> > rk3288) so should probably move to a separate section and not be in the
> > required properties
>
> For RK3399, the grf clock is required, according to the advice provided
> by rob[0],
> put it into "required properties " is better.
> [0] https://patchwork.kernel.org/patch/9220187/
ok, but it should probably be made clear, that this is rk3399-specific
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stübner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] dt-bindings: add the grf clock for dw-mipi-dsi
Date: Wed, 15 Mar 2017 11:10:54 +0100 [thread overview]
Message-ID: <2439695.Eb3n5AH02E@diego> (raw)
In-Reply-To: <58C90F8B.7070102@rock-chips.com>
Am Mittwoch, 15. M?rz 2017, 17:55:23 CET schrieb Chris Zhong:
> Hi Heiko
>
> On 03/15/2017 05:03 PM, Heiko St?bner wrote:
> > Am Mittwoch, 15. M?rz 2017, 16:42:30 CET schrieb Chris Zhong:
> >> For RK3399, the grf clock should be controlled by dw-mipi-dsi driver,
> >> add the description for this clock.
> >>
> >> Signed-off-by: Chris Zhong <zyw@rock-chips.com>
> >> ---
> >>
> >> .../devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip.txt
> >> | 2
> >>
> >> +- 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git
> >> a/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt
> >> b/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt index 188f6f7..7e17a60 100644
> >> ---
> >> a/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >> xt +++
> >> b/Documentation/devicetree/bindings/display/rockchip/dw_mipi_dsi_rockchip
> >> .t
> >>
> >> xt @@ -10,7 +10,7 @@ Required properties:
> >> - interrupts: Represent the controller's interrupt to the CPU(s).
> >> - clocks, clock-names: Phandles to the controller's pll reference
> >>
> >> clock(ref) and APB clock(pclk). For RK3399, a phy config clock
> >>
> >> - (phy_cfg) is additional required. As described in [1].
> >> + (phy_cfg) and a grf clock(grf) are additional required. As described
> >> in
> >> [1].
> >
> > your "grf" clock is optional, as it is not present on all socs (like the
> > rk3288) so should probably move to a separate section and not be in the
> > required properties
>
> For RK3399, the grf clock is required, according to the advice provided
> by rob[0],
> put it into "required properties " is better.
> [0] https://patchwork.kernel.org/patch/9220187/
ok, but it should probably be made clear, that this is rk3399-specific
next prev parent reply other threads:[~2017-03-15 10:11 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-15 8:42 [PATCH 1/3] dt-bindings: add the grf clock for dw-mipi-dsi Chris Zhong
2017-03-15 8:42 ` Chris Zhong
2017-03-15 8:42 ` Chris Zhong
2017-03-15 8:42 ` [PATCH 2/3] drm/rockchip/dsi: enable the grf clk before writing grf registers Chris Zhong
2017-03-15 8:42 ` Chris Zhong
2017-03-15 8:42 ` Chris Zhong
2017-03-15 16:47 ` Brian Norris
2017-03-15 16:47 ` Brian Norris
2017-03-15 16:47 ` Brian Norris
2017-03-15 16:55 ` John Keeping
2017-03-15 16:55 ` John Keeping
2017-03-15 16:55 ` John Keeping
2017-03-15 8:42 ` [PATCH 3/3] drm/rockchip/dsi: correct the grf_switch_reg name Chris Zhong
2017-03-15 8:42 ` Chris Zhong
2017-03-15 8:42 ` Chris Zhong
2017-03-15 9:03 ` [PATCH 1/3] dt-bindings: add the grf clock for dw-mipi-dsi Heiko Stübner
2017-03-15 9:03 ` Heiko Stübner
2017-03-15 9:03 ` Heiko Stübner
2017-03-15 9:55 ` Chris Zhong
2017-03-15 9:55 ` Chris Zhong
2017-03-15 9:55 ` Chris Zhong
2017-03-15 10:10 ` Heiko Stübner [this message]
2017-03-15 10:10 ` Heiko Stübner
2017-03-15 10:10 ` Heiko Stübner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2439695.Eb3n5AH02E@diego \
--to=heiko@sntech.de \
--cc=airlied@linux.ie \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=mark.yao@rock-chips.com \
--cc=robh+dt@kernel.org \
--cc=zyw@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.