Am 07.06.2013 03:03, schrieb Paolo Bonzini:
> Il 05/06/2013 05:50, Andreas Färber ha scritto:
>> Also I notice some patches are accessing parent fields directly - please
>> use BUS(), PCI_DEVICE() etc. to hide this.
>
> I'm always using them. For example:
>
> +static void intel_hda_instance_finalize(Object *obj)
> +{
> + PCIDevice *pci = PCI_DEVICE(obj);
> + IntelHDAState *d = DO_UPCAST(IntelHDAState, pci, pci);
I'll comment inline then. :)
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
