qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v5 19/20] hw/arm/virt-acpi-build: Add PCIe contr


From: Michael S. Tsirkin
Subject: Re: [Qemu-devel] [PATCH v5 19/20] hw/arm/virt-acpi-build: Add PCIe controller in ACPI DSDT table
Date: Tue, 28 Apr 2015 17:54:45 +0200

On Tue, Apr 28, 2015 at 05:13:10PM +0200, Igor Mammedov wrote:
> > Here I need to set the value of buffer to 1 or 0, we could
> > createbitfield, but if we want to set the value to non one or zero and
> > the BufferSize is large, how could we do? CreateByteField? It's a little
> > complex for user.
> that's what one would have to do writing it in ASL if bits
> are flipped on/off dynamically.
> 
> In ASL you also can declare buffer with static initializer
> 
>    Buffer (0x01) { 0x03 }
> 
> and compiler is smart enough to set appropriate bits but it doesn't
> allow you to do so with large values. For example:
> 
>    Buffer (0x01) { 0xAABBCCDD }
> 
> gives error:
> Error 6139 - Constant out of range ^  (0xAABBCCDD, allowable: 0x0-0xFF)
> 
> If one wants to manipulate specific fields in Buffer, ASL has
> a bunch of CreateFOOField operators, so lets follow spec and use
> API consistently to avoid confusion.
> 
> BTW:
> packaging value as int (even without prefix) is wrong since
> its LE encoding will shuffle bytes and you won't get bits in
> positions that you expect if value is more than 1 byte.

I don't care about ASL, we are writing in C
But AML is same:
DefBuffer := BufferOp PkgLength BufferSize ByteList
BufferOp := 0x11
BufferSize := TermArg => Integer

So really just a bytelist.
We don't have any users for aml_buffer, maybe just add
const uint8_t *bytes, unsigned len as parameters.

Would that be enough?


> > 
> > > ifctx1 = aml_if(aml_equal(aml_arg(2), aml_int(0)));
> > > aml_append(ifctx1, aml_store(aml_name("FNEN", aml_int(1)));
> > > ...
> > > /* create bit field for every supported function if supported */
> > > ...
> > > aml_append(method, aml_return(aml_name("RET")));
> > > 
> > > 
> > >>
> > >>>>
> > >>>>> +    aml_append(ifctx1, aml_return(buf));
> > >>>>> +    aml_append(ifctx, ifctx1);
> > >>>>> +    aml_append(method, ifctx);
> > >>>>> +
> > >>>>> +    buf = aml_buffer();
> > >>>>> +    build_append_int_noprefix(buf->buf, 0x00, 1);
> > >>>>> +    aml_append(method, aml_return(buf));
> > >>>>> +    aml_append(dev, method);
> > >>>>> +
> > >>>>> +    Aml *dev_rp0 = aml_device("%s", "RP0");
> > >>>>> +    aml_append(dev_rp0, aml_name_decl("_ADR", aml_int(0)));
> > >>>>> +    aml_append(dev, dev_rp0);
> > >>>>> +    aml_append(scope, dev);
> > >>>>> +}
> > >>>>> +
> > >>>>>  /* RSDP */
> > >>>>>  static GArray *
> > >>>>>  build_rsdp(GArray *rsdp_table, GArray *linker, unsigned rsdt)
> > >>>>> @@ -318,6 +468,8 @@ build_dsdt(GArray *table_data, GArray *linker, 
> > >>>>> VirtGuestInfo *guest_info)
> > >>>>>      acpi_dsdt_add_flash(scope, info->flash_memmap);
> > >>>>>      acpi_dsdt_add_virtio(scope, info->virtio_mmio_memmap,
> > >>>>>               info->virtio_mmio_irq, info->virtio_mmio_num);
> > >>>>> +    acpi_dsdt_add_pci(scope, guest_info->pcie_info);
> > >>>>> +
> > >>>>>      aml_append(dsdt, scope);
> > >>>>>  
> > >>>>>      /* copy AML table into ACPI tables blob and patch header there */
> > >>>
> > >>> .
> > >>>
> > >>
> > >>
> > > 
> > > 
> > > .
> > > 
> > 
> > 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]