All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/5] hw_random: ixp4xx: enable compile-testing
@ 2021-05-11 13:29 Linus Walleij
  2021-05-11 13:29 ` [PATCH 2/5] hw_random: ixp4xx: Use SPDX license tag Linus Walleij
                   ` (6 more replies)
  0 siblings, 7 replies; 13+ messages in thread
From: Linus Walleij @ 2021-05-11 13:29 UTC (permalink / raw)
  To: Matt Mackall, Herbert Xu, Deepak Saxena
  Cc: linux-crypto, Arnd Bergmann, Linus Walleij

From: Arnd Bergmann <arnd@arndb.de>

The driver is almost portable already, it just needs to
include the new header for the cpu definition.

Cc: Deepak Saxena <dsaxena@plexity.net>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
The idea is to apply this through the ARM SoC tree along
with other IXP4xx refactorings.
Please tell me if you prefer another solution.
---
 drivers/char/hw_random/Kconfig      | 2 +-
 drivers/char/hw_random/ixp4xx-rng.c | 3 +--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/char/hw_random/Kconfig b/drivers/char/hw_random/Kconfig
index 1fe006f3f12f..f033a11cc90d 100644
--- a/drivers/char/hw_random/Kconfig
+++ b/drivers/char/hw_random/Kconfig
@@ -152,7 +152,7 @@ config HW_RANDOM_VIA
 
 config HW_RANDOM_IXP4XX
 	tristate "Intel IXP4xx NPU HW Pseudo-Random Number Generator support"
-	depends on ARCH_IXP4XX
+	depends on ARCH_IXP4XX || COMPILE_TEST
 	default HW_RANDOM
 	help
 	  This driver provides kernel-side support for the Pseudo-Random
diff --git a/drivers/char/hw_random/ixp4xx-rng.c b/drivers/char/hw_random/ixp4xx-rng.c
index beec1627db3c..34781028caec 100644
--- a/drivers/char/hw_random/ixp4xx-rng.c
+++ b/drivers/char/hw_random/ixp4xx-rng.c
@@ -21,10 +21,9 @@
 #include <linux/init.h>
 #include <linux/bitops.h>
 #include <linux/hw_random.h>
+#include <linux/soc/ixp4xx/cpu.h>
 
 #include <asm/io.h>
-#include <mach/hardware.h>
-
 
 static int ixp4xx_rng_data_read(struct hwrng *rng, u32 *buffer)
 {
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 2/5] hw_random: ixp4xx: Use SPDX license tag
  2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
@ 2021-05-11 13:29 ` Linus Walleij
  2021-05-21  7:10   ` Herbert Xu
  2021-05-11 13:29 ` [PATCH 3/5] hw_random: ixp4xx: Turn into a module Linus Walleij
                   ` (5 subsequent siblings)
  6 siblings, 1 reply; 13+ messages in thread
From: Linus Walleij @ 2021-05-11 13:29 UTC (permalink / raw)
  To: Matt Mackall, Herbert Xu, Deepak Saxena
  Cc: linux-crypto, Arnd Bergmann, Linus Walleij

This switches the IXP4xx hardware random driver to use
the SPDX type license tag.

Cc: Deepak Saxena <dsaxena@plexity.net>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
The idea is to apply this through the ARM SoC tree along
with other IXP4xx refactorings.
Please tell me if you prefer another solution.
---
 drivers/char/hw_random/ixp4xx-rng.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/char/hw_random/ixp4xx-rng.c b/drivers/char/hw_random/ixp4xx-rng.c
index 34781028caec..defd8176cb68 100644
--- a/drivers/char/hw_random/ixp4xx-rng.c
+++ b/drivers/char/hw_random/ixp4xx-rng.c
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0
 /*
  * drivers/char/hw_random/ixp4xx-rng.c
  *
@@ -8,10 +9,6 @@
  * Copyright 2005 (c) MontaVista Software, Inc.
  *
  * Fixes by Michael Buesch
- *
- * 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 <linux/kernel.h>
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 3/5] hw_random: ixp4xx: Turn into a module
  2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
  2021-05-11 13:29 ` [PATCH 2/5] hw_random: ixp4xx: Use SPDX license tag Linus Walleij
@ 2021-05-11 13:29 ` Linus Walleij
  2021-05-21  7:10   ` Herbert Xu
  2021-05-11 13:29 ` [PATCH 4/5] hw_random: ixp4xx: Add DT bindings Linus Walleij
                   ` (4 subsequent siblings)
  6 siblings, 1 reply; 13+ messages in thread
From: Linus Walleij @ 2021-05-11 13:29 UTC (permalink / raw)
  To: Matt Mackall, Herbert Xu, Deepak Saxena
  Cc: linux-crypto, Arnd Bergmann, Linus Walleij

Instead of just initializing always, which will invariably
create problems on multiplatform builds, turn this driver into
a module and pass a resource with the memory location.
The device only exist on the IXP46x so we only register
the device for the IXP46x SoC.

Cc: Deepak Saxena <dsaxena@plexity.net>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
The idea is to apply this through the ARM SoC tree along
with other IXP4xx refactorings.
Please tell me if you prefer another solution.
---
 arch/arm/mach-ixp4xx/common.c       | 24 ++++++++++++++++++-
 drivers/char/hw_random/ixp4xx-rng.c | 37 +++++++++++++----------------
 2 files changed, 40 insertions(+), 21 deletions(-)

diff --git a/arch/arm/mach-ixp4xx/common.c b/arch/arm/mach-ixp4xx/common.c
index f86c1bf34eea..a10aa14d09b1 100644
--- a/arch/arm/mach-ixp4xx/common.c
+++ b/arch/arm/mach-ixp4xx/common.c
@@ -237,6 +237,27 @@ static struct resource ixp46x_i2c_resources[] = {
 	}
 };
 
+/* A single 32-bit register on IXP46x */
+#define IXP4XX_HWRANDOM_BASE_PHYS	0x70002100
+
+static struct resource ixp46x_hwrandom_resource[] = {
+	{
+		.start = IXP4XX_HWRANDOM_BASE_PHYS,
+		.end = IXP4XX_HWRANDOM_BASE_PHYS + 0x3,
+		.flags = IORESOURCE_MEM,
+	},
+};
+
+static struct platform_device ixp46x_hwrandom_device = {
+	.name           = "ixp4xx-hwrandom",
+	.id             = -1,
+	.dev = {
+		.coherent_dma_mask      = DMA_BIT_MASK(32),
+	},
+	.resource = ixp46x_hwrandom_resource,
+	.num_resources  = ARRAY_SIZE(ixp46x_hwrandom_resource),
+};
+
 /*
  * I2C controller. The IXP46x uses the same block as the IOP3xx, so
  * we just use the same device name.
@@ -249,7 +270,8 @@ static struct platform_device ixp46x_i2c_controller = {
 };
 
 static struct platform_device *ixp46x_devices[] __initdata = {
-	&ixp46x_i2c_controller
+	&ixp46x_hwrandom_device,
+	&ixp46x_i2c_controller,
 };
 
 unsigned long ixp4xx_exp_bus_size;
diff --git a/drivers/char/hw_random/ixp4xx-rng.c b/drivers/char/hw_random/ixp4xx-rng.c
index defd8176cb68..8b59aeefd4a4 100644
--- a/drivers/char/hw_random/ixp4xx-rng.c
+++ b/drivers/char/hw_random/ixp4xx-rng.c
@@ -15,6 +15,7 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/moduleparam.h>
+#include <linux/platform_device.h>
 #include <linux/init.h>
 #include <linux/bitops.h>
 #include <linux/hw_random.h>
@@ -36,35 +37,31 @@ static struct hwrng ixp4xx_rng_ops = {
 	.data_read	= ixp4xx_rng_data_read,
 };
 
-static int __init ixp4xx_rng_init(void)
+static int ixp4xx_rng_probe(struct platform_device *pdev)
 {
 	void __iomem * rng_base;
-	int err;
+	struct device *dev = &pdev->dev;
+	struct resource *res;
 
 	if (!cpu_is_ixp46x()) /* includes IXP455 */
 		return -ENOSYS;
 
-	rng_base = ioremap(0x70002100, 4);
-	if (!rng_base)
-		return -ENOMEM;
-	ixp4xx_rng_ops.priv = (unsigned long)rng_base;
-	err = hwrng_register(&ixp4xx_rng_ops);
-	if (err)
-		iounmap(rng_base);
-
-	return err;
-}
+	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+	rng_base = devm_ioremap_resource(dev, res);
+	if (IS_ERR(rng_base))
+		return PTR_ERR(rng_base);
 
-static void __exit ixp4xx_rng_exit(void)
-{
-	void __iomem * rng_base = (void __iomem *)ixp4xx_rng_ops.priv;
-
-	hwrng_unregister(&ixp4xx_rng_ops);
-	iounmap(rng_base);
+	ixp4xx_rng_ops.priv = (unsigned long)rng_base;
+	return devm_hwrng_register(dev, &ixp4xx_rng_ops);
 }
 
-module_init(ixp4xx_rng_init);
-module_exit(ixp4xx_rng_exit);
+static struct platform_driver ixp4xx_rng_driver = {
+	.driver = {
+		.name = "ixp4xx-hwrandom",
+	},
+	.probe = ixp4xx_rng_probe,
+};
+module_platform_driver(ixp4xx_rng_driver);
 
 MODULE_AUTHOR("Deepak Saxena <dsaxena@plexity.net>");
 MODULE_DESCRIPTION("H/W Pseudo-Random Number Generator (RNG) driver for IXP45x/46x");
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 4/5] hw_random: ixp4xx: Add DT bindings
  2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
  2021-05-11 13:29 ` [PATCH 2/5] hw_random: ixp4xx: Use SPDX license tag Linus Walleij
  2021-05-11 13:29 ` [PATCH 3/5] hw_random: ixp4xx: Turn into a module Linus Walleij
@ 2021-05-11 13:29 ` Linus Walleij
  2021-05-17 22:44   ` Rob Herring
  2021-05-11 13:29 ` [PATCH 5/5] hw_random: ixp4xx: Add OF support Linus Walleij
                   ` (3 subsequent siblings)
  6 siblings, 1 reply; 13+ messages in thread
From: Linus Walleij @ 2021-05-11 13:29 UTC (permalink / raw)
  To: Matt Mackall, Herbert Xu, Deepak Saxena
  Cc: linux-crypto, Arnd Bergmann, Linus Walleij, devicetree

This adds device tree bindings for the simple random number
generator found in the IXP46x SoCs.

Cc: Deepak Saxena <dsaxena@plexity.net>
Cc: devicetree@vger.kernel.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
The idea is to apply this through the ARM SoC tree along
with other IXP4xx refactorings.
Please tell me if you prefer another solution.
---
 .../bindings/rng/intel,ixp46x-rng.yaml        | 36 +++++++++++++++++++
 MAINTAINERS                                   |  1 +
 2 files changed, 37 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/rng/intel,ixp46x-rng.yaml

diff --git a/Documentation/devicetree/bindings/rng/intel,ixp46x-rng.yaml b/Documentation/devicetree/bindings/rng/intel,ixp46x-rng.yaml
new file mode 100644
index 000000000000..61963fa9347e
--- /dev/null
+++ b/Documentation/devicetree/bindings/rng/intel,ixp46x-rng.yaml
@@ -0,0 +1,36 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/rng/intel,ixp46x-rng.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Intel IXP46x RNG bindings
+
+description: |
+  The Intel IXP46x has a random number generator at a fixed physical
+  location in memory. Each read is guaranteed to provide a unique
+  32 bit random number.
+
+maintainers:
+  - Linus Walleij <linus.walleij@linaro.org>
+
+properties:
+  compatible:
+    const: intel,ixp46x-rng
+
+  reg:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    rng@70002100 {
+      compatible = "intel,ixp46x-rng";
+      reg = <0x70002100 4>;
+    };
+
diff --git a/MAINTAINERS b/MAINTAINERS
index bd7aff0c120f..cc4bb4e75f04 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9248,6 +9248,7 @@ F:	include/linux/soc/ixp4xx/qmgr.h
 INTEL IXP4XX RANDOM NUMBER GENERATOR SUPPORT
 M:	Deepak Saxena <dsaxena@plexity.net>
 S:	Maintained
+F:	Documentation/devicetree/bindings/display/intel,ixp46x-rng.yaml
 F:	drivers/char/hw_random/ixp4xx-rng.c
 
 INTEL KEEM BAY DRM DRIVER
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 5/5] hw_random: ixp4xx: Add OF support
  2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
                   ` (2 preceding siblings ...)
  2021-05-11 13:29 ` [PATCH 4/5] hw_random: ixp4xx: Add DT bindings Linus Walleij
