Message ID | 1461119601-4936-12-git-send-email-david@gibson.dropbear.id.au |
---|---|
State | New |
Headers | show |
On 04/20/2016 12:33 PM, David Gibson wrote: > The VIOsPAPRBus structure has some callback pointers in it which aren't > used anywhere in the code. It's not clear exactly why they were there in > the first place, but they certainly have no function now. It is clear - incorrect QOM'fication in 3954d33ab7f82f5a5 when parts of VIOsPAPRDeviceInfo became parts of VIOsPAPRBus, you can adjust the commit log :) Reviewed-by: Alexey Kardashevskiy <aik@ozlabs.ru> > > Signed-off-by: David Gibson <david@gibson.dropbear.id.au> > --- > include/hw/ppc/spapr_vio.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/include/hw/ppc/spapr_vio.h b/include/hw/ppc/spapr_vio.h > index ea00c3b..380fe4f 100644 > --- a/include/hw/ppc/spapr_vio.h > +++ b/include/hw/ppc/spapr_vio.h > @@ -75,8 +75,6 @@ struct VIOsPAPRDevice { > struct VIOsPAPRBus { > BusState bus; > uint32_t next_reg; > - int (*init)(VIOsPAPRDevice *dev); > - int (*devnode)(VIOsPAPRDevice *dev, void *fdt, int node_off); > }; > > extern VIOsPAPRBus *spapr_vio_bus_init(void); >
On Thu, Apr 21, 2016 at 03:31:33PM +1000, Alexey Kardashevskiy wrote: > On 04/20/2016 12:33 PM, David Gibson wrote: > >The VIOsPAPRBus structure has some callback pointers in it which aren't > >used anywhere in the code. It's not clear exactly why they were there in > >the first place, but they certainly have no function now. > > It is clear - incorrect QOM'fication in 3954d33ab7f82f5a5 when parts of > VIOsPAPRDeviceInfo became parts of VIOsPAPRBus, you can adjust the commit > log :) Thanks for the research, I'll update accordingly.
diff --git a/include/hw/ppc/spapr_vio.h b/include/hw/ppc/spapr_vio.h index ea00c3b..380fe4f 100644 --- a/include/hw/ppc/spapr_vio.h +++ b/include/hw/ppc/spapr_vio.h @@ -75,8 +75,6 @@ struct VIOsPAPRDevice { struct VIOsPAPRBus { BusState bus; uint32_t next_reg; - int (*init)(VIOsPAPRDevice *dev); - int (*devnode)(VIOsPAPRDevice *dev, void *fdt, int node_off); }; extern VIOsPAPRBus *spapr_vio_bus_init(void);
The VIOsPAPRBus structure has some callback pointers in it which aren't used anywhere in the code. It's not clear exactly why they were there in the first place, but they certainly have no function now. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> --- include/hw/ppc/spapr_vio.h | 2 -- 1 file changed, 2 deletions(-)