diff mbox series

[07/24] aspeed: Integrate HACE

Message ID 20210407171637.777743-8-clg@kaod.org
State Superseded
Headers show
Series None | expand

Commit Message

Cédric Le Goater April 7, 2021, 5:16 p.m. UTC
From: Joel Stanley <joel@jms.id.au>


Add the hash and crypto engine model to the Aspeed socs.

Reviewed-by: Andrew Jeffery <andrew@aj.id.au>

Signed-off-by: Joel Stanley <joel@jms.id.au>

Reviewed-by: Cédric Le Goater <clg@kaod.org>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Message-Id: <20210324070955.125941-3-joel@jms.id.au>
Signed-off-by: Cédric Le Goater <clg@kaod.org>

---
 docs/system/arm/aspeed.rst  |  2 +-
 include/hw/arm/aspeed_soc.h |  3 +++
 hw/arm/aspeed_ast2600.c     | 15 +++++++++++++++
 hw/arm/aspeed_soc.c         | 16 ++++++++++++++++
 4 files changed, 35 insertions(+), 1 deletion(-)

-- 
2.26.3

Comments

Klaus Heinrich Kiwi April 7, 2021, 7:22 p.m. UTC | #1
On 4/7/2021 2:16 PM, Cédric Le Goater wrote:
> From: Joel Stanley <joel@jms.id.au>

> 

> Add the hash and crypto engine model to the Aspeed socs.

> 

> Reviewed-by: Andrew Jeffery <andrew@aj.id.au>

> Signed-off-by: Joel Stanley <joel@jms.id.au>

> Reviewed-by: Cédric Le Goater <clg@kaod.org>

> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

> Message-Id: <20210324070955.125941-3-joel@jms.id.au>

> Signed-off-by: Cédric Le Goater <clg@kaod.org>


Reviewed-by: Klaus Heinrich Kiwi <klaus@linux.vnet.ibm.com>


> ---

>   docs/system/arm/aspeed.rst  |  2 +-

>   include/hw/arm/aspeed_soc.h |  3 +++

>   hw/arm/aspeed_ast2600.c     | 15 +++++++++++++++

>   hw/arm/aspeed_soc.c         | 16 ++++++++++++++++

>   4 files changed, 35 insertions(+), 1 deletion(-)

> 

> diff --git a/docs/system/arm/aspeed.rst b/docs/system/arm/aspeed.rst

> index d1fb8f25b39c..f9466e6d8245 100644

> --- a/docs/system/arm/aspeed.rst

> +++ b/docs/system/arm/aspeed.rst

> @@ -49,6 +49,7 @@ Supported devices

>    * Ethernet controllers

>    * Front LEDs (PCA9552 on I2C bus)

>    * LPC Peripheral Controller (a subset of subdevices are supported)

> + * Hash/Crypto Engine (HACE) - Hash support only, no scatter-gather

>   

>   

>   Missing devices

> @@ -59,7 +60,6 @@ Missing devices

>    * PWM and Fan Controller

>    * Slave GPIO Controller

>    * Super I/O Controller

> - * Hash/Crypto Engine

>    * PCI-Express 1 Controller

>    * Graphic Display Controller

>    * PECI Controller

> diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h

> index 9359d6da336d..d9161d26d645 100644

> --- a/include/hw/arm/aspeed_soc.h

> +++ b/include/hw/arm/aspeed_soc.h

> @@ -21,6 +21,7 @@

>   #include "hw/rtc/aspeed_rtc.h"

>   #include "hw/i2c/aspeed_i2c.h"

>   #include "hw/ssi/aspeed_smc.h"

> +#include "hw/misc/aspeed_hace.h"

>   #include "hw/watchdog/wdt_aspeed.h"

>   #include "hw/net/ftgmac100.h"

>   #include "target/arm/cpu.h"

> @@ -50,6 +51,7 @@ struct AspeedSoCState {

>       AspeedTimerCtrlState timerctrl;

>       AspeedI2CState i2c;

>       AspeedSCUState scu;

> +    AspeedHACEState hace;

>       AspeedXDMAState xdma;

>       AspeedSMCState fmc;

>       AspeedSMCState spi[ASPEED_SPIS_NUM];

> @@ -133,6 +135,7 @@ enum {

>       ASPEED_DEV_XDMA,

>       ASPEED_DEV_EMMC,

>       ASPEED_DEV_KCS,

> +    ASPEED_DEV_HACE,

>   };

>   

>   #endif /* ASPEED_SOC_H */

> diff --git a/hw/arm/aspeed_ast2600.c b/hw/arm/aspeed_ast2600.c

> index 2a1255b6a042..e0fbb020c770 100644

> --- a/hw/arm/aspeed_ast2600.c

> +++ b/hw/arm/aspeed_ast2600.c

> @@ -42,6 +42,7 @@ static const hwaddr aspeed_soc_ast2600_memmap[] = {

>       [ASPEED_DEV_ETH2]      = 0x1E680000,

>       [ASPEED_DEV_ETH4]      = 0x1E690000,

>       [ASPEED_DEV_VIC]       = 0x1E6C0000,

> +    [ASPEED_DEV_HACE]      = 0x1E6D0000,

>       [ASPEED_DEV_SDMC]      = 0x1E6E0000,

>       [ASPEED_DEV_SCU]       = 0x1E6E2000,

>       [ASPEED_DEV_XDMA]      = 0x1E6E7000,

> @@ -102,6 +103,7 @@ static const int aspeed_soc_ast2600_irqmap[] = {

>       [ASPEED_DEV_I2C]       = 110,   /* 110 -> 125 */

>       [ASPEED_DEV_ETH1]      = 2,

>       [ASPEED_DEV_ETH2]      = 3,

> +    [ASPEED_DEV_HACE]      = 4,

>       [ASPEED_DEV_ETH3]      = 32,

>       [ASPEED_DEV_ETH4]      = 33,

>       [ASPEED_DEV_KCS]       = 138,   /* 138 -> 142 */

> @@ -213,6 +215,9 @@ static void aspeed_soc_ast2600_init(Object *obj)

>                               TYPE_SYSBUS_SDHCI);

>   

>       object_initialize_child(obj, "lpc", &s->lpc, TYPE_ASPEED_LPC);

> +

> +    snprintf(typename, sizeof(typename), "aspeed.hace-%s", socname);

> +    object_initialize_child(obj, "hace", &s->hace, typename);

>   }

>   

>   /*

> @@ -494,6 +499,16 @@ static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)

>       sysbus_connect_irq(SYS_BUS_DEVICE(&s->lpc), 1 + aspeed_lpc_kcs_4,

>                          qdev_get_gpio_in(DEVICE(&s->a7mpcore),

>                                   sc->irqmap[ASPEED_DEV_KCS] + aspeed_lpc_kcs_4));

> +

> +    /* HACE */

> +    object_property_set_link(OBJECT(&s->hace), "dram", OBJECT(s->dram_mr),

> +                             &error_abort);

> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->hace), errp)) {

> +        return;

> +    }

> +    sysbus_mmio_map(SYS_BUS_DEVICE(&s->hace), 0, sc->memmap[ASPEED_DEV_HACE]);

> +    sysbus_connect_irq(SYS_BUS_DEVICE(&s->hace), 0,

> +                       aspeed_soc_get_irq(s, ASPEED_DEV_HACE));

>   }

>   

>   static void aspeed_soc_ast2600_class_init(ObjectClass *oc, void *data)

> diff --git a/hw/arm/aspeed_soc.c b/hw/arm/aspeed_soc.c

> index 817f3ba63dfd..8ed29113f79f 100644

> --- a/hw/arm/aspeed_soc.c

> +++ b/hw/arm/aspeed_soc.c

