Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suravee Suthikulpanit]

Hi

On 10/12/15 15:27, Rafael J. Wysocki wrote:

Just wondering if you are planning to queue this series up for 4.4 as well?

You don't seem to have addressed the Bjorn's comments on patch [2/4].

They need to be addressed before I can take this series.

Thanks,
Rafael



Ah, I missed that one. Sorry. I'll get back on that.

Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Rafael J. Wysocki]

On Monday, October 12, 2015 02:51:33 PM Suravee Suthikulpanit wrote:
> Hi Rafael,
> 
> On 9/9/15 21:48, Suthikulpanit, Suravee wrote:
> > Hi Rafael,
> >
> > On 9/10/2015 3:38 AM, Rafael J. Wysocki wrote:
> >> On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee
> >> wrote:
> >>> >Hi All,
> >>> >
> >>> >Are there any other concerns about this patch series?
> >> I have none, but then it sort of missed the merge window.
> >>
> >> I can easily queue it up for the next one unless it is super-urgent,
> >> but in that case I need to know why that's the case.
> >>
> >
> > This is not urgent. This mainly needed to enable the ACPI PCI supports
> > for ARM64 which is still work in progress.
> >
> > Thanks,
> > Suravee
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> > the body of a message to majord...@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> Just wondering if you are planning to queue this series up for 4.4 as well?

You don't seem to have addressed the Bjorn's comments on patch [2/4].

They need to be addressed before I can take this series.

Thanks,
Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suravee Suthikulpanit]

Hi Rafael,

On 9/9/15 21:48, Suthikulpanit, Suravee wrote:

Hi Rafael,

On 9/10/2015 3:38 AM, Rafael J. Wysocki wrote:

On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee
wrote:

>Hi All,
>
>Are there any other concerns about this patch series?

I have none, but then it sort of missed the merge window.

I can easily queue it up for the next one unless it is super-urgent,
but in that case I need to know why that's the case.



This is not urgent. This mainly needed to enable the ACPI PCI supports
for ARM64 which is still work in progress.

Thanks,
Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Just wondering if you are planning to queue this series up for 4.4 as well?

Thank you,
Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suravee Suthikulpanit]

Hi

On 10/12/15 15:27, Rafael J. Wysocki wrote:

Just wondering if you are planning to queue this series up for 4.4 as well?

You don't seem to have addressed the Bjorn's comments on patch [2/4].

They need to be addressed before I can take this series.

Thanks,
Rafael



Ah, I missed that one. Sorry. I'll get back on that.

Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suravee Suthikulpanit]

Hi Rafael,

On 9/9/15 21:48, Suthikulpanit, Suravee wrote:

Hi Rafael,

On 9/10/2015 3:38 AM, Rafael J. Wysocki wrote:

On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee
wrote:

>Hi All,
>
>Are there any other concerns about this patch series?

I have none, but then it sort of missed the merge window.

I can easily queue it up for the next one unless it is super-urgent,
but in that case I need to know why that's the case.



This is not urgent. This mainly needed to enable the ACPI PCI supports
for ARM64 which is still work in progress.

Thanks,
Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Just wondering if you are planning to queue this series up for 4.4 as well?

Thank you,
Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Rafael J. Wysocki]

On Monday, October 12, 2015 02:51:33 PM Suravee Suthikulpanit wrote:
> Hi Rafael,
> 
> On 9/9/15 21:48, Suthikulpanit, Suravee wrote:
> > Hi Rafael,
> >
> > On 9/10/2015 3:38 AM, Rafael J. Wysocki wrote:
> >> On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee
> >> wrote:
> >>> >Hi All,
> >>> >
> >>> >Are there any other concerns about this patch series?
> >> I have none, but then it sort of missed the merge window.
> >>
> >> I can easily queue it up for the next one unless it is super-urgent,
> >> but in that case I need to know why that's the case.
> >>
> >
> > This is not urgent. This mainly needed to enable the ACPI PCI supports
> > for ARM64 which is still work in progress.
> >
> > Thanks,
> > Suravee
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> > the body of a message to majord...@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> Just wondering if you are planning to queue this series up for 4.4 as well?

You don't seem to have addressed the Bjorn's comments on patch [2/4].

They need to be addressed before I can take this series.

Thanks,
Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Hanjun Guo]

On 08/26/2015 09:54 PM, Suravee Suthikulpanit wrote:

This patch adds support to setup DMA coherency for PCI device using
the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute
is required for ARM64. Therefore, this patch is a pre-req for ACPI PCI
support for ARM64 which is currently in development.

