Message ID | 1425520665.1637.10.camel@phoenix |
---|---|
State | Needs Review / ACK, archived |
Headers | show |
Context | Check | Description |
---|---|---|
robh/checkpatch | warning | total: 1 errors, 0 warnings, 0 lines checked |
robh/patch-applied | success |
On Thu, 05 Mar 2015, Axel Lin wrote: > The defines in phy-miphy365x.h are all covered in phy.h: > > MIPHY_TYPE_SATA == PHY_TYPE_STA > MIPHY_TYPE_PCIE == PHY_TYPE_PCIE > MIPPHY_TYPE_USB == PHY_TYPE_USB2 > > So covert to use phy.h and then delete phy-miphy365x.h. > > Signed-off-by: Axel Lin <axel.lin@ingics.com> > --- > Documentation/devicetree/bindings/ata/ahci-st.txt | 2 +- > Documentation/devicetree/bindings/phy/phy-miphy365x.txt | 8 ++++---- > arch/arm/boot/dts/stih416.dtsi | 4 ++-- > drivers/phy/phy-miphy365x.c | 14 +++++++------- > include/dt-bindings/phy/phy-miphy365x.h | 14 -------------- > 5 files changed, 14 insertions(+), 28 deletions(-) > delete mode 100644 include/dt-bindings/phy/phy-miphy365x.h Acked-by: Lee Jones <lee.jones@linaro.org> > diff --git a/Documentation/devicetree/bindings/ata/ahci-st.txt b/Documentation/devicetree/bindings/ata/ahci-st.txt > index 0574a77..070748b 100644 > --- a/Documentation/devicetree/bindings/ata/ahci-st.txt > +++ b/Documentation/devicetree/bindings/ata/ahci-st.txt > @@ -21,7 +21,7 @@ Example: > reg = <0xfe380000 0x1000>; > interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; > interrupt-names = "hostc"; > - phys = <&miphy365x_phy MIPHY_PORT_0 MIPHY_TYPE_SATA>; > + phys = <&miphy365x_phy MIPHY_PORT_0 PHY_TYPE_SATA>; > phy-names = "ahci_phy"; > resets = <&powerdown STIH416_SATA0_POWERDOWN>, > <&softreset STIH416_SATA0_SOFTRESET>; > diff --git a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt > index 9802d5d..087294d 100644 > --- a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt > +++ b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt > @@ -20,8 +20,8 @@ Required nodes : A sub-node is required for each channel the controller > Required properties (port (child) node): > - #phy-cells : Should be 1 (See second example) > Cell after port phandle is device type from: > - - MIPHY_TYPE_SATA > - - MIPHY_TYPE_PCI > + - PHY_TYPE_SATA > + - PHY_TYPE_PCIE > - reg : Address and length of register sets for each device in > "reg-names" > - reg-names : The names of the register addresses corresponding to the > @@ -68,10 +68,10 @@ property, containing a phandle to the phy port node and a device type. > > Example: > > -#include <dt-bindings/phy/phy-miphy365x.h> > +#include <dt-bindings/phy/phy.h> > > sata0: sata@fe380000 { > ... > - phys = <&phy_port0 MIPHY_TYPE_SATA>; > + phys = <&phy_port0 PHY_TYPE_SATA>; > ... > }; > diff --git a/arch/arm/boot/dts/stih416.dtsi b/arch/arm/boot/dts/stih416.dtsi > index ea28eba..eeb7afe 100644 > --- a/arch/arm/boot/dts/stih416.dtsi > +++ b/arch/arm/boot/dts/stih416.dtsi > @@ -10,7 +10,7 @@ > #include "stih416-clock.dtsi" > #include "stih416-pinctrl.dtsi" > > -#include <dt-bindings/phy/phy-miphy365x.h> > +#include <dt-bindings/phy/phy.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/reset-controller/stih416-resets.h> > / { > @@ -306,7 +306,7 @@ > reg = <0xfe380000 0x1000>; > interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; > interrupt-names = "hostc"; > - phys = <&phy_port0 MIPHY_TYPE_SATA>; > + phys = <&phy_port0 PHY_TYPE_SATA>; > phy-names = "sata-phy"; > resets = <&powerdown STIH416_SATA0_POWERDOWN>, > <&softreset STIH416_SATA0_SOFTRESET>; > diff --git a/drivers/phy/phy-miphy365x.c b/drivers/phy/phy-miphy365x.c > index 51b459d..019c2d7 100644 > --- a/drivers/phy/phy-miphy365x.c > +++ b/drivers/phy/phy-miphy365x.c > @@ -25,7 +25,7 @@ > #include <linux/mfd/syscon.h> > #include <linux/regmap.h> > > -#include <dt-bindings/phy/phy-miphy365x.h> > +#include <dt-bindings/phy/phy.h> > > #define HFC_TIMEOUT 100 > > @@ -177,7 +177,7 @@ static u8 rx_tx_spd[] = { > static int miphy365x_set_path(struct miphy365x_phy *miphy_phy, > struct miphy365x_dev *miphy_dev) > { > - bool sata = (miphy_phy->type == MIPHY_TYPE_SATA); > + bool sata = (miphy_phy->type == PHY_TYPE_SATA); > > return regmap_update_bits(miphy_dev->regmap, > miphy_phy->ctrlreg, > @@ -431,7 +431,7 @@ static int miphy365x_init(struct phy *phy) > } > > /* Initialise Miphy for PCIe or SATA */ > - if (miphy_phy->type == MIPHY_TYPE_PCIE) > + if (miphy_phy->type == PHY_TYPE_PCIE) > ret = miphy365x_init_pcie_port(miphy_phy, miphy_dev); > else > ret = miphy365x_init_sata_port(miphy_phy, miphy_dev); > @@ -455,8 +455,8 @@ int miphy365x_get_addr(struct device *dev, struct miphy365x_phy *miphy_phy, > return ret; > } > > - if (!((!strncmp(name, "sata", 4) && type == MIPHY_TYPE_SATA) || > - (!strncmp(name, "pcie", 4) && type == MIPHY_TYPE_PCIE))) > + if (!((!strncmp(name, "sata", 4) && type == PHY_TYPE_SATA) || > + (!strncmp(name, "pcie", 4) && type == PHY_TYPE_PCIE))) > return 0; > > miphy_phy->base = of_iomap(phynode, index); > @@ -499,8 +499,8 @@ static struct phy *miphy365x_xlate(struct device *dev, > > miphy_phy->type = args->args[0]; > > - if (!(miphy_phy->type == MIPHY_TYPE_SATA || > - miphy_phy->type == MIPHY_TYPE_PCIE)) { > + if (!(miphy_phy->type == PHY_TYPE_SATA || > + miphy_phy->type == PHY_TYPE_PCIE)) { > dev_err(dev, "Unsupported device type: %d\n", miphy_phy->type); > return ERR_PTR(-EINVAL); > } > diff --git a/include/dt-bindings/phy/phy-miphy365x.h b/include/dt-bindings/phy/phy-miphy365x.h > deleted file mode 100644 > index 8ef8aba..0000000 > --- a/include/dt-bindings/phy/phy-miphy365x.h > +++ /dev/null > @@ -1,14 +0,0 @@ > -/* > - * This header provides constants for the phy framework > - * based on the STMicroelectronics MiPHY365x. > - * > - * Author: Lee Jones <lee.jones@linaro.org> > - */ > -#ifndef _DT_BINDINGS_PHY_MIPHY > -#define _DT_BINDINGS_PHY_MIPHY > - > -#define MIPHY_TYPE_SATA 1 > -#define MIPHY_TYPE_PCIE 2 > -#define MIPPHY_TYPE_USB 3 > - > -#endif /* _DT_BINDINGS_PHY_MIPHY */
Hi, On Thursday 05 March 2015 07:27 AM, Axel Lin wrote: > The defines in phy-miphy365x.h are all covered in phy.h: > > MIPHY_TYPE_SATA == PHY_TYPE_STA > MIPHY_TYPE_PCIE == PHY_TYPE_PCIE > MIPPHY_TYPE_USB == PHY_TYPE_USB2 > > So covert to use phy.h and then delete phy-miphy365x.h. > > Signed-off-by: Axel Lin <axel.lin@ingics.com> > --- > Documentation/devicetree/bindings/ata/ahci-st.txt | 2 +- > Documentation/devicetree/bindings/phy/phy-miphy365x.txt | 8 ++++---- > arch/arm/boot/dts/stih416.dtsi | 4 ++-- > drivers/phy/phy-miphy365x.c | 14 +++++++------- > include/dt-bindings/phy/phy-miphy365x.h | 14 -------------- > 5 files changed, 14 insertions(+), 28 deletions(-) > delete mode 100644 include/dt-bindings/phy/phy-miphy365x.h > > diff --git a/Documentation/devicetree/bindings/ata/ahci-st.txt b/Documentation/devicetree/bindings/ata/ahci-st.txt > index 0574a77..070748b 100644 > --- a/Documentation/devicetree/bindings/ata/ahci-st.txt > +++ b/Documentation/devicetree/bindings/ata/ahci-st.txt > @@ -21,7 +21,7 @@ Example: > reg = <0xfe380000 0x1000>; > interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; > interrupt-names = "hostc"; > - phys = <&miphy365x_phy MIPHY_PORT_0 MIPHY_TYPE_SATA>; > + phys = <&miphy365x_phy MIPHY_PORT_0 PHY_TYPE_SATA>; > phy-names = "ahci_phy"; > resets = <&powerdown STIH416_SATA0_POWERDOWN>, > <&softreset STIH416_SATA0_SOFTRESET>; > diff --git a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt > index 9802d5d..087294d 100644 > --- a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt > +++ b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt > @@ -20,8 +20,8 @@ Required nodes : A sub-node is required for each channel the controller > Required properties (port (child) node): > - #phy-cells : Should be 1 (See second example) > Cell after port phandle is device type from: > - - MIPHY_TYPE_SATA > - - MIPHY_TYPE_PCI > + - PHY_TYPE_SATA > + - PHY_TYPE_PCIE > - reg : Address and length of register sets for each device in > "reg-names" > - reg-names : The names of the register addresses corresponding to the > @@ -68,10 +68,10 @@ property, containing a phandle to the phy port node and a device type. > > Example: > > -#include <dt-bindings/phy/phy-miphy365x.h> > +#include <dt-bindings/phy/phy.h> > > sata0: sata@fe380000 { > ... > - phys = <&phy_port0 MIPHY_TYPE_SATA>; > + phys = <&phy_port0 PHY_TYPE_SATA>; > ... > }; > diff --git a/arch/arm/boot/dts/stih416.dtsi b/arch/arm/boot/dts/stih416.dtsi > index ea28eba..eeb7afe 100644 > --- a/arch/arm/boot/dts/stih416.dtsi > +++ b/arch/arm/boot/dts/stih416.dtsi Can you split the patch so that I can queue the PHY part? Cheers Kishon -- 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
2015-03-09 19:44 GMT+08:00 Kishon Vijay Abraham I <kishon@ti.com>: > Hi, > > > On Thursday 05 March 2015 07:27 AM, Axel Lin wrote: >> >> The defines in phy-miphy365x.h are all covered in phy.h: >> >> MIPHY_TYPE_SATA == PHY_TYPE_STA >> MIPHY_TYPE_PCIE == PHY_TYPE_PCIE >> MIPPHY_TYPE_USB == PHY_TYPE_USB2 >> >> So covert to use phy.h and then delete phy-miphy365x.h. >> >> Signed-off-by: Axel Lin <axel.lin@ingics.com> >> --- >> Documentation/devicetree/bindings/ata/ahci-st.txt | 2 +- >> Documentation/devicetree/bindings/phy/phy-miphy365x.txt | 8 ++++---- >> arch/arm/boot/dts/stih416.dtsi | 4 ++-- >> drivers/phy/phy-miphy365x.c | 14 >> +++++++------- >> include/dt-bindings/phy/phy-miphy365x.h | 14 >> -------------- >> 5 files changed, 14 insertions(+), 28 deletions(-) >> delete mode 100644 include/dt-bindings/phy/phy-miphy365x.h >> >> diff --git a/Documentation/devicetree/bindings/ata/ahci-st.txt >> b/Documentation/devicetree/bindings/ata/ahci-st.txt >> index 0574a77..070748b 100644 >> --- a/Documentation/devicetree/bindings/ata/ahci-st.txt >> +++ b/Documentation/devicetree/bindings/ata/ahci-st.txt >> @@ -21,7 +21,7 @@ Example: >> reg = <0xfe380000 0x1000>; >> interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; >> interrupt-names = "hostc"; >> - phys = <&miphy365x_phy MIPHY_PORT_0 >> MIPHY_TYPE_SATA>; >> + phys = <&miphy365x_phy MIPHY_PORT_0 >> PHY_TYPE_SATA>; >> phy-names = "ahci_phy"; >> resets = <&powerdown STIH416_SATA0_POWERDOWN>, >> <&softreset STIH416_SATA0_SOFTRESET>; >> diff --git a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >> b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >> index 9802d5d..087294d 100644 >> --- a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >> +++ b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >> @@ -20,8 +20,8 @@ Required nodes : A sub-node is required for each >> channel the controller >> Required properties (port (child) node): >> - #phy-cells : Should be 1 (See second example) >> Cell after port phandle is device type from: >> - - MIPHY_TYPE_SATA >> - - MIPHY_TYPE_PCI >> + - PHY_TYPE_SATA >> + - PHY_TYPE_PCIE >> - reg : Address and length of register sets for each device in >> "reg-names" >> - reg-names : The names of the register addresses corresponding to >> the >> @@ -68,10 +68,10 @@ property, containing a phandle to the phy port node >> and a device type. >> >> Example: >> >> -#include <dt-bindings/phy/phy-miphy365x.h> >> +#include <dt-bindings/phy/phy.h> >> >> sata0: sata@fe380000 { >> ... >> - phys = <&phy_port0 MIPHY_TYPE_SATA>; >> + phys = <&phy_port0 PHY_TYPE_SATA>; >> ... >> }; >> diff --git a/arch/arm/boot/dts/stih416.dtsi >> b/arch/arm/boot/dts/stih416.dtsi >> index ea28eba..eeb7afe 100644 >> --- a/arch/arm/boot/dts/stih416.dtsi >> +++ b/arch/arm/boot/dts/stih416.dtsi > > > Can you split the patch so that I can queue the PHY part? In my opinion, apply the patch as is should be fine and we don't have any build breakage. I think the change in each file is pretty trivial and won't cause problem. otherwise, I will have to send patches for the conversion in each file and then wait until all patches are merged. Then send another patch to delete include/dt-bindings/phy/phy-miphy365x.h. Regards, Axel -- 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
+Maxime, Srinivas, patrice On Monday 09 March 2015 06:55 PM, Axel Lin wrote: > 2015-03-09 19:44 GMT+08:00 Kishon Vijay Abraham I <kishon@ti.com>: >> Hi, >> >> >> On Thursday 05 March 2015 07:27 AM, Axel Lin wrote: >>> >>> The defines in phy-miphy365x.h are all covered in phy.h: >>> >>> MIPHY_TYPE_SATA == PHY_TYPE_STA >>> MIPHY_TYPE_PCIE == PHY_TYPE_PCIE >>> MIPPHY_TYPE_USB == PHY_TYPE_USB2 >>> >>> So covert to use phy.h and then delete phy-miphy365x.h. >>> >>> Signed-off-by: Axel Lin <axel.lin@ingics.com> >>> --- >>> Documentation/devicetree/bindings/ata/ahci-st.txt | 2 +- >>> Documentation/devicetree/bindings/phy/phy-miphy365x.txt | 8 ++++---- >>> arch/arm/boot/dts/stih416.dtsi | 4 ++-- >>> drivers/phy/phy-miphy365x.c | 14 >>> +++++++------- >>> include/dt-bindings/phy/phy-miphy365x.h | 14 >>> -------------- >>> 5 files changed, 14 insertions(+), 28 deletions(-) >>> delete mode 100644 include/dt-bindings/phy/phy-miphy365x.h >>> >>> diff --git a/Documentation/devicetree/bindings/ata/ahci-st.txt >>> b/Documentation/devicetree/bindings/ata/ahci-st.txt >>> index 0574a77..070748b 100644 >>> --- a/Documentation/devicetree/bindings/ata/ahci-st.txt >>> +++ b/Documentation/devicetree/bindings/ata/ahci-st.txt >>> @@ -21,7 +21,7 @@ Example: >>> reg = <0xfe380000 0x1000>; >>> interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; >>> interrupt-names = "hostc"; >>> - phys = <&miphy365x_phy MIPHY_PORT_0 >>> MIPHY_TYPE_SATA>; >>> + phys = <&miphy365x_phy MIPHY_PORT_0 >>> PHY_TYPE_SATA>; >>> phy-names = "ahci_phy"; >>> resets = <&powerdown STIH416_SATA0_POWERDOWN>, >>> <&softreset STIH416_SATA0_SOFTRESET>; >>> diff --git a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>> b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>> index 9802d5d..087294d 100644 >>> --- a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>> +++ b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>> @@ -20,8 +20,8 @@ Required nodes : A sub-node is required for each >>> channel the controller >>> Required properties (port (child) node): >>> - #phy-cells : Should be 1 (See second example) >>> Cell after port phandle is device type from: >>> - - MIPHY_TYPE_SATA >>> - - MIPHY_TYPE_PCI >>> + - PHY_TYPE_SATA >>> + - PHY_TYPE_PCIE >>> - reg : Address and length of register sets for each device in >>> "reg-names" >>> - reg-names : The names of the register addresses corresponding to >>> the >>> @@ -68,10 +68,10 @@ property, containing a phandle to the phy port node >>> and a device type. >>> >>> Example: >>> >>> -#include <dt-bindings/phy/phy-miphy365x.h> >>> +#include <dt-bindings/phy/phy.h> >>> >>> sata0: sata@fe380000 { >>> ... >>> - phys = <&phy_port0 MIPHY_TYPE_SATA>; >>> + phys = <&phy_port0 PHY_TYPE_SATA>; >>> ... >>> }; >>> diff --git a/arch/arm/boot/dts/stih416.dtsi >>> b/arch/arm/boot/dts/stih416.dtsi >>> index ea28eba..eeb7afe 100644 >>> --- a/arch/arm/boot/dts/stih416.dtsi >>> +++ b/arch/arm/boot/dts/stih416.dtsi >> >> >> Can you split the patch so that I can queue the PHY part? > > In my opinion, apply the patch as is should be fine and we don't have any > build breakage. I think the change in each file is pretty trivial and won't > cause problem. otherwise, I will have to send patches for the conversion > in each file and then wait until all patches are merged. Then send another > patch to delete include/dt-bindings/phy/phy-miphy365x.h. Okay. I can queue after getting Ack from arch/arm/boot/dts/sti* maintainers. Cheers Kishon -- 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
Hi Kishon On 03/26/2015 12:11 AM, Kishon Vijay Abraham I wrote: > +Maxime, Srinivas, patrice > > On Monday 09 March 2015 06:55 PM, Axel Lin wrote: >> 2015-03-09 19:44 GMT+08:00 Kishon Vijay Abraham I <kishon@ti.com>: >>> Hi, >>> >>> >>> On Thursday 05 March 2015 07:27 AM, Axel Lin wrote: >>>> The defines in phy-miphy365x.h are all covered in phy.h: >>>> >>>> MIPHY_TYPE_SATA == PHY_TYPE_STA >>>> MIPHY_TYPE_PCIE == PHY_TYPE_PCIE >>>> MIPPHY_TYPE_USB == PHY_TYPE_USB2 >>>> >>>> So covert to use phy.h and then delete phy-miphy365x.h. >>>> >>>> Signed-off-by: Axel Lin <axel.lin@ingics.com> >>>> --- >>>> Documentation/devicetree/bindings/ata/ahci-st.txt | 2 +- >>>> Documentation/devicetree/bindings/phy/phy-miphy365x.txt | 8 ++++---- >>>> arch/arm/boot/dts/stih416.dtsi | 4 ++-- >>>> drivers/phy/phy-miphy365x.c | 14 >>>> +++++++------- >>>> include/dt-bindings/phy/phy-miphy365x.h | 14 >>>> -------------- >>>> 5 files changed, 14 insertions(+), 28 deletions(-) >>>> delete mode 100644 include/dt-bindings/phy/phy-miphy365x.h >>>> >>>> diff --git a/Documentation/devicetree/bindings/ata/ahci-st.txt >>>> b/Documentation/devicetree/bindings/ata/ahci-st.txt >>>> index 0574a77..070748b 100644 >>>> --- a/Documentation/devicetree/bindings/ata/ahci-st.txt >>>> +++ b/Documentation/devicetree/bindings/ata/ahci-st.txt >>>> @@ -21,7 +21,7 @@ Example: >>>> reg = <0xfe380000 0x1000>; >>>> interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; >>>> interrupt-names = "hostc"; >>>> - phys = <&miphy365x_phy MIPHY_PORT_0 >>>> MIPHY_TYPE_SATA>; >>>> + phys = <&miphy365x_phy MIPHY_PORT_0 >>>> PHY_TYPE_SATA>; >>>> phy-names = "ahci_phy"; >>>> resets = <&powerdown STIH416_SATA0_POWERDOWN>, >>>> <&softreset STIH416_SATA0_SOFTRESET>; >>>> diff --git a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>>> b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>>> index 9802d5d..087294d 100644 >>>> --- a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>>> +++ b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt >>>> @@ -20,8 +20,8 @@ Required nodes : A sub-node is required for each >>>> channel the controller >>>> Required properties (port (child) node): >>>> - #phy-cells : Should be 1 (See second example) >>>> Cell after port phandle is device type from: >>>> - - MIPHY_TYPE_SATA >>>> - - MIPHY_TYPE_PCI >>>> + - PHY_TYPE_SATA >>>> + - PHY_TYPE_PCIE >>>> - reg : Address and length of register sets for each device in >>>> "reg-names" >>>> - reg-names : The names of the register addresses corresponding to >>>> the >>>> @@ -68,10 +68,10 @@ property, containing a phandle to the phy port node >>>> and a device type. >>>> >>>> Example: >>>> >>>> -#include <dt-bindings/phy/phy-miphy365x.h> >>>> +#include <dt-bindings/phy/phy.h> >>>> >>>> sata0: sata@fe380000 { >>>> ... >>>> - phys = <&phy_port0 MIPHY_TYPE_SATA>; >>>> + phys = <&phy_port0 PHY_TYPE_SATA>; >>>> ... >>>> }; >>>> diff --git a/arch/arm/boot/dts/stih416.dtsi >>>> b/arch/arm/boot/dts/stih416.dtsi >>>> index ea28eba..eeb7afe 100644 >>>> --- a/arch/arm/boot/dts/stih416.dtsi >>>> +++ b/arch/arm/boot/dts/stih416.dtsi >>> >>> Can you split the patch so that I can queue the PHY part? >> In my opinion, apply the patch as is should be fine and we don't have any >> build breakage. I think the change in each file is pretty trivial and won't >> cause problem. otherwise, I will have to send patches for the conversion >> in each file and then wait until all patches are merged. Then send another >> patch to delete include/dt-bindings/phy/phy-miphy365x.h. > Okay. I can queue after getting Ack from arch/arm/boot/dts/sti* maintainers. Acked-by: Patrice Chotard <patrice.chotard@st.com> Thanks Patrice > > Cheers > Kishon -- 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/ata/ahci-st.txt b/Documentation/devicetree/bindings/ata/ahci-st.txt index 0574a77..070748b 100644 --- a/Documentation/devicetree/bindings/ata/ahci-st.txt +++ b/Documentation/devicetree/bindings/ata/ahci-st.txt @@ -21,7 +21,7 @@ Example: reg = <0xfe380000 0x1000>; interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; interrupt-names = "hostc"; - phys = <&miphy365x_phy MIPHY_PORT_0 MIPHY_TYPE_SATA>; + phys = <&miphy365x_phy MIPHY_PORT_0 PHY_TYPE_SATA>; phy-names = "ahci_phy"; resets = <&powerdown STIH416_SATA0_POWERDOWN>, <&softreset STIH416_SATA0_SOFTRESET>; diff --git a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt index 9802d5d..087294d 100644 --- a/Documentation/devicetree/bindings/phy/phy-miphy365x.txt +++ b/Documentation/devicetree/bindings/phy/phy-miphy365x.txt @@ -20,8 +20,8 @@ Required nodes : A sub-node is required for each channel the controller Required properties (port (child) node): - #phy-cells : Should be 1 (See second example) Cell after port phandle is device type from: - - MIPHY_TYPE_SATA - - MIPHY_TYPE_PCI + - PHY_TYPE_SATA + - PHY_TYPE_PCIE - reg : Address and length of register sets for each device in "reg-names" - reg-names : The names of the register addresses corresponding to the @@ -68,10 +68,10 @@ property, containing a phandle to the phy port node and a device type. Example: -#include <dt-bindings/phy/phy-miphy365x.h> +#include <dt-bindings/phy/phy.h> sata0: sata@fe380000 { ... - phys = <&phy_port0 MIPHY_TYPE_SATA>; + phys = <&phy_port0 PHY_TYPE_SATA>; ... }; diff --git a/arch/arm/boot/dts/stih416.dtsi b/arch/arm/boot/dts/stih416.dtsi index ea28eba..eeb7afe 100644 --- a/arch/arm/boot/dts/stih416.dtsi +++ b/arch/arm/boot/dts/stih416.dtsi @@ -10,7 +10,7 @@ #include "stih416-clock.dtsi" #include "stih416-pinctrl.dtsi" -#include <dt-bindings/phy/phy-miphy365x.h> +#include <dt-bindings/phy/phy.h> #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/reset-controller/stih416-resets.h> / { @@ -306,7 +306,7 @@ reg = <0xfe380000 0x1000>; interrupts = <GIC_SPI 157 IRQ_TYPE_NONE>; interrupt-names = "hostc"; - phys = <&phy_port0 MIPHY_TYPE_SATA>; + phys = <&phy_port0 PHY_TYPE_SATA>; phy-names = "sata-phy"; resets = <&powerdown STIH416_SATA0_POWERDOWN>, <&softreset STIH416_SATA0_SOFTRESET>; diff --git a/drivers/phy/phy-miphy365x.c b/drivers/phy/phy-miphy365x.c index 51b459d..019c2d7 100644 --- a/drivers/phy/phy-miphy365x.c +++ b/drivers/phy/phy-miphy365x.c @@ -25,7 +25,7 @@ #include <linux/mfd/syscon.h> #include <linux/regmap.h> -#include <dt-bindings/phy/phy-miphy365x.h> +#include <dt-bindings/phy/phy.h> #define HFC_TIMEOUT 100 @@ -177,7 +177,7 @@ static u8 rx_tx_spd[] = { static int miphy365x_set_path(struct miphy365x_phy *miphy_phy, struct miphy365x_dev *miphy_dev) { - bool sata = (miphy_phy->type == MIPHY_TYPE_SATA); + bool sata = (miphy_phy->type == PHY_TYPE_SATA); return regmap_update_bits(miphy_dev->regmap, miphy_phy->ctrlreg, @@ -431,7 +431,7 @@ static int miphy365x_init(struct phy *phy) } /* Initialise Miphy for PCIe or SATA */ - if (miphy_phy->type == MIPHY_TYPE_PCIE) + if (miphy_phy->type == PHY_TYPE_PCIE) ret = miphy365x_init_pcie_port(miphy_phy, miphy_dev); else ret = miphy365x_init_sata_port(miphy_phy, miphy_dev); @@ -455,8 +455,8 @@ int miphy365x_get_addr(struct device *dev, struct miphy365x_phy *miphy_phy, return ret; } - if (!((!strncmp(name, "sata", 4) && type == MIPHY_TYPE_SATA) || - (!strncmp(name, "pcie", 4) && type == MIPHY_TYPE_PCIE))) + if (!((!strncmp(name, "sata", 4) && type == PHY_TYPE_SATA) || + (!strncmp(name, "pcie", 4) && type == PHY_TYPE_PCIE))) return 0; miphy_phy->base = of_iomap(phynode, index); @@ -499,8 +499,8 @@ static struct phy *miphy365x_xlate(struct device *dev, miphy_phy->type = args->args[0]; - if (!(miphy_phy->type == MIPHY_TYPE_SATA || - miphy_phy->type == MIPHY_TYPE_PCIE)) { + if (!(miphy_phy->type == PHY_TYPE_SATA || + miphy_phy->type == PHY_TYPE_PCIE)) { dev_err(dev, "Unsupported device type: %d\n", miphy_phy->type); return ERR_PTR(-EINVAL); } diff --git a/include/dt-bindings/phy/phy-miphy365x.h b/include/dt-bindings/phy/phy-miphy365x.h deleted file mode 100644 index 8ef8aba..0000000 --- a/include/dt-bindings/phy/phy-miphy365x.h +++ /dev/null @@ -1,14 +0,0 @@ -/* - * This header provides constants for the phy framework - * based on the STMicroelectronics MiPHY365x. - * - * Author: Lee Jones <lee.jones@linaro.org> - */ -#ifndef _DT_BINDINGS_PHY_MIPHY -#define _DT_BINDINGS_PHY_MIPHY - -#define MIPHY_TYPE_SATA 1 -#define MIPHY_TYPE_PCIE 2 -#define MIPPHY_TYPE_USB 3 - -#endif /* _DT_BINDINGS_PHY_MIPHY */
The defines in phy-miphy365x.h are all covered in phy.h: MIPHY_TYPE_SATA == PHY_TYPE_STA MIPHY_TYPE_PCIE == PHY_TYPE_PCIE MIPPHY_TYPE_USB == PHY_TYPE_USB2 So covert to use phy.h and then delete phy-miphy365x.h. Signed-off-by: Axel Lin <axel.lin@ingics.com> --- Documentation/devicetree/bindings/ata/ahci-st.txt | 2 +- Documentation/devicetree/bindings/phy/phy-miphy365x.txt | 8 ++++---- arch/arm/boot/dts/stih416.dtsi | 4 ++-- drivers/phy/phy-miphy365x.c | 14 +++++++------- include/dt-bindings/phy/phy-miphy365x.h | 14 -------------- 5 files changed, 14 insertions(+), 28 deletions(-) delete mode 100644 include/dt-bindings/phy/phy-miphy365x.h