| Message ID | 1492769331-5891-4-git-send-email-jjhiblot@ti.com |
|---|---|
| State | Superseded |
| Headers | show |
Hi Jean-Jacques, On 21 April 2017 at 04:08, Jean-Jacques Hiblot <jjhiblot@ti.com> wrote: > Those tests check: > - the ability for a phy-user to get a phy based on its name or its index > - the ability of a phy device (provider) to manage multiple ports > - the ability to perform operations on the phy (init,deinit,on,off) > - the behavior of the uclass when optional operations are not implemented > > Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com> > --- > > changes since v3: > updated the tests to reflect the changes in the PHY framework : > * use several phy devices (provider) > * use several phy ports per phy device > > arch/sandbox/dts/test.dts | 17 ++++++ > configs/sandbox_defconfig | 2 + > configs/sandbox_noblk_defconfig | 2 + > configs/sandbox_spl_defconfig | 3 ++ > drivers/phy/Kconfig | 9 ++++ > drivers/phy/Makefile | 1 + > drivers/phy/sandbox-phy.c | 98 +++++++++++++++++++++++++++++++++++ > test/dm/Makefile | 2 + > test/dm/phy.c | 112 ++++++++++++++++++++++++++++++++++++++++ > 9 files changed, 246 insertions(+) > create mode 100644 drivers/phy/sandbox-phy.c > create mode 100644 test/dm/phy.c Reviewed-by: Simon Glass <sjg@chromium.org> Nits below. > > diff --git a/arch/sandbox/dts/test.dts b/arch/sandbox/dts/test.dts > index fff175d..c845f0b 100644 > --- a/arch/sandbox/dts/test.dts > +++ b/arch/sandbox/dts/test.dts > @@ -59,6 +59,23 @@ > ping-add = <3>; > }; > > + phy_provider0: gen_phy@0 { > + compatible = "sandbox,phy"; > + #phy-cells = <1>; > + }; > + > + phy_provider1: gen_phy@1 { > + compatible = "sandbox,phy"; > + #phy-cells = <0>; > + broken; > + }; > + > + gen_phy_user: gen_phy_user { > + compatible = "simple-bus"; > + phys = <&phy_provider0 0>, <&phy_provider0 1>, <&phy_provider1>; > + phy-names = "phy1", "phy2", "phy3"; > + }; > + > some-bus { > #address-cells = <1>; > #size-cells = <0>; > diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig > index 9814ea3..a7bc08e 100644 > --- a/configs/sandbox_defconfig > +++ b/configs/sandbox_defconfig > @@ -165,6 +165,8 @@ CONFIG_CONSOLE_ROTATION=y > CONFIG_CONSOLE_TRUETYPE=y > CONFIG_CONSOLE_TRUETYPE_CANTORAONE=y > CONFIG_VIDEO_SANDBOX_SDL=y > +CONFIG_GENERIC_PHY=y > +CONFIG_PHY_SANDBOX=y > CONFIG_CMD_DHRYSTONE=y > CONFIG_TPM=y > CONFIG_LZ4=y > diff --git a/configs/sandbox_noblk_defconfig b/configs/sandbox_noblk_defconfig > index bba7443..59a80ef 100644 > --- a/configs/sandbox_noblk_defconfig > +++ b/configs/sandbox_noblk_defconfig > @@ -167,6 +167,8 @@ CONFIG_CONSOLE_ROTATION=y > CONFIG_CONSOLE_TRUETYPE=y > CONFIG_CONSOLE_TRUETYPE_CANTORAONE=y > CONFIG_VIDEO_SANDBOX_SDL=y > +CONFIG_GENERIC_PHY=y > +CONFIG_PHY_SANDBOX=y > CONFIG_CMD_DHRYSTONE=y > CONFIG_TPM=y > CONFIG_LZ4=y > diff --git a/configs/sandbox_spl_defconfig b/configs/sandbox_spl_defconfig > index 6fe2125..b1a2209 100644 > --- a/configs/sandbox_spl_defconfig > +++ b/configs/sandbox_spl_defconfig > @@ -171,6 +171,9 @@ CONFIG_CONSOLE_ROTATION=y > CONFIG_CONSOLE_TRUETYPE=y > CONFIG_CONSOLE_TRUETYPE_CANTORAONE=y > CONFIG_VIDEO_SANDBOX_SDL=y > +CONFIG_GENERIC_PHY=y > +CONFIG_SPL_GENERIC_PHY=y > +CONFIG_PHY_SANDBOX=y > CONFIG_CMD_DHRYSTONE=y > CONFIG_TPM=y > CONFIG_LZ4=y > diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig > index 0a74920..1131d46 100644 > --- a/drivers/phy/Kconfig > +++ b/drivers/phy/Kconfig > @@ -33,4 +33,13 @@ config SPL_PHY > compatible as possible with the equivalent framework found in the > linux kernel. > > +config PHY_SANDBOX > + bool "Sandbox PHY support" > + depends on SANDBOX > + depends on PHY > + help > + This select a dummy sandbox PHY driver. It used only to implement > + the unit tests for the phy framework > + > + Drop extra blank line > endmenu > diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile > index 186f07d..ca67145 100644 > --- a/drivers/phy/Makefile > +++ b/drivers/phy/Makefile > @@ -1,2 +1,3 @@ > obj-$(CONFIG_$(SPL_)PHY) += phy-uclass.o > +obj-$(CONFIG_PHY_SANDBOX) += sandbox-phy.o > > diff --git a/drivers/phy/sandbox-phy.c b/drivers/phy/sandbox-phy.c > new file mode 100644 > index 0000000..892a4a2 > --- /dev/null > +++ b/drivers/phy/sandbox-phy.c > @@ -0,0 +1,98 @@ > +/* > + * Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/ > + * Written by Jean-Jacques Hiblot <jjhiblot@ti.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <dm.h> > +#include <generic-phy.h> > + > +DECLARE_GLOBAL_DATA_PTR; > + > +struct sandbox_phy_priv { > + bool initialized; > + bool on; > + bool broken; > +}; > + > +static int sandbox_phy_power_on(struct phy *phy) > +{ > + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); blank line between decls and code. Please fix below also. > + if (!priv->initialized) > + return -EIO; > + > + if (priv->broken) > + return -EIO; > + > + priv->on = true; Blank line before return, Please fix below also. > + return 0; > +} > + > +static int sandbox_phy_power_off(struct phy *phy) > +{ > + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); > + if (!priv->initialized) > + return -EIO; > + > + if (priv->broken) > + return -EIO; > + > + /* > + * for validation purpose, let's says that power off > + * works only for PHY 0 > + */ > + if (phy->id) > + return -EIO; > + > + priv->on = false; > + return 0; > +} > + > +static int sandbox_phy_init(struct phy *phy) > +{ > + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); > + priv->initialized = true; > + priv->on = true; > + return 0; > +} > + > +static int sandbox_phy_exit(struct phy *phy) > +{ > + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); > + priv->initialized = false; > + priv->on = false; > + return 0; > +} > + > +static int sandbox_phy_probe(struct udevice *dev) > +{ > + struct sandbox_phy_priv *priv = dev_get_priv(dev); > + priv->initialized = false; > + priv->on = false; > + priv->broken = fdtdec_get_bool(gd->fdt_blob, dev_of_offset(dev), > + "broken"); > + return 0; > +} > + > +static struct phy_ops sandbox_phy_ops = { > + .power_on = sandbox_phy_power_on, > + .power_off = sandbox_phy_power_off, > + .init = sandbox_phy_init, > + .exit = sandbox_phy_exit, > +}; > + > +static const struct udevice_id sandbox_phy_ids[] = { > + { .compatible = "sandbox,phy" }, > + { } > +}; > + > +U_BOOT_DRIVER(phy_sandbox) = { > + .name = "phy_sandbox", > + .id = UCLASS_PHY, > + .of_match = sandbox_phy_ids, > + .ops = &sandbox_phy_ops, > + .probe = sandbox_phy_probe, > + .priv_auto_alloc_size = sizeof(struct sandbox_phy_priv), > +}; > diff --git a/test/dm/Makefile b/test/dm/Makefile > index 1885e17..dcd69b3 100644 > --- a/test/dm/Makefile > +++ b/test/dm/Makefile > @@ -24,6 +24,7 @@ obj-$(CONFIG_LED) += led.o > obj-$(CONFIG_DM_MAILBOX) += mailbox.o > obj-$(CONFIG_DM_MMC) += mmc.o > obj-$(CONFIG_DM_PCI) += pci.o > +obj-$(CONFIG_PHY) += phy.o > obj-$(CONFIG_POWER_DOMAIN) += power-domain.o > obj-$(CONFIG_RAM) += ram.o > obj-y += regmap.o > @@ -41,4 +42,5 @@ obj-$(CONFIG_TIMER) += timer.o > obj-$(CONFIG_DM_VIDEO) += video.o > obj-$(CONFIG_ADC) += adc.o > obj-$(CONFIG_SPMI) += spmi.o > + Drop extra blank ine > endif > diff --git a/test/dm/phy.c b/test/dm/phy.c > new file mode 100644 > index 0000000..811045f > --- /dev/null > +++ b/test/dm/phy.c > @@ -0,0 +1,112 @@ > +/* > + * Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/ > + * Written by Jean-Jacques Hiblot <jjhiblot@ti.com> > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <common.h> > +#include <dm.h> > +#include <generic-phy.h> > +#include <dm/test.h> > +#include <test/ut.h> > + > +DECLARE_GLOBAL_DATA_PTR; > + > +/* Base test of the phy uclass */ > +static int dm_test_phy_base(struct unit_test_state *uts) > +{ > + struct udevice *dev; > + struct phy phy1_method1; > + struct phy phy1_method2; > + struct phy phy2; > + struct phy phy3; > + struct udevice *parent; > + > + /* Get the device using the phy device*/ > + ut_assertok(uclass_get_device_by_name(UCLASS_SIMPLE_BUS, > + "gen_phy_user", &parent)); > + /* > + * Get the same phy port in 2 different ways and compare. > + */ > + ut_assertok(generic_phy_get_by_name(parent, "phy1", &phy1_method1)) > + ut_assertok(generic_phy_get_by_index(parent, 0, &phy1_method2)) > + ut_asserteq(phy1_method1.id, phy1_method2.id); > + > + /* > + * Get the second phy port. Check that the same phy provider (device) > + * provides this 2nd phy port, but that the IDs are different > + */ > + ut_assertok(generic_phy_get_by_name(parent, "phy2", &phy2)) > + ut_asserteq_ptr(phy1_method2.dev, phy2.dev); > + ut_assert(phy1_method1.id != phy2.id); > + > + /* > + * Get the third phy port. Check that the phy provider is different > + */ > + ut_assertok(generic_phy_get_by_name(parent, "phy3", &phy3)) > + ut_assert(phy2.dev != phy3.dev); > + > + /* Try to get a non-existing phy */ > + ut_asserteq(-ENODEV, uclass_get_device(UCLASS_PHY, 3, &dev)); > + ut_assert(generic_phy_get_by_name(parent, "phy_not_existing", > + &phy1_method1) < 0) > + > + return 0; > +} > +DM_TEST(dm_test_phy_base, DM_TESTF_SCAN_PDATA | DM_TESTF_SCAN_FDT); > + > +/* Test of the phy uclass using the sandbox phy driver operations */ > +static int dm_test_phy_ops(struct unit_test_state *uts) > +{ > + struct phy phy1; > + struct phy phy2; > + struct phy phy3; > + struct udevice *parent; > + > + ut_assertok(uclass_get_device_by_name(UCLASS_SIMPLE_BUS, > + "gen_phy_user", &parent)); > + > + ut_assertok(generic_phy_get_by_name(parent, "phy1", &phy1)); > + ut_assertok(generic_phy_get_by_name(parent, "phy2", &phy2)); > + ut_assertok(generic_phy_get_by_name(parent, "phy3", &phy3)); > + > + /* test normal operations */ > + ut_assertok(generic_phy_init(&phy1)); > + ut_assertok(generic_phy_power_on(&phy1)); > + ut_assertok(generic_phy_power_off(&phy1)); > + > + /* > + * test operations after exit(). > + * The sandbox phy driver does not allow it. > + */ > + ut_assertok(generic_phy_exit(&phy1)); > + ut_assert(generic_phy_power_on(&phy1) != 0); > + ut_assert(generic_phy_power_off(&phy1) != 0); > + > + /* > + * test normal operations again (after re-init) > + */ > + ut_assertok(generic_phy_init(&phy1)); > + ut_assertok(generic_phy_power_on(&phy1)); > + ut_assertok(generic_phy_power_off(&phy1)); > + > + /* > + * test calling unimplemented feature. > + * The call is expected to succeed > + */ > + ut_assertok(generic_phy_reset(&phy1)); > + > + /* PHY2 has a known problem with power off */ > + ut_assertok(generic_phy_init(&phy2)); > + ut_assertok(generic_phy_power_on(&phy2)); > + ut_assert(generic_phy_power_off(&phy2) == -EIO); > + > + /* PHY3 has a known problem with power off and power on*/ > + ut_assertok(generic_phy_init(&phy3)); > + ut_assert(generic_phy_power_off(&phy3) == -EIO); > + ut_assert(generic_phy_power_off(&phy3) == -EIO); > + > + return 0; > +} > +DM_TEST(dm_test_phy_ops, DM_TESTF_SCAN_PDATA | DM_TESTF_SCAN_FDT); > -- > 1.9.1 > This looks like a good set of tests. Regards, Simon
diff --git a/arch/sandbox/dts/test.dts b/arch/sandbox/dts/test.dts index fff175d..c845f0b 100644 --- a/arch/sandbox/dts/test.dts +++ b/arch/sandbox/dts/test.dts @@ -59,6 +59,23 @@ ping-add = <3>; }; + phy_provider0: gen_phy@0 { + compatible = "sandbox,phy"; + #phy-cells = <1>; + }; + + phy_provider1: gen_phy@1 { + compatible = "sandbox,phy"; + #phy-cells = <0>; + broken; + }; + + gen_phy_user: gen_phy_user { + compatible = "simple-bus"; + phys = <&phy_provider0 0>, <&phy_provider0 1>, <&phy_provider1>; + phy-names = "phy1", "phy2", "phy3"; + }; + some-bus { #address-cells = <1>; #size-cells = <0>; diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig index 9814ea3..a7bc08e 100644 --- a/configs/sandbox_defconfig +++ b/configs/sandbox_defconfig @@ -165,6 +165,8 @@ CONFIG_CONSOLE_ROTATION=y CONFIG_CONSOLE_TRUETYPE=y CONFIG_CONSOLE_TRUETYPE_CANTORAONE=y CONFIG_VIDEO_SANDBOX_SDL=y +CONFIG_GENERIC_PHY=y +CONFIG_PHY_SANDBOX=y CONFIG_CMD_DHRYSTONE=y CONFIG_TPM=y CONFIG_LZ4=y diff --git a/configs/sandbox_noblk_defconfig b/configs/sandbox_noblk_defconfig index bba7443..59a80ef 100644 --- a/configs/sandbox_noblk_defconfig +++ b/configs/sandbox_noblk_defconfig @@ -167,6 +167,8 @@ CONFIG_CONSOLE_ROTATION=y CONFIG_CONSOLE_TRUETYPE=y CONFIG_CONSOLE_TRUETYPE_CANTORAONE=y CONFIG_VIDEO_SANDBOX_SDL=y +CONFIG_GENERIC_PHY=y +CONFIG_PHY_SANDBOX=y CONFIG_CMD_DHRYSTONE=y CONFIG_TPM=y CONFIG_LZ4=y diff --git a/configs/sandbox_spl_defconfig b/configs/sandbox_spl_defconfig index 6fe2125..b1a2209 100644 --- a/configs/sandbox_spl_defconfig +++ b/configs/sandbox_spl_defconfig @@ -171,6 +171,9 @@ CONFIG_CONSOLE_ROTATION=y CONFIG_CONSOLE_TRUETYPE=y CONFIG_CONSOLE_TRUETYPE_CANTORAONE=y CONFIG_VIDEO_SANDBOX_SDL=y +CONFIG_GENERIC_PHY=y +CONFIG_SPL_GENERIC_PHY=y +CONFIG_PHY_SANDBOX=y CONFIG_CMD_DHRYSTONE=y CONFIG_TPM=y CONFIG_LZ4=y diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig index 0a74920..1131d46 100644 --- a/drivers/phy/Kconfig +++ b/drivers/phy/Kconfig @@ -33,4 +33,13 @@ config SPL_PHY compatible as possible with the equivalent framework found in the linux kernel. +config PHY_SANDBOX + bool "Sandbox PHY support" + depends on SANDBOX + depends on PHY + help + This select a dummy sandbox PHY driver. It used only to implement + the unit tests for the phy framework + + endmenu diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile index 186f07d..ca67145 100644 --- a/drivers/phy/Makefile +++ b/drivers/phy/Makefile @@ -1,2 +1,3 @@ obj-$(CONFIG_$(SPL_)PHY) += phy-uclass.o +obj-$(CONFIG_PHY_SANDBOX) += sandbox-phy.o diff --git a/drivers/phy/sandbox-phy.c b/drivers/phy/sandbox-phy.c new file mode 100644 index 0000000..892a4a2 --- /dev/null +++ b/drivers/phy/sandbox-phy.c @@ -0,0 +1,98 @@ +/* + * Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/ + * Written by Jean-Jacques Hiblot <jjhiblot@ti.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <dm.h> +#include <generic-phy.h> + +DECLARE_GLOBAL_DATA_PTR; + +struct sandbox_phy_priv { + bool initialized; + bool on; + bool broken; +}; + +static int sandbox_phy_power_on(struct phy *phy) +{ + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); + if (!priv->initialized) + return -EIO; + + if (priv->broken) + return -EIO; + + priv->on = true; + return 0; +} + +static int sandbox_phy_power_off(struct phy *phy) +{ + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); + if (!priv->initialized) + return -EIO; + + if (priv->broken) + return -EIO; + + /* + * for validation purpose, let's says that power off + * works only for PHY 0 + */ + if (phy->id) + return -EIO; + + priv->on = false; + return 0; +} + +static int sandbox_phy_init(struct phy *phy) +{ + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); + priv->initialized = true; + priv->on = true; + return 0; +} + +static int sandbox_phy_exit(struct phy *phy) +{ + struct sandbox_phy_priv *priv = dev_get_priv(phy->dev); + priv->initialized = false; + priv->on = false; + return 0; +} + +static int sandbox_phy_probe(struct udevice *dev) +{ + struct sandbox_phy_priv *priv = dev_get_priv(dev); + priv->initialized = false; + priv->on = false; + priv->broken = fdtdec_get_bool(gd->fdt_blob, dev_of_offset(dev), + "broken"); + return 0; +} + +static struct phy_ops sandbox_phy_ops = { + .power_on = sandbox_phy_power_on, + .power_off = sandbox_phy_power_off, + .init = sandbox_phy_init, + .exit = sandbox_phy_exit, +}; + +static const struct udevice_id sandbox_phy_ids[] = { + { .compatible = "sandbox,phy" }, + { } +}; + +U_BOOT_DRIVER(phy_sandbox) = { + .name = "phy_sandbox", + .id = UCLASS_PHY, + .of_match = sandbox_phy_ids, + .ops = &sandbox_phy_ops, + .probe = sandbox_phy_probe, + .priv_auto_alloc_size = sizeof(struct sandbox_phy_priv), +}; diff --git a/test/dm/Makefile b/test/dm/Makefile index 1885e17..dcd69b3 100644 --- a/test/dm/Makefile +++ b/test/dm/Makefile @@ -24,6 +24,7 @@ obj-$(CONFIG_LED) += led.o obj-$(CONFIG_DM_MAILBOX) += mailbox.o obj-$(CONFIG_DM_MMC) += mmc.o obj-$(CONFIG_DM_PCI) += pci.o +obj-$(CONFIG_PHY) += phy.o obj-$(CONFIG_POWER_DOMAIN) += power-domain.o obj-$(CONFIG_RAM) += ram.o obj-y += regmap.o @@ -41,4 +42,5 @@ obj-$(CONFIG_TIMER) += timer.o obj-$(CONFIG_DM_VIDEO) += video.o obj-$(CONFIG_ADC) += adc.o obj-$(CONFIG_SPMI) += spmi.o + endif diff --git a/test/dm/phy.c b/test/dm/phy.c new file mode 100644 index 0000000..811045f --- /dev/null +++ b/test/dm/phy.c @@ -0,0 +1,112 @@ +/* + * Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/ + * Written by Jean-Jacques Hiblot <jjhiblot@ti.com> + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <dm.h> +#include <generic-phy.h> +#include <dm/test.h> +#include <test/ut.h> + +DECLARE_GLOBAL_DATA_PTR; + +/* Base test of the phy uclass */ +static int dm_test_phy_base(struct unit_test_state *uts) +{ + struct udevice *dev; + struct phy phy1_method1; + struct phy phy1_method2; + struct phy phy2; + struct phy phy3; + struct udevice *parent; + + /* Get the device using the phy device*/ + ut_assertok(uclass_get_device_by_name(UCLASS_SIMPLE_BUS, + "gen_phy_user", &parent)); + /* + * Get the same phy port in 2 different ways and compare. + */ + ut_assertok(generic_phy_get_by_name(parent, "phy1", &phy1_method1)) + ut_assertok(generic_phy_get_by_index(parent, 0, &phy1_method2)) + ut_asserteq(phy1_method1.id, phy1_method2.id); + + /* + * Get the second phy port. Check that the same phy provider (device) + * provides this 2nd phy port, but that the IDs are different + */ + ut_assertok(generic_phy_get_by_name(parent, "phy2", &phy2)) + ut_asserteq_ptr(phy1_method2.dev, phy2.dev); + ut_assert(phy1_method1.id != phy2.id); + + /* + * Get the third phy port. Check that the phy provider is different + */ + ut_assertok(generic_phy_get_by_name(parent, "phy3", &phy3)) + ut_assert(phy2.dev != phy3.dev); + + /* Try to get a non-existing phy */ + ut_asserteq(-ENODEV, uclass_get_device(UCLASS_PHY, 3, &dev)); + ut_assert(generic_phy_get_by_name(parent, "phy_not_existing", + &phy1_method1) < 0) + + return 0; +} +DM_TEST(dm_test_phy_base, DM_TESTF_SCAN_PDATA | DM_TESTF_SCAN_FDT); + +/* Test of the phy uclass using the sandbox phy driver operations */ +static int dm_test_phy_ops(struct unit_test_state *uts) +{ + struct phy phy1; + struct phy phy2; + struct phy phy3; + struct udevice *parent; + + ut_assertok(uclass_get_device_by_name(UCLASS_SIMPLE_BUS, + "gen_phy_user", &parent)); + + ut_assertok(generic_phy_get_by_name(parent, "phy1", &phy1)); + ut_assertok(generic_phy_get_by_name(parent, "phy2", &phy2)); + ut_assertok(generic_phy_get_by_name(parent, "phy3", &phy3)); + + /* test normal operations */ + ut_assertok(generic_phy_init(&phy1)); + ut_assertok(generic_phy_power_on(&phy1)); + ut_assertok(generic_phy_power_off(&phy1)); + + /* + * test operations after exit(). + * The sandbox phy driver does not allow it. + */ + ut_assertok(generic_phy_exit(&phy1)); + ut_assert(generic_phy_power_on(&phy1) != 0); + ut_assert(generic_phy_power_off(&phy1) != 0); + + /* + * test normal operations again (after re-init) + */ + ut_assertok(generic_phy_init(&phy1)); + ut_assertok(generic_phy_power_on(&phy1)); + ut_assertok(generic_phy_power_off(&phy1)); + + /* + * test calling unimplemented feature. + * The call is expected to succeed + */ + ut_assertok(generic_phy_reset(&phy1)); + + /* PHY2 has a known problem with power off */ + ut_assertok(generic_phy_init(&phy2)); + ut_assertok(generic_phy_power_on(&phy2)); + ut_assert(generic_phy_power_off(&phy2) == -EIO); + + /* PHY3 has a known problem with power off and power on*/ + ut_assertok(generic_phy_init(&phy3)); + ut_assert(generic_phy_power_off(&phy3) == -EIO); + ut_assert(generic_phy_power_off(&phy3) == -EIO); + + return 0; +} +DM_TEST(dm_test_phy_ops, DM_TESTF_SCAN_PDATA | DM_TESTF_SCAN_FDT);
Those tests check: - the ability for a phy-user to get a phy based on its name or its index - the ability of a phy device (provider) to manage multiple ports - the ability to perform operations on the phy (init,deinit,on,off) - the behavior of the uclass when optional operations are not implemented Signed-off-by: Jean-Jacques Hiblot <jjhiblot@ti.com> --- changes since v3: updated the tests to reflect the changes in the PHY framework : * use several phy devices (provider) * use several phy ports per phy device arch/sandbox/dts/test.dts | 17 ++++++ configs/sandbox_defconfig | 2 + configs/sandbox_noblk_defconfig | 2 + configs/sandbox_spl_defconfig | 3 ++ drivers/phy/Kconfig | 9 ++++ drivers/phy/Makefile | 1 + drivers/phy/sandbox-phy.c | 98 +++++++++++++++++++++++++++++++++++ test/dm/Makefile | 2 + test/dm/phy.c | 112 ++++++++++++++++++++++++++++++++++++++++ 9 files changed, 246 insertions(+) create mode 100644 drivers/phy/sandbox-phy.c create mode 100644 test/dm/phy.c