[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2
From: |
sundeep subbaraya |
Subject: |
Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block |
Date: |
Mon, 18 Sep 2017 18:57:02 +0530 |
Hi Peter,
On Mon, Sep 18, 2017 at 3:47 PM, sundeep subbaraya <address@hidden>
wrote:
> Hi Philippe,
>
> On Mon, Sep 18, 2017 at 6:31 AM, Philippe Mathieu-Daudé <address@hidden>
> wrote:
>
>> Hi Sundeep, Peter,
>>
>>
>> On 09/15/2017 01:59 PM, Subbaraya Sundeep wrote:
>>
>>> Added Sytem register block of Smartfusion2.
>>> This block has PLL registers which are accessed by guest.
>>>
>>> Signed-off-by: Subbaraya Sundeep <address@hidden>
>>> Reviewed-by: Alistair Francis <address@hidden>
>>> ---
>>> hw/misc/Makefile.objs | 1 +
>>> hw/misc/msf2-sysreg.c | 168 ++++++++++++++++++++++++++++++
>>> ++++++++++++
>>> hw/misc/trace-events | 5 ++
>>> include/hw/misc/msf2-sysreg.h | 77 +++++++++++++++++++
>>> 4 files changed, 251 insertions(+)
>>> create mode 100644 hw/misc/msf2-sysreg.c
>>> create mode 100644 include/hw/misc/msf2-sysreg.h
>>>
>>> diff --git a/hw/misc/Makefile.objs b/hw/misc/Makefile.objs
>>> index 29fb922..e8f0a02 100644
>>> --- a/hw/misc/Makefile.objs
>>> +++ b/hw/misc/Makefile.objs
>>> @@ -59,3 +59,4 @@ obj-$(CONFIG_HYPERV_TESTDEV) += hyperv_testdev.o
>>> obj-$(CONFIG_AUX) += auxbus.o
>>> obj-$(CONFIG_ASPEED_SOC) += aspeed_scu.o aspeed_sdmc.o
>>> obj-y += mmio_interface.o
>>> +obj-$(CONFIG_MSF2) += msf2-sysreg.o
>>> diff --git a/hw/misc/msf2-sysreg.c b/hw/misc/msf2-sysreg.c
>>> new file mode 100644
>>> index 0000000..dc3597b
>>> --- /dev/null
>>> +++ b/hw/misc/msf2-sysreg.c
>>> @@ -0,0 +1,168 @@
>>> +/*
>>> + * System Register block model of Microsemi SmartFusion2.
>>> + *
>>> + * Copyright (c) 2017 Subbaraya Sundeep <address@hidden>
>>> + *
>>> + * This program is free software; you can redistribute it and/or
>>> + * modify it under the terms of the GNU General Public License
>>> + * as published by the Free Software Foundation; either version
>>> + * 2 of the License, or (at your option) any later version.
>>> + *
>>> + * You should have received a copy of the GNU General Public License
>>> along
>>> + * with this program; if not, see <http://www.gnu.org/licenses/>.
>>> + */
>>> +
>>> +#include "qemu/osdep.h"
>>> +#include "qemu/log.h"
>>> +#include "hw/misc/msf2-sysreg.h"
>>> +#include "trace.h"
>>> +
>>> +static inline int msf2_divbits(uint32_t div)
>>> +{
>>> + int ret = 0;
>>> +
>>> + switch (div) {
>>> + case 1:
>>> + ret = 0;
>>> + break;
>>> + case 2:
>>> + ret = 1;
>>> + break;
>>> + case 4:
>>> + ret = 2;
>>> + break;
>>> + case 8:
>>> + ret = 4;
>>> + break;
>>> + case 16:
>>> + ret = 5;
>>> + break;
>>> + case 32:
>>> + ret = 6;
>>> + break;
>>> + default:
>>> + break;
>>>
>>
>> eh?
>>
>> + }
>>> +
>>> + return ret;
>>> +}
>>> +
>>> +static void msf2_sysreg_reset(DeviceState *d)
>>> +{
>>> + MSF2SysregState *s = MSF2_SYSREG(d);
>>> +
>>> + s->regs[MSSDDR_PLL_STATUS_LOW_CR] = 0x021A2358;
>>> + s->regs[MSSDDR_PLL_STATUS] = 0x3;
>>> + s->regs[MSSDDR_FACC1_CR] = msf2_divbits(s->apb0div) << 5 |
>>>
>>
>> ctz32(s->apb0div) << 5 ...
>
>
> I will modify like below:
> div0 = s->apb0div < 8 ? ctz32(s->apb0div) : ctz32(s->apb0div) + 1;
> div1 = s->apb1div < 8 ? ctz32(s->apb1div) : ctz32(s->apb1div) + 1;
> s->regs[MSSDDR_FACC1_CR] = div0 << 5 | div1 << 2;
>
>
>>
>> + msf2_divbits(s->apb1div) << 2;
>>> +}
>>> +
>>> +static uint64_t msf2_sysreg_read(void *opaque, hwaddr offset,
>>> + unsigned size)
>>> +{
>>> + MSF2SysregState *s = opaque;
>>> + uint32_t ret = 0;
>>> +
>>> + offset >>= 2;
>>> + if (offset < ARRAY_SIZE(s->regs)) {
>>> + ret = s->regs[offset];
>>> + trace_msf2_sysreg_read(offset << 2, ret);
>>> + } else {
>>> + qemu_log_mask(LOG_GUEST_ERROR,
>>> + "%s: Bad offset 0x%08" HWADDR_PRIx "\n", __func__,
>>> + offset << 2);
>>> + }
>>> +
>>> + return ret;
>>> +}
>>> +
>>> +static void msf2_sysreg_write(void *opaque, hwaddr offset,
>>> + uint64_t val, unsigned size)
>>> +{
>>> + MSF2SysregState *s = opaque;
>>> + uint32_t newval = val;
>>> +
>>> + offset >>= 2;
>>> +
>>> + switch (offset) {
>>> + case MSSDDR_PLL_STATUS:
>>> + trace_msf2_sysreg_write_pll_status();
>>> + break;
>>> +
>>> + case ESRAM_CR:
>>> + case DDR_CR:
>>> + case ENVM_REMAP_BASE_CR:
>>> + if (newval != s->regs[offset]) {
>>> + qemu_log_mask(LOG_UNIMP,
>>> + TYPE_MSF2_SYSREG": remapping not supported\n");
>>>
>>
>> I'd rather exit here than continue with inconsistent cpu, Peter what do
>> you recommend?
>
>
Could you please suggest.
Thanks,
Sundeep
>
>>
>> + }
>>> + break;
>>> +
>>> + default:
>>> + if (offset < ARRAY_SIZE(s->regs)) {
>>> + trace_msf2_sysreg_write(offset << 2, newval,
>>> s->regs[offset]);
>>> + s->regs[offset] = newval;
>>> + } else {
>>> + qemu_log_mask(LOG_GUEST_ERROR,
>>> + "%s: Bad offset 0x%08" HWADDR_PRIx "\n",
>>> __func__,
>>> + offset << 2);
>>> + }
>>> + break;
>>> + }
>>> +}
>>> +
>>> +static const MemoryRegionOps sysreg_ops = {
>>> + .read = msf2_sysreg_read,
>>> + .write = msf2_sysreg_write,
>>> + .endianness = DEVICE_NATIVE_ENDIAN,
>>> +};
>>> +
>>> +static void msf2_sysreg_init(Object *obj)
>>> +{
>>> + MSF2SysregState *s = MSF2_SYSREG(obj);
>>> +
>>> + memory_region_init_io(&s->iomem, obj, &sysreg_ops, s,
>>> TYPE_MSF2_SYSREG,
>>> + MSF2_SYSREG_MMIO_SIZE);
>>> + sysbus_init_mmio(SYS_BUS_DEVICE(obj), &s->iomem);
>>> +}
>>> +
>>> +static const VMStateDescription vmstate_msf2_sysreg = {
>>> + .name = TYPE_MSF2_SYSREG,
>>> + .version_id = 1,
>>> + .minimum_version_id = 1,
>>> + .fields = (VMStateField[]) {
>>> + VMSTATE_UINT32_ARRAY(regs, MSF2SysregState,
>>> MSF2_SYSREG_MMIO_SIZE / 4),
>>> + VMSTATE_END_OF_LIST()
>>> + }
>>> +};
>>> +
>>> +static Property msf2_sysreg_properties[] = {
>>> + /* default divisors in Libero GUI */
>>> + DEFINE_PROP_UINT32("apb0divisor", MSF2SysregState, apb0div, 2),
>>> + DEFINE_PROP_UINT32("apb1divisor", MSF2SysregState, apb1div, 2),
>>>
>>
>> DEFINE_PROP_UINT8() is enough for both.
>>
>> Beware these divisors must be power of 2, and maximum 32.
>
> This can be checked in msf2_sysreg_realize() ...
>
>
> Ok
>
>>
>>
>> + DEFINE_PROP_END_OF_LIST(),
>>> +};
>>> +
>>> +static void msf2_sysreg_class_init(ObjectClass *klass, void *data)
>>> +{
>>> + DeviceClass *dc = DEVICE_CLASS(klass);
>>> +
>>> + dc->vmsd = &vmstate_msf2_sysreg;
>>> + dc->reset = msf2_sysreg_reset;
>>>
>>
>> ... which would go here:
>>
>> dc->realize = msf2_sysreg_realize;
>>
>> Ok.
>
>>
>> + dc->props = msf2_sysreg_properties;
>>> +}
>>> +
>>> +static const TypeInfo msf2_sysreg_info = {
>>> + .name = TYPE_MSF2_SYSREG,
>>> + .parent = TYPE_SYS_BUS_DEVICE,
>>> + .class_init = msf2_sysreg_class_init,
>>> + .instance_size = sizeof(MSF2SysregState),
>>> + .instance_init = msf2_sysreg_init,
>>> +};
>>> +
>>> +static void msf2_sysreg_register_types(void)
>>> +{
>>> + type_register_static(&msf2_sysreg_info);
>>> +}
>>> +
>>> +type_init(msf2_sysreg_register_types)
>>> diff --git a/hw/misc/trace-events b/hw/misc/trace-events
>>> index 3313585..616579a 100644
>>> --- a/hw/misc/trace-events
>>> +++ b/hw/misc/trace-events
>>> @@ -61,3 +61,8 @@ mps2_scc_reset(void) "MPS2 SCC: reset"
>>> mps2_scc_leds(char led7, char led6, char led5, char led4, char led3,
>>> char led2, char led1, char led0) "MPS2 SCC LEDs: %c%c%c%c%c%c%c%c"
>>> mps2_scc_cfg_write(unsigned function, unsigned device, uint32_t value)
>>> "MPS2 SCC config write: function %d device %d data 0x%" PRIx32
>>> mps2_scc_cfg_read(unsigned function, unsigned device, uint32_t value)
>>> "MPS2 SCC config read: function %d device %d data 0x%" PRIx32
>>> +
>>> +# hw/misc/msf2-sysreg.c
>>> +msf2_sysreg_write(uint64_t offset, uint32_t val, uint32_t prev)
>>> "msf2-sysreg write: addr 0x%08" HWADDR_PRIx " data 0x%" PRIx32 " prev 0x%"
>>> PRIx32
>>> +msf2_sysreg_read(uint64_t offset, uint32_t val) "msf2-sysreg read: addr
>>> 0x%08" HWADDR_PRIx " data 0x%08" PRIx32
>>> +msf2_sysreg_write_pll_status(void) "Invalid write to read only PLL
>>> status register"
>>> diff --git a/include/hw/misc/msf2-sysreg.h
>>> b/include/hw/misc/msf2-sysreg.h
>>> new file mode 100644
>>> index 0000000..231f827
>>> --- /dev/null
>>> +++ b/include/hw/misc/msf2-sysreg.h
>>> @@ -0,0 +1,77 @@
>>> +/*
>>> + * Microsemi SmartFusion2 SYSREG
>>> + *
>>> + * Copyright (c) 2017 Subbaraya Sundeep <address@hidden>
>>> + *
>>> + * Permission is hereby granted, free of charge, to any person
>>> obtaining a copy
>>> + * of this software and associated documentation files (the
>>> "Software"), to deal
>>> + * in the Software without restriction, including without limitation
>>> the rights
>>> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or
>>> sell
>>> + * copies of the Software, and to permit persons to whom the Software is
>>> + * furnished to do so, subject to the following conditions:
>>> + *
>>> + * The above copyright notice and this permission notice shall be
>>> included in
>>> + * all copies or substantial portions of the Software.
>>> + *
>>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>>> EXPRESS OR
>>> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>>> MERCHANTABILITY,
>>> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT
>>> SHALL
>>> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>>> OTHER
>>> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>>> ARISING FROM,
>>> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>>> DEALINGS IN
>>> + * THE SOFTWARE.
>>> + */
>>> +
>>> +#ifndef HW_MSF2_SYSREG_H
>>> +#define HW_MSF2_SYSREG_H
>>> +
>>> +#include "hw/sysbus.h"
>>> +
>>> +enum {
>>> + ESRAM_CR = 0x00 / 4,
>>> + ESRAM_MAX_LAT,
>>> + DDR_CR,
>>> + ENVM_CR,
>>> + ENVM_REMAP_BASE_CR,
>>> + ENVM_REMAP_FAB_CR,
>>> + CC_CR,
>>> + CC_REGION_CR,
>>> + CC_LOCK_BASE_ADDR_CR,
>>> + CC_FLUSH_INDX_CR,
>>> + DDRB_BUF_TIMER_CR,
>>> + DDRB_NB_ADDR_CR,
>>> + DDRB_NB_SIZE_CR,
>>> + DDRB_CR,
>>> +
>>> + SOFT_RESET_CR = 0x48 / 4,
>>> + M3_CR,
>>> +
>>> + GPIO_SYSRESET_SEL_CR = 0x58 / 4,
>>> +
>>> + MDDR_CR = 0x60 / 4,
>>> +
>>> + MSSDDR_PLL_STATUS_LOW_CR = 0x90 / 4,
>>> + MSSDDR_PLL_STATUS_HIGH_CR,
>>> + MSSDDR_FACC1_CR,
>>> + MSSDDR_FACC2_CR,
>>> +
>>> + MSSDDR_PLL_STATUS = 0x150 / 4,
>>> +};
>>> +
>>> +#define MSF2_SYSREG_MMIO_SIZE 0x300
>>> +
>>> +#define TYPE_MSF2_SYSREG "msf2-sysreg"
>>> +#define MSF2_SYSREG(obj) OBJECT_CHECK(MSF2SysregState, (obj),
>>> TYPE_MSF2_SYSREG)
>>> +
>>> +typedef struct MSF2SysregState {
>>> + SysBusDevice parent_obj;
>>> +
>>> + MemoryRegion iomem;
>>> +
>>> + uint32_t apb0div;
>>> + uint32_t apb1div;
>>>
>>
>> uint8_t for both
>
>
> Ok.
>
>>
>>
>> +
>>> + uint32_t regs[MSF2_SYSREG_MMIO_SIZE / 4];
>>> +} MSF2SysregState;
>>> +
>>> +#endif /* HW_MSF2_SYSREG_H */
>>>
>>>
>> Acked-by: Philippe Mathieu-Daudé <address@hidden>
>> Tested-by: Philippe Mathieu-Daudé <address@hidden>
>>
>
> Thank you,
> Sundeep
>
- [Qemu-devel] [Qemu devel v9 PATCH 0/5] Add support for Smartfusion2 SoC, Subbaraya Sundeep, 2017/09/15
- [Qemu-devel] [Qemu devel v9 PATCH 1/5] msf2: Add Smartfusion2 System timer, Subbaraya Sundeep, 2017/09/15
- [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block, Subbaraya Sundeep, 2017/09/15
- Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block, Philippe Mathieu-Daudé, 2017/09/17
- Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block,
sundeep subbaraya <=
- Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block, Peter Maydell, 2017/09/18
- Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block, Philippe Mathieu-Daudé, 2017/09/18
- Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block, Peter Maydell, 2017/09/18
- Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block, Philippe Mathieu-Daudé, 2017/09/18
- Re: [Qemu-devel] [Qemu devel v9 PATCH 2/5] msf2: Microsemi Smartfusion2 System Register block, Peter Maydell, 2017/09/18
[Qemu-devel] [Qemu devel v9 PATCH 3/5] msf2: Add Smartfusion2 SPI controller, Subbaraya Sundeep, 2017/09/15
[Qemu-devel] [Qemu devel v9 PATCH 4/5] msf2: Add Smartfusion2 SoC, Subbaraya Sundeep, 2017/09/15