Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1269936 > unrolled thread
| Started by | huangdaode <huangdaode@hisilicon.com> |
|---|---|
| First post | 2015-11-16 10:30 +0100 |
| Last post | 2015-11-18 02:50 +0100 |
| Articles | 5 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH v2 net-next] net: hisilicon: fix binding document of mdio huangdaode <huangdaode@hisilicon.com> - 2015-11-16 10:30 +0100
Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio Arnd Bergmann <arnd@arndb.de> - 2015-11-16 10:50 +0100
Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio huangdaode <huangdaode@hisilicon.com> - 2015-11-18 02:50 +0100
Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio Rob Herring <robh@kernel.org> - 2015-11-16 16:30 +0100
Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio huangdaode <huangdaode@hisilicon.com> - 2015-11-18 02:50 +0100
| From | huangdaode <huangdaode@hisilicon.com> |
|---|---|
| Date | 2015-11-16 10:30 +0100 |
| Subject | [PATCH v2 net-next] net: hisilicon: fix binding document of mdio |
| Message-ID | <qvogO-5va-1@gated-at.bofh.it> |
This patch explains the occasion of "hisilcon,mdio" and "hisilicon,hns-mdio" according to Arnd's comments. and reformat it according to comments from Rob<robh@kernel.org>. chang log: v2: 1) reformat the style. 2) make it more clearity. v1: initial version. Signed-off-by: huangdaode <huangdaode@hisilicon.com> --- Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt index 9c23fdf..4942d20 100644 --- a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt +++ b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt @@ -1,7 +1,12 @@ Hisilicon MDIO bus controller Properties: -- compatible: "hisilicon,mdio","hisilicon,hns-mdio". +- compatible: can be one of: + "hisilicon,hns-mdio" + "hisilicon,mdio" + "hisilicon,hns-mdio" is recommendly to be used for hip05 and later SOCs, + while "hisilicon,mdio" is optional for backwards compatibility, only + support for hip04 Soc. - reg: The base address of the MDIO bus controller register bank. - #address-cells: Must be <1>. - #size-cells: Must be <0>. MDIO addresses have no size component. -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2015-11-16 10:50 +0100 |
| Message-ID | <qvoAa-5D7-13@gated-at.bofh.it> |
| In reply to | #1269936 |
On Monday 16 November 2015 17:38:24 huangdaode wrote: > This patch explains the occasion of "hisilcon,mdio" and > "hisilicon,hns-mdio" according to Arnd's comments. > and reformat it according to comments from Rob<robh@kernel.org>. > > chang log: > > v2: > 1) reformat the style. > 2) make it more clearity. > > v1: > initial version. > > Signed-off-by: huangdaode <huangdaode@hisilicon.com> > --- The new version looks ok, but your changelog above is formatted incorrectly. Please put only the contents you want in the git history above the '---' line and your Signed-off-by, but put the parts that refer to previous versions of the submission below the ---. > Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt > index 9c23fdf..4942d20 100644 > --- a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt > +++ b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt > @@ -1,7 +1,12 @@ > Hisilicon MDIO bus controller > > Properties: > -- compatible: "hisilicon,mdio","hisilicon,hns-mdio". > +- compatible: can be one of: > + "hisilicon,hns-mdio" > + "hisilicon,mdio" > + "hisilicon,hns-mdio" is recommendly to be used for hip05 and later SOCs, > + while "hisilicon,mdio" is optional for backwards compatibility, only > + support for hip04 Soc. > - reg: The base address of the MDIO bus controller register bank. > - #address-cells: Must be <1>. > - #size-cells: Must be <0>. MDIO addresses have no size component. > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | huangdaode <huangdaode@hisilicon.com> |
|---|---|
| Date | 2015-11-18 02:50 +0100 |
| Message-ID | <qw02J-4w8-1@gated-at.bofh.it> |
| In reply to | #1269946 |
On 2015/11/16 17:38, Arnd Bergmann wrote: > On Monday 16 November 2015 17:38:24 huangdaode wrote: >> This patch explains the occasion of "hisilcon,mdio" and >> "hisilicon,hns-mdio" according to Arnd's comments. >> and reformat it according to comments from Rob<robh@kernel.org>. >> >> chang log: >> >> v2: >> 1) reformat the style. >> 2) make it more clearity. >> >> v1: >> initial version. >> >> Signed-off-by: huangdaode <huangdaode@hisilicon.com> >> --- > The new version looks ok, but your changelog above is formatted > incorrectly. Please put only the contents you want in the git history > above the '---' line and your Signed-off-by, but put the parts that > refer to previous versions of the submission below the ---. Got it. thanks Arnd. Daode >> Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt | 7 ++++++- >> 1 file changed, 6 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt >> index 9c23fdf..4942d20 100644 >> --- a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt >> +++ b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt >> @@ -1,7 +1,12 @@ >> Hisilicon MDIO bus controller >> >> Properties: >> -- compatible: "hisilicon,mdio","hisilicon,hns-mdio". >> +- compatible: can be one of: >> + "hisilicon,hns-mdio" >> + "hisilicon,mdio" >> + "hisilicon,hns-mdio" is recommendly to be used for hip05 and later SOCs, >> + while "hisilicon,mdio" is optional for backwards compatibility, only >> + support for hip04 Soc. >> - reg: The base address of the MDIO bus controller register bank. >> - #address-cells: Must be <1>. >> - #size-cells: Must be <0>. MDIO addresses have no size component. >> > > . > -- Best Regards Daode Huang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Rob Herring <robh@kernel.org> |
|---|---|
| Date | 2015-11-16 16:30 +0100 |
| Message-ID | <qvtTd-AH-35@gated-at.bofh.it> |
| In reply to | #1269936 |
On Mon, Nov 16, 2015 at 05:38:24PM +0800, huangdaode wrote: > This patch explains the occasion of "hisilcon,mdio" and > "hisilicon,hns-mdio" according to Arnd's comments. > and reformat it according to comments from Rob<robh@kernel.org>. > > chang log: > > v2: > 1) reformat the style. > 2) make it more clearity. > > v1: > initial version. > > Signed-off-by: huangdaode <huangdaode@hisilicon.com> > --- > Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt > index 9c23fdf..4942d20 100644 > --- a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt > +++ b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt > @@ -1,7 +1,12 @@ > Hisilicon MDIO bus controller > > Properties: > -- compatible: "hisilicon,mdio","hisilicon,hns-mdio". > +- compatible: can be one of: > + "hisilicon,hns-mdio" > + "hisilicon,mdio" > + "hisilicon,hns-mdio" is recommendly to be used for hip05 and later SOCs, s/recommendly/recommended/ > + while "hisilicon,mdio" is optional for backwards compatibility, only > + support for hip04 Soc. ...compatibility only on hip04 SOC. > - reg: The base address of the MDIO bus controller register bank. > - #address-cells: Must be <1>. > - #size-cells: Must be <0>. MDIO addresses have no size component. > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | huangdaode <huangdaode@hisilicon.com> |
|---|---|
| Date | 2015-11-18 02:50 +0100 |
| Message-ID | <qw02J-4w8-3@gated-at.bofh.it> |
| In reply to | #1270284 |
On 2015/11/16 23:20, Rob Herring wrote: > On Mon, Nov 16, 2015 at 05:38:24PM +0800, huangdaode wrote: >> This patch explains the occasion of "hisilcon,mdio" and >> "hisilicon,hns-mdio" according to Arnd's comments. >> and reformat it according to comments from Rob<robh@kernel.org>. >> >> chang log: >> >> v2: >> 1) reformat the style. >> 2) make it more clearity. >> >> v1: >> initial version. >> >> Signed-off-by: huangdaode <huangdaode@hisilicon.com> >> --- >> Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt | 7 ++++++- >> 1 file changed, 6 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt >> index 9c23fdf..4942d20 100644 >> --- a/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt >> +++ b/Documentation/devicetree/bindings/net/hisilicon-hns-mdio.txt >> @@ -1,7 +1,12 @@ >> Hisilicon MDIO bus controller >> >> Properties: >> -- compatible: "hisilicon,mdio","hisilicon,hns-mdio". >> +- compatible: can be one of: >> + "hisilicon,hns-mdio" >> + "hisilicon,mdio" >> + "hisilicon,hns-mdio" is recommendly to be used for hip05 and later SOCs, > s/recommendly/recommended/ > >> + while "hisilicon,mdio" is optional for backwards compatibility, only >> + support for hip04 Soc. > ...compatibility only on hip04 SOC. > >> - reg: The base address of the MDIO bus controller register bank. >> - #address-cells: Must be <1>. >> - #size-cells: Must be <0>. MDIO addresses have no size component. >> -- >> 1.9.1 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe devicetree" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html Agreed. Will fix this in v3 patch. thanks Daode. > . > -- Best Regards Daode Huang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web