Re: [Qemu-devel] [PATCH 4/6] [RFC] Header file for the GRLIB components.

2010-12-07 Thread Fabien Chouteau

On 12/06/2010 06:31 PM, Blue Swirl wrote:

On Mon, Dec 6, 2010 at 9:26 AM, Fabien Chouteauchout...@adacore.com  wrote:


Signed-off-by: Fabien Chouteauchout...@adacore.com
---
  hw/grlib.h |   27 +++
  1 files changed, 27 insertions(+), 0 deletions(-)

diff --git a/hw/grlib.h b/hw/grlib.h
new file mode 100644
index 000..eab7974
--- /dev/null
+++ b/hw/grlib.h
@@ -0,0 +1,27 @@
+#ifndef _GRLIB_H_
+#define _GRLIB_H_
+
+#include qdev.h
+
+/* IRQMP */
+
+DeviceState *grlib_irqmp_create(target_phys_addr_t   base,
+CPUState*env,
+qemu_irq   **cpu_irqs,
+uint32_t nr_irqs);
+
+/* GPTimer */
+
+DeviceState *grlib_gptimer_create(target_phys_addr_t  base,
+  uint32_tnr_timers,
+  uint32_tfreq,
+  qemu_irq   *cpu_irqs,
+  int base_irq);
+
+/* APB UART */
+
+DeviceState *grlib_apbuart_create(target_phys_addr_t  base,
+  CharDriverState*serial,
+  qemu_irqirq);
+


The file should not be needed when you move the functions to leon3.c.


These functions can be used to create more machines with GrLib 
components, that's why they are implemented in the grlib_* files.


--
Fabien Chouteau




[Qemu-devel] [PATCH 4/6] [RFC] Header file for the GRLIB components.

2010-12-06 Thread Fabien Chouteau

Signed-off-by: Fabien Chouteau chout...@adacore.com
---
 hw/grlib.h |   27 +++
 1 files changed, 27 insertions(+), 0 deletions(-)

diff --git a/hw/grlib.h b/hw/grlib.h
new file mode 100644
index 000..eab7974
--- /dev/null
+++ b/hw/grlib.h
@@ -0,0 +1,27 @@
+#ifndef _GRLIB_H_
+#define _GRLIB_H_
+
+#include qdev.h
+
+/* IRQMP */
+
+DeviceState *grlib_irqmp_create(target_phys_addr_t   base,
+CPUState*env,
+qemu_irq   **cpu_irqs,
+uint32_t nr_irqs);
+
+/* GPTimer */
+
+DeviceState *grlib_gptimer_create(target_phys_addr_t  base,
+  uint32_tnr_timers,
+  uint32_tfreq,
+  qemu_irq   *cpu_irqs,
+  int base_irq);
+
+/* APB UART */
+
+DeviceState *grlib_apbuart_create(target_phys_addr_t  base,
+  CharDriverState*serial,
+  qemu_irqirq);
+
+#endif /* ! _GRLIB_H_ */
-- 
1.7.1




Re: [Qemu-devel] [PATCH 4/6] [RFC] Header file for the GRLIB components.

2010-12-06 Thread Blue Swirl
On Mon, Dec 6, 2010 at 9:26 AM, Fabien Chouteau chout...@adacore.com wrote:

 Signed-off-by: Fabien Chouteau chout...@adacore.com
 ---
  hw/grlib.h |   27 +++
  1 files changed, 27 insertions(+), 0 deletions(-)

 diff --git a/hw/grlib.h b/hw/grlib.h
 new file mode 100644
 index 000..eab7974
 --- /dev/null
 +++ b/hw/grlib.h
 @@ -0,0 +1,27 @@
 +#ifndef _GRLIB_H_
 +#define _GRLIB_H_
 +
 +#include qdev.h
 +
 +/* IRQMP */
 +
 +DeviceState *grlib_irqmp_create(target_phys_addr_t   base,
 +                                CPUState            *env,
 +                                qemu_irq           **cpu_irqs,
 +                                uint32_t             nr_irqs);
 +
 +/* GPTimer */
 +
 +DeviceState *grlib_gptimer_create(target_phys_addr_t  base,
 +                                  uint32_t            nr_timers,
 +                                  uint32_t            freq,
 +                                  qemu_irq           *cpu_irqs,
 +                                  int                 base_irq);
 +
 +/* APB UART */
 +
 +DeviceState *grlib_apbuart_create(target_phys_addr_t  base,
 +                                  CharDriverState    *serial,
 +                                  qemu_irq            irq);
 +

The file should not be needed when you move the functions to leon3.c.

By the way, the patch series is now broken because earlier patches in
the series needed this file. This would break bisection which is bad.