Re: [resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-10-23 Thread Felipe Balbi
On Wed, Oct 22, 2014 at 04:19:45PM +0200, Rafael J. Wysocki wrote:
 On Wednesday, October 22, 2014 10:33:08 AM Adrian Hunter wrote:
  On 21/10/14 16:13, Adrian Hunter wrote:
   On 21/10/14 16:08, Rafael J. Wysocki wrote:
   On Tuesday, October 21, 2014 03:27:45 PM Adrian Hunter wrote:
   On 24/09/14 16:49, Rafael J. Wysocki wrote:
   On Wednesday, September 24, 2014 11:00:37 AM Heikki Krogerus wrote:
   Most devices are configured for 32-bit DMA addresses.
   Setting the mask to 32-bit here removes the need for the
   drivers to do it separately.
  
   Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
   Cc: Rafael J. Wysocki r...@rjwysocki.net
  
   ACK
  
   Hi
  
   I need this for another driver.  Did this patch go anywhere?
  
   It's not in 3.18-rc1 as far as I can say.
  
   Note Heikki is away at the moment.
  
   That's OK.  I guess I can queue up this one for you.
  
   When do you need this to get into the Linus' tree?
   
   My patches are aiming for 3.19
  
  Actually it would be easier for me if Heikki's patch is already in
  3.18, but all I need to know is when it will hit?
 
 I've applied the Heiki's patch and I'm going to push it for 3.18-rc2.

I had this queued for v3.19 with your Ack. It didn't feel like it should
be in v3.18-rc as it's not really fixing any bug. In any case, fine by
me; I'll drop it from my queue.

-- 
balbi


signature.asc
Description: Digital signature


Re: [resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-10-23 Thread Rafael J. Wysocki
On Thursday, October 23, 2014 10:06:05 AM Felipe Balbi wrote:
 
 --HSQ3hISbU3Um6hch
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 Content-Transfer-Encoding: quoted-printable
 
 On Wed, Oct 22, 2014 at 04:19:45PM +0200, Rafael J. Wysocki wrote:
  On Wednesday, October 22, 2014 10:33:08 AM Adrian Hunter wrote:
   On 21/10/14 16:13, Adrian Hunter wrote:
On 21/10/14 16:08, Rafael J. Wysocki wrote:
On Tuesday, October 21, 2014 03:27:45 PM Adrian Hunter wrote:
On 24/09/14 16:49, Rafael J. Wysocki wrote:
On Wednesday, September 24, 2014 11:00:37 AM Heikki Krogerus wrote:
Most devices are configured for 32-bit DMA addresses.
Setting the mask to 32-bit here removes the need for the
drivers to do it separately.
   
Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
Cc: Rafael J. Wysocki r...@rjwysocki.net
   
ACK
   
Hi
   
I need this for another driver.  Did this patch go anywhere?
   
It's not in 3.18-rc1 as far as I can say.
   
Note Heikki is away at the moment.
   
That's OK.  I guess I can queue up this one for you.
   
When do you need this to get into the Linus' tree?
   =20
My patches are aiming for 3.19
  =20
   Actually it would be easier for me if Heikki's patch is already in
   3.18, but all I need to know is when it will hit?
 =20
  I've applied the Heiki's patch and I'm going to push it for 3.18-rc2.
 
 I had this queued for v3.19 with your Ack. It didn't feel like it should
 be in v3.18-rc as it's not really fixing any bug. In any case, fine by
 me; I'll drop it from my queue.

OK, thanks!

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


Re: [resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-10-22 Thread Adrian Hunter
On 21/10/14 16:13, Adrian Hunter wrote:
 On 21/10/14 16:08, Rafael J. Wysocki wrote:
 On Tuesday, October 21, 2014 03:27:45 PM Adrian Hunter wrote:
 On 24/09/14 16:49, Rafael J. Wysocki wrote:
 On Wednesday, September 24, 2014 11:00:37 AM Heikki Krogerus wrote:
 Most devices are configured for 32-bit DMA addresses.
 Setting the mask to 32-bit here removes the need for the
 drivers to do it separately.

 Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
 Cc: Rafael J. Wysocki r...@rjwysocki.net

 ACK

 Hi

 I need this for another driver.  Did this patch go anywhere?

 It's not in 3.18-rc1 as far as I can say.

 Note Heikki is away at the moment.

 That's OK.  I guess I can queue up this one for you.

 When do you need this to get into the Linus' tree?
 
 My patches are aiming for 3.19

Actually it would be easier for me if Heikki's patch is already in
3.18, but all I need to know is when it will hit?

 


 ---
  drivers/acpi/acpi_platform.c | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/drivers/acpi/acpi_platform.c b/drivers/acpi/acpi_platform.c
 index 2bf9082..8d099e6 100644
 --- a/drivers/acpi/acpi_platform.c
 +++ b/drivers/acpi/acpi_platform.c
 @@ -16,6 +16,7 @@
  #include linux/err.h
  #include linux/kernel.h
  #include linux/module.h
 +#include linux/dma-mapping.h
  #include linux/platform_device.h
  
  #include internal.h
 @@ -102,6 +103,7 @@ struct platform_device 
 *acpi_create_platform_device(struct acpi_device *adev)
   pdevinfo.res = resources;
   pdevinfo.num_res = count;
   pdevinfo.acpi_node.companion = adev;
 + pdevinfo.dma_mask = DMA_BIT_MASK(32);
   pdev = platform_device_register_full(pdevinfo);
   if (IS_ERR(pdev))
   dev_err(adev-dev, platform device creation failed: %ld\n,




 
 
 

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


Re: [resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-10-21 Thread Adrian Hunter
On 24/09/14 16:49, Rafael J. Wysocki wrote:
 On Wednesday, September 24, 2014 11:00:37 AM Heikki Krogerus wrote:
 Most devices are configured for 32-bit DMA addresses.
 Setting the mask to 32-bit here removes the need for the
 drivers to do it separately.

 Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
 Cc: Rafael J. Wysocki r...@rjwysocki.net
 
 ACK

Hi

I need this for another driver.  Did this patch go anywhere?
Note Heikki is away at the moment.

Regards
Adrian

 
 ---
  drivers/acpi/acpi_platform.c | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/drivers/acpi/acpi_platform.c b/drivers/acpi/acpi_platform.c
 index 2bf9082..8d099e6 100644
 --- a/drivers/acpi/acpi_platform.c
 +++ b/drivers/acpi/acpi_platform.c
 @@ -16,6 +16,7 @@
  #include linux/err.h
  #include linux/kernel.h
  #include linux/module.h
 +#include linux/dma-mapping.h
  #include linux/platform_device.h
  
  #include internal.h
 @@ -102,6 +103,7 @@ struct platform_device 
 *acpi_create_platform_device(struct acpi_device *adev)
  pdevinfo.res = resources;
  pdevinfo.num_res = count;
  pdevinfo.acpi_node.companion = adev;
 +pdevinfo.dma_mask = DMA_BIT_MASK(32);
  pdev = platform_device_register_full(pdevinfo);
  if (IS_ERR(pdev))
  dev_err(adev-dev, platform device creation failed: %ld\n,

 

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


Re: [resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-10-21 Thread Rafael J. Wysocki
On Tuesday, October 21, 2014 03:27:45 PM Adrian Hunter wrote:
 On 24/09/14 16:49, Rafael J. Wysocki wrote:
  On Wednesday, September 24, 2014 11:00:37 AM Heikki Krogerus wrote:
  Most devices are configured for 32-bit DMA addresses.
  Setting the mask to 32-bit here removes the need for the
  drivers to do it separately.
 
  Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
  Cc: Rafael J. Wysocki r...@rjwysocki.net
  
  ACK
 
 Hi
 
 I need this for another driver.  Did this patch go anywhere?

It's not in 3.18-rc1 as far as I can say.

 Note Heikki is away at the moment.

That's OK.  I guess I can queue up this one for you.

When do you need this to get into the Linus' tree?


  ---
   drivers/acpi/acpi_platform.c | 2 ++
   1 file changed, 2 insertions(+)
 
  diff --git a/drivers/acpi/acpi_platform.c b/drivers/acpi/acpi_platform.c
  index 2bf9082..8d099e6 100644
  --- a/drivers/acpi/acpi_platform.c
  +++ b/drivers/acpi/acpi_platform.c
  @@ -16,6 +16,7 @@
   #include linux/err.h
   #include linux/kernel.h
   #include linux/module.h
  +#include linux/dma-mapping.h
   #include linux/platform_device.h
   
   #include internal.h
  @@ -102,6 +103,7 @@ struct platform_device 
  *acpi_create_platform_device(struct acpi_device *adev)
 pdevinfo.res = resources;
 pdevinfo.num_res = count;
 pdevinfo.acpi_node.companion = adev;
  +  pdevinfo.dma_mask = DMA_BIT_MASK(32);
 pdev = platform_device_register_full(pdevinfo);
 if (IS_ERR(pdev))
 dev_err(adev-dev, platform device creation failed: %ld\n,
 
  
 

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


Re: [resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-10-21 Thread Adrian Hunter
On 21/10/14 16:08, Rafael J. Wysocki wrote:
 On Tuesday, October 21, 2014 03:27:45 PM Adrian Hunter wrote:
 On 24/09/14 16:49, Rafael J. Wysocki wrote:
 On Wednesday, September 24, 2014 11:00:37 AM Heikki Krogerus wrote:
 Most devices are configured for 32-bit DMA addresses.
 Setting the mask to 32-bit here removes the need for the
 drivers to do it separately.

 Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
 Cc: Rafael J. Wysocki r...@rjwysocki.net

 ACK

 Hi

 I need this for another driver.  Did this patch go anywhere?
 
 It's not in 3.18-rc1 as far as I can say.
 
 Note Heikki is away at the moment.
 
 That's OK.  I guess I can queue up this one for you.
 
 When do you need this to get into the Linus' tree?

My patches are aiming for 3.19

 
 
 ---
  drivers/acpi/acpi_platform.c | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/drivers/acpi/acpi_platform.c b/drivers/acpi/acpi_platform.c
 index 2bf9082..8d099e6 100644
 --- a/drivers/acpi/acpi_platform.c
 +++ b/drivers/acpi/acpi_platform.c
 @@ -16,6 +16,7 @@
  #include linux/err.h
  #include linux/kernel.h
  #include linux/module.h
 +#include linux/dma-mapping.h
  #include linux/platform_device.h
  
  #include internal.h
 @@ -102,6 +103,7 @@ struct platform_device 
 *acpi_create_platform_device(struct acpi_device *adev)
pdevinfo.res = resources;
pdevinfo.num_res = count;
pdevinfo.acpi_node.companion = adev;
 +  pdevinfo.dma_mask = DMA_BIT_MASK(32);
pdev = platform_device_register_full(pdevinfo);
if (IS_ERR(pdev))
dev_err(adev-dev, platform device creation failed: %ld\n,



 

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


[resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-09-24 Thread Heikki Krogerus
Most devices are configured for 32-bit DMA addresses.
Setting the mask to 32-bit here removes the need for the
drivers to do it separately.

Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
Cc: Rafael J. Wysocki r...@rjwysocki.net
---
 drivers/acpi/acpi_platform.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/acpi/acpi_platform.c b/drivers/acpi/acpi_platform.c
index 2bf9082..8d099e6 100644
--- a/drivers/acpi/acpi_platform.c
+++ b/drivers/acpi/acpi_platform.c
@@ -16,6 +16,7 @@
 #include linux/err.h
 #include linux/kernel.h
 #include linux/module.h
+#include linux/dma-mapping.h
 #include linux/platform_device.h
 
 #include internal.h
@@ -102,6 +103,7 @@ struct platform_device *acpi_create_platform_device(struct 
acpi_device *adev)
pdevinfo.res = resources;
pdevinfo.num_res = count;
pdevinfo.acpi_node.companion = adev;
+   pdevinfo.dma_mask = DMA_BIT_MASK(32);
pdev = platform_device_register_full(pdevinfo);
if (IS_ERR(pdev))
dev_err(adev-dev, platform device creation failed: %ld\n,
-- 
2.1.0

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


Re: [resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-09-24 Thread Rafael J. Wysocki
On Wednesday, September 24, 2014 11:00:37 AM Heikki Krogerus wrote:
 Most devices are configured for 32-bit DMA addresses.
 Setting the mask to 32-bit here removes the need for the
 drivers to do it separately.
 
 Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com
 Cc: Rafael J. Wysocki r...@rjwysocki.net

ACK

 ---
  drivers/acpi/acpi_platform.c | 2 ++
  1 file changed, 2 insertions(+)
 
 diff --git a/drivers/acpi/acpi_platform.c b/drivers/acpi/acpi_platform.c
 index 2bf9082..8d099e6 100644
 --- a/drivers/acpi/acpi_platform.c
 +++ b/drivers/acpi/acpi_platform.c
 @@ -16,6 +16,7 @@
  #include linux/err.h
  #include linux/kernel.h
  #include linux/module.h
 +#include linux/dma-mapping.h
  #include linux/platform_device.h
  
  #include internal.h
 @@ -102,6 +103,7 @@ struct platform_device 
 *acpi_create_platform_device(struct acpi_device *adev)
   pdevinfo.res = resources;
   pdevinfo.num_res = count;
   pdevinfo.acpi_node.companion = adev;
 + pdevinfo.dma_mask = DMA_BIT_MASK(32);
   pdev = platform_device_register_full(pdevinfo);
   if (IS_ERR(pdev))
   dev_err(adev-dev, platform device creation failed: %ld\n,
 

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