[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [XEN/ARM PATCH v6 1/1] Add OdroidXU board (Exynos 5410)



On Thu, 2014-09-04 at 15:57 -0700, Suriyan Ramasami wrote:

>      BUILD_BUG_ON(EXYNOS5_MCT_G_TCON >= PAGE_SIZE);
>  
> -    mct = ioremap_attr(EXYNOS5_MCT_BASE, PAGE_SIZE, PAGE_HYPERVISOR_NOCACHE);
> +    node = dt_find_compatible_node(NULL, NULL, "samsung,exynos4210-mct");
> +    if ( !node )
> +    {
> +        dprintk(XENLOG_ERR, "samsung,exynos4210-mct missing in DT\n");
> +        return -ENXIO;
> +    }
> +
> +    rc = dt_device_get_address(node, 0, &mct_base_addr, &size);
> +    if ( rc )
> +    {
> +        dprintk(XENLOG_ERR, "Error in \"samsung,exynos4210-mct\"\n");
> +        return -ENXIO;
> +    }
> +
> +    dprintk(XENLOG_INFO, "mct_base_addr: %016llx size: %016llx\n",
> +            mct_base_addr, size);
> +
> +    mct = ioremap_attr(mct_base_addr, PAGE_SIZE, PAGE_HYPERVISOR_NOCACHE);

I noticed this while applying, it wasn't enough to stop me but it would
be nice to fix:

We should drop that BUILD_BUG_ON and use the size retrieved from the DT
for the mapping instead.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.