From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kishon Vijay Abraham I Subject: Re: [PATCH v2 1/6] phy: add a driver for the Berlin SATA PHY Date: Tue, 13 May 2014 10:58:36 +0530 Message-ID: <5371AD84.4070600@ti.com> References: <1399886217-20474-1-git-send-email-antoine.tenart@free-electrons.com> <1399886217-20474-2-git-send-email-antoine.tenart@free-electrons.com> <5370C2B6.8030908@ti.com> <5370D84A.9030405@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from bear.ext.ti.com ([192.94.94.41]:35264 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750776AbaEMF3R (ORCPT ); Tue, 13 May 2014 01:29:17 -0400 In-Reply-To: <5370D84A.9030405@gmail.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Sebastian Hesselbarth , =?UTF-8?B?QW50b2luZSBUw6luYXJ0?= , tj@kernel.org Cc: alexandre.belloni@free-electrons.com, thomas.petazzoni@free-electrons.com, zmxu@marvell.com, jszhang@marvell.com, linux-arm-kernel@lists.infradead.org, linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org Hi, On Monday 12 May 2014 07:48 PM, Sebastian Hesselbarth wrote: > On 05/12/2014 02:46 PM, Kishon Vijay Abraham I wrote: >> Hi, >> >> On Monday 12 May 2014 02:46 PM, Antoine T=C3=A9nart wrote: >>> The Berlin SoC has a two SATA ports. Add a PHY driver to handle the= m. >>> >>> Signed-off-by: Antoine T=C3=A9nart >>> --- >>> drivers/phy/Kconfig | 5 ++ >>> drivers/phy/Makefile | 1 + >>> drivers/phy/phy-berlin-sata.c | 179 >>> ++++++++++++++++++++++++++++++++++++++++++ >>> 3 files changed, 185 insertions(+) >>> create mode 100644 drivers/phy/phy-berlin-sata.c >>> >>> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig >>> index 4906c27fa3bd..b31b1986fda4 100644 >>> --- a/drivers/phy/Kconfig >>> +++ b/drivers/phy/Kconfig >>> @@ -15,6 +15,11 @@ config GENERIC_PHY >>> phy users can obtain reference to the PHY. All the users of= this >>> framework should select this config. >>> >>> +config PHY_BERLIN_SATA >>> + bool >>> + depends on ARCH_BERLIN && OF >>> + select GENERIC_PHY >>> + >>> config PHY_EXYNOS_MIPI_VIDEO >>> tristate "S5P/EXYNOS SoC series MIPI CSI-2/DSI PHY driver" >>> depends on HAS_IOMEM >>> diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile >>> index 7728518572a4..40278706ac1b 100644 >>> --- a/drivers/phy/Makefile >>> +++ b/drivers/phy/Makefile >>> @@ -3,6 +3,7 @@ >>> # >>> >>> obj-$(CONFIG_GENERIC_PHY) +=3D phy-core.o >>> +obj-$(CONFIG_PHY_BERLIN_SATA) +=3D phy-berlin-sata.o >>> obj-$(CONFIG_BCM_KONA_USB2_PHY) +=3D phy-bcm-kona-usb2.o >>> obj-$(CONFIG_PHY_EXYNOS_DP_VIDEO) +=3D phy-exynos-dp-video.o >>> obj-$(CONFIG_PHY_EXYNOS_MIPI_VIDEO) +=3D phy-exynos-mipi-video= =2Eo >>> diff --git a/drivers/phy/phy-berlin-sata.c b/drivers/phy/phy-berlin= -sata.c >>> new file mode 100644 >>> index 000000000000..f20f5ece1a7f >>> --- /dev/null >>> +++ b/drivers/phy/phy-berlin-sata.c >>> @@ -0,0 +1,179 @@ >>> +/* >>> + * Marvell Berlin SATA PHY driver >>> + * >>> + * Copyright (C) 2014 Marvell Technology Group Ltd. >>> + * >>> + * Antoine T=C3=A9nart >>> + * >>> + * This file is licensed under the terms of the GNU General Public >>> + * License version 2. This program is licensed "as is" without any >>> + * warranty of any kind, whether express or implied. >>> + */ >>> + >>> +#include >>> +#include >>> +#include >>> +#include >>> + >>> +#define HOST_VSA_ADDR 0x0 >>> +#define HOST_VSA_DATA 0x4 >>> + >>> +#define CONTROL_REGISTER 0x0 >>> +#define MBUS_SIZE_CONTROL 0x4 >>> + >>> +#define POWER_DOWN_SATA0 BIT(6) >>> +#define POWER_DOWN_SATA1 BIT(14) >>> +#define MBUS_WRITE_REQUEST_SIZE_128 (BIT(2) << 16) >>> +#define MBUS_READ_REQUEST_SIZE_128 (BIT(2) << 19) >>> + >>> +#define BERLIN_SATA_PHY_NB 2 >>> + >>> +#define to_berlin_sata_phy_priv(desc) \ >>> + container_of((desc), struct priv, phys[(desc)->index]) >>> + >>> +struct phy_desc { >> >> to be consistent, lets name it phy_berlin_desc. >>> + struct phy *phy; >>> + u32 val; >>> + unsigned index; >>> +}; >>> + >>> +struct priv { >=20 > And phy_berlin_priv then? >=20 > [...] >>> +static int phy_berlin_sata_probe(struct platform_device *pdev) >>> +{ >>> + struct phy *phy; >>> + struct phy_provider *phy_provider; >>> + struct priv *priv; >>> + struct resource *res; >>> + int i; >>> + >>> + priv =3D devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); >>> + if (!priv) >>> + return -ENOMEM; >>> + >>> + res =3D platform_get_resource(pdev, IORESOURCE_MEM, 0); >>> + priv->base =3D devm_ioremap(&pdev->dev, res->start, resource_s= ize(res)); >>> + if (IS_ERR(priv->base)) >>> + return PTR_ERR(priv->base); >>> + >>> + phy =3D devm_phy_create(&pdev->dev, &phy_berlin_sata_ops, NULL= ); >>> + if (IS_ERR(phy)) >>> + return PTR_ERR(phy); >>> + >>> + dev_set_drvdata(&pdev->dev, priv); >>> + spin_lock_init(&priv->lock); >>> + >>> + for (i =3D 0; i < BERLIN_SATA_PHY_NB; i++) { >> >> huh.. this should come from dt data. For devices which have multiple= PHYs, each >> PHY should be modelled as the sub-node of the *PHY provider* device = node. >=20 > Just to make sure we get it right: >=20 > You want the PHY (provider) node look like this: >=20 > sata_phy: phy@f7e900a0 { > compatible =3D "marvell,berlin-sata-phy"; > reg =3D <0xf7e900a0 0x10>; > #address-cells =3D <1>; > #size-cells =3D <0>; > #phy-cells =3D <1>; > status =3D "okay"; >=20 > sata_phy0: phy@0 { reg =3D <0>; }; >=20 > sata_phy1: phy@1 { reg =3D <1>; }; > }; >=20 > and parse the number of individual PHYs with > for_each_child_of_node()? yeah.. make sure you cc devicetree list when you send the patch. Cheers Kishon >=20 > Sebastian >=20 >>> + struct phy *phy =3D devm_phy_create(&pdev->dev, >>> + &phy_berlin_sata_ops, NULL); >>> + if (IS_ERR(phy)) { >>> + dev_err(&pdev->dev, "failed to create PHY %d\n", i); >>> + return PTR_ERR(phy); >>> + } >>> + >>> + priv->phys[i].phy =3D phy; >>> + priv->phys[i].val =3D desc[i].val; >>> + priv->phys[i].index =3D i; >>> + phy_set_drvdata(phy, &priv->phys[i]); >>> + >>> + /* Make sure the PHY is off */ >>> + phy_berlin_sata_power_off(phy); >>> + } >>> + >>> + phy_provider =3D devm_of_phy_provider_register(&pdev->dev, >>> + berlin_sata_phy_xlate); >>> + if (IS_ERR(phy_provider)) >>> + return PTR_ERR(phy_provider); >>> + >>> + return 0; >>> +} >>> + >>> +static const struct of_device_id phy_berlin_sata_of_match[] =3D { >>> + { .compatible =3D "marvell,berlin-sata-phy" }, >>> + { }, >>> +}; >>> + >>> +static struct platform_driver phy_berlin_sata_driver =3D { >>> + .probe =3D phy_berlin_sata_probe, >>> + .driver =3D { >>> + .name =3D "phy-berlin-sata", >>> + .owner =3D THIS_MODULE, >>> + .of_match_table =3D phy_berlin_sata_of_match, >>> + }, >>> +}; >>> +module_platform_driver(phy_berlin_sata_driver); >>> + >>> +MODULE_DESCRIPTION("Marvell Berlin SATA PHY driver"); >>> +MODULE_AUTHOR("Antoine T=C3=A9nart "); >>> +MODULE_LICENSE("GPL"); >> >> GPL v2 as the file header states so? >> >> Thanks >> Kishon >> >=20