| Message ID | 1402991881-27676-2-git-send-email-zhangfei.gao@linaro.org |
|---|---|
| State | Changes Requested |
| Headers | show |
On Tue, Jun 17, 2014 at 08:58:00AM +0100, Zhangfei Gao wrote: > From: Jiancheng Xue <xuejiancheng@huawei.com> > > Add necessary binding documentation SATA PHY on Hisilicon hix5hd2 soc. > > Signed-off-by: Jiancheng Xue <xuejiancheng@huawei.com> > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> > --- > .../devicetree/bindings/phy/hix5hd2-sata-phy.txt | 26 ++++++++++++++++++++ > 1 file changed, 26 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt > > diff --git a/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt b/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt > new file mode 100644 > index 0000000..ed15123 > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt > @@ -0,0 +1,26 @@ > +Hisilicon hix5hd2 SATA PHY > +----------------------- > + > +Required properties: > +- compatible: should be "hisilicon,hix5hd2-sata-phy" > +- reg: offset and length of the PHY registers > +- #phy-cells: must be 0 > +Refer to phy/phy-bindings.txt for the generic PHY binding properties > + > +Optional Properties: > +- hisilicon,peri-syscon: phandle of syscon used to control peripheral. "peri" is a rather strange contraction of "peripheral". > +- hisilicon,power-reg: offset and bit number of the sata power supply register. > + Only effective when hisilicon,peri-syscon is supplied. This is the offset within the system controller? It would be good to state that. > +- hisilicon,reg-init: one of more sets of 4 cells. The first cell > + is the register offset address, the second cell is the start bit in register, > + the third cell means the bit width, and the fourth cell is the value to set. Which registers is this meant to be initialising? Why does this need to be in the DT? How much is this expected to vary? Mark. -- 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
Dear Mark On 06/18/2014 01:13 AM, Mark Rutland wrote: > On Tue, Jun 17, 2014 at 08:58:00AM +0100, Zhangfei Gao wrote: >> From: Jiancheng Xue <xuejiancheng@huawei.com> >> >> Add necessary binding documentation SATA PHY on Hisilicon hix5hd2 soc. >> >> Signed-off-by: Jiancheng Xue <xuejiancheng@huawei.com> >> Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org> >> --- >> .../devicetree/bindings/phy/hix5hd2-sata-phy.txt | 26 ++++++++++++++++++++ >> 1 file changed, 26 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt >> >> diff --git a/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt b/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt >> new file mode 100644 >> index 0000000..ed15123 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt >> @@ -0,0 +1,26 @@ >> +Hisilicon hix5hd2 SATA PHY >> +----------------------- >> + >> +Required properties: >> +- compatible: should be "hisilicon,hix5hd2-sata-phy" >> +- reg: offset and length of the PHY registers >> +- #phy-cells: must be 0 >> +Refer to phy/phy-bindings.txt for the generic PHY binding properties >> + >> +Optional Properties: >> +- hisilicon,peri-syscon: phandle of syscon used to control peripheral. > > "peri" is a rather strange contraction of "peripheral". Will use hisilicon,peripheral-syscon instead, just thought it is too long. > >> +- hisilicon,power-reg: offset and bit number of the sata power supply register. >> + Only effective when hisilicon,peri-syscon is supplied. > > This is the offset within the system controller? It would be good to > state that. Change to - hisilicon,power-reg: offset and bit number within peripheral-syscon, register of controlling sata power supply. > >> +- hisilicon,reg-init: one of more sets of 4 cells. The first cell >> + is the register offset address, the second cell is the start bit in register, >> + the third cell means the bit width, and the fourth cell is the value to set. > > Which registers is this meant to be initialising? > > Why does this need to be in the DT? How much is this expected to vary? Double checked with hardware guy, the registers required to be set are same in different board, the value maybe different. Will move to phy-hix5hd2-sata.c directly and remove hisilicon,reg-init. Thanks for the suggestions. -- 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
diff --git a/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt b/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt new file mode 100644 index 0000000..ed15123 --- /dev/null +++ b/Documentation/devicetree/bindings/phy/hix5hd2-sata-phy.txt @@ -0,0 +1,26 @@ +Hisilicon hix5hd2 SATA PHY +----------------------- + +Required properties: +- compatible: should be "hisilicon,hix5hd2-sata-phy" +- reg: offset and length of the PHY registers +- #phy-cells: must be 0 +Refer to phy/phy-bindings.txt for the generic PHY binding properties + +Optional Properties: +- hisilicon,peri-syscon: phandle of syscon used to control peripheral. +- hisilicon,power-reg: offset and bit number of the sata power supply register. + Only effective when hisilicon,peri-syscon is supplied. +- hisilicon,reg-init: one of more sets of 4 cells. The first cell + is the register offset address, the second cell is the start bit in register, + the third cell means the bit width, and the fourth cell is the value to set. + +Example: + sata_phy: phy@f9900000 { + compatible = "hisilicon,hix5hd2-sata-phy"; + reg = <0xf9900000 0x10000>; + #phy-cells = <0>; + hisilicon,peri-syscon = <&peri_ctrl>; + hisilicon,power-reg = <0x8 10>; + hisilicon,reg-init = <0x148 0 32 0x345cb8>,<0x14c 0 32 0x20545>; + };