Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1331073

Re: [RFC PATCH v3 3/3] PCI/ACPI: hisi: Add ACPI support for HiSilicon SoCs Host Controllers

From Mark Rutland <mark.rutland@arm.com>
Newsgroups linux.kernel
Subject Re: [RFC PATCH v3 3/3] PCI/ACPI: hisi: Add ACPI support for HiSilicon SoCs Host Controllers
Date 2016-02-10 12:20 +0100
Message-ID <r0AYp-1sa-5@gated-at.bofh.it> (permalink)
References <r0kqD-7hS-25@gated-at.bofh.it> <r0kqD-7hS-39@gated-at.bofh.it> <r0ld0-7On-11@gated-at.bofh.it> <r0zJ0-vX-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Feb 10, 2016 at 09:52:36AM +0000, Gabriele Paoloni wrote:
> Hi Mark
> 
> > On Tue, Feb 09, 2016 at 05:34:20PM +0000, Gabriele Paoloni wrote:
> > > From: gabriele paoloni <gabriele.paoloni@huawei.com>
> > > +/*
> > > + * Retrieve rc_dbi base and size from _DSD
> > > + * Name (_DSD, Package () {
> > > + *	ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"),
> > > + *	Package () {
> > > + *	Package () {"rc-dbi", Package () { 0x0, 0xb0080000, 0x0, 0x10000
> > }},
> > > + *	}
> > > + *	})
> > > + */
> > 
> > As above, this does not look right. ACPI has standard mechanisms for
> > describing addresses. Making something up like this is not a good idea.
>  
> I am quite new to ACPI, may I ask you to explain a bit? 

ACPI has standard mechanisms for describing certain resources, and these
should not be described in _DSD. Memory or IO address regions are such
resources (in _CRS, IIRC), and should not be described in _DSD.

Thanks,
Mark.

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[RFC PATCH v3 3/3] PCI/ACPI: hisi: Add ACPI support for HiSilicon SoCs Host Controllers Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2016-02-09 18:40 +0100
  Re: [RFC PATCH v3 3/3] PCI/ACPI: hisi: Add ACPI support for  HiSilicon SoCs Host Controllers Mark Rutland <mark.rutland@arm.com> - 2016-02-09 19:30 +0100
    RE: [RFC PATCH v3 3/3] PCI/ACPI: hisi: Add ACPI support for  HiSilicon SoCs Host Controllers Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2016-02-10 11:00 +0100
      Re: [RFC PATCH v3 3/3] PCI/ACPI: hisi: Add ACPI support for  HiSilicon SoCs Host Controllers Mark Rutland <mark.rutland@arm.com> - 2016-02-10 12:20 +0100
        RE: [RFC PATCH v3 3/3] PCI/ACPI: hisi: Add ACPI support for  HiSilicon SoCs Host Controllers Gabriele Paoloni <gabriele.paoloni@huawei.com> - 2016-02-10 15:50 +0100

csiph-web