@ 2021-05-11 13:29 ` Linus Walleij
  2021-05-21  7:11   ` Herbert Xu
  2021-05-11 16:22 ` [PATCH 1/5] hw_random: ixp4xx: enable compile-testing kernel test robot
                   ` (2 subsequent siblings)
  6 siblings, 1 reply; 13+ messages in thread
From: Linus Walleij @ 2021-05-11 13:29 UTC (permalink / raw)
  To: Matt Mackall, Herbert Xu, Deepak Saxena
  Cc: linux-crypto, Arnd Bergmann, Linus Walleij

This makes the hardware random number generator found in
the IXP46x SoCs probe from the device tree.

Cc: Deepak Saxena <dsaxena@plexity.net>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
The idea is to apply this through the ARM SoC tree along
with other IXP4xx refactorings.
Please tell me if you prefer another solution.
---
 drivers/char/hw_random/ixp4xx-rng.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/drivers/char/hw_random/ixp4xx-rng.c b/drivers/char/hw_random/ixp4xx-rng.c
index 8b59aeefd4a4..188854dd16a9 100644
--- a/drivers/char/hw_random/ixp4xx-rng.c
+++ b/drivers/char/hw_random/ixp4xx-rng.c
@@ -19,6 +19,7 @@
 #include <linux/init.h>
 #include <linux/bitops.h>
 #include <linux/hw_random.h>
+#include <linux/of.h>
 #include <linux/soc/ixp4xx/cpu.h>
 
 #include <asm/io.h>
@@ -55,9 +56,18 @@ static int ixp4xx_rng_probe(struct platform_device *pdev)
 	return devm_hwrng_register(dev, &ixp4xx_rng_ops);
 }
 
+static const struct of_device_id ixp4xx_rng_of_match[] = {
+	{
+		.compatible = "intel,ixp46x-rng",
+	},
+	{},
+};
+MODULE_DEVICE_TABLE(of, ixp4xx_rng_of_match);
+
 static struct platform_driver ixp4xx_rng_driver = {
 	.driver = {
 		.name = "ixp4xx-hwrandom",
+		.of_match_table = ixp4xx_rng_of_match,
 	},
 	.probe = ixp4xx_rng_probe,
 };
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/5] hw_random: ixp4xx: enable compile-testing
  2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
                   ` (3 preceding siblings ...)
  2021-05-11 13:29 ` [PATCH 5/5] hw_random: ixp4xx: Add OF support Linus Walleij
@ 2021-05-11 16:22 ` kernel test robot
  2021-05-17 22:06   ` Linus Walleij
  2021-05-11 18:26 ` kernel test robot
  2021-05-21  7:10 ` Herbert Xu
  6 siblings, 1 reply; 13+ messages in thread
From: kernel test robot @ 2021-05-11 16:22 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 2135 bytes --]

Hi Linus,

I love your patch! Yet something to improve:

[auto build test ERROR on char-misc/char-misc-testing]
[also build test ERROR on soc/for-next robh/for-next linus/master v5.13-rc1 next-20210511]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Linus-Walleij/hw_random-ixp4xx-enable-compile-testing/20210511-213010
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git 6efb943b8616ec53a5e444193dccf1af9ad627b5
config: microblaze-randconfig-s032-20210511 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 9.3.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # apt-get install sparse
        # sparse version: v0.6.3-341-g8af24329-dirty
        # https://github.com/0day-ci/linux/commit/3df0348614ea703ac178e060dd1663801c2d02cd
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Linus-Walleij/hw_random-ixp4xx-enable-compile-testing/20210511-213010
        git checkout 3df0348614ea703ac178e060dd1663801c2d02cd
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' W=1 ARCH=microblaze 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> drivers/char/hw_random/ixp4xx-rng.c:24:10: fatal error: linux/soc/ixp4xx/cpu.h: No such file or directory
      24 | #include <linux/soc/ixp4xx/cpu.h>
         |          ^~~~~~~~~~~~~~~~~~~~~~~~
   compilation terminated.


vim +24 drivers/char/hw_random/ixp4xx-rng.c

  > 24	#include <linux/soc/ixp4xx/cpu.h>
    25	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 30936 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/5] hw_random: ixp4xx: enable compile-testing
  2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
                   ` (4 preceding siblings ...)
  2021-05-11 16:22 ` [PATCH 1/5] hw_random: ixp4xx: enable compile-testing kernel test robot
