[PATCH 1/1] Changes style to look more like list iteration.

2018-10-22 Thread Leonardo Brás
Changes the code in order to make it looks more like its a list iteration. Signed-off-by: Leonardo Brás --- It's a remainer from a patch from the -Wshadow patchset. Since it doesn't really belongs to that patchset anymore, I decided to release it just as a code refactor for file2alias.c I

[PATCH 1/1] Changes style to look more like list iteration.

2018-10-22 Thread Leonardo Brás
Changes the code in order to make it looks more like its a list iteration. Signed-off-by: Leonardo Brás --- It's a remainer from a patch from the -Wshadow patchset. Since it doesn't really belongs to that patchset anymore, I decided to release it just as a code refactor for file2alias.c I

[PATCH v2 0/5] Adds -Wshadow on KBUILD_HOSTCFLAGS

2018-10-22 Thread Leonardo Brás
- Renames variables to meaningful names in vdso2c.h - The previous logic in file2alias.c will be available on a separated patch Leonardo Brás (5): x86/vdso: Renames variable to fix shadow warning. kbuild: Removes unnecessary shadowed local variable. Creates macro to avoid variable shadowing

[PATCH v2 0/5] Adds -Wshadow on KBUILD_HOSTCFLAGS

2018-10-22 Thread Leonardo Brás
- Renames variables to meaningful names in vdso2c.h - The previous logic in file2alias.c will be available on a separated patch Leonardo Brás (5): x86/vdso: Renames variable to fix shadow warning. kbuild: Removes unnecessary shadowed local variable. Creates macro to avoid variable shadowing

[PATCH v2 5/5] Adds -Wshadow on KBUILD_HOSTCFLAGS

2018-10-22 Thread Leonardo Brás
Adds -Wshadow on KBUILD_HOSTCFLAGS to show shadow warnings on tools built for HOST. Signed-off-by: Leonardo Brás --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index e8b599b4dcde..3edae5d359b5 100644 --- a/Makefile +++ b/Makefile @@ -360,7

[PATCH v2 5/5] Adds -Wshadow on KBUILD_HOSTCFLAGS

2018-10-22 Thread Leonardo Brás
Adds -Wshadow on KBUILD_HOSTCFLAGS to show shadow warnings on tools built for HOST. Signed-off-by: Leonardo Brás --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index e8b599b4dcde..3edae5d359b5 100644 --- a/Makefile +++ b/Makefile @@ -360,7

[PATCH v2 4/5] modpost: Changes parameter name to avoid shadowing.

2018-10-22 Thread Leonardo Brás
Changes the parameter name to avoid shadowing a variable. Signed-off-by: Leonardo Brás --- scripts/mod/modpost.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 0d998c54564d..368fe42340df 100644 --- a/scripts/mod

[PATCH v2 4/5] modpost: Changes parameter name to avoid shadowing.

2018-10-22 Thread Leonardo Brás
Changes the parameter name to avoid shadowing a variable. Signed-off-by: Leonardo Brás --- scripts/mod/modpost.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 0d998c54564d..368fe42340df 100644 --- a/scripts/mod

[PATCH v2 3/5] Creates macro to avoid variable shadowing

2018-10-22 Thread Leonardo Brás
-by: Leonardo Brás --- scripts/mod/file2alias.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/scripts/mod/file2alias.c b/scripts/mod/file2alias.c index 7be43697ff84..3015c0bdecb2 100644 --- a/scripts/mod/file2alias.c +++ b/scripts/mod/file2alias.c @@ -95,12

[PATCH v2 2/5] kbuild: Removes unnecessary shadowed local variable.

2018-10-22 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). It was used only once, with the same value it was started before the if block. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/asn1_compiler.c b/scripts

[PATCH v2 1/5] x86/vdso: Renames variable to fix shadow warning.

2018-10-22 Thread Leonardo Brás
Renames the char variable to avoid shadowing a variable previously declared on this function. Signed-off-by: Leonardo Brás --- arch/x86/entry/vdso/vdso2c.h | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/x86/entry/vdso/vdso2c.h b/arch/x86/entry/vdso

[PATCH v2 1/5] x86/vdso: Renames variable to fix shadow warning.

2018-10-22 Thread Leonardo Brás
Renames the char variable to avoid shadowing a variable previously declared on this function. Signed-off-by: Leonardo Brás --- arch/x86/entry/vdso/vdso2c.h | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/x86/entry/vdso/vdso2c.h b/arch/x86/entry/vdso

[PATCH v2 3/5] Creates macro to avoid variable shadowing

