diff mbox

pinctrl: mediatek: mt8135/mt8173: Fix build error due to missing include file

Message ID 1426143074.26140.0.camel@phoenix (mailing list archive)
State New, archived
Headers show

Commit Message

Axel Lin March 12, 2015, 6:51 a.m. UTC
Fix below build error:
  CC      drivers/pinctrl/mediatek/pinctrl-mt8135.o
In file included from drivers/pinctrl/mediatek/pinctrl-mt8135.c:24:0:
drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h:19:32: fatal error: pinctrl-mtk-common.h: No such file or directory
compilation terminated.
make[3]: *** [drivers/pinctrl/mediatek/pinctrl-mt8135.o] Error 1
make[2]: *** [drivers/pinctrl/mediatek] Error 2
make[1]: *** [drivers/pinctrl] Error 2
make: *** [drivers] Error 2

Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
 drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h | 2 +-
 drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Matthias Brugger March 12, 2015, 10:32 a.m. UTC | #1
2015-03-12 7:51 GMT+01:00 Axel Lin <axel.lin@ingics.com>:
> Fix below build error:
>   CC      drivers/pinctrl/mediatek/pinctrl-mt8135.o
> In file included from drivers/pinctrl/mediatek/pinctrl-mt8135.c:24:0:
> drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h:19:32: fatal error: pinctrl-mtk-common.h: No such file or directory
> compilation terminated.
> make[3]: *** [drivers/pinctrl/mediatek/pinctrl-mt8135.o] Error 1
> make[2]: *** [drivers/pinctrl/mediatek] Error 2
> make[1]: *** [drivers/pinctrl] Error 2
> make: *** [drivers] Error 2
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

The fix from Guenter already addresses mt8135 case [0].
Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>

for the mt8173 part.

Cheers,
Matthias

[0] https://lkml.org/lkml/2015/3/11/510

