Re: [PATCH v2 2/7] dt-bindings: hypervisor: Add MediaTek GenieZone hypervisor

On Fri, 2023-05-12 at 12:01 +0100, Conor Dooley wrote:

On Fri, May 12, 2023 at 06:42:51AM +0000, Yi-De Wu (吳一德) wrote:

quoted

On Fri, 2023-04-28 at 16:24 -0500, Rob Herring wrote:

quoted

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:

quoted

From: "Yingshiuan Pan" <redacted>

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.

Noted, we would stop updating newer version until there's some
consensus to do so.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel