On Mon, 16 Mar 2020 21:06:40 +0900 <masahiroy@xxxxxxxxxx> wrote:
> On Mon, Mar 16, 2020 at 7:43 PM Kunihiko Hayashi
> <hayashi.kunihiko@xxxxxxxxxxxxx> wrote:
> >
> > On Sat, 14 Mar 2020 11:14:09 +0900 <yamada.masahiro@xxxxxxxxxxxxx> wrote:
> >
> > > On Fri, Mar 13, 2020 at 9:58 AM Kunihiko Hayashi
> > > <hayashi.kunihiko@xxxxxxxxxxxxx> wrote:
> > > >
> > > > Add external DMA controller support implemented in UniPhier SoCs.
> > > > This supports for Pro4, Pro5 and PXs2.
> > > >
> > > > Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@xxxxxxxxxxxxx>
> > > > ---
> > > > arch/arm/boot/dts/uniphier-pro4.dtsi | 8 ++++++++
> > > > arch/arm/boot/dts/uniphier-pro5.dtsi | 8 ++++++++
> > > > arch/arm/boot/dts/uniphier-pxs2.dtsi | 8 ++++++++
> > > > 3 files changed, 24 insertions(+)
> > > >
> > > > diff --git a/arch/arm/boot/dts/uniphier-pro4.dtsi b/arch/arm/boot/dts/uniphier-pro4.dtsi
> > > > index 2ec04d7..a1bfe0f 100644
> > > > --- a/arch/arm/boot/dts/uniphier-pro4.dtsi
> > > > +++ b/arch/arm/boot/dts/uniphier-pro4.dtsi
> > > > @@ -426,6 +426,14 @@
> > > > };
> > > > };
> > > >
> > > > + xdmac: dma-controller@5fc10000 {
> > > > + compatible = "socionext,uniphier-xdmac";
> > > > + reg = <0x5fc10000 0x1000>, <0x5fc20000 0x800>;
> > >
> > >
> > >
> > > This is odd.
> > > <0x5fc20000 0x800> causes reg overwrap with
> > > aidet@5fc20000 below.
> >
> > Thank you for pointing out.
> >
> > Indeed, the address is wrong. I'll fix it.
> > And I'll also fix the example of dt-bindings.
> >
> > Thank you,
>
>
>
> You did not notice the over-wrap
> because the second region is not used
> from the driver.
>
> Why did you define the unused region?
There is an extention register area for xdmac, however, the current driver
doesn't have any features using this area.
The description of the area will be suppressed until some features that uses
the area are implemented.
Thank you,
---
Best Regards,
Kunihiko Hayashi
