Messages in this thread | | | Date | Fri, 12 May 2023 12:01:18 +0100 | From | Conor Dooley <> | Subject | Re: [PATCH v2 2/7] dt-bindings: hypervisor: Add MediaTek GenieZone hypervisor |
| |
On Fri, May 12, 2023 at 06:42:51AM +0000, Yi-De Wu (吳一德) wrote: > On Fri, 2023-04-28 at 16:24 -0500, Rob Herring wrote: > > External email : Please do not click links or open attachments until > > you have verified the sender or the content. > > > > > > On Fri, Apr 28, 2023 at 06:36:17PM +0800, Yi-De Wu wrote: > > > From: "Yingshiuan Pan" <yingshiuan.pan@mediatek.com> > > > > > > Add documentation for GenieZone(gzvm) node. This node informs gzvm > > > driver to start probing if geniezone hypervisor is available and > > > able to do virtual machine operations. > > > > Why can't the driver just try and do virtual machine operations to > > see > > if the hypervisor is there? IOW, make your software interfaces > > discoverable. DT is for non-discoverable hardware. > > > > Rob > > Can do, our hypervisor is discoverable through invoking probing > hypercall, and we use the device tree to prevent unnecessary module > loading on all systems.
Please do not wait until immediately prior to submitting version N+1 before replying to any of the comments on version N. This creates a confusing scenario, where some review comments may be missed due to parallel discussion.
Thanks, Conor.
[unhandled content-type:application/pgp-signature] | |