| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <HE1PR04MB3196FE21A068CB251775C48395240@HE1PR04MB3196.eurprd04.prod.outlook.com>
Date: Fri, 11 Sep 2020 13:25:57 +0000
From: Ashish Kumar <ashish.kumar@....com>
To: kuldip dwivedi <kuldip.dwivedi@...esoftware.com>,
Yogesh Gaur <yogeshgaur.83@...il.com>,
Mark Brown <broonie@...nel.org>,
"linux-spi@...r.kernel.org" <linux-spi@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC: Varun Sethi <V.Sethi@....com>, Arokia Samy <arokia.samy@....com>,
Ard Biesheuvel <Ard.Biesheuvel@....com>,
Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@....com>,
Paul Yang <Paul.Yang@....com>,
kuldip dwivedi <kuldip.dwivedi@...esoftware.com>
Subject: RE: [EXT] [PATCH v3] spi: spi-nxp-fspi: Add ACPI support
> -----Original Message-----
> From: kuldip dwivedi <kuldip.dwivedi@...esoftware.com>
> Sent: Friday, September 11, 2020 6:34 PM
> To: Ashish Kumar <ashish.kumar@....com>; Yogesh Gaur
> <yogeshgaur.83@...il.com>; Mark Brown <broonie@...nel.org>; linux-
> spi@...r.kernel.org; linux-kernel@...r.kernel.org
> Cc: Varun Sethi <V.Sethi@....com>; Arokia Samy <arokia.samy@....com>;
> Ard Biesheuvel <Ard.Biesheuvel@....com>; Samer El-Haj-Mahmoud
> <Samer.El-Haj-Mahmoud@....com>; Paul Yang <Paul.Yang@....com>;
> kuldip dwivedi <kuldip.dwivedi@...esoftware.com>
> Subject: [EXT] [PATCH v3] spi: spi-nxp-fspi: Add ACPI support
>
> Caution: EXT Email
>
> Currently NXP fspi driver has support of DT only. Adding ACPI support to the
> driver so that it can be used by UEFI firmware booting in ACPI mode. This
> driver will be probed if any firmware will expose HID "NXP0009" in DSDT
> table.
>
> Signed-off-by: kuldip dwivedi <kuldip.dwivedi@...esoftware.com>
Reviewed-by: Ashish Kumar <Ashish.Kumar@....com>
Regards
Ashish
> ---
>
> Notes:
> 1. Add ACPI match table, NXP members are added to confirm HID for FSPI
> 2. Change the DT specific APIs to device property APIs
> so that same API can be used in DT and ACPi mode.
> 3. Add node specific checks to use indexed based lookup API in case of
> ACPI.
> if (is_acpi_node(f->dev->fwnode))
> res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> else
> res = platform_get_resource_byname(pdev, IORESOURCE_MEM,
> "fspi_base");
> 4. Omit clock configuration part - in ACPI world, the firmware
> is responsible for clock maintenance.
> 5. This patch is tested on LX2160A platform
>
> drivers/spi/spi-nxp-fspi.c | 69 +++++++++++++++++++++++++++-----------
> 1 file changed, 50 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/spi/spi-nxp-fspi.c b/drivers/spi/spi-nxp-fspi.c index
> 1ccda82da206..0d41406c036d 100644
> --- a/drivers/spi/spi-nxp-fspi.c
> +++ b/drivers/spi/spi-nxp-fspi.c
> @@ -3,7 +3,8 @@
> /*
> * NXP FlexSPI(FSPI) controller driver.
> *
> - * Copyright 2019 NXP.
> + * Copyright 2019-2020 NXP
> + * Copyright 2020 Puresoftware Ltd.
> *
> * FlexSPI is a flexsible SPI host controller which supports two SPI
> * channels and up to 4 external devices. Each channel supports @@ -30,6
> +31,7 @@
> * Frieder Schrempf <frieder.schrempf@...tron.de>
> */
>
> +#include <linux/acpi.h>
> #include <linux/bitops.h>
> #include <linux/clk.h>
> #include <linux/completion.h>
> @@ -563,6 +565,9 @@ static int nxp_fspi_clk_prep_enable(struct nxp_fspi
> *f) {
> int ret;
>
> + if (is_acpi_node(f->dev->fwnode))
> + return 0;
> +
> ret = clk_prepare_enable(f->clk_en);
> if (ret)
> return ret;
> @@ -576,10 +581,15 @@ static int nxp_fspi_clk_prep_enable(struct
> nxp_fspi *f)
> return 0;
> }
>
> -static void nxp_fspi_clk_disable_unprep(struct nxp_fspi *f)
> +static int nxp_fspi_clk_disable_unprep(struct nxp_fspi *f)
> {
> + if (is_acpi_node(f->dev->fwnode))
> + return 0;
> +
> clk_disable_unprepare(f->clk);
> clk_disable_unprepare(f->clk_en);
> +
> + return 0;
> }
>
> /*
> @@ -1001,7 +1011,7 @@ static int nxp_fspi_probe(struct platform_device
> *pdev)
>
> f = spi_controller_get_devdata(ctlr);
> f->dev = dev;
> - f->devtype_data = of_device_get_match_data(dev);
> + f->devtype_data = device_get_match_data(dev);
> if (!f->devtype_data) {
> ret = -ENODEV;
> goto err_put_ctrl;
> @@ -1010,7 +1020,12 @@ static int nxp_fspi_probe(struct platform_device
> *pdev)
> platform_set_drvdata(pdev, f);
>
> /* find the resources - configuration register address space */
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM,
> "fspi_base");
> + if (is_acpi_node(f->dev->fwnode))
> + res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> + else
> + res = platform_get_resource_byname(pdev,
> + IORESOURCE_MEM, "fspi_base");
> +
> f->iobase = devm_ioremap_resource(dev, res);
> if (IS_ERR(f->iobase)) {
> ret = PTR_ERR(f->iobase); @@ -1018,7 +1033,12 @@ static int
> nxp_fspi_probe(struct platform_device *pdev)
> }
>
> /* find the resources - controller memory mapped space */
> - res = platform_get_resource_byname(pdev, IORESOURCE_MEM,
> "fspi_mmap");
> + if (is_acpi_node(f->dev->fwnode))
> + res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
> + else
> + res = platform_get_resource_byname(pdev,
> + IORESOURCE_MEM, "fspi_mmap");
> +
> if (!res) {
> ret = -ENODEV;
> goto err_put_ctrl;
> @@ -1029,22 +1049,24 @@ static int nxp_fspi_probe(struct platform_device
> *pdev)
> f->memmap_phy_size = resource_size(res);
>
> /* find the clocks */
> - f->clk_en = devm_clk_get(dev, "fspi_en");
> - if (IS_ERR(f->clk_en)) {
> - ret = PTR_ERR(f->clk_en);
> - goto err_put_ctrl;
> - }
> + if (dev_of_node(&pdev->dev)) {
> + f->clk_en = devm_clk_get(dev, "fspi_en");
> + if (IS_ERR(f->clk_en)) {
> + ret = PTR_ERR(f->clk_en);
> + goto err_put_ctrl;
> + }
>
> - f->clk = devm_clk_get(dev, "fspi");
> - if (IS_ERR(f->clk)) {
> - ret = PTR_ERR(f->clk);
> - goto err_put_ctrl;
> - }
> + f->clk = devm_clk_get(dev, "fspi");
> + if (IS_ERR(f->clk)) {
> + ret = PTR_ERR(f->clk);
> + goto err_put_ctrl;
> + }
>
> - ret = nxp_fspi_clk_prep_enable(f);
> - if (ret) {
> - dev_err(dev, "can not enable the clock\n");
> - goto err_put_ctrl;
> + ret = nxp_fspi_clk_prep_enable(f);
> + if (ret) {
> + dev_err(dev, "can not enable the clock\n");
> + goto err_put_ctrl;
> + }
> }
>
> /* find the irq */
> @@ -1127,6 +1149,14 @@ static const struct of_device_id nxp_fspi_dt_ids[]
> = { }; MODULE_DEVICE_TABLE(of, nxp_fspi_dt_ids);
>
> +#ifdef CONFIG_ACPI
> +static const struct acpi_device_id nxp_fspi_acpi_ids[] = {
> + { "NXP0009", .driver_data = (kernel_ulong_t)&lx2160a_data, },
> + {}
> +};
> +MODULE_DEVICE_TABLE(acpi, nxp_fspi_acpi_ids); #endif
> +
> static const struct dev_pm_ops nxp_fspi_pm_ops = {
> .suspend = nxp_fspi_suspend,
> .resume = nxp_fspi_resume,
> @@ -1136,6 +1166,7 @@ static struct platform_driver nxp_fspi_driver = {
> .driver = {
> .name = "nxp-fspi",
> .of_match_table = nxp_fspi_dt_ids,
> + .acpi_match_table = ACPI_PTR(nxp_fspi_acpi_ids),
> .pm = &nxp_fspi_pm_ops,
> },
> .probe = nxp_fspi_probe,
> --
> 2.17.1
Powered by blists - more mailing lists