[PATCH] net/mlx5e: Fix trailing semicolon

2018-01-17 Thread Luis de Bethencourt
The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything. Signed-off-by: Luis de Bethencourt <lui...@kernel.org> --- Hi, After fixing the same thing in drivers/staging/rtl8723bs/, Joe Perches suggested I fix it treewide [0]. Best r

[PATCH] rtl8xxxu: Fix trailing semicolon

2018-01-17 Thread Luis de Bethencourt
The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything. Signed-off-by: Luis de Bethencourt <lui...@kernel.org> --- Hi, After fixing the same thing in drivers/staging/rtl8723bs/, Joe Perches suggested I fix it treewide [0]. Best r

[PATCH] vxlan: Fix trailing semicolon

2018-01-16 Thread Luis de Bethencourt
The trailing semicolon is an empty statement that does no operation. It is completely stripped out by the compiler. Removing it since it doesn't do anything. Signed-off-by: Luis de Bethencourt <lui...@kernel.org> --- Hi, An other trailing semicolon to remove. Best regards Luis [0]

[PATCH] netfilter: nf_tables: Fix trailing semicolon

2018-01-16 Thread Luis de Bethencourt
The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything. Signed-off-by: Luis de Bethencourt <lui...@kernel.org> --- Hi, After fixing the same thing in drivers/staging/rtl8723bs/, Joe Perches suggested I fix it treewide [0]. Best r

[PATCH] samples/bpf: Fix trailing semicolon

2018-01-16 Thread Luis de Bethencourt
The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything. Signed-off-by: Luis de Bethencourt <lui...@kernel.org> --- Hi, After fixing the same thing in drivers/staging/rtl8723bs/, Joe Perches suggested I fix it treewide [0]. Best r

Re: [PATCH 13/36] befs: Define usercopy region in befs_inode_cache slab cache

2018-01-10 Thread Luis de Bethencourt
am's PAX_USERCOPY > whitelisting code in the last public patch of grsecurity/PaX based on my > understanding of the code. Changes or omissions from the original code are > mine and don't reflect the original grsecurity/PaX code. > > Signed-off-by: David Windsor <d...@nullcore.net>

Re: [PATCH v3 10/31] befs: Define usercopy region in befs_inode_cache slab cache

2017-09-21 Thread Luis de Bethencourt
off-by: David Windsor <d...@nullcore.net> [kees: adjust commit log, provide usage trace] Cc: Luis de Bethencourt <lui...@kernel.org> Cc: Salah Triki <salah.tr...@gmail.com> Signed-off-by: Kees Cook <keesc...@chromium.org> Acked-by: Luis de Bethencourt <lui...@kernel.org>

[PATCH] rtl8xxxu: remove unneeded assignments

2016-06-10 Thread Luis de Bethencourt
reg_eac and reg_ecc are only used if candidate is bigger than 0, and in that case new values will be given to them. Removing the unused assignments. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c | 2 -- 1 file chan

Re: [PATCH] net: Add missing kernel-doc for netdev ptype lists

2016-03-21 Thread Luis de Bethencourt
On 21/03/16 21:08, Benjamin Poirier wrote: > .//include/linux/netdevice.h:1826: warning: No description found for > parameter 'ptype_all' > .//include/linux/netdevice.h:1826: warning: No description found for > parameter 'ptype_specific' > > Introduced by commit 7866a621043f ("dev: add per

Re: [PATCH] net: add missing descriptions in net_device_priv_flags

2016-03-21 Thread Luis de Bethencourt
On 21/03/16 20:41, Benjamin Poirier wrote: > On 2016/03/21 20:20, Luis de Bethencourt wrote: >> The flags IFF_XMIT_DST_RELEASE_PERM, IFF_IPVLAN_MASTER and >> IFF_IPVLAN_SLAVE are missing descriptions for the Documentation. Adding >> them. >> >> Signed

[PATCH v2] net: add missing descriptions in net_device_priv_flags

2016-03-21 Thread Luis de Bethencourt
The flags IFF_XMIT_DST_RELEASE_PERM, IFF_IPVLAN_MASTER and IFF_IPVLAN_SLAVE are missing descriptions for the Documentation. Adding them. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> Suggested-by: Benjamin Poirier <benjamin.poir...@gmail.com> --- Hi, Changing the

