From: Rafał Miłecki Date: Wed, 11 Jan 2017 14:21:13 +0000 (+0100) Subject: bcm53xx: backport upstream bcm53xx spi driver changes X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=4ce3c416965e1d08728833cc7d074688041c163e;p=openwrt%2Fstaging%2Fluka.git bcm53xx: backport upstream bcm53xx spi driver changes One replaces our local patch and another is just for keeping driver in sync (license change). Signed-off-by: Rafał Miłecki --- diff --git a/target/linux/bcm53xx/patches-4.4/084-0001-spi-bcm53xx-set-of_node-to-let-DT-specify-device-s.patch b/target/linux/bcm53xx/patches-4.4/084-0001-spi-bcm53xx-set-of_node-to-let-DT-specify-device-s.patch new file mode 100644 index 0000000000..39ebf7286b --- /dev/null +++ b/target/linux/bcm53xx/patches-4.4/084-0001-spi-bcm53xx-set-of_node-to-let-DT-specify-device-s.patch @@ -0,0 +1,52 @@ +From 78d759daceaf0a7058f37c4142bdca9948b6d987 Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= +Date: Thu, 29 Dec 2016 17:27:55 +0100 +Subject: [PATCH] spi: bcm53xx: set of_node to let DT specify device(s) +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Setting of_node of master's dev seems to be a common way of letting it +work nicely with DT. This allows specifying device there instead of +hardcoding one in the driver code. + +This was successfully tested with commit 1b47b98acce2 ("ARM: BCM5301X: +Add DT entry for SPI controller and NOR flash") + +Signed-off-by: Rafał Miłecki +Signed-off-by: Mark Brown +--- + drivers/spi/spi-bcm53xx.c | 8 +------- + 1 file changed, 1 insertion(+), 7 deletions(-) + +--- a/drivers/spi/spi-bcm53xx.c ++++ b/drivers/spi/spi-bcm53xx.c +@@ -275,10 +275,6 @@ static int bcm53xxspi_flash_read(struct + * BCMA + **************************************************/ + +-static struct spi_board_info bcm53xx_info = { +- .modalias = "bcm53xxspiflash", +-}; +- + static const struct bcma_device_id bcm53xxspi_bcma_tbl[] = { + BCMA_CORE(BCMA_MANUF_BCM, BCMA_CORE_NS_QSPI, BCMA_ANY_REV, BCMA_ANY_CLASS), + {}, +@@ -311,6 +307,7 @@ static int bcm53xxspi_bcma_probe(struct + b53spi->bspi = true; + bcm53xxspi_disable_bspi(b53spi); + ++ master->dev.of_node = dev->of_node; + master->transfer_one = bcm53xxspi_transfer_one; + if (b53spi->mmio_base) + master->spi_flash_read = bcm53xxspi_flash_read; +@@ -324,9 +321,6 @@ static int bcm53xxspi_bcma_probe(struct + return err; + } + +- /* Broadcom SoCs (at least with the CC rev 42) use SPI for flash only */ +- spi_new_device(master, &bcm53xx_info); +- + return 0; + } + diff --git a/target/linux/bcm53xx/patches-4.4/084-0002-spi-bcm53xx-re-license-code-to-the-GPL-v2.patch b/target/linux/bcm53xx/patches-4.4/084-0002-spi-bcm53xx-re-license-code-to-the-GPL-v2.patch new file mode 100644 index 0000000000..37fb6df2b9 --- /dev/null +++ b/target/linux/bcm53xx/patches-4.4/084-0002-spi-bcm53xx-re-license-code-to-the-GPL-v2.patch @@ -0,0 +1,50 @@ +From e360e72e715f228e426edf0fc99ffa34027ab0ad Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= +Date: Thu, 29 Dec 2016 20:13:13 +0100 +Subject: [PATCH] spi: bcm53xx: (re)license code to the GPL v2 +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +My intention was to release this code under GPL v2 license. For some +reason my initial commit 0fc6a323e191 ("spi: bcm53xx: driver for SPI +controller on Broadcom bcma SoC") totally missed licensing info. +MODULE_LICENSE was later added by Axel specifying "GNU Public License +v2 or later". + +This patch clarifies situation by adding a proper header (with Copyright +line) and adjusting MODULE_LICENSE. It should be acked by every driver +contributor. + +Signed-off-by: Rafał Miłecki +Acked-by: Nicholas Mc Guire +Reviewed-by: Jingoo Han +Acked-by: Jingoo Han +Acked-by: Joe Perches +Acked-by: Axel Lin +Acked-by: Vaishali Thakkar +Signed-off-by: Mark Brown +--- + drivers/spi/spi-bcm53xx.c | 10 +++++++++- + 1 file changed, 9 insertions(+), 1 deletion(-) + +--- a/drivers/spi/spi-bcm53xx.c ++++ b/drivers/spi/spi-bcm53xx.c +@@ -1,3 +1,11 @@ ++/* ++ * Copyright (C) 2014-2016 Rafał Miłecki ++ * ++ * This program is free software; you can redistribute it and/or modify ++ * it under the terms of the GNU General Public License version 2 as ++ * published by the Free Software Foundation. ++ */ ++ + #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + + #include +@@ -355,4 +363,4 @@ module_exit(bcm53xxspi_module_exit); + + MODULE_DESCRIPTION("Broadcom BCM53xx SPI Controller driver"); + MODULE_AUTHOR("Rafał Miłecki "); +-MODULE_LICENSE("GPL"); ++MODULE_LICENSE("GPL v2"); diff --git a/target/linux/bcm53xx/patches-4.4/820-spi-bcm53xx-let-DT-specify-SPI-device-s-instead-hard.patch b/target/linux/bcm53xx/patches-4.4/820-spi-bcm53xx-let-DT-specify-SPI-device-s-instead-hard.patch deleted file mode 100644 index c60eb1954a..0000000000 --- a/target/linux/bcm53xx/patches-4.4/820-spi-bcm53xx-let-DT-specify-SPI-device-s-instead-hard.patch +++ /dev/null @@ -1,41 +0,0 @@ -From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= -Subject: [PATCH] spi: bcm53xx: let DT specify SPI device(s) instead hardcoding - them -MIME-Version: 1.0 -Content-Type: text/plain; charset=UTF-8 -Content-Transfer-Encoding: 8bit - -Signed-off-by: Rafał Miłecki ---- - ---- a/drivers/spi/spi-bcm53xx.c -+++ b/drivers/spi/spi-bcm53xx.c -@@ -275,10 +275,6 @@ static int bcm53xxspi_flash_read(struct - * BCMA - **************************************************/ - --static struct spi_board_info bcm53xx_info = { -- .modalias = "bcm53xxspiflash", --}; -- - static const struct bcma_device_id bcm53xxspi_bcma_tbl[] = { - BCMA_CORE(BCMA_MANUF_BCM, BCMA_CORE_NS_QSPI, BCMA_ANY_REV, BCMA_ANY_CLASS), - {}, -@@ -300,6 +296,7 @@ static int bcm53xxspi_bcma_probe(struct - master = spi_alloc_master(dev, sizeof(*b53spi)); - if (!master) - return -ENOMEM; -+ master->dev.of_node = dev->of_node; - - b53spi = spi_master_get_devdata(master); - b53spi->master = master; -@@ -324,9 +321,6 @@ static int bcm53xxspi_bcma_probe(struct - return err; - } - -- /* Broadcom SoCs (at least with the CC rev 42) use SPI for flash only */ -- spi_new_device(master, &bcm53xx_info); -- - return 0; - } -