> -----Original Message----- > From: Peter Maydell [mailto:peter.mayd...@linaro.org] > Sent: Thursday, November 12, 2020 7:20 PM > To: ganqixin <ganqi...@huawei.com> > Cc: QEMU Developers <qemu-devel@nongnu.org>; QEMU Trivial > <qemu-triv...@nongnu.org>; Thomas Huth <th...@redhat.com>; > Zhanghailiang <zhang.zhanghaili...@huawei.com>; Michael S. Tsirkin > <m...@redhat.com>; Laurent Vivier <laur...@vivier.eu>; Markus Armbruster > <arm...@redhat.com>; Chenqun (kuhn) <kuhn.chen...@huawei.com>; > Philippe Mathieu-Daudé <phi...@redhat.com> > Subject: Re: [PATCH v2 2/4] ads7846: put it into the 'display' category > > On Thu, 12 Nov 2020 at 07:25, Gan Qixin <ganqi...@huawei.com> wrote: > > > > The category of the ads7846 device is not set, put it into the 'display' > > category. > > > > Signed-off-by: Gan Qixin <ganqi...@huawei.com> > > --- > > hw/display/ads7846.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/hw/display/ads7846.c b/hw/display/ads7846.c index > > 023165b2a3..b455cb5acd 100644 > > --- a/hw/display/ads7846.c > > +++ b/hw/display/ads7846.c > > @@ -163,10 +163,12 @@ static void ads7846_realize(SSISlave *d, Error > > **errp) > > > > static void ads7846_class_init(ObjectClass *klass, void *data) { > > + DeviceClass *dc = DEVICE_CLASS(klass); > > SSISlaveClass *k = SSI_SLAVE_CLASS(klass); > > > > k->realize = ads7846_realize; > > k->transfer = ads7846_transfer; > > + set_bit(DEVICE_CATEGORY_DISPLAY, dc->categories); > > } > > This isn't a display, despite being in hw/display. It's a touch-screen > controller, > so it ought to be in hw/input and be DEVICE_CATEGORY_INPUT. > Thanks for your reply, I will recategorize ads7846.
Gan Qixin