|
From: | Aleksandar Markovic |
Subject: | Re: [PATCH rc3 01/30] target/avr: Add basic parameters for new AVR platform |
Date: | Wed, 29 Jan 2020 11:09:02 +0100 |
08:38 Sre, 29.01.2020. Philippe Mathieu-Daudé <address@hidden> је написао/ла:
>
> On 1/28/20 7:01 PM, Aleksandar Markovic wrote:
> > Works for me too.
> >
> >
> > 16:10 Uto, 28.01.2020. Michael Rolnik <address@hidden
> > <mailto:address@hidden>> је написао/ла:
> >
> > Sounds good to me.
> >
> > On Tue, Jan 28, 2020 at 3:49 PM Aleksandar Markovic
> > <address@hidden <mailto:address@hidden>>
> > wrote:
> >
> >
> >
> > On Tuesday, January 28, 2020, Michael Rolnik <address@hidden
> > <mailto:address@hidden>> wrote:
> >
> > Hi all.
> >
> > I am totally lost in the email.
> > Are there any action items for me / someone else ?
> >
> > Regards,
> > Michael Rolnik
> >
> >
> > I think it would be helpful if we have rc4 with two action items:
> >
> > - remove CONFIG_USER_ONLY references, and all dead code thay
> > may have come with it + add check in cpu.h that Philippe suggested.
> > - remove "Atmel" word from all elements of the series (patch
> > names, messages, filenames, structure names, ...)
> >
> > Needless to say that this must be done with extreme care.
> >
> > I propose that Philippe do rc4.
>
> I won't have time to work on AVR this week-end and the next one (family
> plans). If I find time I prefer prioritize the raspi machine because I
> want to see some work merged before Igor "use memdev for RAM" series get
> in. So if you guys have time & motivation, go for it!
>
OK, I am going to do and send rc4.
Yours,
Aleksandar
> >
> > Sincerely,
> > Aleksandar
> >
> >
> > On Mon, Jan 27, 2020 at 10:54 AM Michael Rolnik
> > <address@hidden <mailto:address@hidden>> wrote:
> >
> > Thanks for you help guys.
> >
> > On Mon, Jan 27, 2020 at 12:55 AM Aleksandar Markovic
> > <address@hidden
> > <mailto:address@hidden>> wrote:
> >
> > From: Michael Rolnik <address@hidden
> > <mailto:address@hidden>>
> >
> > This includes definitions of various basic
> > parameters needed
> > for integration of a new platform into QEMU.
> >
> > Co-developed-by: Michael Rolnik <address@hidden
> > <mailto:address@hidden>>
> > Co-developed-by: Sarah Harris <address@hidden
> > <mailto:address@hidden>>
> > Signed-off-by: Michael Rolnik <address@hidden
> > <mailto:address@hidden>>
> > Signed-off-by: Sarah Harris <address@hidden
> > <mailto:address@hidden>>
> > Signed-off-by: Michael Rolnik <address@hidden
> > <mailto:address@hidden>>
> > Acked-by: Igor Mammedov <address@hidden
> > <mailto:address@hidden>>
> > Tested-by: Philippe Mathieu-Daudé <address@hidden
> > <mailto:address@hidden>>
> > Signed-off-by: Richard Henderson
> > <address@hidden
> > <mailto:address@hidden>>
> > Signed-off-by: Aleksandar Markovic
> > <address@hidden
> > <mailto:address@hidden>>
> > ---
> > target/avr/cpu-param.h | 37 ++++++++++++++++++++++++++
> > target/avr/cpu.h | 72
> > ++++++++++++++++++++++++++++++++++++++++++++++++++
> > 2 files changed, 109 insertions(+)
> > create mode 100644 target/avr/cpu-param.h
> > create mode 100644 target/avr/cpu.h
> >
> > diff --git a/target/avr/cpu-param.h
> > b/target/avr/cpu-param.h
> > new file mode 100644
> > index 0000000..0c29ce4
> > --- /dev/null
> > +++ b/target/avr/cpu-param.h
> > @@ -0,0 +1,37 @@
> > +/*
> > + * QEMU AVR CPU
> > + *
> > + * Copyright (c) 2019 Michael Rolnik
> > + *
> > + * This library is free software; you can
> > redistribute it and/or
> > + * modify it under the terms of the GNU Lesser
> > General Public
> > + * License as published by the Free Software
> > Foundation; either
> > + * version 2.1 of the License, or (at your option)
> > any later version.
> > + *
> > + * This library is distributed in the hope that it
> > will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the
> > implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR
> > PURPOSE. See the GNU
> > + * Lesser General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU
> > Lesser General Public
> > + * License along with this library; if not, see
> > + * <http://www.gnu.org/licenses/lgpl-2.1.html>
> > + */
> > +
> > +#ifndef AVR_CPU_PARAM_H
> > +#define AVR_CPU_PARAM_H
> > +
> > +#define TARGET_LONG_BITS 32
> > +/*
> > + * TARGET_PAGE_BITS cannot be more than 8 bits because
> > + * 1. all IO registers occupy [0x0000 .. 0x00ff]
> > address range, and they
> > + * should be implemented as a device and not memory
> > + * 2. SRAM starts at the address 0x0100
> > + */
> > +#define TARGET_PAGE_BITS 8
> > +#define TARGET_PHYS_ADDR_SPACE_BITS 24
> > +#define TARGET_VIRT_ADDR_SPACE_BITS 24
> > +#define NB_MMU_MODES 2
> > +
> > +
> > +#endif
> > diff --git a/target/avr/cpu.h b/target/avr/cpu.h
> > new file mode 100644
> > index 0000000..d122611
> > --- /dev/null
> > +++ b/target/avr/cpu.h
> > @@ -0,0 +1,72 @@
> > +/*
> > + * QEMU AVR CPU
> > + *
> > + * Copyright (c) 2019 Michael Rolnik
> > + *
> > + * This library is free software; you can
> > redistribute it and/or
> > + * modify it under the terms of the GNU Lesser
> > General Public
> > + * License as published by the Free Software
> > Foundation; either
> > + * version 2.1 of the License, or (at your option)
> > any later version.
> > + *
> > + * This library is distributed in the hope that it
> > will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the
> > implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR
> > PURPOSE. See the GNU
> > + * Lesser General Public License for more details.
> > + *
> > + * You should have received a copy of the GNU
> > Lesser General Public
> > + * License along with this library; if not, see
> > + * <http://www.gnu.org/licenses/lgpl-2.1.html>
> > + */
> > +
> > +#ifndef QEMU_AVR_CPU_H
> > +#define QEMU_AVR_CPU_H
> > +
> > +#include "cpu-qom.h"
> > +#include "exec/cpu-defs.h"
> > +
> > +#define TCG_GUEST_DEFAULT_MO 0
> > +#define AVR_CPU_TYPE_SUFFIX "-" TYPE_AVR_CPU
> > +#define AVR_CPU_TYPE_NAME(name) (name
> > AVR_CPU_TYPE_SUFFIX)
> > +#define CPU_RESOLVING_TYPE TYPE_AVR_CPU
> > +
> > +/*
> > + * AVR has two memory spaces, data & code.
> > + * e.g. both have 0 address
> > + * ST/LD instructions access data space
> > + * LPM/SPM and instruction fetching access code
> > memory space
> > + */
> > +#define MMU_CODE_IDX 0
> > +#define MMU_DATA_IDX 1
> > +
> > +#define EXCP_RESET 1
> > +#define EXCP_INT(n) (EXCP_RESET + (n) + 1)
> > +
> > +/* Number of CPU registers */
> > +#define NUMBER_OF_CPU_REGISTERS 32
> > +/* Number of IO registers accessible by ld/st/in/out */
> > +#define NUMBER_OF_IO_REGISTERS 64
> > +
> > +/*
> > + * Offsets of AVR memory regions in host memory space.
> > + *
> > + * This is needed because the AVR has separate code
> > and data address
> > + * spaces that both have start from zero but have
> > to go somewhere in
> > + * host memory.
> > + *
> > + * It's also useful to know where some things are,
> > like the IO registers.
> > + */
> > +/* Flash program memory */
> > +#define OFFSET_CODE 0x00000000
> > +/* CPU registers, IO registers, and SRAM */
> > +#define OFFSET_DATA 0x00800000
> > +/* CPU registers specifically, these are mapped at
> > the start of data */
> > +#define OFFSET_CPU_REGISTERS OFFSET_DATA
> > +/*
> > + * IO registers, including status register, stack
> > pointer, and memory
> > + * mapped peripherals, mapped just after CPU registers
> > + */
> > +#define OFFSET_IO_REGISTERS (OFFSET_DATA +
> > NUMBER_OF_CPU_REGISTERS)
> > +
> > +#define EF_AVR_MACH 0x7F
> > +
> > +#endif /* !defined (QEMU_AVR_CPU_H) */
> > --
> > 2.7.4
> >
> >
> >
> > --
> > Best Regards,
> > Michael Rolnik
> >
> >
> >
> > --
> > Best Regards,
> > Michael Rolnik
> >
> >
> >
> > --
> > Best Regards,
> > Michael Rolnik
> >
>
[Prev in Thread] | Current Thread | [Next in Thread] |