2018-10-22 Thread Leonardo Brás
-by: Leonardo Brás --- scripts/mod/file2alias.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/scripts/mod/file2alias.c b/scripts/mod/file2alias.c index 7be43697ff84..3015c0bdecb2 100644 --- a/scripts/mod/file2alias.c +++ b/scripts/mod/file2alias.c @@ -95,12

[PATCH v2 2/5] kbuild: Removes unnecessary shadowed local variable.

2018-10-22 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). It was used only once, with the same value it was started before the if block. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/asn1_compiler.c b/scripts

[PATCH 4/4] Changes macro usage to avoid shadowing a variable.

2018-10-16 Thread Leonardo Brás
Changes the usage of DEF_FIELD_ADDR in this function to create a reference and operate over it using an aux variable. It also changes the loop logic used to find duplicates, to avoid creating another variable. Signed-off-by: Leonardo Brás --- scripts/mod/file2alias.c | 14 -- 1 file

[PATCH 4/4] Changes macro usage to avoid shadowing a variable.

2018-10-16 Thread Leonardo Brás
Changes the usage of DEF_FIELD_ADDR in this function to create a reference and operate over it using an aux variable. It also changes the loop logic used to find duplicates, to avoid creating another variable. Signed-off-by: Leonardo Brás --- scripts/mod/file2alias.c | 14 -- 1 file

[PATCH 3/4] kbuild: Removes unnecessary shadowed local variable and optimize testing.

2018-10-16 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). Optimize test of isdigit: - If isalpha returns true, isdigit will return false, so no need to test. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff

[PATCH 3/4] kbuild: Removes unnecessary shadowed local variable and optimize testing.

2018-10-16 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). Optimize test of isdigit: - If isalpha returns true, isdigit will return false, so no need to test. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff

[PATCH 2/4] Renames variable to fix shadow warning.

2018-10-16 Thread Leonardo Brás
Renames the char variable to avoid shadowing a variable previously declared on this function. Signed-off-by: Leonardo Brás --- arch/x86/entry/vdso/vdso2c.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/entry/vdso/vdso2c.h b/arch/x86/entry/vdso/vdso2c.h index

