Re: [PATCH] xen/arm: domain_build: DT: add clocks node to the hypervisor node

[Julien Grall <julien.grall@arm.com>]

Hi Dirk,

On 23/06/16 16:59, Dirk Behme wrote:
> On 23.06.2016 17:02, Julien Grall wrote:
>> On 22/06/16 16:58, Julien Grall wrote:
>>> On 21/06/16 11:15, Dirk Behme wrote:
>>>> +            printk("Failed to remember the clock node of %s\n",
>>>> path);
>>>> +            printk("Use the Linux kernel command
>>>> 'clk_ignore_unused'\n");
>>>> +            return 0;
>>>
>>> I don't think this is tolerable. We need to fix it  once and for all.
>>>
>>> I understand that xen does not provide a realloc function. Is there
>>> another way we can get a rid of this limit?
>>
>> Note that I would wait that we agree on the device tree bindings
>> before reworking this patch.
>
>
> You mean the 'clocks' binding for the hypervisor node? I.e. as proposed in
>
> https://lists.xen.org/archives/html/xen-devel/2016-06/msg02885.html
>
> create a patch for
>
> - linux/Documentation/devicetree/bindings/arm/xen.txt
> - xen/docs/misc/arm/device-tree/guest.txt
>
> and proceed only with the code changes when this is accepted?

The second patch (xen/docs/misc/*) will be a verbatim copy of the first 
one. So writing only the first one is good for now.

I will defer the implementation in Xen but keep the Linux one to give an 
overview to the maintainers (ARM and clocks)

Regards,

-- 
Julien Grall

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