Re: [PATCH] platform/x86: Add IIO dependency to ACPI_TOSHIBA

2016-06-28 Thread Azael Avalos 
2016-06-28 23:16 GMT-06:00 Darren Hart :
> On Tue, Jun 28, 2016 at 01:58:55PM -0600, Azael Avalos wrote:
>> The toshiba_acpi driver added support to the accelerometer axis
>> via the IIO subsystem, but failed to add the dependency to the
>> Kconfig file, making the driver fail to compile if the IIO
>> subsystem is not selected.
>>
>> This patch adds a depedency to the IIO subsystem to the
>> ACPI_TOSHIBA entry in Kconfig to avoid such build failure.
>>
>> Signed-off-by: Azael Avalos 
>> ---
>> Darren, Jonathan:
>>  I just received a few mails from the kbuild test robot informing
>>  the build failure, as a fix for this I just added a hard dependency
>>  to IIO, but maybe Jonathan can weigh in and provide me with a better
>>  alternative (perhaps some ifdefs) as to the hard IIO dependency.
>
> Don't you just love the giant kernel builder in the sky?

I certainly do, the ever watching eye XD

>
> Rather than add this separately and leave a potential build failure in the
> kernel, I'll add this line to:
>
> toshiba_acpi: Add IIO interface for accelerometer axis data

Great :-)

>
>>
>>  drivers/platform/x86/Kconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
>> index c06bb85..633fc5c 100644
>> --- a/drivers/platform/x86/Kconfig
>> +++ b/drivers/platform/x86/Kconfig
>> @@ -669,6 +669,7 @@ config ACPI_TOSHIBA
>>   depends on SERIO_I8042 || SERIO_I8042 = n
>>   depends on ACPI_VIDEO || ACPI_VIDEO = n
>>   depends on RFKILL || RFKILL = n
>> + depends on IIO
>>   select INPUT_POLLDEV
>>   select INPUT_SPARSEKMAP
>>   ---help---
>> --
>> 2.8.4
>>
>>
>
> --
> Darren Hart
> Intel Open Source Technology Center



-- 
-- El mundo apesta y vosotros apestais tambien --

Re: [PATCH] platform/x86: Add IIO dependency to ACPI_TOSHIBA

2016-06-28 Thread Azael Avalos 
2016-06-28 23:16 GMT-06:00 Darren Hart :
> On Tue, Jun 28, 2016 at 01:58:55PM -0600, Azael Avalos wrote:
>> The toshiba_acpi driver added support to the accelerometer axis
>> via the IIO subsystem, but failed to add the dependency to the
>> Kconfig file, making the driver fail to compile if the IIO
>> subsystem is not selected.
>>
>> This patch adds a depedency to the IIO subsystem to the
>> ACPI_TOSHIBA entry in Kconfig to avoid such build failure.
>>
>> Signed-off-by: Azael Avalos 
>> ---
>> Darren, Jonathan:
>>  I just received a few mails from the kbuild test robot informing
>>  the build failure, as a fix for this I just added a hard dependency
>>  to IIO, but maybe Jonathan can weigh in and provide me with a better
>>  alternative (perhaps some ifdefs) as to the hard IIO dependency.
>
> Don't you just love the giant kernel builder in the sky?

I certainly do, the ever watching eye XD

>
> Rather than add this separately and leave a potential build failure in the
> kernel, I'll add this line to:
>
> toshiba_acpi: Add IIO interface for accelerometer axis data

Great :-)

>
>>
>>  drivers/platform/x86/Kconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
>> index c06bb85..633fc5c 100644
>> --- a/drivers/platform/x86/Kconfig
>> +++ b/drivers/platform/x86/Kconfig
>> @@ -669,6 +669,7 @@ config ACPI_TOSHIBA
>>   depends on SERIO_I8042 || SERIO_I8042 = n
>>   depends on ACPI_VIDEO || ACPI_VIDEO = n
>>   depends on RFKILL || RFKILL = n
>> + depends on IIO
>>   select INPUT_POLLDEV
>>   select INPUT_SPARSEKMAP
>>   ---help---
>> --
>> 2.8.4
>>
>>
>
> --
> Darren Hart
> Intel Open Source Technology Center



-- 
-- El mundo apesta y vosotros apestais tambien --

Re: [PATCH] platform/x86: Add IIO dependency to ACPI_TOSHIBA

2016-06-28 Thread Darren Hart 
On Tue, Jun 28, 2016 at 01:58:55PM -0600, Azael Avalos wrote:
> The toshiba_acpi driver added support to the accelerometer axis
> via the IIO subsystem, but failed to add the dependency to the
> Kconfig file, making the driver fail to compile if the IIO
> subsystem is not selected.
> 
> This patch adds a depedency to the IIO subsystem to the
> ACPI_TOSHIBA entry in Kconfig to avoid such build failure.
> 
> Signed-off-by: Azael Avalos 
> ---
> Darren, Jonathan:
>  I just received a few mails from the kbuild test robot informing
>  the build failure, as a fix for this I just added a hard dependency
>  to IIO, but maybe Jonathan can weigh in and provide me with a better
>  alternative (perhaps some ifdefs) as to the hard IIO dependency.