[PATCH 1/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Leonardo Brás
Adds -Wshadow=local on KBUILD_HOSTCFLAGS to show shadow warnings on tools built for HOST. Signed-off-by: Leonardo Brás --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index e8b599b4dcde..fb0a9ac195e7 100644 --- a/Makefile +++ b/Makefile

[PATCH 2/4] Renames variable to fix shadow warning.

2018-10-16 Thread Leonardo Brás
Renames the char variable to avoid shadowing a variable previously declared on this function. Signed-off-by: Leonardo Brás --- arch/x86/entry/vdso/vdso2c.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/entry/vdso/vdso2c.h b/arch/x86/entry/vdso/vdso2c.h index

[PATCH 1/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Leonardo Brás
Adds -Wshadow=local on KBUILD_HOSTCFLAGS to show shadow warnings on tools built for HOST. Signed-off-by: Leonardo Brás --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index e8b599b4dcde..fb0a9ac195e7 100644 --- a/Makefile +++ b/Makefile

[PATCH 0/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Leonardo Brás
This patchset add -Wshadow=local on KBUILD_HOSTCFLAGS and fixes all code that show this warning. The third patch was already submitted, but was not merged yet. I like to think it's part of this patchset, but if it was already merged, please ignore it. Leonardo Brás (4): Adds -Wshadow=local

[PATCH 0/4] Adds -Wshadow=local on KBUILD_HOSTCFLAGS

2018-10-16 Thread Leonardo Brás
This patchset add -Wshadow=local on KBUILD_HOSTCFLAGS and fixes all code that show this warning. The third patch was already submitted, but was not merged yet. I like to think it's part of this patchset, but if it was already merged, please ignore it. Leonardo Brás (4): Adds -Wshadow=local

[PATCH v3 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 1 + 1 file changed, 1 insertion(+) create mode

[PATCH v3 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-27 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH v3 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 1 + 1 file changed, 1 insertion(+) create mode

[PATCH v3 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-27 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH v3 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-27 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH v3 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-27 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH v3 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro

[PATCH v3 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro

[PATCH v3 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH v3 0/7] Remove errors building drivers/DRIVERNAME

2018-09-27 Thread Leonardo Brás
to community. If there is any interest helping/using this, I have a prototype in: https://gitlab.com/LeoBras/linux-next Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc

[PATCH v3 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH v3 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH v3 0/7] Remove errors building drivers/DRIVERNAME

2018-09-27 Thread Leonardo Brás
to community. If there is any interest helping/using this, I have a prototype in: https://gitlab.com/LeoBras/linux-next Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc

[PATCH v3 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH v3 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..8fc80e805b90 100644

[PATCH v3 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..8fc80e805b90 100644

Re: [PATCH v2 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Hello Eike, > > Oh, it's a Makefile, and as all text files, it have to end with newline. > > If I am wrong, please let me know. > > No, that's fine. But it means it has to have one LF after "tree.". It had > none, now it hat 2. > > Eike Oh, i haven't noticed. Sorry for the trouble. Thanks for

Re: [PATCH v2 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Hello Eike, > > Oh, it's a Makefile, and as all text files, it have to end with newline. > > If I am wrong, please let me know. > > No, that's fine. But it means it has to have one LF after "tree.". It had > none, now it hat 2. > > Eike Oh, i haven't noticed. Sorry for the trouble. Thanks for

[PATCH v3 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-27 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH v3 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-27 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH v3 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-27 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH v3 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-27 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH v3 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 1 + 1 file changed, 1 insertion(+) create mode

[PATCH v3 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 1 + 1 file changed, 1 insertion(+) create mode

[PATCH v3 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro

[PATCH v3 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro

[PATCH v3 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH v3 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH v3 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH v3 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH v3 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..8fc80e805b90 100644

[PATCH v3 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-27 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..8fc80e805b90 100644

[PATCH v3 0/7] Remove errors building drivers/DRIVERNAME

2018-09-27 Thread Leonardo Brás
to community. If there is any interest helping/using this, I have a prototype in: https://gitlab.com/LeoBras/linux-next Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc

[PATCH v3 0/7] Remove errors building drivers/DRIVERNAME

2018-09-27 Thread Leonardo Brás
to community. If there is any interest helping/using this, I have a prototype in: https://gitlab.com/LeoBras/linux-next Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc

Re: [PATCH v2 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Hello Eike, > No, that's fine. But it means it has to have one LF after "tree.". It had > none, now it hat 2. > > Eike Oh, i haven't noticed. Sorry for the trouble. Thanks for the feedback! Leonardo Brás

Re: [PATCH v2 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-27 Thread Leonardo Brás
Hello Eike, > No, that's fine. But it means it has to have one LF after "tree.". It had > none, now it hat 2. > > Eike Oh, i haven't noticed. Sorry for the trouble. Thanks for the feedback! Leonardo Brás

[PATCH v2 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-26 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH v2 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-26 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH v2 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-26 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 2 ++ 1 file changed, 2 insertions(+) create

[PATCH v2 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-26 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 2 ++ 1 file changed, 2 insertions(+) create

[PATCH v2 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-26 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH v2 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-26 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH v2 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-26 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro

[PATCH v2 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-26 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro

[PATCH v2 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-26 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH v2 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-26 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH v2 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-26 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH v2 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-26 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

[PATCH v2 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-26 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..f5cffe232448 100644

[PATCH v2 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-26 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..f5cffe232448 100644

[PATCH v2 0/7] Remove errors building drivers/DRIVERNAME

2018-09-26 Thread Leonardo Brás
submitting to community. If there is any interest helping/using this, I have a prototype in: https://gitlab.com/LeoBras/linux-next Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc

[PATCH v2 0/7] Remove errors building drivers/DRIVERNAME

2018-09-26 Thread Leonardo Brás
submitting to community. If there is any interest helping/using this, I have a prototype in: https://gitlab.com/LeoBras/linux-next Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc

[PATCH 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-25 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH 7/7] drivers: hwtracing: Adds Makefile to enable building from directory.

2018-09-25 Thread Leonardo Brás
Adds Makefile to enable building the driver using 'make drivers/hwtracing/'. Changes drivers/Makefile to call the new Makefile directly. It enables user building this driver without building the whole drivers/ subtree. Signed-off-by: Leonardo Brás --- drivers/Makefile | 4

[PATCH 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-25 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 1 + 1 file changed, 1 insertion(+) create mode

[PATCH 6/7] drivers: oprofile: Avoids building driver from direct make command

2018-09-25 Thread Leonardo Brás
Creates new Makefile to avoid building driver if 'make drivers/oprofile/' is called directly. This driver is usually built from arch/$ARCH and seems to have no meaning building alone. Signed-off-by: Leonardo Brás --- drivers/oprofile/Makefile | 1 + 1 file changed, 1 insertion(+) create mode

[PATCH 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-25 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH 5/7] drivers: s390: Avoids building drivers if ARCH is not s390.

2018-09-25 Thread Leonardo Brás
Avoids building s390 drivers if 'make drivers/s390/' is called but ARCH is not s390. Signed-off-by: Leonardo Brás --- drivers/s390/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/s390/Makefile b/drivers/s390/Makefile index a863b0462b43..0575f02dba45

[PATCH 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-25 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro/Makefile

[PATCH 4/7] drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

2018-09-25 Thread Leonardo Brás
Avoids building proc.o if 'make drivers/zorro/' is called and CONFIG_ZORRO is disabled, even if CONFIG_PROC_FS is enabled. Signed-off-by: Leonardo Brás --- drivers/zorro/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/zorro/Makefile b/drivers/zorro/Makefile

[PATCH 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-25 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-25 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH 2/7] drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled

2018-09-25 Thread Leonardo Brás
Avoids building driver if 'make drivers/nubus/' is called and CONFIG_NUBUS is disabled. Avoids building proc.o if CONFIG_PROC_FS is enabled but CONFIG_NUBUS is disabled. Signed-off-by: Leonardo Brás --- drivers/nubus/Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH 3/7] drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled

2018-09-25 Thread Leonardo Brás
Avoids building driver if 'make drivers/parisc/' is called and CONFIG_PARISC is disabled. Signed-off-by: Leonardo Brás --- drivers/parisc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/parisc/Makefile b/drivers/parisc/Makefile index 3cd5e6cb8478

[PATCH 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-25 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..f5cffe232448 100644

[PATCH 1/7] drivers: dio: Avoids building driver if CONFIG_DIO is disabled

2018-09-25 Thread Leonardo Brás
Avoids building driver if 'make drivers/dio/' is called and CONFIG_DIO is disabled. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio/Makefile b/drivers/dio/Makefile index ae92d17083f2..f5cffe232448 100644

[PATCH 0/7] Remove errors building drivers/DRIVERNAME

2018-09-25 Thread Leonardo Brás
-next. Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

[PATCH 0/7] Remove errors building drivers/DRIVERNAME

2018-09-25 Thread Leonardo Brás
-next. Leonardo Brás (7): drivers: dio: Avoids building driver if CONFIG_DIO is disabled drivers: nubus: Avoids building driver if CONFIG_NUBUS is disabled drivers: parisc: Avoids building driver if CONFIG_PARISC is disabled drivers: zorro: Avoids building proc.o if CONFIG_ZORRO is disabled

[PATCH 1/1] drivers/dio : Changes Makefile to avoid acciental compilation.

2018-09-20 Thread Leonardo Brás
Changes Makefile so 'make drivers/dio/' won't compile this driver unless CONFIG_DIO is enabled. Previously, it would compile independently of .config file. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio

[PATCH 1/1] drivers/dio : Changes Makefile to avoid acciental compilation.

2018-09-20 Thread Leonardo Brás
Changes Makefile so 'make drivers/dio/' won't compile this driver unless CONFIG_DIO is enabled. Previously, it would compile independently of .config file. Signed-off-by: Leonardo Brás --- drivers/dio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dio

[PATCH 1/1] kbuild: Optimize tests and remove shadowed local variable.

2018-09-18 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). Optimize test of isdigit: - If isalpha returns true, isdigit will return false, so no need to test. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 1/1] kbuild: Optimize tests and remove shadowed local variable.

2018-09-18 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). Optimize test of isdigit: - If isalpha returns true, isdigit will return false, so no need to test. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH] Optimize tests and remove shadowed local variable.

2018-09-12 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). Optimize test of isdigit: - If isalpha returns true, isdigit will return false, so no need to test. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH] Optimize tests and remove shadowed local variable.

2018-09-12 Thread Leonardo Brás
Removes an unnecessary shadowed local variable (start). Optimize test of isdigit: - If isalpha returns true, isdigit will return false, so no need to test. Signed-off-by: Leonardo Brás --- scripts/asn1_compiler.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 1/1] staging: fbtft: Removes ";" from very used macro definition.

2018-08-09 Thread Leonardo Brás
All usages of this macro append the ";" at the end of line. It is not recommended nor necessary to append a ";" at the end of this macro definition. Signed-off-by: Leonardo Brás --- drivers/staging/fbtft/fbtft.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

[PATCH 1/1] staging: fbtft: Removes ";" from very used macro definition.

2018-08-09 Thread Leonardo Brás
All usages of this macro append the ";" at the end of line. It is not recommended nor necessary to append a ";" at the end of this macro definition. Signed-off-by: Leonardo Brás --- drivers/staging/fbtft/fbtft.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

  1   2   >