> @@ -34,6 +34,7 @@ static const hwaddr aspeed_soc_ast2400_memmap[] = {

>       [ASPEED_DEV_VIC]    = 0x1E6C0000,

>       [ASPEED_DEV_SDMC]   = 0x1E6E0000,

>       [ASPEED_DEV_SCU]    = 0x1E6E2000,

> +    [ASPEED_DEV_HACE]   = 0x1E6E3000,

>       [ASPEED_DEV_XDMA]   = 0x1E6E7000,

>       [ASPEED_DEV_VIDEO]  = 0x1E700000,

>       [ASPEED_DEV_ADC]    = 0x1E6E9000,

> @@ -65,6 +66,7 @@ static const hwaddr aspeed_soc_ast2500_memmap[] = {

>       [ASPEED_DEV_VIC]    = 0x1E6C0000,

>       [ASPEED_DEV_SDMC]   = 0x1E6E0000,

>       [ASPEED_DEV_SCU]    = 0x1E6E2000,

> +    [ASPEED_DEV_HACE]   = 0x1E6E3000,

>       [ASPEED_DEV_XDMA]   = 0x1E6E7000,

>       [ASPEED_DEV_ADC]    = 0x1E6E9000,

>       [ASPEED_DEV_VIDEO]  = 0x1E700000,

> @@ -117,6 +119,7 @@ static const int aspeed_soc_ast2400_irqmap[] = {

>       [ASPEED_DEV_ETH2]   = 3,

>       [ASPEED_DEV_XDMA]   = 6,

>       [ASPEED_DEV_SDHCI]  = 26,

> +    [ASPEED_DEV_HACE]   = 4,

>   };

>   

>   #define aspeed_soc_ast2500_irqmap aspeed_soc_ast2400_irqmap

> @@ -212,6 +215,9 @@ static void aspeed_soc_init(Object *obj)

>       }

>   

>       object_initialize_child(obj, "lpc", &s->lpc, TYPE_ASPEED_LPC);

> +

> +    snprintf(typename, sizeof(typename), "aspeed.hace-%s", socname);

> +    object_initialize_child(obj, "hace", &s->hace, typename);

>   }

>   

>   static void aspeed_soc_realize(DeviceState *dev, Error **errp)

> @@ -421,6 +427,16 @@ static void aspeed_soc_realize(DeviceState *dev, Error **errp)

>   

>       sysbus_connect_irq(SYS_BUS_DEVICE(&s->lpc), 1 + aspeed_lpc_kcs_4,

>                          qdev_get_gpio_in(DEVICE(&s->lpc), aspeed_lpc_kcs_4));

> +

> +    /* HACE */

> +    object_property_set_link(OBJECT(&s->hace), "dram", OBJECT(s->dram_mr),

> +                             &error_abort);

> +    if (!sysbus_realize(SYS_BUS_DEVICE(&s->hace), errp)) {

> +        return;

> +    }

> +    sysbus_mmio_map(SYS_BUS_DEVICE(&s->hace), 0, sc->memmap[ASPEED_DEV_HACE]);

> +    sysbus_connect_irq(SYS_BUS_DEVICE(&s->hace), 0,

> +                       aspeed_soc_get_irq(s, ASPEED_DEV_HACE));

>   }

>   static Property aspeed_soc_properties[] = {

>       DEFINE_PROP_LINK("dram", AspeedSoCState, dram_mr, TYPE_MEMORY_REGION,

> 


-- 
Klaus Heinrich Kiwi <klaus@linux.vnet.ibm.com>
diff mbox series

Patch

diff --git a/docs/system/arm/aspeed.rst b/docs/system/arm/aspeed.rst
index d1fb8f25b39c..f9466e6d8245 100644
--- a/docs/system/arm/aspeed.rst
+++ b/docs/system/arm/aspeed.rst
@@ -49,6 +49,7 @@  Supported devices
  * Ethernet controllers
  * Front LEDs (PCA9552 on I2C bus)
  * LPC Peripheral Controller (a subset of subdevices are supported)
+ * Hash/Crypto Engine (HACE) - Hash support only, no scatter-gather
 
 
 Missing devices
@@ -59,7 +60,6 @@  Missing devices
  * PWM and Fan Controller
  * Slave GPIO Controller
  * Super I/O Controller
- * Hash/Crypto Engine
  * PCI-Express 1 Controller
  * Graphic Display Controller
  * PECI Controller
diff --git a/include/hw/arm/aspeed_soc.h b/include/hw/arm/aspeed_soc.h
index 9359d6da336d..d9161d26d645 100644
--- a/include/hw/arm/aspeed_soc.h
+++ b/include/hw/arm/aspeed_soc.h
@@ -21,6 +21,7 @@ 
 #include "hw/rtc/aspeed_rtc.h"
 #include "hw/i2c/aspeed_i2c.h"
 #include "hw/ssi/aspeed_smc.h"
+#include "hw/misc/aspeed_hace.h"
 #include "hw/watchdog/wdt_aspeed.h"
 #include "hw/net/ftgmac100.h"
 #include "target/arm/cpu.h"
@@ -50,6 +51,7 @@  struct AspeedSoCState {
     AspeedTimerCtrlState timerctrl;
     AspeedI2CState i2c;
     AspeedSCUState scu;
+    AspeedHACEState hace;
     AspeedXDMAState xdma;
     AspeedSMCState fmc;
     AspeedSMCState spi[ASPEED_SPIS_NUM];
@@ -133,6 +135,7 @@  enum {
     ASPEED_DEV_XDMA,
     ASPEED_DEV_EMMC,
     ASPEED_DEV_KCS,
+    ASPEED_DEV_HACE,
 };
 
 #endif /* ASPEED_SOC_H */
diff --git a/hw/arm/aspeed_ast2600.c b/hw/arm/aspeed_ast2600.c
index 2a1255b6a042..e0fbb020c770 100644
--- a/hw/arm/aspeed_ast2600.c
+++ b/hw/arm/aspeed_ast2600.c
@@ -42,6 +42,7 @@  static const hwaddr aspeed_soc_ast2600_memmap[] = {
     [ASPEED_DEV_ETH2]      = 0x1E680000,
     [ASPEED_DEV_ETH4]      = 0x1E690000,
     [ASPEED_DEV_VIC]       = 0x1E6C0000,
+    [ASPEED_DEV_HACE]      = 0x1E6D0000,
     [ASPEED_DEV_SDMC]      = 0x1E6E0000,
     [ASPEED_DEV_SCU]       = 0x1E6E2000,
     [ASPEED_DEV_XDMA]      = 0x1E6E7000,
@@ -102,6 +103,7 @@  static const int aspeed_soc_ast2600_irqmap[] = {
     [ASPEED_DEV_I2C]       = 110,   /* 110 -> 125 */
     [ASPEED_DEV_ETH1]      = 2,
     [ASPEED_DEV_ETH2]      = 3,
+    [ASPEED_DEV_HACE]      = 4,
     [ASPEED_DEV_ETH3]      = 32,
     [ASPEED_DEV_ETH4]      = 33,
     [ASPEED_DEV_KCS]       = 138,   /* 138 -> 142 */
@@ -213,6 +215,9 @@  static void aspeed_soc_ast2600_init(Object *obj)
                             TYPE_SYSBUS_SDHCI);
 
     object_initialize_child(obj, "lpc", &s->lpc, TYPE_ASPEED_LPC);
+
+    snprintf(typename, sizeof(typename), "aspeed.hace-%s", socname);
+    object_initialize_child(obj, "hace", &s->hace, typename);
 }
 
 /*
@@ -494,6 +499,16 @@  static void aspeed_soc_ast2600_realize(DeviceState *dev, Error **errp)
     sysbus_connect_irq(SYS_BUS_DEVICE(&s->lpc), 1 + aspeed_lpc_kcs_4,
                        qdev_get_gpio_in(DEVICE(&s->a7mpcore),
                                 sc->irqmap[ASPEED_DEV_KCS] + aspeed_lpc_kcs_4));
+
+    /* HACE */
+    object_property_set_link(OBJECT(&s->hace), "dram", OBJECT(s->dram_mr),
+                             &error_abort);
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->hace), errp)) {
+        return;
+    }
+    sysbus_mmio_map(SYS_BUS_DEVICE(&s->hace), 0, sc->memmap[ASPEED_DEV_HACE]);
+    sysbus_connect_irq(SYS_BUS_DEVICE(&s->hace), 0,
+                       aspeed_soc_get_irq(s, ASPEED_DEV_HACE));
 }
 
 static void aspeed_soc_ast2600_class_init(ObjectClass *oc, void *data)
