[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 01/24] arm/stm32f405: Fix realization of "stm32f2xx-adc" devi
From: |
Alistair Francis |
Subject: |
Re: [PATCH 01/24] arm/stm32f405: Fix realization of "stm32f2xx-adc" devices |
Date: |
Tue, 19 May 2020 15:20:15 -0700 |
On Mon, May 18, 2020 at 10:08 PM Markus Armbruster <address@hidden> wrote:
>
> Alistair Francis <address@hidden> writes:
>
> > On Sun, May 17, 2020 at 10:06 PM Markus Armbruster <address@hidden> wrote:
> >>
> >> stm32f405_soc_initfn() creates six such devices, but
> >> stm32f405_soc_realize() realizes only one. Affects machine
> >> netduinoplus2.
> >>
> >> I wonder how this ever worked. If the "device becomes real only on
> >> realize" thing actually works, then we've always been missing five of
> >> six such devices, yet nobody noticed.
> >
> > I must have just been testing the first ADC.
> >
> >>
> >> Fix stm32f405_soc_realize() to realize all six. Visible in "info
> >> qtree":
> >>
> >> bus: main-system-bus
> >> type System
> >> dev: stm32f405-soc, id ""
> >> cpu-type = "cortex-m4-arm-cpu"
> >> dev: stm32f2xx-adc, id ""
> >> gpio-out "sysbus-irq" 1
> >> - mmio ffffffffffffffff/00000000000000ff
> >> + mmio 0000000040012000/00000000000000ff
> >> dev: stm32f2xx-adc, id ""
> >> gpio-out "sysbus-irq" 1
> >> - mmio ffffffffffffffff/00000000000000ff
> >> + mmio 0000000040012000/00000000000000ff
> >> dev: stm32f2xx-adc, id ""
> >> gpio-out "sysbus-irq" 1
> >> - mmio ffffffffffffffff/00000000000000ff
> >> + mmio 0000000040012000/00000000000000ff
> >> dev: stm32f2xx-adc, id ""
> >> gpio-out "sysbus-irq" 1
> >> - mmio ffffffffffffffff/00000000000000ff
> >> + mmio 0000000040012000/00000000000000ff
> >> dev: stm32f2xx-adc, id ""
> >> gpio-out "sysbus-irq" 1
> >> mmio 0000000040012000/00000000000000ff
> >> dev: stm32f2xx-adc, id ""
> >> gpio-out "sysbus-irq" 1
> >> - mmio ffffffffffffffff/00000000000000ff
> >> + mmio 0000000040012000/00000000000000ff
> >> dev: armv7m, id ""
> >>
> >> The mmio addresses look suspicious.
> >
> > Good catch, thanks :)
>
> I'd love to squash in corrections, but I don't know the correct
> addresses. Can you help?
Yep, thanks for squashing it in.
The three addresses are:
0x40012000
0x40012100
0x40012200
and they all share interrupt number 18.
Let me know if you want me to do it.
Alistair
>
> >>
> >> Fixes: 529fc5fd3e18ace8f739afd02dc0953354f39442
> >> Cc: Alistair Francis <address@hidden>
> >> Cc: Peter Maydell <address@hidden>
> >> Cc: address@hidden
> >> Signed-off-by: Markus Armbruster <address@hidden>
> >
> > Reviewed-by: Alistair Francis <address@hidden>
>
> Thanks!
>