mbox series

[v3,0/2] hw: vmmouse: use link property instead of DEFINE_PROP_PTR

Message ID 1543467132-2873-1-git-send-email-liq3ea@gmail.com (mailing list archive)
Headers show
Series hw: vmmouse: use link property instead of DEFINE_PROP_PTR | expand

Message

Li Qiang Nov. 29, 2018, 4:52 a.m. UTC 
According https://wiki.qemu.org/Contribute/BiteSizedTasks
the 'DEFINE_PROP_PTR' should be replaced by QOM link property.
The first patch replace constant strings with TYPE_XXXX and move some
definition to pc.h header file so that the second patch can work.

Change since v2: detail commit message
Change since v1: fix some issues per Markus' review

Li Qiang (2):
  hw: pc: use TYPE_XXX instead of constant strings
  hw: vmmouse: Use link instead of pointer property

 hw/i386/pc.c         |  9 +++++----
 hw/i386/vmmouse.c    | 18 +++++++++++-------
 include/hw/i386/pc.h |  3 +++
 3 files changed, 19 insertions(+), 11 deletions(-)

Comments

Li Qiang Dec. 17, 2018, 1:10 a.m. UTC | #1 
Ping....Paolo, could these patches go to your misc tree?

Thanks,
Li Qiang

Li Qiang <liq3ea@gmail.com> 于2018年11月29日周四 下午12:53写道：

> According https://wiki.qemu.org/Contribute/BiteSizedTasks
> the 'DEFINE_PROP_PTR' should be replaced by QOM link property.
> The first patch replace constant strings with TYPE_XXXX and move some
> definition to pc.h header file so that the second patch can work.
>
> Change since v2: detail commit message
> Change since v1: fix some issues per Markus' review
>
> Li Qiang (2):
>   hw: pc: use TYPE_XXX instead of constant strings
>   hw: vmmouse: Use link instead of pointer property
>
>  hw/i386/pc.c         |  9 +++++----
>  hw/i386/vmmouse.c    | 18 +++++++++++-------
>  include/hw/i386/pc.h |  3 +++
>  3 files changed, 19 insertions(+), 11 deletions(-)
>
> --
> 2.11.0
>
>
Michael S. Tsirkin Dec. 17, 2018, 2:25 a.m. UTC | #2 
On Mon, Dec 17, 2018 at 09:10:06AM +0800, Li Qiang wrote:
> Ping....Paolo, could these patches go to your misc tree?
> 
> Thanks,
> Li Qiang


Fine by me

Reviewed-by: Michael S. Tsirkin <mst@redhat.com>

> Li Qiang <liq3ea@gmail.com> 于2018年11月29日周四下午12:53写道：
> 
>     According https://wiki.qemu.org/Contribute/BiteSizedTasks
>     the 'DEFINE_PROP_PTR' should be replaced by QOM link property.
>     The first patch replace constant strings with TYPE_XXXX and move some
>     definition to pc.h header file so that the second patch can work.
> 
>     Change since v2: detail commit message
>     Change since v1: fix some issues per Markus' review
> 
>     Li Qiang (2):
>       hw: pc: use TYPE_XXX instead of constant strings
>       hw: vmmouse: Use link instead of pointer property
> 
>      hw/i386/pc.c         |  9 +++++----
>      hw/i386/vmmouse.c    | 18 +++++++++++-------
>      include/hw/i386/pc.h |  3 +++
>      3 files changed, 19 insertions(+), 11 deletions(-)
> 
>     --
>     2.11.0
> 
>