[PATCH] net: add missing descriptions in net_device_priv_flags

2016-03-21 Thread Luis de Bethencourt
The flags IFF_XMIT_DST_RELEASE_PERM, IFF_IPVLAN_MASTER and IFF_IPVLAN_SLAVE are missing descriptions for the Documentation. Adding them. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- Hi, I also noticed this issue when running make htmldocs. Having better documentation is

Re: [PATCH] ipv6: remove unused in6_addr struct

2016-03-21 Thread Luis de Bethencourt
On 21/03/16 18:13, David Miller wrote: > From: Luis de Bethencourt <lui...@osg.samsung.com> > Date: Mon, 21 Mar 2016 17:37:28 + > >> struct in6_addr isn't used anymore in inet6_connection_sock.h, removing >> the forward declaration. >> >> Fixes: 1b3

[PATCH] ipv6: remove unused in6_addr struct

2016-03-21 Thread Luis de Bethencourt
struct in6_addr isn't used anymore in inet6_connection_sock.h, removing the forward declaration. Fixes: 1b33bc3e9e90 ("ipv6: remove obsolete inet6 functions") Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- include/net/inet6_connection_sock.h | 1 - 1 file cha

[PATCH] net: add description for len argument of dev_get_phys_port_name

2016-03-21 Thread Luis de Bethencourt
When the function dev_get_phys_port_name was added it missed a description for it's len argument. Adding it. Fixes: db24a9044ee1 ("net: add support for phys_port_name") Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- Hi, Noticed this when running make ht

[PATCH] mac80211: add doc for RX_FLAG_DUP_VALIDATED flag

2016-03-20 Thread Luis de Bethencourt
Add documentation for the flag for duplication check. Fixes the following warning when running make htmldocs: warning: Enum value 'RX_FLAG_DUP_VALIDATED' not described in enum 'mac80211_rx_flags' Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- Hi, While running make html

Re: [PATCH] mac80211: fix order of flag descriptions

2016-03-19 Thread Luis de Bethencourt
On 18/03/16 16:49, Johannes Berg wrote: > On Fri, 2016-03-18 at 16:35 +0000, Luis de Bethencourt wrote: >> Fix order of mac80211_rx_flags description to match the enum. >> >> Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> >> --- >> Hi,

[PATCH] mac80211: fix order of flag descriptions

2016-03-19 Thread Luis de Bethencourt
Fix order of mac80211_rx_flags description to match the enum. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- Hi, I want ahead and fixed the order of the descriptions. checkpatch.pl was giving a warning to my previous patch and I had a hunch it was because the wrong order

Re: [PATCH] mac80211: add doc for RX_FLAG_DUP_VALIDATED flag

2016-03-19 Thread Luis de Bethencourt
On 18/03/16 16:09, Luis de Bethencourt wrote: > Add documentation for the flag for duplication check. > > Fixes the following warning when running make htmldocs: > warning: Enum value 'RX_FLAG_DUP_VALIDATED' not described in enum > 'mac80211_rx_flags' > > Signed-off-by: Lui

[PATCH] mac80211: remove description of dropped member

2016-03-19 Thread Luis de Bethencourt
Commit 976bd9efdae6 ("mac80211: move beacon_loss_count into ifmgd") removed the member from the sta_info struct but the description stayed lingering. Remove it. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- Hi, A second patch fixing a warning in make htmldocs.

Re: [PATCH] mac80211: fix order of flag descriptions

2016-03-19 Thread Luis de Bethencourt
On 18/03/16 17:46, Joe Perches wrote: > On Fri, 2016-03-18 at 17:40 +0000, Luis de Bethencourt wrote: >> On 18/03/16 17:29, Joe Perches wrote: >>> On Fri, 2016-03-18 at 16:35 +, Luis de Bethencourt wrote: >>>> Fix order of mac80211_rx_flags description to match th

[PATCH] net: sched: Add description for cpu_bstats argument