> ---
>  drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h | 2 +-
>  drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
> index 5019cef..e17aedb 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
> @@ -16,7 +16,7 @@
>  #define __PINCTRL_MTK_MT8135_H
>
>  #include <linux/pinctrl/pinctrl.h>
> -#include <pinctrl-mtk-common.h>
> +#include "pinctrl-mtk-common.h"
>
>  static const struct mtk_desc_pin mtk_pins_mt8135[] = {
>         MTK_PIN(
> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
> index c1a3ca0..13e5b68 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
> @@ -16,7 +16,7 @@
>  #define __PINCTRL_MTK_MT8173_H
>
>  #include <linux/pinctrl/pinctrl.h>
> -#include <pinctrl-mtk-common.h>
> +#include "pinctrl-mtk-common.h"
>
>  static const struct mtk_desc_pin mtk_pins_mt8173[] = {
>         MTK_PIN(
> --
> 1.9.1
>
>
>
Guenter Roeck March 12, 2015, 10:41 a.m. UTC | #2
On 03/12/2015 03:32 AM, Matthias Brugger wrote:
> 2015-03-12 7:51 GMT+01:00 Axel Lin <axel.lin@ingics.com>:
>> Fix below build error:
>>    CC      drivers/pinctrl/mediatek/pinctrl-mt8135.o
>> In file included from drivers/pinctrl/mediatek/pinctrl-mt8135.c:24:0:
>> drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h:19:32: fatal error: pinctrl-mtk-common.h: No such file or directory
>> compilation terminated.
>> make[3]: *** [drivers/pinctrl/mediatek/pinctrl-mt8135.o] Error 1
>> make[2]: *** [drivers/pinctrl/mediatek] Error 2
>> make[1]: *** [drivers/pinctrl] Error 2
>> make: *** [drivers] Error 2
>>
>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
>
> The fix from Guenter already addresses mt8135 case [0].
> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
>
> for the mt8173 part.
>

Axel's patch is more complete, so if my patch has not been applied yet
I would suggest to take this one instead.

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Guenter

> Cheers,
> Matthias
>
> [0] https://lkml.org/lkml/2015/3/11/510
>
>> ---
>>   drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h | 2 +-
>>   drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h | 2 +-
>>   2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
>> index 5019cef..e17aedb 100644
>> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
>> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
>> @@ -16,7 +16,7 @@
>>   #define __PINCTRL_MTK_MT8135_H
>>
>>   #include <linux/pinctrl/pinctrl.h>
>> -#include <pinctrl-mtk-common.h>
>> +#include "pinctrl-mtk-common.h"
>>
>>   static const struct mtk_desc_pin mtk_pins_mt8135[] = {
>>          MTK_PIN(
>> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
>> index c1a3ca0..13e5b68 100644
>> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
>> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
>> @@ -16,7 +16,7 @@
>>   #define __PINCTRL_MTK_MT8173_H
>>
>>   #include <linux/pinctrl/pinctrl.h>
>> -#include <pinctrl-mtk-common.h>
>> +#include "pinctrl-mtk-common.h"
>>
>>   static const struct mtk_desc_pin mtk_pins_mt8173[] = {
>>          MTK_PIN(
>> --
>> 1.9.1
>>
>>
>>
>
>
>
Hongzhou Yang March 13, 2015, 7:44 p.m. UTC | #3
On Thu, 2015-03-12 at 14:51 +0800, Axel Lin wrote:
> Fix below build error:
>   CC      drivers/pinctrl/mediatek/pinctrl-mt8135.o
> In file included from drivers/pinctrl/mediatek/pinctrl-mt8135.c:24:0:
> drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h:19:32: fatal error: pinctrl-mtk-common.h: No such file or directory
> compilation terminated.
> make[3]: *** [drivers/pinctrl/mediatek/pinctrl-mt8135.o] Error 1
> make[2]: *** [drivers/pinctrl/mediatek] Error 2
> make[1]: *** [drivers/pinctrl] Error 2
> make: *** [drivers] Error 2
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
>  drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h | 2 +-
>  drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
> index 5019cef..e17aedb 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
> @@ -16,7 +16,7 @@
>  #define __PINCTRL_MTK_MT8135_H
>  
>  #include <linux/pinctrl/pinctrl.h>
> -#include <pinctrl-mtk-common.h>
> +#include "pinctrl-mtk-common.h"
>  
>  static const struct mtk_desc_pin mtk_pins_mt8135[] = {
>  	MTK_PIN(
> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
> index c1a3ca0..13e5b68 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
> @@ -16,7 +16,7 @@
>  #define __PINCTRL_MTK_MT8173_H
>  
>  #include <linux/pinctrl/pinctrl.h>
> -#include <pinctrl-mtk-common.h>
> +#include "pinctrl-mtk-common.h"
>  
>  static const struct mtk_desc_pin mtk_pins_mt8173[] = {
>  	MTK_PIN(

Hi Axel,

Thank you very much.

Acked-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

Hongzhou
Linus Walleij March 18, 2015, 12:07 p.m. UTC | #4
On Thu, Mar 12, 2015 at 7:51 AM, Axel Lin <axel.lin@ingics.com> wrote:

> Fix below build error:
>   CC      drivers/pinctrl/mediatek/pinctrl-mt8135.o
> In file included from drivers/pinctrl/mediatek/pinctrl-mt8135.c:24:0:
> drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h:19:32: fatal error: pinctrl-mtk-common.h: No such file or directory
> compilation terminated.
> make[3]: *** [drivers/pinctrl/mediatek/pinctrl-mt8135.o] Error 1
> make[2]: *** [drivers/pinctrl/mediatek] Error 2
> make[1]: *** [drivers/pinctrl] Error 2
> make: *** [drivers] Error 2
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

Patch applied. Partially, since Guenther already fixed one oneliner.

Yours,
Linus Walleij
diff mbox

Patch

diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
index 5019cef..e17aedb 100644
--- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
+++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8135.h
@@ -16,7 +16,7 @@ 
 #define __PINCTRL_MTK_MT8135_H
 
 #include <linux/pinctrl/pinctrl.h>
-#include <pinctrl-mtk-common.h>
+#include "pinctrl-mtk-common.h"
 
 static const struct mtk_desc_pin mtk_pins_mt8135[] = {
 	MTK_PIN(
diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
index c1a3ca0..13e5b68 100644
--- a/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
+++ b/drivers/pinctrl/mediatek/pinctrl-mtk-mt8173.h
@@ -16,7 +16,7 @@ 
 #define __PINCTRL_MTK_MT8173_H
 
 #include <linux/pinctrl/pinctrl.h>
-#include <pinctrl-mtk-common.h>
+#include "pinctrl-mtk-common.h"
 
 static const struct mtk_desc_pin mtk_pins_mt8173[] = {
 	MTK_PIN(