Also, this should not affect other architectures that does not define
CONFIG_ACPI_CCA_REQUIRED, since the default value is coherent.

I have included the following patch from Jeremy before cleaning up
the acpi_check_dma() function (in patch 2).

 http://www.spinics.net/lists/linux-usb/msg128582.html)

Changes from V2: (https://lkml.org/lkml/2015/8/25/549)
 * Return -ENOSUPP instead of -1 (per Rafael's suggestion)
 * Add WARN() when fail to setup DMA for PCI device when booting
   ACPI (per Arnd's suggestion)
 * Added Acked-by from Rob.
 * Minor clean up

Changes from V1: (https://lkml.org/lkml/2015/8/13/182)
 * Include patch 1 from Jeremy to enable support for _CCA=0
 * Clean up acpi_check_dma() per Bjorn suggestions
 * Split the original V1 patch into two patches (patch 3 and 4)


Jeremy Linton (1):
   Honor ACPI _CCA attribute setting

Suravee Suthikulpanit (3):
   ACPI/scan: Clean up acpi_check_dma
   PCI: OF: Move of_pci_dma_configure() to pci_dma_configure()
   PCI: ACPI: Add support for PCI device DMA coherency


For this series,

Reviewed-by: Hanjun Guo 

Thanks
Hanjun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Hanjun Guo]

On 08/26/2015 09:54 PM, Suravee Suthikulpanit wrote:

This patch adds support to setup DMA coherency for PCI device using
the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute
is required for ARM64. Therefore, this patch is a pre-req for ACPI PCI
support for ARM64 which is currently in development.

Also, this should not affect other architectures that does not define
CONFIG_ACPI_CCA_REQUIRED, since the default value is coherent.

I have included the following patch from Jeremy before cleaning up
the acpi_check_dma() function (in patch 2).

 http://www.spinics.net/lists/linux-usb/msg128582.html)

Changes from V2: (https://lkml.org/lkml/2015/8/25/549)
 * Return -ENOSUPP instead of -1 (per Rafael's suggestion)
 * Add WARN() when fail to setup DMA for PCI device when booting
   ACPI (per Arnd's suggestion)
 * Added Acked-by from Rob.
 * Minor clean up

Changes from V1: (https://lkml.org/lkml/2015/8/13/182)
 * Include patch 1 from Jeremy to enable support for _CCA=0
 * Clean up acpi_check_dma() per Bjorn suggestions
 * Split the original V1 patch into two patches (patch 3 and 4)


Jeremy Linton (1):
   Honor ACPI _CCA attribute setting

Suravee Suthikulpanit (3):
   ACPI/scan: Clean up acpi_check_dma
   PCI: OF: Move of_pci_dma_configure() to pci_dma_configure()
   PCI: ACPI: Add support for PCI device DMA coherency


For this series,

Reviewed-by: Hanjun Guo 

Thanks
Hanjun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suthikulpanit, Suravee]

Hi Rafael,

On 9/10/2015 3:38 AM, Rafael J. Wysocki wrote:

On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee wrote:

>Hi All,
>
>Are there any other concerns about this patch series?

I have none, but then it sort of missed the merge window.

I can easily queue it up for the next one unless it is super-urgent,
but in that case I need to know why that's the case.



This is not urgent. This mainly needed to enable the ACPI PCI supports 
for ARM64 which is still work in progress.


Thanks,
Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Rafael J. Wysocki]

On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee wrote:
> Hi All,
> 
> Are there any other concerns about this patch series?

I have none, but then it sort of missed the merge window.

I can easily queue it up for the next one unless it is super-urgent,
but in that case I need to know why that's the case.


> On 8/26/2015 8:54 PM, Suravee Suthikulpanit wrote:
> > This patch adds support to setup DMA coherency for PCI device using
> > the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute
> > is required for ARM64. Therefore, this patch is a pre-req for ACPI PCI
> > support for ARM64 which is currently in development.
> >
> > Also, this should not affect other architectures that does not define
> > CONFIG_ACPI_CCA_REQUIRED, since the default value is coherent.
> >
> > I have included the following patch from Jeremy before cleaning up
> > the acpi_check_dma() function (in patch 2).
> >
> >  http://www.spinics.net/lists/linux-usb/msg128582.html)
> >
> > Changes from V2: (https://lkml.org/lkml/2015/8/25/549)
> >  * Return -ENOSUPP instead of -1 (per Rafael's suggestion)
> >  * Add WARN() when fail to setup DMA for PCI device when booting
> >ACPI (per Arnd's suggestion)
> >  * Added Acked-by from Rob.
> >  * Minor clean up
> >
> > Changes from V1: (https://lkml.org/lkml/2015/8/13/182)
> >  * Include patch 1 from Jeremy to enable support for _CCA=0
> >  * Clean up acpi_check_dma() per Bjorn suggestions
> >  * Split the original V1 patch into two patches (patch 3 and 4)
> >
> >
> > Jeremy Linton (1):
> >Honor ACPI _CCA attribute setting
> >
> > Suravee Suthikulpanit (3):
> >ACPI/scan: Clean up acpi_check_dma
> >PCI: OF: Move of_pci_dma_configure() to pci_dma_configure()
> >PCI: ACPI: Add support for PCI device DMA coherency
> >
> >   drivers/acpi/acpi_platform.c |  7 ++-
> >   drivers/acpi/glue.c  |  6 +++---
> >   drivers/acpi/scan.c  | 39 +++
> >   drivers/base/property.c  | 10 +++---
> >   drivers/of/of_pci.c  | 20 
> >   drivers/pci/probe.c  | 35 +--
> >   include/acpi/acpi_bus.h  | 35 ++-
> >   include/linux/acpi.h |  4 ++--
> >   include/linux/of_pci.h   |  3 ---
> >   9 files changed, 92 insertions(+), 67 deletions(-)
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suthikulpanit, Suravee]

Hi All,

Are there any other concerns about this patch series?

Thanks,
Suravee

On 8/26/2015 8:54 PM, Suravee Suthikulpanit wrote:

This patch adds support to setup DMA coherency for PCI device using
the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute
is required for ARM64. Therefore, this patch is a pre-req for ACPI PCI
support for ARM64 which is currently in development.

Also, this should not affect other architectures that does not define
CONFIG_ACPI_CCA_REQUIRED, since the default value is coherent.

I have included the following patch from Jeremy before cleaning up
the acpi_check_dma() function (in patch 2).

 http://www.spinics.net/lists/linux-usb/msg128582.html)

Changes from V2: (https://lkml.org/lkml/2015/8/25/549)
 * Return -ENOSUPP instead of -1 (per Rafael's suggestion)
 * Add WARN() when fail to setup DMA for PCI device when booting
   ACPI (per Arnd's suggestion)
 * Added Acked-by from Rob.
 * Minor clean up

Changes from V1: (https://lkml.org/lkml/2015/8/13/182)
 * Include patch 1 from Jeremy to enable support for _CCA=0
 * Clean up acpi_check_dma() per Bjorn suggestions
 * Split the original V1 patch into two patches (patch 3 and 4)


Jeremy Linton (1):
   Honor ACPI _CCA attribute setting

Suravee Suthikulpanit (3):
   ACPI/scan: Clean up acpi_check_dma
   PCI: OF: Move of_pci_dma_configure() to pci_dma_configure()
   PCI: ACPI: Add support for PCI device DMA coherency

  drivers/acpi/acpi_platform.c |  7 ++-
  drivers/acpi/glue.c  |  6 +++---
  drivers/acpi/scan.c  | 39 +++
  drivers/base/property.c  | 10 +++---
  drivers/of/of_pci.c  | 20 
  drivers/pci/probe.c  | 35 +--
  include/acpi/acpi_bus.h  | 35 ++-
  include/linux/acpi.h |  4 ++--
  include/linux/of_pci.h   |  3 ---
  9 files changed, 92 insertions(+), 67 deletions(-)


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suthikulpanit, Suravee]

Hi All,

Are there any other concerns about this patch series?

Thanks,
Suravee

On 8/26/2015 8:54 PM, Suravee Suthikulpanit wrote:

This patch adds support to setup DMA coherency for PCI device using
the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute
is required for ARM64. Therefore, this patch is a pre-req for ACPI PCI
support for ARM64 which is currently in development.

Also, this should not affect other architectures that does not define
CONFIG_ACPI_CCA_REQUIRED, since the default value is coherent.

I have included the following patch from Jeremy before cleaning up
the acpi_check_dma() function (in patch 2).

 http://www.spinics.net/lists/linux-usb/msg128582.html)

Changes from V2: (https://lkml.org/lkml/2015/8/25/549)
 * Return -ENOSUPP instead of -1 (per Rafael's suggestion)
 * Add WARN() when fail to setup DMA for PCI device when booting
   ACPI (per Arnd's suggestion)
 * Added Acked-by from Rob.
 * Minor clean up

Changes from V1: (https://lkml.org/lkml/2015/8/13/182)
 * Include patch 1 from Jeremy to enable support for _CCA=0
 * Clean up acpi_check_dma() per Bjorn suggestions
 * Split the original V1 patch into two patches (patch 3 and 4)


Jeremy Linton (1):
   Honor ACPI _CCA attribute setting

Suravee Suthikulpanit (3):
   ACPI/scan: Clean up acpi_check_dma
   PCI: OF: Move of_pci_dma_configure() to pci_dma_configure()
   PCI: ACPI: Add support for PCI device DMA coherency

  drivers/acpi/acpi_platform.c |  7 ++-
  drivers/acpi/glue.c  |  6 +++---
  drivers/acpi/scan.c  | 39 +++
  drivers/base/property.c  | 10 +++---
  drivers/of/of_pci.c  | 20 
  drivers/pci/probe.c  | 35 +--
  include/acpi/acpi_bus.h  | 35 ++-
  include/linux/acpi.h |  4 ++--
  include/linux/of_pci.h   |  3 ---
  9 files changed, 92 insertions(+), 67 deletions(-)


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Suthikulpanit, Suravee]

Hi Rafael,

On 9/10/2015 3:38 AM, Rafael J. Wysocki wrote:

On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee wrote:

>Hi All,
>
>Are there any other concerns about this patch series?

I have none, but then it sort of missed the merge window.

I can easily queue it up for the next one unless it is super-urgent,
but in that case I need to know why that's the case.



This is not urgent. This mainly needed to enable the ACPI PCI supports 
for ARM64 which is still work in progress.


Thanks,
Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH V3 0/4] PCI: ACPI: Setting up DMA coherency for PCI device from _CCA attribute

[Rafael J. Wysocki]

On Wednesday, September 09, 2015 07:16:49 PM Suthikulpanit, Suravee wrote:
> Hi All,
> 
> Are there any other concerns about this patch series?

I have none, but then it sort of missed the merge window.

I can easily queue it up for the next one unless it is super-urgent,
but in that case I need to know why that's the case.


> On 8/26/2015 8:54 PM, Suravee Suthikulpanit wrote:
> > This patch adds support to setup DMA coherency for PCI device using
> > the ACPI _CCA attribute. According to the ACPI spec, the _CCA attribute
> > is required for ARM64. Therefore, this patch is a pre-req for ACPI PCI
> > support for ARM64 which is currently in development.
> >
> > Also, this should not affect other architectures that does not define
> > CONFIG_ACPI_CCA_REQUIRED, since the default value is coherent.
> >
> > I have included the following patch from Jeremy before cleaning up
> > the acpi_check_dma() function (in patch 2).
> >
> >  http://www.spinics.net/lists/linux-usb/msg128582.html)
> >
> > Changes from V2: (https://lkml.org/lkml/2015/8/25/549)
> >  * Return -ENOSUPP instead of -1 (per Rafael's suggestion)
> >  * Add WARN() when fail to setup DMA for PCI device when booting
> >ACPI (per Arnd's suggestion)
> >  * Added Acked-by from Rob.
> >  * Minor clean up
> >
> > Changes from V1: (https://lkml.org/lkml/2015/8/13/182)
> >  * Include patch 1 from Jeremy to enable support for _CCA=0
> >  * Clean up acpi_check_dma() per Bjorn suggestions
> >  * Split the original V1 patch into two patches (patch 3 and 4)
> >
> >
> > Jeremy Linton (1):
> >Honor ACPI _CCA attribute setting
> >
> > Suravee Suthikulpanit (3):
> >ACPI/scan: Clean up acpi_check_dma
> >PCI: OF: Move of_pci_dma_configure() to pci_dma_configure()
> >PCI: ACPI: Add support for PCI device DMA coherency
> >
> >   drivers/acpi/acpi_platform.c |  7 ++-
> >   drivers/acpi/glue.c  |  6 +++---
> >   drivers/acpi/scan.c  | 39 +++
> >   drivers/base/property.c  | 10 +++---
> >   drivers/of/of_pci.c  | 20 
> >   drivers/pci/probe.c  | 35 +--
> >   include/acpi/acpi_bus.h  | 35 ++-
> >   include/linux/acpi.h |  4 ++--
> >   include/linux/of_pci.h   |  3 ---
> >   9 files changed, 92 insertions(+), 67 deletions(-)
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/