diff options
author | Heyi Guo | 2020-02-04 02:43:24 +0100 |
---|---|---|
committer | Peter Maydell | 2020-02-13 15:14:53 +0100 |
commit | b43bd40ba27c3a9ebd246f7294dc8983d96fe7e4 (patch) | |
tree | 031889469dc0d02eaa2ed125042aeb7959880da8 /tests | |
parent | arm/acpi: fix duplicated _UID of PCI interrupt link devices (diff) | |
download | qemu-b43bd40ba27c3a9ebd246f7294dc8983d96fe7e4.tar.gz qemu-b43bd40ba27c3a9ebd246f7294dc8983d96fe7e4.tar.xz qemu-b43bd40ba27c3a9ebd246f7294dc8983d96fe7e4.zip |
arm/acpi: simplify the description of PCI _CRS
The original code defines a named object for the resource template but
then returns the resource template object itself; the resulted output
is like below:
Method (_CRS, 0, NotSerialized) // _CRS: Current Resource Settings
{
Name (RBUF, ResourceTemplate ()
{
WordBusNumber (ResourceProducer, MinFixed, MaxFixed, PosDecode,
0x0000, // Granularity
0x0000, // Range Minimum
0x00FF, // Range Maximum
0x0000, // Translation Offset
0x0100, // Length
,, )
......
})
Return (ResourceTemplate ()
{
WordBusNumber (ResourceProducer, MinFixed, MaxFixed, PosDecode,
0x0000, // Granularity
0x0000, // Range Minimum
0x00FF, // Range Maximum
0x0000, // Translation Offset
0x0100, // Length
,, )
......
})
}
So the named object "RBUF" is actually useless. The more natural way
is to return RBUF instead, or simply drop RBUF definition.
Choose the latter one to simplify the code.
Signed-off-by: Heyi Guo <guoheyi@huawei.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Message-id: 20200204014325.16279-7-guoheyi@huawei.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions