Checkpatch emits WARNING: Avoid multiple line dereference. It is
possible to reduce the level of indentation by inverting a conditional
and continuing loop if new conditional evaluates to true.
Invert conditional. Continue loop if new conditional evaluates to
true. Reduce subsequent code indentati
On Sun, 26 Feb 2017, Scott Branden wrote:
>
>
> On 17-02-26 01:56 PM, Joe Perches wrote:
> > On Sun, 2017-02-26 at 20:40 +0100, Julia Lawall wrote:
> > > On Sun, 26 Feb 2017, Joe Perches wrote:
> > > > On Sun, 2017-02-26 at 19:59 +0100, Julia Lawall wrote:
> > > > > On Sun, 26 Feb 2017, Joe Perc
On Sun, Feb 26, 2017 at 06:00:19PM -0800, Matthew Giassa wrote:
> ---
> drivers/staging/ks7010/ks_wlan_ioctl.h | 64
> +-
> 1 file changed, 32 insertions(+), 32 deletions(-)
>
> diff --git a/drivers/staging/ks7010/ks_wlan_ioctl.h
> b/drivers/staging/ks7010/ks_wla
This patch adds compatibility wrappers for the ioctls
exposed by vchiq/vc04_services. The compat ioctls are
completely implemented on top of the native ioctls. No
existing lines are modified.
While the ideal approach would be to cleanup the existing
code, this path is simplier and easier to revi
Function call parameters are split over more lines than
necessary. Also assignment statements are split after the '=' sign.
This adds extra lines to the function and may also reduces
readability.
Refactor function call parameters and reduce the number of lines
used. Put assignment statements onto
Switch statement uses one [trivial] case and a default case holding
bulk of code. We can swap the switch statement with an if/return
statement as replacement for the original switch. This can be done
without changing the program logic.
Remove switch statement. Use original switch parameter as cond
Logic operator (&&) is place at the start of the line. Kernel
standards suggest that logical operators should be placed at the end
of the line.
Move logical operator to the end of the previous line.
Signed-off-by: Tobin C. Harding
---
drivers/staging/ks7010/ks7010_sdio.c | 6 +++---
1 file chan
Function _ks_wlan_hw_power_save has 5 levels of indentation. Reducing
the amount of indentation may make code clearer to read. Also a number
of other checkpatch warnings may be removed if we first reduce the
level of indentation in this function.
There are a couple of split line warnings and vario
Checkpatch emits WARNING: Avoid multiple line dereference.
Move dereference onto single line.
Signed-off-by: Tobin C. Harding
---
drivers/staging/ks7010/ks7010_sdio.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/ks7010/ks7010_sdio.c
b/drivers/stagin
Function _ks_wlan_hw_power_save has 5 levels of indentation. Reducing
the amount of indentation may make code clearer to read. Also a number
of other checkpatch warnings may be removed if we first reduce the
level of indentation in this function.
Invert conditional and return from function if new
Assembling a set of patches to address all outstanding checkpatch issues in
staging/drivers/ks7010/ks_wlan_ioctl.h to replace my earlier v2 work which
attempted to do all of this in a single patch. This includes wrapping macros
with complex values in parentheses, removing extraneous/mixed whitespac
---
drivers/staging/ks7010/ks_wlan_ioctl.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ks7010/ks_wlan_ioctl.h
b/drivers/staging/ks7010/ks_wlan_ioctl.h
index 9f71f01..adc935d 100644
--- a/drivers/staging/ks7010/ks_wlan_ioctl.h
+++ b/drivers/staging/ks7010/ks
---
drivers/staging/ks7010/ks_wlan_ioctl.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ks7010/ks_wlan_ioctl.h
b/drivers/staging/ks7010/ks_wlan_ioctl.h
index 47c8015..0aa402d 100644
--- a/drivers/staging/ks7010/ks_wlan_ioctl.h
+++ b/drivers/staging/ks7010/ks
---
drivers/staging/ks7010/ks_wlan_ioctl.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/ks7010/ks_wlan_ioctl.h
b/drivers/staging/ks7010/ks_wlan_ioctl.h
index 0aa402d..9f71f01 100644
--- a/drivers/staging/ks7010/ks_wlan_ioctl.h
+++ b/drivers/staging/ks701
---
drivers/staging/ks7010/ks_wlan_ioctl.h | 64 +-
1 file changed, 32 insertions(+), 32 deletions(-)
diff --git a/drivers/staging/ks7010/ks_wlan_ioctl.h
b/drivers/staging/ks7010/ks_wlan_ioctl.h
index 8e62b10..47c8015 100644
--- a/drivers/staging/ks7010/ks_wlan_io
The typedef lnet_md_iovec_t is never used so kill it
off.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lust
Change lnet_process_id_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lus
Change lnet_event_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/i
Change lnet_counter_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre
Change lnet_ins_pos_t from typedef to proper enum.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/incl
Kill off lnet_kiov_t and use struct bio_vec directly.
Signed-off-by: James Simmons
---
drivers/staging/lustre/include/linux/lnet/lib-lnet.h| 8
drivers/staging/lustre/include/linux/lnet/lib-types.h | 6 +++---
drivers/staging/lustre/include/linux/lnet/types.h | 7 +++
Change lnet_md_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/incl
Change lnet_ack_req_t from typedef to proper enum.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/incl
Change lnet_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/include
Change lnet_msg_type_t from typedef to proper enum.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/inc
The LNET_MATCH* flags are an enum without a name.
Lets label that enum.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
driver
The lnet_seq_t is a simple unsigned long so lets
simplify it.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/
Change lnet_ni_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
.../staging/lustre/include/
Change lnet_remotenet_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
.../staging/lustre/i
Change lnet_event_kind_t from typedef to proper enum.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/i
Change lnet_unlink_t from typedef to proper enum.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/inclu
Change lnet_peer_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
.../staging/lustre/includ
Change lnet_route_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
.../staging/lustre/inclu
Change lnet_rtrbuf_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/
Change lnet_rtrbufpool_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lus
Change lnet_portal_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/
Change lnet_libmd_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
.../staging/lustre/inclu
Change lnet_me_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/incl
Change lnet_eq_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/incl
Change lnet_msg_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
.../staging/lustre/include
Change lnet_rc_data_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre
Change lnet_handle_me_t from a typedef of another typedef into
a proper stand alone structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: Jam
Change lnet_test_peer_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lust
With the creation of unique handlers we can remove the
barely used generic handlers.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
Change lnd_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lustre/include/
Change lnet_libhandle_t from typedef to proper structure.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: Olaf Weber
Reviewed-by: Doug Oucharek
Reviewed-by: Dmitry Eremin
Reviewed-by: Oleg Drokin
Signed-off-by: James Simmons
---
drivers/staging/lust
Change lnet_handle_md_t from a typedef of another typedef into
a proper stand alone structure. Create the inline functions
LNetInvalidateMDHandle and LNetMDHandleIsInvalid to handle this
new piece of data.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: O
Remove the majority of typedefs from the LNet headers.
Change them into structures or enums instead. Currently
only lnet_nid_t are lnet_pid_t are left. Removed the
rest of the typedefs in socklnd.h as well
James Simmons (34):
staging: lustre: socklnd: change UAPI typedefs to proper structure
s
The upstream kernel requires proper structures so
convert all the UAPI typedefs in socklnd.h.
Signed-off-by: James Simmons
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142
Reviewed-on: https://review.whamcloud.com/18506
Reviewed-by: Dmitry Eremin
Reviewed-by: Doug Oucharek
Reviewed-by:
Change lnet_handle_eq_t from a typedef of another typedef into
a proper stand alone structure. Create the inline functions
LNetInvalidateEQHandle and LNetEQHandleIsInvalid to handle this
new piece of data.
Signed-off-by: James Simmons
Reviewed-on: https://review.whamcloud.com/20831
Reviewed-by: O
On 17-02-26 01:56 PM, Joe Perches wrote:
On Sun, 2017-02-26 at 20:40 +0100, Julia Lawall wrote:
On Sun, 26 Feb 2017, Joe Perches wrote:
On Sun, 2017-02-26 at 19:59 +0100, Julia Lawall wrote:
On Sun, 26 Feb 2017, Joe Perches wrot
Mailing lists _should_ be copied on patch submissions.
The id
On Sun, 2017-02-26 at 20:40 +0100, Julia Lawall wrote:
> On Sun, 26 Feb 2017, Joe Perches wrote:
> > On Sun, 2017-02-26 at 19:59 +0100, Julia Lawall wrote:
> > > On Sun, 26 Feb 2017, Joe Perches wrot
> > > > Mailing lists _should_ be copied on patch submissions.
> > > The idea of outreachy is that
On Sun, 2017-02-26 at 20:36 +0100, Noralf Trønnes wrote:
> Den 26.02.2017 19.11, skrev Stefan Wahren:
> > > Eric Anholt hat am 26. Februar 2017 um 18:16
> > > geschrieben:
> > >
> > > ...
> > >
> > > For staging, Greg has been taking patches without platform
> > > maintainer
> > > ack. I think
On Sun, 26 Feb 2017, Joe Perches wrote:
> On Sun, 2017-02-26 at 19:59 +0100, Julia Lawall wrote:
> > On Sun, 26 Feb 2017, Joe Perches wrote:
> > > Mailing lists _should_ be copied on patch submissions.
> >
> > The idea of outreachy is that it should be a place for people to get
> > started witho
On Sun, 2017-02-26 at 19:59 +0100, Julia Lawall wrote:
> On Sun, 26 Feb 2017, Joe Perches wrote:
> > Mailing lists _should_ be copied on patch submissions.
>
> The idea of outreachy is that it should be a place for people to get
> started without too much criticism from the outside. Originally, o
Den 26.02.2017 19.11, skrev Stefan Wahren:
Eric Anholt hat am 26. Februar 2017 um 18:16 geschrieben:
...
For staging, Greg has been taking patches without platform maintainer
ack. I think this is great -- the staging code needs *lots* of work,
and it generally doesn't need any platform knowl
On Sun, 26 Feb 2017, Joe Perches wrote:
> On Sun, 2017-02-26 at 19:03 +0100, Julia Lawall wrote:
> >
> > On Sun, 26 Feb 2017, Narcisa Ana Maria Vasile wrote:
> >
> > > This was reported by checkpatch.pl
> > >
> > > Signed-off-by: Narcisa Ana Maria Vasile
> >
> > Acked-by: Julia Lawall
> >
> >
On Sun, 2017-02-26 at 19:03 +0100, Julia Lawall wrote:
>
> On Sun, 26 Feb 2017, Narcisa Ana Maria Vasile wrote:
>
> > This was reported by checkpatch.pl
> >
> > Signed-off-by: Narcisa Ana Maria Vasile
>
> Acked-by: Julia Lawall
>
> Actually, the outreachy tutorial says not to send patches to
> Eric Anholt hat am 26. Februar 2017 um 18:16 geschrieben:
>
> ...
>
> For staging, Greg has been taking patches without platform maintainer
> ack. I think this is great -- the staging code needs *lots* of work,
> and it generally doesn't need any platform knowledge.
>
> As far as this patch
On Sun, 26 Feb 2017, Narcisa Ana Maria Vasile wrote:
> This was reported by checkpatch.pl
>
> Signed-off-by: Narcisa Ana Maria Vasile
Acked-by: Julia Lawall
Actually, the outreachy tutorial says not to send patches to mailing
lists, only to people (except in the case of IIO, where the mainta
This was reported by checkpatch.pl
Signed-off-by: Narcisa Ana Maria Vasile
---
drivers/staging/bcm2835-audio/bcm2835-pcm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/bcm2835-audio/bcm2835-pcm.c
b/drivers/staging/bcm2835-audio/bcm2835-pcm.c
index 9e79236.
On Sun, 26 Feb 2017, Arushi Singhal wrote:
> Unnecessary parentheses are removed as reported by checkpatch.pl
> to make coder nicer and to improve readability.
> Also coding style is improved.For example:-
> It's often nicer to read if &(foo[0]) is converted to foo like:
> memcpy(&(ap->bssid[
On Sun, 26 Feb 2017, Arushi Singhal wrote:
> Unnecessary parentheses are removed as reported by checkpatch.pl
> to make coder nicer and to improve readability.
> Also coding style is improved.For example:-
> It's often nicer to read if &(foo[0]) is converted to foo like:
> memcpy(&(ap->bssid[
Unnecessary parentheses are removed as reported by checkpatch.pl
to make coder nicer and to improve readability.
Also coding style is improved.For example:-
It's often nicer to read if &(foo[0]) is converted to foo like:
memcpy(&(ap->bssid[0]), &(ap_info->bssid[0]), ETH_ALEN);
memcpy(ap->
Unnecessary parentheses are removed as reported by checkpatch.pl
to make coder nicer and to improve readability.
Also coding style is improved.For example:-
It's often nicer to read if &(foo[0]) is converted to foo like:
memcpy(&(ap->bssid[0]), &(ap_info->bssid[0]), ETH_ALEN);
memcpy(ap->
Stefan Wahren writes:
> [add Eric]
>
>> Michael Zoran hat am 18. Februar 2017 um 12:59
>> geschrieben:
>>
>>
>> On Sat, 2017-02-18 at 03:22 -0800, Michael Zoran wrote:
>> > The original github source allowed for the cache-line-size property
>> > to be missing. Since recent firmwares also req
On Sun, 2017-02-26 at 21:23 +0530, Arushi Singhal wrote:
> Error was reported by checkpatch.pl as "Avoid multiple line
> dereference".And If there is boolean operator then it is
> fixed by Splitting line at boolean operator to satisfy coding
> style.
[]
> diff --git a/drivers/staging/xgifb/XGI_ma
On Sun, 26 Feb 2017, Arushi Singhal wrote:
> Error was reported by checkpatch.pl as "Avoid multiple line
> dereference".And If there is boolean operator then it is
> fixed by Splitting line at boolean operator to satisfy coding
> style.
Please improve the commit message. Capital letters should o
Error was reported by checkpatch.pl as "Avoid multiple line
dereference".And If there is boolean operator then it is
fixed by Splitting line at boolean operator to satisfy coding
style.
Signed-off-by: Arushi Singhal
---
changes in v4
- changes done such that no other errors can generate.
- Im
Commenting Code Is a Bad Idea.
Comments are their to explain the code and how the code achieves its
goal and as codes in the comments does not explain what the code is
doing so there is no use of commenting them.
So in this patch codes in the comments are removed.
Signed-off-by: Arushi Singhal
-
On Sun, Feb 26, 2017 at 03:44:42PM +0100, Noralf Trønnes wrote:
> Due to personal reasons I'm unable to continue as fbtft maintainer.
>
> Signed-off-by: Noralf Trønnes
> ---
>
> For a long time I have wanted to convert the fbtft drivers to drm.
> There's now a foundation in place to do so in dri
Error was reported by checkpatch.pl as "Avoid multiple line
dereference".And If there is boolean operator then it is
fixed by Splitting line at boolean operator to satisfy coding
style.
Signed-off-by: Arushi Singhal
---
changes in v3
- changes done such that no other errors can generate.
-
Hello,
On Sun, 26 Feb 2017 15:44:42 +0100, Noralf Trønnes wrote:
> Due to personal reasons I'm unable to continue as fbtft maintainer.
>
> Signed-off-by: Noralf Trønnes
Sad, but I understand. That leaves just me as the "maintainer", but
I've never done anything special on fbtft. My aim with a s
Due to personal reasons I'm unable to continue as fbtft maintainer.
Signed-off-by: Noralf Trønnes
---
For a long time I have wanted to convert the fbtft drivers to drm.
There's now a foundation in place to do so in drivers/gpu/drm/tinydrm.
Because of a worsening fatigue illness, I'm unable to t
Fix the following checkpatch.pl warning:
Avoid multiple line dereference - prefer
'XGI330_RefIndex[RefreshRateTableIndex+(*i)].Ext_InfoFlag'
Signed-off-by: Manoj Sawai
---
drivers/staging/xgifb/vb_setmode.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/stagin
Fix the following checkpatch.pl warnings:
1: Avoid multiple line dereference - prefer
'XGI330_RefIndex[RefreshRateTableIndex+i].ModeID'
2: Avoid multiple line dereference - prefer
'XGI330_RefIndex[RefreshRateTableIndex+(*i)].Ext_InfoFlag'
Signed-off-by: Manoj Sawai
---
drivers/staging/xg
76 matches
Mail list logo
