[PATCH 1/3] powerpc/dt_cpu_ftrs: Remove unused macro ISA_V2_07B

2020-06-10 Thread Murilo Opsfelder Araujo
Macro ISA_V2_07B is defined but not used anywhere else in the code. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/dt_cpu_ftrs.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/kernel/dt_cpu_ftrs.c b/arch/powerpc/kernel/dt_cpu_ftrs.c index 3a409517c031

[PATCH 0/3] powerpc/dt_cpu_ftrs: Make use of ISA_V3_* macros

2020-06-10 Thread Murilo Opsfelder Araujo
The first patch removes unused macro ISA_V2_07B. The second and third patches make use of macros ISA_V3_0B and ISA_V3_1, respectively, instead their corresponding literals. Murilo Opsfelder Araujo (3): powerpc/dt_cpu_ftrs: Remove unused macro ISA_V2_07B powerpc/dt_cpu_ftrs: Make use of macro

[PATCH 3/3] powerpc/dt_cpu_ftrs: Make use of macro ISA_V3_1

2020-06-10 Thread Murilo Opsfelder Araujo
Macro ISA_V3_1 was defined but never used. Use it instead of literal. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/dt_cpu_ftrs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kernel/dt_cpu_ftrs.c b/arch/powerpc/kernel/dt_cpu_ftrs.c index

[PATCH 2/3] powerpc/dt_cpu_ftrs: Make use of macro ISA_V3_0B

2020-06-10 Thread Murilo Opsfelder Araujo
Macro ISA_V3_0B was defined but never used. Use it instead of literal. Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/dt_cpu_ftrs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kernel/dt_cpu_ftrs.c b/arch/powerpc/kernel/dt_cpu_ftrs.c index

Re: [PATCH v3 1/2] mm: add probe_user_read()

2019-02-05 Thread Murilo Opsfelder Araujo
user_read(void *dst, const void __user > *src, > + size_t size) > +{ > + long ret; > + > + if (!access_ok(src, size)) > + return -EFAULT; Hopefully, there is still time for a minor comment. Do we need to differ

Re: [PATCH kernel v7 20/20] vfio_pci: Add NVIDIA GV100GL [Tesla V100 SXM2] subdriver

2018-12-20 Thread Murilo Opsfelder Araujo
On Thu, Dec 20, 2018 at 07:23:50PM +1100, Alexey Kardashevskiy wrote: > POWER9 Witherspoon machines come with 4 or 6 V100 GPUs which are not > pluggable PCIe devices but still have PCIe links which are used > for config space and MMIO. In addition to that the GPUs have 6 NVLinks > which are connect

[PATCH v3 9/9] powerpc/traps: Add line prefix in show_instructions()

2018-07-31 Thread Murilo Opsfelder Araujo
88d0 f93f0020 e93f0020 39400048 <9949> 3920 7d234b78 383f0040 Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/process.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index e7879

[PATCH v3 5/9] powerpc/traps: Print signal name for unhandled signals

2018-07-31 Thread Murilo Opsfelder Araujo
]: segfault (11) at 13a2a09f8 nip 13a2a086c lr 7fffb63e5100 code 2 in pandafault[13a2a+1] Signed-off-by: Murilo Opsfelder Araujo --- arch/powerpc/kernel/traps.c | 39 +++-- 1 file changed, 37 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel

[PATCH v3 7/9] powerpc: Add stacktrace.h header

2018-07-31 Thread Murilo Opsfelder Araujo
Move show_instructions() declaration to arch/powerpc/include/asm/stacktrace.h and include asm/stracktrace.h in arch/powerpc/kernel/process.c, which contains the implementation. This allows show_instructions() to be called on, for example, show_signal_msg(). Signed-off-by: Murilo Opsfelder Araujo

Re: [PATCH 7/7] powerpc/traps: Show instructions on exceptions

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Wed, Jul 25, 2018 at 06:01:34PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > Move show_instructions() declaration to > > arch/powerpc/include/asm/stacktrace.h > > and include asm/stracktrace.h in arch/powerpc/kernel/proce

