[v2] selftests: gpio: add config fragment for gpio-mockup

Message ID 20170331084130.7973-1-fathi.boudra@linaro.org
State New
Headers show

Commit Message

Fathi Boudra March 31, 2017, 8:41 a.m.
The gpio test requires to insert the gpio mockup module (CONFIG_GPIO_MOCKUP).
The gpio mockup driver depends on gpiolib (CONFIG_GPIOLIB).
CONFIG_GPIO_SYSFS is selected automatically by the gpio mockup driver.

Tested on x86_64 and arm64 with defconfig and kselftest-merge.

Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>

---

Changes in v2:
 * Per Bamvor Zhang Jian review, add gpiolib dependency (CONFIG_GPIOLIB).
   Skipped CONFIG_DEBUG_GPIO as it isn't necessary, and CONFIG_GPIO_SYSFS
   as it's selected automatically by the gpio mockup driver.

 tools/testing/selftests/gpio/config | 2 ++
 1 file changed, 2 insertions(+)
 create mode 100644 tools/testing/selftests/gpio/config

-- 
2.11.0

Comments

Bamvor Zhang Jian April 4, 2017, 12:52 a.m. | #1
Hi,

On 31 March 2017 at 16:41, Fathi Boudra <fathi.boudra@linaro.org> wrote:
> The gpio test requires to insert the gpio mockup module (CONFIG_GPIO_MOCKUP).

> The gpio mockup driver depends on gpiolib (CONFIG_GPIOLIB).

> CONFIG_GPIO_SYSFS is selected automatically by the gpio mockup driver.

>

> Tested on x86_64 and arm64 with defconfig and kselftest-merge.

>

> Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>

Reviewed-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>

> ---

>

> Changes in v2:

>  * Per Bamvor Zhang Jian review, add gpiolib dependency (CONFIG_GPIOLIB).

>    Skipped CONFIG_DEBUG_GPIO as it isn't necessary, and CONFIG_GPIO_SYSFS

>    as it's selected automatically by the gpio mockup driver.

>

>  tools/testing/selftests/gpio/config | 2 ++

>  1 file changed, 2 insertions(+)

>  create mode 100644 tools/testing/selftests/gpio/config

>

> diff --git a/tools/testing/selftests/gpio/config b/tools/testing/selftests/gpio/config

> new file mode 100644

> index 000000000000..abaa6902b7b6

> --- /dev/null

> +++ b/tools/testing/selftests/gpio/config

> @@ -0,0 +1,2 @@

> +CONFIG_GPIOLIB=y

> +CONFIG_GPIO_MOCKUP=m

> --

> 2.11.0
Fathi Boudra April 19, 2017, 5:32 p.m. | #2
gentle ping

On 4 April 2017 at 03:52, Bamvor Zhang Jian <bamvor.zhangjian@linaro.org> wrote:
> Hi,

>

> On 31 March 2017 at 16:41, Fathi Boudra <fathi.boudra@linaro.org> wrote:

>> The gpio test requires to insert the gpio mockup module (CONFIG_GPIO_MOCKUP).

>> The gpio mockup driver depends on gpiolib (CONFIG_GPIOLIB).

>> CONFIG_GPIO_SYSFS is selected automatically by the gpio mockup driver.

>>

>> Tested on x86_64 and arm64 with defconfig and kselftest-merge.

>>

>> Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>

> Reviewed-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>

>> ---

>>

>> Changes in v2:

>>  * Per Bamvor Zhang Jian review, add gpiolib dependency (CONFIG_GPIOLIB).

>>    Skipped CONFIG_DEBUG_GPIO as it isn't necessary, and CONFIG_GPIO_SYSFS

>>    as it's selected automatically by the gpio mockup driver.

>>

>>  tools/testing/selftests/gpio/config | 2 ++

>>  1 file changed, 2 insertions(+)

>>  create mode 100644 tools/testing/selftests/gpio/config

>>

>> diff --git a/tools/testing/selftests/gpio/config b/tools/testing/selftests/gpio/config

>> new file mode 100644

>> index 000000000000..abaa6902b7b6

>> --- /dev/null

>> +++ b/tools/testing/selftests/gpio/config

>> @@ -0,0 +1,2 @@

>> +CONFIG_GPIOLIB=y

>> +CONFIG_GPIO_MOCKUP=m

>> --

>> 2.11.0
Shuah Khan April 27, 2017, 1:46 p.m. | #3
On 04/19/2017 11:32 AM, Fathi Boudra wrote:
> gentle ping

> 

> On 4 April 2017 at 03:52, Bamvor Zhang Jian <bamvor.zhangjian@linaro.org> wrote:

>> Hi,

>>

>> On 31 March 2017 at 16:41, Fathi Boudra <fathi.boudra@linaro.org> wrote:

>>> The gpio test requires to insert the gpio mockup module (CONFIG_GPIO_MOCKUP).

>>> The gpio mockup driver depends on gpiolib (CONFIG_GPIOLIB).

>>> CONFIG_GPIO_SYSFS is selected automatically by the gpio mockup driver.

>>>

>>> Tested on x86_64 and arm64 with defconfig and kselftest-merge.

>>>

>>> Signed-off-by: Fathi Boudra <fathi.boudra@linaro.org>

>> Reviewed-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>


Applied to linux-kselftest next for 4.12-rc1

thanks for the patch and review.

-- Shuah

Patch hide | download patch | download mbox

diff --git a/tools/testing/selftests/gpio/config b/tools/testing/selftests/gpio/config
new file mode 100644
index 000000000000..abaa6902b7b6
--- /dev/null
+++ b/tools/testing/selftests/gpio/config
@@ -0,0 +1,2 @@ 
+CONFIG_GPIOLIB=y
+CONFIG_GPIO_MOCKUP=m