Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1423914 > unrolled thread
| Started by | Xinliang Liu <xinliang.liu@linaro.org> |
|---|---|
| First post | 2016-06-16 12:40 +0200 |
| Last post | 2016-06-17 03:30 +0200 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH v2 1/3] reset: hisilicon: Add media reset controller binding Xinliang Liu <xinliang.liu@linaro.org> - 2016-06-16 12:40 +0200
Re: [PATCH v2 1/3] reset: hisilicon: Add media reset controller binding Philipp Zabel <p.zabel@pengutronix.de> - 2016-06-16 12:50 +0200
Re: [PATCH v2 1/3] reset: hisilicon: Add media reset controller binding Xinliang Liu <xinliang.liu@linaro.org> - 2016-06-17 03:30 +0200
| From | Xinliang Liu <xinliang.liu@linaro.org> |
|---|---|
| Date | 2016-06-16 12:40 +0200 |
| Subject | [PATCH v2 1/3] reset: hisilicon: Add media reset controller binding |
| Message-ID | <rKCSm-7c3-23@gated-at.bofh.it> |
Add compatible for media reset controller. Signed-off-by: Xinliang Liu <xinliang.liu@linaro.org> --- Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt index e0b185a944ba..c25da39df707 100644 --- a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt +++ b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt @@ -8,7 +8,9 @@ The reset controller registers are part of the system-ctl block on hi6220 SoC. Required properties: -- compatible: may be "hisilicon,hi6220-sysctrl" +- compatible: should be one of the following: + - "hisilicon,hi6220-sysctrl", "syscon" : For peripheral reset controller. + - "hisilicon,hi6220-mediactrl", "syscon" : For media reset controller. - reg: should be register base and length as documented in the datasheet - #reset-cells: 1, see below -- 2.8.3
[toc] | [next] | [standalone]
| From | Philipp Zabel <p.zabel@pengutronix.de> |
|---|---|
| Date | 2016-06-16 12:50 +0200 |
| Subject | Re: [PATCH v2 1/3] reset: hisilicon: Add media reset controller binding |
| Message-ID | <rKD22-7fn-21@gated-at.bofh.it> |
| In reply to | #1423914 |
Am Donnerstag, den 16.06.2016, 18:29 +0800 schrieb Xinliang Liu: > Add compatible for media reset controller. > > Signed-off-by: Xinliang Liu <xinliang.liu@linaro.org> > --- > Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt > index e0b185a944ba..c25da39df707 100644 > --- a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt > +++ b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt > @@ -8,7 +8,9 @@ The reset controller registers are part of the system-ctl block on > hi6220 SoC. > > Required properties: > -- compatible: may be "hisilicon,hi6220-sysctrl" > +- compatible: should be one of the following: > + - "hisilicon,hi6220-sysctrl", "syscon" : For peripheral reset controller. > + - "hisilicon,hi6220-mediactrl", "syscon" : For media reset controller. This does more than the commit message says. Why do you add syscon here? Do the sysctrl and mediactrl register ranges have bits other than reset controls that will be used by other drivers via the syscon mechanism? regards Philipp
[toc] | [prev] | [next] | [standalone]
| From | Xinliang Liu <xinliang.liu@linaro.org> |
|---|---|
| Date | 2016-06-17 03:30 +0200 |
| Message-ID | <rKQLD-8aN-1@gated-at.bofh.it> |
| In reply to | #1423932 |
Hi, On 16 June 2016 at 18:46, Philipp Zabel <p.zabel@pengutronix.de> wrote: > Am Donnerstag, den 16.06.2016, 18:29 +0800 schrieb Xinliang Liu: >> Add compatible for media reset controller. >> >> Signed-off-by: Xinliang Liu <xinliang.liu@linaro.org> >> --- >> Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt >> index e0b185a944ba..c25da39df707 100644 >> --- a/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt >> +++ b/Documentation/devicetree/bindings/reset/hisilicon,hi6220-reset.txt >> @@ -8,7 +8,9 @@ The reset controller registers are part of the system-ctl block on >> hi6220 SoC. >> >> Required properties: >> -- compatible: may be "hisilicon,hi6220-sysctrl" >> +- compatible: should be one of the following: >> + - "hisilicon,hi6220-sysctrl", "syscon" : For peripheral reset controller. >> + - "hisilicon,hi6220-mediactrl", "syscon" : For media reset controller. > > This does more than the commit message says. Why do you add syscon here? > Do the sysctrl and mediactrl register ranges have bits other than reset > controls that will be used by other drivers via the syscon mechanism? yes, that's right. Actually, ahead of the Required properties part, there is the sentence description "The reset controller registers are part of the system-ctl block" has said that reset bits are inside system controller. Thanks, -xinliang > > regards > Philipp >
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web