2016-03-19 Thread Luis de Bethencourt
Commit 22e0f8b9322c ("net: sched: make bstats per cpu and estimator RCU safe") added the argument cpu_bstats to functions gen_new_estimator and gen_replace_estimator and now the descriptions of these are missing for the documentation. Adding them. Signed-off-by: Luis de Bethen

[PATCH] gen_stats.c: Add description for cpu argument

2016-03-19 Thread Luis de Bethencourt
Function gnet_stats_copy_basic is missing the description of the cpu argument in the documentation. Adding it. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- Hi, Noticed this issue when running make htmldocs. I got the following warning: .//net/core/gen_stats.c:155: w

Re: [PATCH] mac80211: fix order of flag descriptions

2016-03-19 Thread Luis de Bethencourt
On 18/03/16 17:29, Joe Perches wrote: > On Fri, 2016-03-18 at 16:35 +0000, Luis de Bethencourt wrote: >> Fix order of mac80211_rx_flags description to match the enum. >> >> Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> >> --- >> Hi,

Re: [PATCH] mac80211: fix order of flag descriptions

2016-03-18 Thread Luis de Bethencourt
On 18/03/16 17:46, Luis de Bethencourt wrote: > On 18/03/16 16:49, Johannes Berg wrote: >> On Fri, 2016-03-18 at 16:35 +0000, Luis de Bethencourt wrote: >>> Fix order of mac80211_rx_flags description to match the enum. >>> >>> Signed-off-by: Luis de

[PATCH] ssb: extif: Compile with COMPILE_TEST

2015-10-14 Thread Luis de Bethencourt
, when building with allyesconfig. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/ssb/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/ssb/Kconfig b/drivers/ssb/Kconfig index f0d22cd..1819a54 100644 --- a/drivers/ssb/Kconfig +++ b/drive

Re: [PATCH] net: gianfar_ptp: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
Please ignore this. It has the wrong [PATCH] tag. Apologies, Luis On Fri, Sep 18, 2015 at 09:52:44PM +0200, Luis de Bethencourt wrote: > This platform driver has a OF device ID table but the OF module > alias information is not created so module autoloading won't work. > > Signed-

[PATCH] net: gianfar_ptp: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- Corrected the subject line as requested by Richard. https://lkml.org/lkml/2015/9/18/717

[PATCH 0/2] net: phy: Fix module autoload for OF platform drivers

2015-09-18 Thread Luis de Bethencourt
should've been part of the ethernet series: https://lkml.org/lkml/2015/9/18/567 Thanks, Luis Luis de Bethencourt (2): net: phy: mdio-bcm-unimac: Fix module autoload for OF platform driver net: phy: mdio-gpio: Fix module autoload for OF platform driver drivers/net/phy/mdio-bcm-unimac.c | 1

[PATCH 1/2] net: phy: mdio-bcm-unimac: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/phy/mdio-bcm-unimac.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/n

[PATCH 2/2] net: phy: mdio-gpio: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/phy/mdio-gpio.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ph

[PATCH 0/6] net: Fix module autoload for OF platform drivers

2015-09-18 Thread Luis de Bethencourt
Hi, These patches add the missing MODULE_DEVICE_TABLE() for OF to export the information so modules have the correct aliases built-in and autoloading works correctly. A longer explanation by Javier Canillas can be found here: https://lkml.org/lkml/2015/7/30/519 Thanks, Luis Luis de Bethencourt

[PATCH 2/6] net: systemport: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/ethernet/broadcom/bcmsysport.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH 3/6] net: bcmgenet: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <l...@osg.samsung.com> --- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH 1/6] net: arc: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/ethernet/arc/emac_arc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drive

[PATCH 4/6] net: freescale: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/ethernet/freescale/gianfar_ptp.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH 5/6] net: ks8851: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/ethernet/micrel/ks8851.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drive

[PATCH 6/6] net: moxa: Fix module autoload for OF platform driver

2015-09-18 Thread Luis de Bethencourt
This platform driver has a OF device ID table but the OF module alias information is not created so module autoloading won't work. Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com> --- drivers/net/ethernet/moxa/moxart_ether.c | 1 + 1 file changed, 1 insertion(+) diff --git a/d