Re: [Xen-devel] [PATCH V3 31/41] xen/arm: Allow Xen to run on multiple platform without recompilation

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2013-05-10 at 03:18 +0100, Julien Grall wrote:
> Xen can include various platform support (ie: exynos5, versatile express...)
> and choose during boot time a set of callbacks for the current board.
> These callbacks will be called in places where each board can have specific
> code. For the moment the callbacks are:
>     - platform_init: additional initialization for the platform
>     - platform_init_time: some platform (ie: Exynos 5) needs to initialize
>     the timer with an uncommon way
>     - platform_specific_mapping: add mapping to dom0 which are not specified
>     in the device tree
>     - platform_reset: reset the platform
>     - platform_poweroff: poweroff the platform
>     - platform_quirks: list of quirks for a specific board.
> 
> Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
[...]
> +/* List of possible platform */
> +static void dump_platform_table(void)
> +{
> +    const struct platform_desc *p;
> +
> +    printk("Available platform support:\n");
> +
> +    for ( p = _splatform; p != _eplatform; p++ )
> +        printk("    - %s\n", p->name);
> +}
> +
> +int __init platform_init(void)
> +{
[...]
> +    /* We don't have specific operations for this platform */
> +    if ( platform == _eplatform )
> +    {
> +        /* TODO: List compatible */
> +        printk(XENLOG_WARNING "WARNING: Unrecognized/unsupported device tree 
> "
> +              "compatible list\n");
> +        dump_platform_table();

Looks like the TODO is obsolete?

Ian.



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