* [PATCH v2 net-next] net: hisilicon: fix binding document of mdio
@ 2015-11-16 9:38 huangdaode
2015-11-16 9:38 ` Arnd Bergmann
2015-11-16 15:20 ` Rob Herring
0 siblings, 2 replies; 5+ messages in thread
From: huangdaode @ 2015-11-16 9:38 UTC (permalink / raw)
To: davem, robh+dt
Cc: pawel.moll, mark.rutland, ijc+devicetree, galak, liguozhu,
Yisen.Zhuang, dingtianhong, zhangfei.gao, devicetree,
linux-kernel, linux-arm-kernel, netdev, linuxarm, salil.mehta,
huangdaode, kenneth-lee-2012, xuwei5, lisheng011
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio
2015-11-16 9:38 [PATCH v2 net-next] net: hisilicon: fix binding document of mdio huangdaode
@ 2015-11-16 9:38 ` Arnd Bergmann
2015-11-18 1:42 ` huangdaode
2015-11-16 15:20 ` Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Arnd Bergmann @ 2015-11-16 9:38 UTC (permalink / raw)
To: huangdaode
Cc: davem, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
liguozhu, Yisen.Zhuang, dingtianhong, zhangfei.gao, devicetree,
linux-kernel, linux-arm-kernel, netdev, linuxarm, salil.mehta,
kenneth-lee-2012, xuwei5, lisheng011
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.
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio
2015-11-16 9:38 ` Arnd Bergmann
@ 2015-11-18 1:42 ` huangdaode
0 siblings, 0 replies; 5+ messages in thread
From: huangdaode @ 2015-11-18 1:42 UTC (permalink / raw)
To: Arnd Bergmann
Cc: davem-fT/PcQaiUtIeIZ0/mPfg9Q, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8,
ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, liguozhu-C8/M+/jPZTeaMJb+Lgu22Q,
Yisen.Zhuang-hv44wF8Li93QT0dZR+AlfA,
dingtianhong-hv44wF8Li93QT0dZR+AlfA,
zhangfei.gao-QSEj5FYQhm4dnm+yROfE0A,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
netdev-u79uwXL29TY76Z2rM5mHXA, linuxarm-hv44wF8Li93QT0dZR+AlfA,
salil.mehta-hv44wF8Li93QT0dZR+AlfA,
kenneth-lee-2012-H32Fclmsjq1BDgjK7y7TUQ,
xuwei5-C8/M+/jPZTeaMJb+Lgu22Q, lisheng011-hv44wF8Li93QT0dZR+AlfA
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-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>.
>>
>> chang log:
>>
>> v2:
>> 1) reformat the style.
>> 2) make it more clearity.
>>
>> v1:
>> initial version.
>>
>> Signed-off-by: huangdaode <huangdaode-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org>
>> ---
> 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 devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio
2015-11-16 9:38 [PATCH v2 net-next] net: hisilicon: fix binding document of mdio huangdaode
2015-11-16 9:38 ` Arnd Bergmann
@ 2015-11-16 15:20 ` Rob Herring
2015-11-18 1:43 ` huangdaode
1 sibling, 1 reply; 5+ messages in thread
From: Rob Herring @ 2015-11-16 15:20 UTC (permalink / raw)
To: huangdaode
Cc: davem, pawel.moll, mark.rutland, ijc+devicetree, galak, liguozhu,
Yisen.Zhuang, dingtianhong, zhangfei.gao, devicetree,
linux-kernel, linux-arm-kernel, netdev, linuxarm, salil.mehta,
kenneth-lee-2012, xuwei5, lisheng011
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 net-next] net: hisilicon: fix binding document of mdio
2015-11-16 15:20 ` Rob Herring
@ 2015-11-18 1:43 ` huangdaode
0 siblings, 0 replies; 5+ messages in thread
From: huangdaode @ 2015-11-18 1:43 UTC (permalink / raw)
To: Rob Herring
Cc: davem-fT/PcQaiUtIeIZ0/mPfg9Q, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, liguozhu-C8/M+/jPZTeaMJb+Lgu22Q,
Yisen.Zhuang-hv44wF8Li93QT0dZR+AlfA,
dingtianhong-hv44wF8Li93QT0dZR+AlfA,
zhangfei.gao-QSEj5FYQhm4dnm+yROfE0A,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
netdev-u79uwXL29TY76Z2rM5mHXA, linuxarm-hv44wF8Li93QT0dZR+AlfA,
salil.mehta-hv44wF8Li93QT0dZR+AlfA,
kenneth-lee-2012-H32Fclmsjq1BDgjK7y7TUQ,
xuwei5-C8/M+/jPZTeaMJb+Lgu22Q, lisheng011-hv44wF8Li93QT0dZR+AlfA
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-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>.
>>
>> chang log:
>>
>> v2:
>> 1) reformat the style.
>> 2) make it more clearity.
>>
>> v1:
>> initial version.
>>
>> Signed-off-by: huangdaode <huangdaode-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org>
>> ---
>> 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-u79uwXL29TY76Z2rM5mHXA@public.gmane.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 devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-11-18 1:43 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-16 9:38 [PATCH v2 net-next] net: hisilicon: fix binding document of mdio huangdaode
2015-11-16 9:38 ` Arnd Bergmann
2015-11-18 1:42 ` huangdaode
2015-11-16 15:20 ` Rob Herring
2015-11-18 1:43 ` huangdaode
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).