Re: [PATCH 2/7] powerpc/traps: Return early in show_signal_msg()

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Wed, Jul 25, 2018 at 05:42:28PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > Modify logic of show_signal_msg() to return early, if possible. Replace > > printk_ratelimited() by printk() and a default rate limit burst to li

Re: [PATCH 6/7] powerpc/traps: Print signal name for unhandled signals

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Christophe. On Wed, Jul 25, 2018 at 05:49:27PM +0200, LEROY Christophe wrote: > Murilo Opsfelder Araujo a écrit : > > > This adds a human-readable name in the unhandled signal message. > > > > Before this patch, a page fault looked like: > > > >

Re: [PATCH 6/7] powerpc/traps: Print signal name for unhandled signals

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Gustavo. On Wed, Jul 25, 2018 at 12:19:00PM -0300, Gustavo Romero wrote: > Hi Murilo, > > LGTM. > > Just a comment: > > On 07/24/2018 04:27 PM, Murilo Opsfelder Araujo wrote: > > This adds a human-readable name in the unhandled signal message. > > &g

Re: [PATCH 0/7] powerpc: Modernize unhandled signals message

2018-07-25 Thread Murilo Opsfelder Araujo
Hi, Mikey. On Wed, Jul 25, 2018 at 05:00:21PM +1000, Michael Neuling wrote: > On Tue, 2018-07-24 at 16:27 -0300, Murilo Opsfelder Araujo wrote: > > Hi, everyone. > > > > This series was inspired by the need to modernize and display more > > informative messa

Re: [v2 PATCH 2/2] powerpc: Enable CPU_FTR_ASYM_SMT for interleaved big-cores

