Re: [Xen-devel] [PATCH] xen/arm: iommu: Panic if not all IOMMUs are initialized

[Oleksandr <olekstysh@xxxxxxxxx>]

On 20.09.19 03:31, Stefano Stabellini wrote:

Hi, Stefano.

> On Tue, 20 Aug 2019, Oleksandr wrote:
> > On 20.08.19 15:22, Julien Grall wrote:
> >
> > Hi, Julien
> >
> > >    -    iommu_setup();
> > > +    rc = iommu_setup();
> > > +    if ( !iommu_enabled && rc != -ENODEV )
> > > +        panic("Couldn't configure correctly all the IOMMUs.");
> > >    
> > Please add "\n"
> >
> >
> > You can add:
> >
> > Tested-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx>
> Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
>
> I added the "\n", fixed a typo in the commit message, and committed the
> patch.

Thank you, I will re-base and drop dependency from the cover letter for 
the coming V5 (IPMMU+iommu_fwspec).

--
Regards,

Oleksandr Tyshchenko


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel