The device uses serial_hds in its realize function and thus can't be
used twice. Apart from that, the comma in its name makes it quite hard
to use for the user anyway, since a comma is normally used to separate
the device name from its properties when using the "-device" parameter
or the "device_add" HMP command.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/arm/xlnx-zynqmp.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c
index 2b27daf..d4b6560 100644
--- a/hw/arm/xlnx-zynqmp.c
+++ b/hw/arm/xlnx-zynqmp.c
@@ -440,6 +440,8 @@ static void xlnx_zynqmp_class_init(ObjectClass *oc, void *data)
dc->props = xlnx_zynqmp_props;
dc->realize = xlnx_zynqmp_realize;
+ /* Reason: Uses serial_hds in realize function, thus can't be used twice */
+ dc->user_creatable = false;
}
static const TypeInfo xlnx_zynqmp_type_info = {
--
1.8.3.1
On Tue, Sep 26, 2017 at 8:51 AM, Thomas Huth <thuth@redhat.com> wrote: > The device uses serial_hds in its realize function and thus can't be > used twice. Apart from that, the comma in its name makes it quite hard > to use for the user anyway, since a comma is normally used to separate > the device name from its properties when using the "-device" parameter > or the "device_add" HMP command. Is it worth changing the name now then as well? > > Signed-off-by: Thomas Huth <thuth@redhat.com> Reviewed-by: Alistair Francis <alistair.francis@xilinx.com> Thanks, Alistair > --- > hw/arm/xlnx-zynqmp.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c > index 2b27daf..d4b6560 100644 > --- a/hw/arm/xlnx-zynqmp.c > +++ b/hw/arm/xlnx-zynqmp.c > @@ -440,6 +440,8 @@ static void xlnx_zynqmp_class_init(ObjectClass *oc, void *data) > > dc->props = xlnx_zynqmp_props; > dc->realize = xlnx_zynqmp_realize; > + /* Reason: Uses serial_hds in realize function, thus can't be used twice */ > + dc->user_creatable = false; > } > > static const TypeInfo xlnx_zynqmp_type_info = { > -- > 1.8.3.1 > >
On 26.09.2017 18:36, Alistair Francis wrote: > On Tue, Sep 26, 2017 at 8:51 AM, Thomas Huth <thuth@redhat.com> wrote: >> The device uses serial_hds in its realize function and thus can't be >> used twice. Apart from that, the comma in its name makes it quite hard >> to use for the user anyway, since a comma is normally used to separate >> the device name from its properties when using the "-device" parameter >> or the "device_add" HMP command. > > Is it worth changing the name now then as well? Not sure, was there a reason for this name? E.g. is it used in a device tree somewhere somehow? Anyway, with user_creatable = false, I think it is not so urgent anymore to change it. >> Signed-off-by: Thomas Huth <thuth@redhat.com> > > Reviewed-by: Alistair Francis <alistair.francis@xilinx.com> Thanks! Thomas
On Tue, Sep 26, 2017 at 9:52 AM, Thomas Huth <thuth@redhat.com> wrote: > On 26.09.2017 18:36, Alistair Francis wrote: >> On Tue, Sep 26, 2017 at 8:51 AM, Thomas Huth <thuth@redhat.com> wrote: >>> The device uses serial_hds in its realize function and thus can't be >>> used twice. Apart from that, the comma in its name makes it quite hard >>> to use for the user anyway, since a comma is normally used to separate >>> the device name from its properties when using the "-device" parameter >>> or the "device_add" HMP command. >> >> Is it worth changing the name now then as well? > > Not sure, was there a reason for this name? E.g. is it used in a device > tree somewhere somehow? Anyway, with user_creatable = false, I think it > is not so urgent anymore to change it. I don't think there was a specific reason. Maybe it was copied from a device tree, but I'm not tied to keeping the comma. Maybe that's something we can investigate in the future. Thanks, Alistair > >>> Signed-off-by: Thomas Huth <thuth@redhat.com> >> >> Reviewed-by: Alistair Francis <alistair.francis@xilinx.com> > > Thanks! > > Thomas >
On 26 September 2017 at 18:03, Alistair Francis <alistair.francis@xilinx.com> wrote: > On Tue, Sep 26, 2017 at 9:52 AM, Thomas Huth <thuth@redhat.com> wrote: >> On 26.09.2017 18:36, Alistair Francis wrote: >>> On Tue, Sep 26, 2017 at 8:51 AM, Thomas Huth <thuth@redhat.com> wrote: >>>> The device uses serial_hds in its realize function and thus can't be >>>> used twice. Apart from that, the comma in its name makes it quite hard >>>> to use for the user anyway, since a comma is normally used to separate >>>> the device name from its properties when using the "-device" parameter >>>> or the "device_add" HMP command. >>> >>> Is it worth changing the name now then as well? >> >> Not sure, was there a reason for this name? E.g. is it used in a device >> tree somewhere somehow? Anyway, with user_creatable = false, I think it >> is not so urgent anymore to change it. > > I don't think there was a specific reason. Maybe it was copied from a > device tree, but I'm not tied to keeping the comma. Maybe that's > something we can investigate in the future. I vaguely recall that the comma was part of an attempt to be consistent about device names by making them follow the device tree names for them rather than just making up our own. Some of the imx devices and boards follow the same convention. thanks -- PMM
On 26 September 2017 at 16:51, Thomas Huth <thuth@redhat.com> wrote: > The device uses serial_hds in its realize function and thus can't be > used twice. Apart from that, the comma in its name makes it quite hard > to use for the user anyway, since a comma is normally used to separate > the device name from its properties when using the "-device" parameter > or the "device_add" HMP command. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > hw/arm/xlnx-zynqmp.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c > index 2b27daf..d4b6560 100644 > --- a/hw/arm/xlnx-zynqmp.c > +++ b/hw/arm/xlnx-zynqmp.c > @@ -440,6 +440,8 @@ static void xlnx_zynqmp_class_init(ObjectClass *oc, void *data) > > dc->props = xlnx_zynqmp_props; > dc->realize = xlnx_zynqmp_realize; > + /* Reason: Uses serial_hds in realize function, thus can't be used twice */ > + dc->user_creatable = false; > } > > static const TypeInfo xlnx_zynqmp_type_info = { Applied to target-arm.next, thanks. -- PMM
© 2016 - 2024 Red Hat, Inc.