Don't you just love the giant kernel builder in the sky?

Rather than add this separately and leave a potential build failure in the
kernel, I'll add this line to:

toshiba_acpi: Add IIO interface for accelerometer axis data

> 
>  drivers/platform/x86/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index c06bb85..633fc5c 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -669,6 +669,7 @@ config ACPI_TOSHIBA
>   depends on SERIO_I8042 || SERIO_I8042 = n
>   depends on ACPI_VIDEO || ACPI_VIDEO = n
>   depends on RFKILL || RFKILL = n
> + depends on IIO
>   select INPUT_POLLDEV
>   select INPUT_SPARSEKMAP
>   ---help---
> -- 
> 2.8.4
> 
> 

-- 
Darren Hart
Intel Open Source Technology Center

Re: [PATCH] platform/x86: Add IIO dependency to ACPI_TOSHIBA

2016-06-28 Thread Darren Hart 
On Tue, Jun 28, 2016 at 01:58:55PM -0600, Azael Avalos wrote:
> The toshiba_acpi driver added support to the accelerometer axis
> via the IIO subsystem, but failed to add the dependency to the
> Kconfig file, making the driver fail to compile if the IIO
> subsystem is not selected.
> 
> This patch adds a depedency to the IIO subsystem to the
> ACPI_TOSHIBA entry in Kconfig to avoid such build failure.
> 
> Signed-off-by: Azael Avalos 
> ---
> Darren, Jonathan:
>  I just received a few mails from the kbuild test robot informing
>  the build failure, as a fix for this I just added a hard dependency
>  to IIO, but maybe Jonathan can weigh in and provide me with a better
>  alternative (perhaps some ifdefs) as to the hard IIO dependency.

Don't you just love the giant kernel builder in the sky?

Rather than add this separately and leave a potential build failure in the
kernel, I'll add this line to:

toshiba_acpi: Add IIO interface for accelerometer axis data

> 
>  drivers/platform/x86/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index c06bb85..633fc5c 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -669,6 +669,7 @@ config ACPI_TOSHIBA
>   depends on SERIO_I8042 || SERIO_I8042 = n
>   depends on ACPI_VIDEO || ACPI_VIDEO = n
>   depends on RFKILL || RFKILL = n
> + depends on IIO
>   select INPUT_POLLDEV
>   select INPUT_SPARSEKMAP
>   ---help---
> -- 
> 2.8.4
> 
> 

-- 
Darren Hart
Intel Open Source Technology Center

[PATCH] platform/x86: Add IIO dependency to ACPI_TOSHIBA

2016-06-28 Thread Azael Avalos 
The toshiba_acpi driver added support to the accelerometer axis
via the IIO subsystem, but failed to add the dependency to the
Kconfig file, making the driver fail to compile if the IIO
subsystem is not selected.

This patch adds a depedency to the IIO subsystem to the
ACPI_TOSHIBA entry in Kconfig to avoid such build failure.

Signed-off-by: Azael Avalos 
---
Darren, Jonathan:
 I just received a few mails from the kbuild test robot informing
 the build failure, as a fix for this I just added a hard dependency
 to IIO, but maybe Jonathan can weigh in and provide me with a better
 alternative (perhaps some ifdefs) as to the hard IIO dependency.

 drivers/platform/x86/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
index c06bb85..633fc5c 100644
--- a/drivers/platform/x86/Kconfig
+++ b/drivers/platform/x86/Kconfig
@@ -669,6 +669,7 @@ config ACPI_TOSHIBA
depends on SERIO_I8042 || SERIO_I8042 = n
depends on ACPI_VIDEO || ACPI_VIDEO = n
depends on RFKILL || RFKILL = n
+   depends on IIO
select INPUT_POLLDEV
select INPUT_SPARSEKMAP
---help---
-- 
2.8.4

[PATCH] platform/x86: Add IIO dependency to ACPI_TOSHIBA

2016-06-28 Thread Azael Avalos 
The toshiba_acpi driver added support to the accelerometer axis
via the IIO subsystem, but failed to add the dependency to the
Kconfig file, making the driver fail to compile if the IIO
subsystem is not selected.

This patch adds a depedency to the IIO subsystem to the
ACPI_TOSHIBA entry in Kconfig to avoid such build failure.

Signed-off-by: Azael Avalos 
---
Darren, Jonathan:
 I just received a few mails from the kbuild test robot informing
 the build failure, as a fix for this I just added a hard dependency
 to IIO, but maybe Jonathan can weigh in and provide me with a better
 alternative (perhaps some ifdefs) as to the hard IIO dependency.

 drivers/platform/x86/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
index c06bb85..633fc5c 100644
--- a/drivers/platform/x86/Kconfig
+++ b/drivers/platform/x86/Kconfig
@@ -669,6 +669,7 @@ config ACPI_TOSHIBA
depends on SERIO_I8042 || SERIO_I8042 = n
depends on ACPI_VIDEO || ACPI_VIDEO = n
depends on RFKILL || RFKILL = n
+   depends on IIO
select INPUT_POLLDEV
select INPUT_SPARSEKMAP
---help---
-- 
2.8.4