diff --git a/hw/arm/aspeed_soc.c b/hw/arm/aspeed_soc.c
index 817f3ba63dfd..8ed29113f79f 100644
--- a/hw/arm/aspeed_soc.c
+++ b/hw/arm/aspeed_soc.c
@@ -34,6 +34,7 @@  static const hwaddr aspeed_soc_ast2400_memmap[] = {
     [ASPEED_DEV_VIC]    = 0x1E6C0000,
     [ASPEED_DEV_SDMC]   = 0x1E6E0000,
     [ASPEED_DEV_SCU]    = 0x1E6E2000,
+    [ASPEED_DEV_HACE]   = 0x1E6E3000,
     [ASPEED_DEV_XDMA]   = 0x1E6E7000,
     [ASPEED_DEV_VIDEO]  = 0x1E700000,
     [ASPEED_DEV_ADC]    = 0x1E6E9000,
@@ -65,6 +66,7 @@  static const hwaddr aspeed_soc_ast2500_memmap[] = {
     [ASPEED_DEV_VIC]    = 0x1E6C0000,
     [ASPEED_DEV_SDMC]   = 0x1E6E0000,
     [ASPEED_DEV_SCU]    = 0x1E6E2000,
+    [ASPEED_DEV_HACE]   = 0x1E6E3000,
     [ASPEED_DEV_XDMA]   = 0x1E6E7000,
     [ASPEED_DEV_ADC]    = 0x1E6E9000,
     [ASPEED_DEV_VIDEO]  = 0x1E700000,
@@ -117,6 +119,7 @@  static const int aspeed_soc_ast2400_irqmap[] = {
     [ASPEED_DEV_ETH2]   = 3,
     [ASPEED_DEV_XDMA]   = 6,
     [ASPEED_DEV_SDHCI]  = 26,
+    [ASPEED_DEV_HACE]   = 4,
 };
 
 #define aspeed_soc_ast2500_irqmap aspeed_soc_ast2400_irqmap
@@ -212,6 +215,9 @@  static void aspeed_soc_init(Object *obj)
     }
 
     object_initialize_child(obj, "lpc", &s->lpc, TYPE_ASPEED_LPC);
+
+    snprintf(typename, sizeof(typename), "aspeed.hace-%s", socname);
+    object_initialize_child(obj, "hace", &s->hace, typename);
 }
 
 static void aspeed_soc_realize(DeviceState *dev, Error **errp)
@@ -421,6 +427,16 @@  static void aspeed_soc_realize(DeviceState *dev, Error **errp)
 
     sysbus_connect_irq(SYS_BUS_DEVICE(&s->lpc), 1 + aspeed_lpc_kcs_4,
                        qdev_get_gpio_in(DEVICE(&s->lpc), aspeed_lpc_kcs_4));
+
+    /* HACE */
+    object_property_set_link(OBJECT(&s->hace), "dram", OBJECT(s->dram_mr),
+                             &error_abort);
+    if (!sysbus_realize(SYS_BUS_DEVICE(&s->hace), errp)) {
+        return;
+    }
+    sysbus_mmio_map(SYS_BUS_DEVICE(&s->hace), 0, sc->memmap[ASPEED_DEV_HACE]);
+    sysbus_connect_irq(SYS_BUS_DEVICE(&s->hace), 0,
+                       aspeed_soc_get_irq(s, ASPEED_DEV_HACE));
 }
 static Property aspeed_soc_properties[] = {
     DEFINE_PROP_LINK("dram", AspeedSoCState, dram_mr, TYPE_MEMORY_REGION,