2018-07-04 Thread Murilo Opsfelder Araujo
On Wed, Jul 04, 2018 at 01:45:05PM +0530, Gautham R Shenoy wrote: > Hi Murilo, > > Thanks for the review. > > On Tue, Jul 03, 2018 at 02:53:46PM -0300, Murilo Opsfelder Araujo wrote: > [..snip..] > > > > -/* Initialize CPU <=> thread mapping/

Re: [PATCH] perf session: Fix undeclared 'oe'

2018-04-30 Thread Murilo Opsfelder Araujo
On 04/30/2018 02:49 PM, Greg Kroah-Hartman wrote: > On Mon, Apr 30, 2018 at 10:20:08AM -0700, Greg Kroah-Hartman wrote: >> On Mon, Apr 30, 2018 at 02:11:55PM -0300, Murilo Opsfelder Araujo wrote: >>> On 04/11/2018 05:21 PM, Murilo Opsfelder Araújo wrote: >>>> On 0

Re: [PATCH] perf session: Fix undeclared 'oe'

2018-04-30 Thread Murilo Opsfelder Araujo
On 04/11/2018 05:21 PM, Murilo Opsfelder Araújo wrote: > On 04/11/2018 04:33 PM, Greg Kroah-Hartman wrote: >> On Wed, Apr 11, 2018 at 03:41:35PM -0300, Murilo Opsfelder Araujo wrote: >>> Using linux-3.18.y branch, perf build fails with the following: >>> >>>

[PATCH] perf session: Fix undeclared 'oe'

2018-04-11 Thread Murilo Opsfelder Araujo
Kuang Cc: Ingo Molnar Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Paul Mackerras Cc: Paul Turner Cc: Peter Zijlstra Cc: Sasha Levin Cc: Simon Que Cc: Stephane Eranian Cc: Wang Nan Signed-off-by: Murilo Opsfelder Araujo --- tools/perf/util/session.c | 1 + 1 file changed, 1 insertion(+) di

[PATCH v2] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

2017-07-18 Thread Murilo Opsfelder Araujo
are here: http://kisskb.ellerman.id.au/kisskb/buildresult/12982362/ Signed-off-by: Murilo Opsfelder Araujo --- Changes from v1: - Rebased on top of next-20170718. include/linux/vfio.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/vfio.h b/include/linux/vfio.h in

[PATCH v2] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

2017-06-07 Thread Murilo Opsfelder Araujo
are here: http://kisskb.ellerman.id.au/kisskb/buildresult/12982362/ Signed-off-by: Murilo Opsfelder Araujo --- v1..v2: - Make use of IS_ENABLED() macro because CONFIG_VFIO_SPAPR_EEH is a tristate option (fix http://www.spinics.net/lists/kvm/msg151032.html). include/linux/vfio.h | 4 ++-- 1 file changed

[PATCH] include/linux/vfio.h: Guard powerpc-specific functions with CONFIG_VFIO_SPAPR_EEH

2017-06-06 Thread Murilo Opsfelder Araujo
_VFIO_SPAPR_EEH, the symbol that controls whether vfio_spapr_eeh.c is built, which is where the non-empty versions of these functions are. This issue was found during a randconfig build. Logs are here: http://kisskb.ellerman.id.au/kisskb/buildresult/12982362/ Signed-off-by: Murilo Opsfelder Araujo

[PATCH] Update MAINTAINERS

2017-06-01 Thread Murilo Opsfelder Araujo
drivers/watchdog/wdrtas.c is of insterest of linuxppc maintainers. Signed-off-by: Murilo Opsfelder Araujo --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 9609ca6..3f05927 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7670,6 +7670,7 @@ F

[PATCH v2] drivers/watchdog/Kconfig: Update CONFIG_WATCHDOG_RTAS dependencies

2017-05-29 Thread Murilo Opsfelder Araujo
ed! This was identified during a randconfig build where CONFIG_WATCHDOG_RTAS=m and CONFIG_PPC_RTAS was not set. Logs are here: http://kisskb.ellerman.id.au/kisskb/buildresult/12982152/ This patch fixes the issue by updating CONFIG_WATCHDOG_RTAS to depend on just CONFIG_PPC_RTAS, removing COMPI

[PATCH] drivers/watchdog/Kconfig: Update CONFIG_WATCHDOG_RTAS dependencies

2017-05-26 Thread Murilo Opsfelder Araujo
ed! This was identified during a randconfig build where CONFIG_WATCHDOG_RTAS=m and CONFIG_PPC_RTAS was not set. Logs are here: http://kisskb.ellerman.id.au/kisskb/buildresult/12982152/ This patch fixes the issue by selecting CONFIG_PPC_RTAS when CONFIG_WATCHDOG_RTAS is set. Signed-off-by: Muri

[PATCH] xen/tmem: Pass page instead of pfn to xen_tmem_get_page()

2015-08-19 Thread Murilo Opsfelder Araujo
ment is of type ‘long unsigned int’ static int xen_tmem_get_page(u32 pool_id, struct tmem_oid oid, ^ Signed-off-by: Murilo Opsfelder Araujo --- drivers/xen/tmem.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/xen/tmem.c b/drivers/xen/tmem.c index b0c895

[PATCH] security/smack/smack_lsm.c: Fix build error when CONFIG_SECURITY_SMACK_BRINGUP is not defined

2015-08-13 Thread Murilo Opsfelder Araujo
mount data". Signed-off-by: Murilo Opsfelder Araujo --- security/smack/smack_lsm.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c index 54fb3a1..6201907 100644 --- a/security/smack/smack_lsm.c +++

[PATCH 1/2] drivers: clk: clk.c: Add comments indicating the matching #ifdef blocks

2015-05-06 Thread Murilo Opsfelder Araujo
Signed-off-by: Murilo Opsfelder Araujo --- drivers/clk/clk.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index 087a092..a961eb6 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -2226,7 +2226,7 @@ static inline void

[PATCH 0/2] Small adjustments in drivers/clk/clk.c

2015-05-06 Thread Murilo Opsfelder Araujo
The first patch just improves code readability by addig comments after #endif statements; the second one gets rid of a warning message during kernel build. Murilo Opsfelder Araujo (2): drivers: clk: clk.c: Add comments indicating the matching #ifdef blocks drivers: clk: clk.c: Make

[PATCH 2/2] drivers: clk: clk.c: Make clk_is_orphan() dependent of CONFIG_OF

2015-05-06 Thread Murilo Opsfelder Araujo
[-Wunused-function] Signed-off-by: Murilo Opsfelder Araujo --- drivers/clk/clk.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index a961eb6..d748aa2 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -2228,14

[PATCH] staging: rtl8192u: r8192U_core: Fix compile issue when CONFIG_WIRELESS_EXT is not defined

2015-01-24 Thread Murilo Opsfelder Araujo
‘wireless_handlers’ dev->wireless_handlers = (struct iw_handler_def *) &r8192_wx_handlers_def; ^ Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/rtl8192u/r8192U_core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/r

[PATCH] gpio: sx150x: Fix compile issue when CONFIG_OF_GPIO is not defined

2015-01-22 Thread Murilo Opsfelder Araujo
s issue was introduced by the commit 04d2264c3bf07f5c3d18165ba78de0a93360c6c0 "gpio: sx150x: add dts support for sx150x driver". Signed-off-by: Murilo Opsfelder Araujo --- drivers/gpio/gpio-sx150x.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpio/gpio-sx150x.c b/d

[PATCH 2/2] staging: olpc_dcon: fix sparse symbol not declared warning

2015-01-15 Thread Murilo Opsfelder Araujo
This patch gets rid of the following sparse warning: drivers/staging/olpc_dcon/olpc_dcon.c:787:19: warning: symbol 'dcon_driver' was not declared. Should it be static? Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/olpc_dcon/olpc_dcon.c | 2 +- 1 file changed, 1 inser

[PATCH 0/2] staging: olpc_dcon: fix sparse warnings and compile errors

2015-01-15 Thread Murilo Opsfelder Araujo
These two patches fix sparse warnings and make olpc_dcon.c build again when CONFIG_OLPC is not set. Murilo Opsfelder Araujo (2): staging: olpc_dcon: check for CONFIG_OLPC before calling olpc_board_at_least() staging: olpc_dcon: fix sparse symbol not declared warning drivers/staging

[PATCH 1/2] staging: olpc_dcon: check for CONFIG_OLPC before calling olpc_board_at_least()

2015-01-15 Thread Murilo Opsfelder Araujo
7; This patch fixes these errors. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/olpc_dcon/olpc_dcon.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c b/drivers/staging/olpc_dcon/olpc_dcon.c index 6a9a881..3708f1e 100644 --- a/drive

Re: next-20141204 crashing on ext4_fill_super()

2014-12-07 Thread Murilo Opsfelder Araujo
On Saturday, December 06, 2014 05:16:21 PM Theodore Ts'o wrote: > Please revert fdfe07398761 and that should the crashes; this will be > fixed in the next linux-next release. My apologies for the inconvenience. > > - Ted Hello, Ted. Rever

next-20141204 crashing on ext4_fill_super()

2014-12-06 Thread Murilo Opsfelder Araujo
Hello, everyone. next-20141204 started crashing on my laptop with the following error right after I enter my disk encryption password: http://opsfelder.com/~murilo/lkml/next-20141204_crash.jpg The previous version next-20141203 was working just fine. next-20141205 is also crashing. Git-bisec

Re: [PATCH] rtlwifi: Add more checks for get_btc_status callback

2014-11-06 Thread Murilo Opsfelder Araujo
On 11/06/2014 09:40 AM, Murilo Opsfelder Araujo wrote: > On 11/05/2014 04:12 PM, Larry Finger wrote: >> On 11/05/2014 03:16 AM, Mike Galbraith wrote: >>> On Wed, 2014-10-29 at 23:30 -0500, Larry Finger wrote: >>>> On 10/29/2014 06:28 PM, Murilo Opsfelder Araujo wrote:

Re: [PATCH] rtlwifi: Add more checks for get_btc_status callback

2014-11-06 Thread Murilo Opsfelder Araujo
On 11/05/2014 04:12 PM, Larry Finger wrote: > On 11/05/2014 03:16 AM, Mike Galbraith wrote: >> On Wed, 2014-10-29 at 23:30 -0500, Larry Finger wrote: >>> On 10/29/2014 06:28 PM, Murilo Opsfelder Araujo wrote: >>>> This is a complement of commit 0805420011

[PATCH] rtlwifi: Add more checks for get_btc_status callback

2014-10-29 Thread Murilo Opsfelder Araujo
This is a complement of commit 08054200117a95afc14c3d2ed3a38bf4e345bf78 "rtlwifi: Add check for get_btc_status callback". With this patch, next-20141029 at least does not panic with rtl8192se device. Signed-off-by: Murilo Opsfelder Araujo --- Hello, everyone. Some days ago, I r

Re: Possible wireless issue introduced in next-20140930

2014-10-23 Thread Murilo Opsfelder Araujo
On 10/23/2014 03:23 AM, Mike Galbraith wrote: On Thu, 2014-10-23 at 01:17 -0200, Murilo Opsfelder Araujo wrote: Hello, everyone. With next-20140930 my laptop does not work, i.e. after I enter my login and password in KDM, the entire system becomes unresponsive and I need to reset it in order

Possible wireless issue introduced in next-20140930

2014-10-22 Thread Murilo Opsfelder Araujo
Hello, everyone. With next-20140930 my laptop does not work, i.e. after I enter my login and password in KDM, the entire system becomes unresponsive and I need to reset it in order to reboot (it does not even show the KDE splash screen). It was working pretty fine with next-20140926. I've a

[PATCH] fs: Initialize mountpoint list head in new_mountpoint()

2014-08-18 Thread Murilo Opsfelder Araujo
vfs: Keep a list of mounts on a mount point To spot any possible problems call BUG if a mountpoint is put when it's list of mounts is not empty. AV: use hlist instead of list_head Signed-off-by: Murilo Opsfelder Araujo --- fs/namespace.c | 1 + 1 file changed, 1 inse

[PATCH] mm: ksm: Remove unused function process_timeout()

2014-08-11 Thread Murilo Opsfelder Araujo
This patch fixes compilation warning: mm/ksm.c:1711:13: warning: ‘process_timeout’ defined but not used [-Wunused-function] Signed-off-by: Murilo Opsfelder Araujo --- mm/ksm.c | 5 - 1 file changed, 5 deletions(-) diff --git a/mm/ksm.c b/mm/ksm.c index f7de4c0..434a50a 100644 --- a/mm

[PATCH v3 1/2] staging: iio: accel: Add blank lines between declarations and code

2014-08-06 Thread Murilo Opsfelder Araujo
This patch adds missing blank lines between declarations and code and fixes lines starting by spaces, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16201_core.c | 5 +++-- drivers/staging/iio/accel/adis16203_core.c | 2 ++ drivers/staging/iio

[PATCH v3 2/2] staging: iio: accel: sca3000_core.c: Adjust code to fit 80-chars limit

2014-08-06 Thread Murilo Opsfelder Araujo
Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/sca3000_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel/sca3000_core.c b/drivers/staging/iio/accel/sca3000_core.c index bc53fedb..e4e5639 100644 --- a/drivers/staging/iio

[PATCH v3 0/2] staging: iio: accel: Multiple coding style fixes

2014-08-06 Thread Murilo Opsfelder Araujo
The following patches fix almost all warnings reported by checkpatch.pl. v2 -> v3 changes: - Folded blank line patches into a single patch. - Fit code in 80-chars limit and still be human-readable. Murilo Opsfelder Araujo (2): staging: iio: accel: Add blank lines between declarations and c

[PATCH v2 0/8] staging: iio: accel: Multiple coding style fixes

2014-08-04 Thread Murilo Opsfelder Araujo
The following patches fix almost all warnings reported by checkpatch.pl. Changes from v1: - Updated commit messages to better reflect changes. Murilo Opsfelder Araujo (8): staging: iio: accel: adis16203_core.c: Add blank lines between declarations and code staging: iio: accel

[PATCH v2 6/8] staging: iio: accel: lis3l02dq_core.c: Add blank lines between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_core.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_core.c b/drivers/staging

[PATCH v2 5/8] staging: iio: accel: adis16240_core.c: Add blank line between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16240_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16240_core.c b/drivers/staging/iio

[PATCH v2 3/8] staging: iio: accel: adis16204_core.c: Add blank line between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16204_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16204_core.c b/drivers/staging/iio

[PATCH v2 4/8] staging: iio: accel: adis16209_core.c: Add blank line between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16209_core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16209_core.c b/drivers/staging/iio

[PATCH v2 2/8] staging: iio: accel: adis16201_core.c: Add blank line between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank line between declarations and code, and also fixes lines starting by space, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16201_core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a

[PATCH v2 8/8] staging: iio: accel: sca3000_core.c: Add blank line between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank line between declarations and code, and ajust code to fit 80-chars limit. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/sca3000_core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel

[PATCH v2 7/8] staging: iio: accel: lis3l02dq_ring.c: Add blank line between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank line between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_ring.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_ring.c b/drivers/staging/iio

[PATCH v2 1/8] staging: iio: accel: adis16203_core.c: Add blank lines between declarations and code

2014-08-04 Thread Murilo Opsfelder Araujo
This patch adds missing blank lines between declarations and code, satisfying checkpatch.pl. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16203_core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/iio/accel/adis16203_core.c b/drivers/staging

[PATCH 1/8] staging: iio: accel: adis16203_core.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank line after declarations. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16203_core.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/iio/accel/adis16203_core.c b/drivers/staging/iio/accel/adis16203_core.c index

[PATCH 2/8] staging: iio: accel: adis16201_core.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank line after declaration and fixes lines starting by space. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16201_core.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/accel/adis16201_core.c b

[PATCH 5/8] staging: iio: accel: adis16240_core.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank line after declaration. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16240_core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16240_core.c b/drivers/staging/iio/accel/adis16240_core.c index

[PATCH 3/8] staging: iio: accel: adis16204_core.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank line after declaration. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16204_core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16204_core.c b/drivers/staging/iio/accel/adis16204_core.c index

[PATCH 8/8] staging: iio: accel: sca3000_core.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank line after declaration and keep line in 80-chars limit. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/sca3000_core.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel/sca3000_core.c b/drivers

[PATCH 7/8] staging: iio: accel: lis3l02dq_ring.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank lines after declarations. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_ring.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_ring.c b/drivers/staging/iio/accel/lis3l02dq_ring.c index

[PATCH 4/8] staging: iio: accel: adis16209_core.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank line after declaration. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/adis16209_core.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16209_core.c b/drivers/staging/iio/accel/adis16209_core.c index

[PATCH 6/8] staging: iio: accel: lis3l02dq_core.c: fix coding style

2014-07-31 Thread Murilo Opsfelder Araujo
This patch adds missing blank line after declarations. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/iio/accel/lis3l02dq_core.c |4 1 file changed, 4 insertions(+) diff --git a/drivers/staging/iio/accel/lis3l02dq_core.c b/drivers/staging/iio/accel/lis3l02dq_core.c index

[PATCH 0/8] staging: iio: accel: multiple coding style fixes

2014-07-31 Thread Murilo Opsfelder Araujo
The following patches fix (almost) all warnings reported by checkpatch.pl. Murilo Opsfelder Araujo (8): staging: iio: accel: adis16203_core.c: fix coding style staging: iio: accel: adis16201_core.c: fix coding style staging: iio: accel: adis16204_core.c: fix coding style staging: iio

Re: [PATCH] Staging: android: sync.c: fix missing blank line after declaration

2014-07-31 Thread Murilo Opsfelder Araujo
On 07/30/2014 09:17 PM, Greg KH wrote: Also doesn't apply, are you sure you are using the staging-next branch of staging.git on git.kernel.org? I was using linux-next, not staging-next. sync.c has no checkpatch.pl warnings in staging-next. Sorry for the false alarm, guys. -- Murilo -- To u

Re: [PATCH] Staging: android: sw_sync.c: fix missing blank line after declaration

2014-07-31 Thread Murilo Opsfelder Araujo
On 07/30/2014 09:17 PM, Greg KH wrote: Does not apply to my tree at all :( I was using linux-next. No checkpatch.pl warnings for sw_sync.c in staging-next. -- Murilo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.

[PATCH v2] Staging: android: timed_gpio.c: improved logic of gpio_get_time()

2014-07-31 Thread Murilo Opsfelder Araujo
This patch improves the logic of gpio_get_time() and, thereafter, makes checkpatch.pl happy. Signed-off-by: Murilo Opsfelder Araujo --- Thanks for reviweing my patch, Dan. I think I got the proper way without using option --scissors in git-am. What about this new commit message? drivers

[PATCH] Staging: android: timed_gpio.c: improved logic of gpio_get_time()

2014-07-30 Thread Murilo Opsfelder Araujo
Thanks for reviewing my first patch, Joe. How about this new one, guys? -- >8 -- Consequently, made checkpatch.pl happy. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/timed_gpio.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --gi

[PATCH] Staging: bcm: Bcmchar.c: remove else statement after return

2014-07-28 Thread Murilo Opsfelder Araujo
This patch makes checkpatch.pl script happier by fixing all warnings related to else statement after break or return. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/bcm/Bcmchar.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers

[PATCH] Staging: android: timed_output.c: use kstrtoint() instead of sscanf()

2014-07-28 Thread Murilo Opsfelder Araujo
This patch makes checkpatch.pl happy by fixing the following warning: WARNING: Prefer kstrto to single variable sscanf Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/timed_output.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH] Staging: android: timed_gpio.c: remove else statement after return

2014-07-28 Thread Murilo Opsfelder Araujo
This patch makes checkpatch.pl script happy by fixing the following warning: WARNING: else is not generally useful after a break or return Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/timed_gpio.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a

[PATCH] Staging: android: sync.c: fix missing blank line after declaration

2014-07-28 Thread Murilo Opsfelder Araujo
Fix coding style issue. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/sync.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index e7b2e02..69c16cb 100644 --- a/drivers/staging/android

[PATCH] Staging: android: sw_sync.c: fix missing blank line after declaration

2014-07-28 Thread Murilo Opsfelder Araujo
Fix coding style issue. Signed-off-by: Murilo Opsfelder Araujo --- drivers/staging/android/sw_sync.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/android/sw_sync.c b/drivers/staging/android/sw_sync.c index a76db3f..863d4b1 100644 --- a/drivers/staging/android

[PATCH v2] trivial: fix a typo in Documentation/00-INDEX

2013-08-20 Thread Murilo Opsfelder Araujo
Signed-off-by: Murilo Opsfelder Araujo --- Documentation/00-INDEX | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/00-INDEX b/Documentation/00-INDEX index 0c4cc68..38f8444 100644 --- a/Documentation/00-INDEX +++ b/Documentation/00-INDEX @@ -40,7 +40,7 @@ IPMI.txt

[PATCH] trivial: fix a typo in Documentation/00-INDEX

2013-08-19 Thread Murilo Opsfelder Araujo
--- Documentation/00-INDEX | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/00-INDEX b/Documentation/00-INDEX index 0c4cc68..38f8444 100644 --- a/Documentation/00-INDEX +++ b/Documentation/00-INDEX @@ -40,7 +40,7 @@ IPMI.txt IRQ-affinity.txt - how to selec