diff mbox

ata: ahci: tegra: fix build error

Message ID 20140821190824.GM11146@saruman.home
State New
Headers show

Commit Message

Felipe Balbi Aug. 21, 2014, 7:08 p.m. UTC
Currently, ahci_tegra doesn't build because it includes a file that
doesn't exist.

The problem is that first commit 306a7f9 (ARM: tegra: Move includes to
include/soc/tegra) renamed include/linux/tegra-powergate.h to
include/soc/tegra/powergate.h.

Later commit 7232398 (ARM: tegra: Convert PMC to a driver) renamed
include/soc/tegra/powergate.h to include/soc/tegra/pmc.h and neither
commits fixed drivers/ata/ahci_tegra.c.

Fix that so the driver has a chance of building. With current kernels.

Cc: <stable@vger.kernel.org> # v3.16
Signed-of-by: Felipe Balbi <balbi@ti.com>
---
 drivers/ata/ahci_tegra.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.0.1.563.g66f467c

Comments

Stephen Warren Aug. 21, 2014, 7:17 p.m. UTC | #1
On 08/21/2014 01:08 PM, Felipe Balbi wrote:
> Currently, ahci_tegra doesn't build because it includes a file that
> doesn't exist.
>
> The problem is that first commit 306a7f9 (ARM: tegra: Move includes to
> include/soc/tegra) renamed include/linux/tegra-powergate.h to
> include/soc/tegra/powergate.h.
>
> Later commit 7232398 (ARM: tegra: Convert PMC to a driver) renamed
> include/soc/tegra/powergate.h to include/soc/tegra/pmc.h and neither
> commits fixed drivers/ata/ahci_tegra.c.
>
> Fix that so the driver has a chance of building. With current kernels.
>
> Cc: <stable@vger.kernel.org> # v3.16

This driver first appeared in v3.17-rc1.

This patch has already been sent, and apparently applied:

https://lkml.org/lkml/2014/8/16/36

I wonder if libata/for-3.17-fixes isn't in linux-next?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
Felipe Balbi Aug. 21, 2014, 7:18 p.m. UTC | #2
On Thu, Aug 21, 2014 at 02:08:24PM -0500, Felipe Balbi wrote:
> Currently, ahci_tegra doesn't build because it includes a file that
> doesn't exist.
> 
> The problem is that first commit 306a7f9 (ARM: tegra: Move includes to
> include/soc/tegra) renamed include/linux/tegra-powergate.h to
> include/soc/tegra/powergate.h.
> 
> Later commit 7232398 (ARM: tegra: Convert PMC to a driver) renamed
> include/soc/tegra/powergate.h to include/soc/tegra/pmc.h and neither
> commits fixed drivers/ata/ahci_tegra.c.
> 
> Fix that so the driver has a chance of building. With current kernels.
                                                 ^
						 this period shouldn't
						 exist :-p
> 
> Cc: <stable@vger.kernel.org> # v3.16
> Signed-of-by: Felipe Balbi <balbi@ti.com>
> ---
>  drivers/ata/ahci_tegra.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/ata/ahci_tegra.c b/drivers/ata/ahci_tegra.c
> index fc3df47..7194835 100644
> --- a/drivers/ata/ahci_tegra.c
> +++ b/drivers/ata/ahci_tegra.c
> @@ -24,7 +24,7 @@
>  #include <linux/module.h>
>  #include <linux/of_device.h>
>  #include <linux/platform_device.h>
> -#include <linux/tegra-powergate.h>
> +#include <soc/tegra/pmc.h>
>  #include <linux/regulator/consumer.h>
>  #include "ahci.h"
>  
> --
> 2.0.1.563.g66f467c
> 
> -- 
> balbi
Felipe Balbi Aug. 21, 2014, 7:21 p.m. UTC | #3
On Thu, Aug 21, 2014 at 01:17:14PM -0600, Stephen Warren wrote:
> On 08/21/2014 01:08 PM, Felipe Balbi wrote:
> >Currently, ahci_tegra doesn't build because it includes a file that
> >doesn't exist.
> >
> >The problem is that first commit 306a7f9 (ARM: tegra: Move includes to
> >include/soc/tegra) renamed include/linux/tegra-powergate.h to
> >include/soc/tegra/powergate.h.
> >
> >Later commit 7232398 (ARM: tegra: Convert PMC to a driver) renamed
> >include/soc/tegra/powergate.h to include/soc/tegra/pmc.h and neither
> >commits fixed drivers/ata/ahci_tegra.c.
> >
> >Fix that so the driver has a chance of building. With current kernels.
> >
> >Cc: <stable@vger.kernel.org> # v3.16
> 
> This driver first appeared in v3.17-rc1.

hmm... weird, git describe told v3.16-rc1 :-s

> This patch has already been sent, and apparently applied:
> 
> https://lkml.org/lkml/2014/8/16/36

aha, thanks.

> I wonder if libata/for-3.17-fixes isn't in linux-next?

it is, it's just that I was build testing my branch which is based off
of v3.17-rc1 as of now.
Tejun Heo Aug. 21, 2014, 7:31 p.m. UTC | #4
Hello,

On Thu, Aug 21, 2014 at 02:21:34PM -0500, Felipe Balbi wrote:
> > I wonder if libata/for-3.17-fixes isn't in linux-next?
> 
> it is, it's just that I was build testing my branch which is based off
> of v3.17-rc1 as of now.

Yeah, I'm traveling and haven't had the chance to push it.  I'll try
to push it right now.

Thanks.
diff mbox

Patch

diff --git a/drivers/ata/ahci_tegra.c b/drivers/ata/ahci_tegra.c
index fc3df47..7194835 100644
--- a/drivers/ata/ahci_tegra.c
+++ b/drivers/ata/ahci_tegra.c
@@ -24,7 +24,7 @@ 
 #include <linux/module.h>
 #include <linux/of_device.h>
 #include <linux/platform_device.h>
-#include <linux/tegra-powergate.h>
+#include <soc/tegra/pmc.h>
 #include <linux/regulator/consumer.h>
 #include "ahci.h"