* Re: [Devel] [PATCH] ResourceMgr: Allow _DMA method in walk resources
@ 2017-07-26 17:04 Lorenzo Pieralisi
0 siblings, 0 replies; 3+ messages in thread
From: Lorenzo Pieralisi @ 2017-07-26 17:04 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 2670 bytes --]
Hi Robert,
do you pull patches from the list or you require an explicit github pull
request for that ?
Plase let me know.
Thanks,
Lorenzo
On Fri, Jul 21, 2017 at 04:28:15PM +0100, Lorenzo Pieralisi wrote:
> The _DMA object contains a resource template, this change adds support
> for the walk resources function so that ACPI devices containing a _DMA
> object can actually parse it to detect DMA ranges for the respective
> bus.
>
> Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi(a)arm.com>
> ---
> source/components/resources/rsxface.c | 7 ++++---
> source/include/acnames.h | 1 +
> 2 files changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/source/components/resources/rsxface.c b/source/components/resources/rsxface.c
> index 5e3f735..2c27eaf 100644
> --- a/source/components/resources/rsxface.c
> +++ b/source/components/resources/rsxface.c
> @@ -796,7 +796,7 @@ ACPI_EXPORT_SYMBOL (AcpiWalkResourceBuffer)
> * device we are querying
> * Name - Method name of the resources we want.
> * (METHOD_NAME__CRS, METHOD_NAME__PRS, or
> - * METHOD_NAME__AEI)
> + * METHOD_NAME__AEI or METHOD_NAME__DMA)
> * UserFunction - Called for each resource
> * Context - Passed to UserFunction
> *
> @@ -827,12 +827,13 @@ AcpiWalkResources (
> if (!DeviceHandle || !UserFunction || !Name ||
> (!ACPI_COMPARE_NAME (Name, METHOD_NAME__CRS) &&
> !ACPI_COMPARE_NAME (Name, METHOD_NAME__PRS) &&
> - !ACPI_COMPARE_NAME (Name, METHOD_NAME__AEI)))
> + !ACPI_COMPARE_NAME (Name, METHOD_NAME__AEI) &&
> + !ACPI_COMPARE_NAME (Name, METHOD_NAME__DMA)))
> {
> return_ACPI_STATUS (AE_BAD_PARAMETER);
> }
>
> - /* Get the _CRS/_PRS/_AEI resource list */
> + /* Get the _CRS/_PRS/_AEI/_DMA resource list */
>
> Buffer.Length = ACPI_ALLOCATE_LOCAL_BUFFER;
> Status = AcpiRsGetMethodData (DeviceHandle, Name, &Buffer);
> diff --git a/source/include/acnames.h b/source/include/acnames.h
> index 27b0b97..69f8884 100644
> --- a/source/include/acnames.h
> +++ b/source/include/acnames.h
> @@ -162,6 +162,7 @@
> #define METHOD_NAME__CLS "_CLS"
> #define METHOD_NAME__CRS "_CRS"
> #define METHOD_NAME__DDN "_DDN"
> +#define METHOD_NAME__DMA "_DMA"
> #define METHOD_NAME__HID "_HID"
> #define METHOD_NAME__INI "_INI"
> #define METHOD_NAME__PLD "_PLD"
> --
> 2.10.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Devel] [PATCH] ResourceMgr: Allow _DMA method in walk resources
@ 2017-07-27 15:02 Moore, Robert
0 siblings, 0 replies; 3+ messages in thread
From: Moore, Robert @ 2017-07-27 15:02 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 3134 bytes --]
We will just take the patch as posted.
> -----Original Message-----
> From: Lorenzo Pieralisi [mailto:lorenzo.pieralisi(a)arm.com]
> Sent: Wednesday, July 26, 2017 10:04 AM
> To: devel(a)acpica.org; Moore, Robert <robert.moore(a)intel.com>
> Subject: Re: [PATCH] ResourceMgr: Allow _DMA method in walk resources
>
> Hi Robert,
>
> do you pull patches from the list or you require an explicit github pull
> request for that ?
>
> Plase let me know.
>
> Thanks,
> Lorenzo
>
> On Fri, Jul 21, 2017 at 04:28:15PM +0100, Lorenzo Pieralisi wrote:
> > The _DMA object contains a resource template, this change adds support
> > for the walk resources function so that ACPI devices containing a _DMA
> > object can actually parse it to detect DMA ranges for the respective
> > bus.
> >
> > Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi(a)arm.com>
> > ---
> > source/components/resources/rsxface.c | 7 ++++---
> > source/include/acnames.h | 1 +
> > 2 files changed, 5 insertions(+), 3 deletions(-)
> >
> > diff --git a/source/components/resources/rsxface.c
> > b/source/components/resources/rsxface.c
> > index 5e3f735..2c27eaf 100644
> > --- a/source/components/resources/rsxface.c
> > +++ b/source/components/resources/rsxface.c
> > @@ -796,7 +796,7 @@ ACPI_EXPORT_SYMBOL (AcpiWalkResourceBuffer)
> > * device we are querying
> > * Name - Method name of the resources we
> want.
> > * (METHOD_NAME__CRS,
> METHOD_NAME__PRS, or
> > - * METHOD_NAME__AEI)
> > + * METHOD_NAME__AEI or
> METHOD_NAME__DMA)
> > * UserFunction - Called for each resource
> > * Context - Passed to UserFunction
> > *
> > @@ -827,12 +827,13 @@ AcpiWalkResources (
> > if (!DeviceHandle || !UserFunction || !Name ||
> > (!ACPI_COMPARE_NAME (Name, METHOD_NAME__CRS) &&
> > !ACPI_COMPARE_NAME (Name, METHOD_NAME__PRS) &&
> > - !ACPI_COMPARE_NAME (Name, METHOD_NAME__AEI)))
> > + !ACPI_COMPARE_NAME (Name, METHOD_NAME__AEI) &&
> > + !ACPI_COMPARE_NAME (Name, METHOD_NAME__DMA)))
> > {
> > return_ACPI_STATUS (AE_BAD_PARAMETER);
> > }
> >
> > - /* Get the _CRS/_PRS/_AEI resource list */
> > + /* Get the _CRS/_PRS/_AEI/_DMA resource list */
> >
> > Buffer.Length = ACPI_ALLOCATE_LOCAL_BUFFER;
> > Status = AcpiRsGetMethodData (DeviceHandle, Name, &Buffer); diff
> > --git a/source/include/acnames.h b/source/include/acnames.h index
> > 27b0b97..69f8884 100644
> > --- a/source/include/acnames.h
> > +++ b/source/include/acnames.h
> > @@ -162,6 +162,7 @@
> > #define METHOD_NAME__CLS "_CLS"
> > #define METHOD_NAME__CRS "_CRS"
> > #define METHOD_NAME__DDN "_DDN"
> > +#define METHOD_NAME__DMA "_DMA"
> > #define METHOD_NAME__HID "_HID"
> > #define METHOD_NAME__INI "_INI"
> > #define METHOD_NAME__PLD "_PLD"
> > --
> > 2.10.0
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Devel] [PATCH] ResourceMgr: Allow _DMA method in walk resources
@ 2017-07-21 15:28 Lorenzo Pieralisi
0 siblings, 0 replies; 3+ messages in thread
From: Lorenzo Pieralisi @ 2017-07-21 15:28 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 2340 bytes --]
The _DMA object contains a resource template, this change adds support
for the walk resources function so that ACPI devices containing a _DMA
object can actually parse it to detect DMA ranges for the respective
bus.
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi(a)arm.com>
---
source/components/resources/rsxface.c | 7 ++++---
source/include/acnames.h | 1 +
2 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/source/components/resources/rsxface.c b/source/components/resources/rsxface.c
index 5e3f735..2c27eaf 100644
--- a/source/components/resources/rsxface.c
+++ b/source/components/resources/rsxface.c
@@ -796,7 +796,7 @@ ACPI_EXPORT_SYMBOL (AcpiWalkResourceBuffer)
* device we are querying
* Name - Method name of the resources we want.
* (METHOD_NAME__CRS, METHOD_NAME__PRS, or
- * METHOD_NAME__AEI)
+ * METHOD_NAME__AEI or METHOD_NAME__DMA)
* UserFunction - Called for each resource
* Context - Passed to UserFunction
*
@@ -827,12 +827,13 @@ AcpiWalkResources (
if (!DeviceHandle || !UserFunction || !Name ||
(!ACPI_COMPARE_NAME (Name, METHOD_NAME__CRS) &&
!ACPI_COMPARE_NAME (Name, METHOD_NAME__PRS) &&
- !ACPI_COMPARE_NAME (Name, METHOD_NAME__AEI)))
+ !ACPI_COMPARE_NAME (Name, METHOD_NAME__AEI) &&
+ !ACPI_COMPARE_NAME (Name, METHOD_NAME__DMA)))
{
return_ACPI_STATUS (AE_BAD_PARAMETER);
}
- /* Get the _CRS/_PRS/_AEI resource list */
+ /* Get the _CRS/_PRS/_AEI/_DMA resource list */
Buffer.Length = ACPI_ALLOCATE_LOCAL_BUFFER;
Status = AcpiRsGetMethodData (DeviceHandle, Name, &Buffer);
diff --git a/source/include/acnames.h b/source/include/acnames.h
index 27b0b97..69f8884 100644
--- a/source/include/acnames.h
+++ b/source/include/acnames.h
@@ -162,6 +162,7 @@
#define METHOD_NAME__CLS "_CLS"
#define METHOD_NAME__CRS "_CRS"
#define METHOD_NAME__DDN "_DDN"
+#define METHOD_NAME__DMA "_DMA"
#define METHOD_NAME__HID "_HID"
#define METHOD_NAME__INI "_INI"
#define METHOD_NAME__PLD "_PLD"
--
2.10.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-07-27 15:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-26 17:04 [Devel] [PATCH] ResourceMgr: Allow _DMA method in walk resources Lorenzo Pieralisi
-- strict thread matches above, loose matches on Subject: below --
2017-07-27 15:02 Moore, Robert
2017-07-21 15:28 Lorenzo Pieralisi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.