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

[Julien Grall <julien.grall@xxxxxxx>]

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@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel