- 01 Sep, 2016 40 commits
-
-
Stuart Yoder authored
Instead of relying on mc-sys.h to include misc system headers, have the source files that need them explicitly do it themselves. Signed-off-by:
Stuart Yoder <stuart.yoder@nxp.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Stuart Yoder authored
An mc-io is a software abstraction of a dpmcp object (an MC portal). Routines to create/initialize/reset this portal abstraction were split between the allocator and mc-sys.c (MC command interface). Move mc-io/portal related definitions into one source file. Signed-off-by:
-
Stuart Yoder authored
Over time we've accumulated some includes that are no longer needed. Remove them. Signed-off-by:
-
Stuart Yoder authored
move public bus-related definitions to the proper mc-bus.h header. Signed-off-by:
-
Stuart Yoder authored
Remove unneeded and duplicated forward declarations. Signed-off-by:
-
Stuart Yoder authored
fsl_mc_get_root_dprc is needed by other components (e.g. vfio) to find the root dprc Signed-off-by:
-
Stuart Yoder authored
Move definitions that are private to fsl-mc-bus.c out of the public mc-bus.h and mc.h headers. Signed-off-by:
-
Stuart Yoder authored
Move definitions that are private to dprc-driver.c out of the public mc-bus.h header. Signed-off-by:
-
Stuart Yoder authored
The mc-bus.h header file is intended to be public for system related drivers that may need to make fsl-mc bus specific calls-- gic its, smmu, vfio. But, currently it contains definitions private/internal to the fsl-mc bus itself. Split those definitions out into a new fsl-mc bus private header. Signed-off-by:
-
Stuart Yoder authored
The resource pool init/cleanup functions logically belong in the allocator. Move them to the allocator and rename to reflect the move out of the dprc-driver. Signed-off-by:
-
Stuart Yoder authored
move definitions private to the allocator out of mc-bus.h and into allocator.c Signed-off-by:
-
Stuart Yoder authored
The mc- prefix on the source files names of some internal components of the fsl-mc bus driver makes things less clear that they could be. The string "mc" generally refers to hardware-- the physical DPAA 'management complex'. Names like "mc-allocator.c" have nothing to with the "MC" hardware per se. Improve clarity by using the prefix string "fsl-mc" instead which is what we are calling this bus. Signed-off-by:
-
Stuart Yoder authored
The definitions in mc-private.h are not strictly speaking 'private', they are bus-related definitions expected to be referenced by bus driver components as well as system components (gic, smmu, vfio). Rename mc-private.h to mc-bus.h to more accurately reflect expected use. Signed-off-by:
-
Wei Yongjun authored
Add the missing unlock before return from function rtw_resume_process() in the error handling case. Signed-off-by:
Wei Yongjun <weiyongjun1@huawei.com> Signed-off-by:
-
Anson Jacob authored
Fix checkpatch.pl warning Block comments use * on subsequent lines Signed-off-by:
Anson Jacob <ansonjacob.aj@gmail.com> Signed-off-by:
-
Julia Lawall authored
Check for ethtool_ops structures that are only stored in the ethtool_ops field of a net_device structure or passed as the second argument to netdev_set_default_ethtool_ops. These contexts are declared const, so ethtool_ops structures that have these properties can be declared as const also. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r disable optional_qualifier@ identifier i; position p; @@ static struct ethtool_ops i@p = { ... }; @ok1@ identifier r.i; struct net_device e; position p; @@ e.ethtool_ops = &i@p; @ok2@ identifier r.i; expression e; position p; @@ netdev_set_default_ethtool_ops(e, &i@p) @bad@ position p != {r.p,ok1.p,ok2.p}; identifier r.i; @@ i@p @depends on !bad disable optional_qualifier@ identifier r.i; @@ static +const struct ethtool_ops i = { ... }; // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Claudiu Beznea authored
This patch removes multiple switch-case statements with a new macro. The macro will generate the corresponding bit mask based on the key index received as input. Chances since v1: Corrected patch title Signed-off-by:
Claudiu Beznea <claudiu.beznea@gmail.com> Signed-off-by:
-
Claudiu Beznea authored
This patch removes duplicate code in p80211req_mibset_mibget() by adding p80211req_handle_action() function. Changes since v1: Removed prototype of p80211req_handle_action() from the prototype list and add directly the function definition. Signed-off-by:
-
Claudiu Beznea authored
This patch improves code from p80211req_mibset_mibget() function by taking into account that every DIDmib_dot11smt_dot11WEPDefaultKeysTable_dot11WEPDefaultKeyX with X in {0, 1, 2, 3} is a bit mask where the (P80211DID_MASK_ITEM << P80211DID_LSB_ITEM) mask aka 0x0x0003f000 mask keeps bits representing the DID item number. To get this item number use P80211DID_ITEM(). Signed-off-by: -
Colin Ian King authored
Trivial fix to spelling mistake in dev_dbg message. Signed-off-by:
Colin Ian King <colin.king@canonical.com> Signed-off-by:
-
Anson Jacob authored
Fix checkpatch.pl warning for trailing */ on a separate line Remove '+' postfix and '-' prefix from the start and end of block comments Signed-off-by:
-
Anson Jacob authored
Fix checkpatch.pl warning for trailing */ on a separate line Remove '+' postfix and '-' prefix from the start and end of block comments Signed-off-by:
-
MingChia Chung authored
This patch fixes a minor checkpatch warnings: "WARNING: Block comments use a trailing */ on a separate line" Signed-off-by:
Ming-Chia Chung <Quexint@gmail.com> Signed-off-by:
-
Matthias Beyer authored
This patch removes the pointless `else if` test. Signed-off-by:
Matthias Beyer <mail@beyermatthias.de> Reported-by:
David Binderman <linuxdev.baldrick@gmail.com> Signed-off-by:
-
Binoy Jayan authored
The semaphore 'lock' in 'pwrctrl_priv' is used as a simple mutex, so it should be written as one. Semaphores are going away in the future. _enter_pwrlock was using down_interruptible(), so the lock could be broken by sending a signal. This could be a bug, because nothing checks the return code here. Hence, using mutex_lock instead of the interruptible version. Removing the now unused _enter_pwrlock and _down_sema. Signed-off-by:
Binoy Jayan <binoy.jayan@linaro.org> Reviewed-by:
Arnd Bergmann <arnd@arndb.de> Tested-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Didik Setiawan authored
Fix checkpatch.pl warning about "Alignment should match open parenthesis". Signed-off-by:
Didik Setiawan <ds@didiksetiawan.com> Signed-off-by:
-
Anson Jacob authored
Fix checkpatch.pl 'line over 80 characters' warning Signed-off-by:
-
Johanna Abrahamsson authored
Remove the function ion_handle_buffer since it is not used anywhere. Signed-off-by:
Johanna Abrahamsson <johanna@mjao.org> Acked-by:
Laura Abbott <labbott@redhat.com> Signed-off-by:
-
Johanna Abrahamsson authored
It is not necessary to save the return value of kref_put since it is directly returned. Signed-off-by:
-
Ben LeMasurier authored
This fixes remaining checkpatch.pl "Alignment should match open parenthesis" issues. Signed-off-by:
Ben LeMasurier <ben@crypt.ly> Signed-off-by:
-
Johanna Abrahamsson authored
Alignment should match open parenthesis as per checkpatch.pl. Signed-off-by:
-
Aaro Koskinen authored
Configure rx-delay/tx-delay when available. Signed-off-by:
Aaro Koskinen <aaro.koskinen@iki.fi> Signed-off-by:
-
Johanna Abrahamsson authored
It is not neccessary to save the value of ion_handle_validate since it is only used once. Signed-off-by:
-
Johanna Abrahamsson authored
The ion_free_nolock() function should not BUG on a handle client mismatch. Signed-off-by:
-
Colin Ian King authored
trivial fix to spelling mistake in dev_err message Signed-off-by:
-
Julia Lawall authored
Check for ethtool_ops structures that are only stored in the ethtool_ops field of a net_device structure or passed as the second argument to netdev_set_default_ethtool_ops. These contexts are declared const, so ethtool_ops structures that have these properties can be declared as const also. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r disable optional_qualifier@ identifier i; position p; @@ static struct ethtool_ops i@p = { ... }; @ok1@ identifier r.i; struct net_device e; position p; @@ e.ethtool_ops = &i@p; @ok2@ identifier r.i; expression e; position p; @@ netdev_set_default_ethtool_ops(e, &i@p) @bad@ position p != {r.p,ok1.p,ok2.p}; identifier r.i; @@ i@p @depends on !bad disable optional_qualifier@ identifier r.i; @@ static +const struct ethtool_ops i = { ... }; // </smpl> Suggested-by:
Stephen Hemminger <stephen@networkplumber.org> Signed-off-by:
-
Oleg Drokin authored
%ul definitely was supposed to be %lu in the format string, so we print long unsigned int value, not just unsigned int with a letter l added at the end. Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
-
Julia Lawall authored
lmv_proc_target_fops, of type struct file_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by:
-
Julia Lawall authored
sops, of type struct seq_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by:
-
Julia Lawall authored
lprocfs_generic_fops, of type struct file_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by:
-