@ 2021-05-11 18:26 ` kernel test robot
  2021-05-21  7:10 ` Herbert Xu
  6 siblings, 0 replies; 13+ messages in thread
From: kernel test robot @ 2021-05-11 18:26 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 2154 bytes --]

Hi Linus,

I love your patch! Yet something to improve:

[auto build test ERROR on char-misc/char-misc-testing]
[also build test ERROR on soc/for-next robh/for-next linus/master v5.13-rc1 next-20210511]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Linus-Walleij/hw_random-ixp4xx-enable-compile-testing/20210511-213010
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git 6efb943b8616ec53a5e444193dccf1af9ad627b5
config: arm-randconfig-r025-20210511 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project a0fed635fe1701470062495a6ffee1c608f3f1bc)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # https://github.com/0day-ci/linux/commit/3df0348614ea703ac178e060dd1663801c2d02cd
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Linus-Walleij/hw_random-ixp4xx-enable-compile-testing/20210511-213010
        git checkout 3df0348614ea703ac178e060dd1663801c2d02cd
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=arm 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

>> drivers/char/hw_random/ixp4xx-rng.c:24:10: fatal error: 'linux/soc/ixp4xx/cpu.h' file not found
   #include <linux/soc/ixp4xx/cpu.h>
            ^~~~~~~~~~~~~~~~~~~~~~~~
   1 error generated.


vim +24 drivers/char/hw_random/ixp4xx-rng.c

  > 24	#include <linux/soc/ixp4xx/cpu.h>
    25	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 30661 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/5] hw_random: ixp4xx: enable compile-testing
  2021-05-11 16:22 ` [PATCH 1/5] hw_random: ixp4xx: enable compile-testing kernel test robot
@ 2021-05-17 22:06   ` Linus Walleij
  0 siblings, 0 replies; 13+ messages in thread
From: Linus Walleij @ 2021-05-17 22:06 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 438 bytes --]

On Tue, May 11, 2021 at 6:22 PM kernel test robot <lkp@intel.com> wrote:

> >> drivers/char/hw_random/ixp4xx-rng.c:24:10: fatal error: linux/soc/ixp4xx/cpu.h: No such file or directory
>       24 | #include <linux/soc/ixp4xx/cpu.h>
>          |          ^~~~~~~~~~~~~~~~~~~~~~~~
>    compilation terminated.

This is just an implicit dependency on another patch which will
be included in the pull request.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 4/5] hw_random: ixp4xx: Add DT bindings
  2021-05-11 13:29 ` [PATCH 4/5] hw_random: ixp4xx: Add DT bindings Linus Walleij
@ 2021-05-17 22:44   ` Rob Herring
  0 siblings, 0 replies; 13+ messages in thread
From: Rob Herring @ 2021-05-17 22:44 UTC (permalink / raw)
  To: Linus Walleij
  Cc: devicetree, Deepak Saxena, linux-crypto, Herbert Xu,
	Arnd Bergmann, Matt Mackall

On Tue, 11 May 2021 15:29:27 +0200, Linus Walleij wrote:
> This adds device tree bindings for the simple random number
> generator found in the IXP46x SoCs.
> 
> Cc: Deepak Saxena <dsaxena@plexity.net>
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> The idea is to apply this through the ARM SoC tree along
> with other IXP4xx refactorings.
> Please tell me if you prefer another solution.
> ---
>  .../bindings/rng/intel,ixp46x-rng.yaml        | 36 +++++++++++++++++++
>  MAINTAINERS                                   |  1 +
>  2 files changed, 37 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/rng/intel,ixp46x-rng.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/5] hw_random: ixp4xx: enable compile-testing
  2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
                   ` (5 preceding siblings ...)
  2021-05-11 18:26 ` kernel test robot
@ 2021-05-21  7:10 ` Herbert Xu
  6 siblings, 0 replies; 13+ messages in thread
From: Herbert Xu @ 2021-05-21  7:10 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Matt Mackall, Deepak Saxena, linux-crypto, Arnd Bergmann

On Tue, May 11, 2021 at 03:29:24PM +0200, Linus Walleij wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> The driver is almost portable already, it just needs to
> include the new header for the cpu definition.
> 
> Cc: Deepak Saxena <dsaxena@plexity.net>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> The idea is to apply this through the ARM SoC tree along
> with other IXP4xx refactorings.
> Please tell me if you prefer another solution.
> ---
>  drivers/char/hw_random/Kconfig      | 2 +-
>  drivers/char/hw_random/ixp4xx-rng.c | 3 +--
>  2 files changed, 2 insertions(+), 3 deletions(-)

Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/5] hw_random: ixp4xx: Use SPDX license tag
  2021-05-11 13:29 ` [PATCH 2/5] hw_random: ixp4xx: Use SPDX license tag Linus Walleij
@ 2021-05-21  7:10   ` Herbert Xu
  0 siblings, 0 replies; 13+ messages in thread
From: Herbert Xu @ 2021-05-21  7:10 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Matt Mackall, Deepak Saxena, linux-crypto, Arnd Bergmann

On Tue, May 11, 2021 at 03:29:25PM +0200, Linus Walleij wrote:
> This switches the IXP4xx hardware random driver to use
> the SPDX type license tag.
> 
> Cc: Deepak Saxena <dsaxena@plexity.net>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> The idea is to apply this through the ARM SoC tree along
> with other IXP4xx refactorings.
> Please tell me if you prefer another solution.
> ---
>  drivers/char/hw_random/ixp4xx-rng.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)

Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/5] hw_random: ixp4xx: Turn into a module
  2021-05-11 13:29 ` [PATCH 3/5] hw_random: ixp4xx: Turn into a module Linus Walleij
@ 2021-05-21  7:10   ` Herbert Xu
  0 siblings, 0 replies; 13+ messages in thread
From: Herbert Xu @ 2021-05-21  7:10 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Matt Mackall, Deepak Saxena, linux-crypto, Arnd Bergmann

On Tue, May 11, 2021 at 03:29:26PM +0200, Linus Walleij wrote:
> Instead of just initializing always, which will invariably
> create problems on multiplatform builds, turn this driver into
> a module and pass a resource with the memory location.
> The device only exist on the IXP46x so we only register
> the device for the IXP46x SoC.
> 
> Cc: Deepak Saxena <dsaxena@plexity.net>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> The idea is to apply this through the ARM SoC tree along
> with other IXP4xx refactorings.
> Please tell me if you prefer another solution.
> ---
>  arch/arm/mach-ixp4xx/common.c       | 24 ++++++++++++++++++-
>  drivers/char/hw_random/ixp4xx-rng.c | 37 +++++++++++++----------------
>  2 files changed, 40 insertions(+), 21 deletions(-)

Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 5/5] hw_random: ixp4xx: Add OF support
  2021-05-11 13:29 ` [PATCH 5/5] hw_random: ixp4xx: Add OF support Linus Walleij
@ 2021-05-21  7:11   ` Herbert Xu
  0 siblings, 0 replies; 13+ messages in thread
From: Herbert Xu @ 2021-05-21  7:11 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Matt Mackall, Deepak Saxena, linux-crypto, Arnd Bergmann

On Tue, May 11, 2021 at 03:29:28PM +0200, Linus Walleij wrote:
> This makes the hardware random number generator found in
> the IXP46x SoCs probe from the device tree.
> 
> Cc: Deepak Saxena <dsaxena@plexity.net>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> The idea is to apply this through the ARM SoC tree along
> with other IXP4xx refactorings.
> Please tell me if you prefer another solution.
> ---
>  drivers/char/hw_random/ixp4xx-rng.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)

Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2021-05-21  7:11 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-11 13:29 [PATCH 1/5] hw_random: ixp4xx: enable compile-testing Linus Walleij
2021-05-11 13:29 ` [PATCH 2/5] hw_random: ixp4xx: Use SPDX license tag Linus Walleij
2021-05-21  7:10   ` Herbert Xu
2021-05-11 13:29 ` [PATCH 3/5] hw_random: ixp4xx: Turn into a module Linus Walleij
2021-05-21  7:10   ` Herbert Xu
2021-05-11 13:29 ` [PATCH 4/5] hw_random: ixp4xx: Add DT bindings Linus Walleij
2021-05-17 22:44   ` Rob Herring
2021-05-11 13:29 ` [PATCH 5/5] hw_random: ixp4xx: Add OF support Linus Walleij
2021-05-21  7:11   ` Herbert Xu
2021-05-11 16:22 ` [PATCH 1/5] hw_random: ixp4xx: enable compile-testing kernel test robot
2021-05-17 22:06   ` Linus Walleij
2021-05-11 18:26 ` kernel test robot
2021-05-21  7:10 ` Herbert Xu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.