linux-iio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/19] staging: Remove unnecessary externs
@ 2015-08-10 21:51 Joe Perches
  2015-08-10 21:51 ` [PATCH 05/19] staging: iio: " Joe Perches
  0 siblings, 1 reply; 6+ messages in thread
From: Joe Perches @ 2015-08-10 21:51 UTC (permalink / raw)
  To: devel, driverdev-devel, ac100, linux-tegra, speakup,
	linux-wireless
  Cc: linux-kernel, linux-iio, HPDD-discuss

trivial neatening of prototypes

Joe Perches (19):
  staging: android: Remove unnecessary externs
  staging: dgnc: Remove unnecessary externs
  staging: fbtft: Remove unnecessary externs
  staging: ft1000: Remove unnecessary externs
  staging: iio: Remove unnecessary externs
  staging: lustre: Remove unnecessary externs
  staging: most: Remove unnecessary externs
  staging: mt29f_spinand: Remove unnecessary externs
  staging: netlogic: Remove unnecessary externs
  staging: nvec: Remove unnecessary externs
  staging: octeon: Remove unnecessary externs
  staging: olpc_dcon: Remove unnecessary externs
  staging: rtl8188eu: Remove unnecessary externs
  staging: rtl8192e: Remove unnecessary externs
  staging: rtl8192u: Remove unnecessary externs
  staging: rtl8712: Remove unnecessary externs
  staging: speakup: Remove unnecessary externs
  staging: wilc1000: Remove unnecessary externs
  staging: xgifb: Remove unnecessary externs

 drivers/staging/android/sync.h                     |  10 +-
 drivers/staging/dgnc/dgnc_sysfs.h                  |  16 +-
 drivers/staging/fbtft/fbtft.h                      |  52 ++-
 drivers/staging/ft1000/ft1000-pcmcia/ft1000.h      |  19 +-
 drivers/staging/ft1000/ft1000-usb/ft1000_usb.h     |   4 +-
 drivers/staging/iio/meter/ade7854.h                |   4 +-
 .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.h    |   2 +-
 .../staging/lustre/lnet/klnds/socklnd/socklnd.h    | 141 +++++----
 drivers/staging/lustre/lnet/selftest/console.h     |  90 +++---
 .../staging/lustre/lustre/include/lprocfs_status.h | 313 +++++++++----------
 .../lustre/lustre/include/lustre/lustre_idl.h      | 133 ++++----
 .../lustre/lustre/include/lustre/lustre_user.h     |   2 +-
 drivers/staging/lustre/lustre/include/lustre_dlm.h |   2 +-
 .../staging/lustre/lustre/include/lustre_export.h  |   4 +-
 .../staging/lustre/lustre/include/lustre_import.h  |   4 +-
 drivers/staging/lustre/lustre/include/lustre_net.h |  16 +-
 drivers/staging/lustre/lustre/include/obd_class.h  |  16 +-
 .../staging/lustre/lustre/include/obd_support.h    |   6 +-
 drivers/staging/lustre/lustre/ldlm/ldlm_internal.h |   2 +-
 drivers/staging/lustre/lustre/libcfs/tracefile.h   |  32 +-
 .../staging/lustre/lustre/llite/llite_internal.h   |  16 +-
 drivers/staging/most/hdm-dim2/dim2_hal.h           |   8 +-
 drivers/staging/mt29f_spinand/mt29f_spinand.h      |   4 +-
 drivers/staging/netlogic/xlr_net.h                 |   2 +-
 drivers/staging/nvec/nvec.h                        |  19 +-
 drivers/staging/octeon/ethernet-mdio.h             |   2 +-
 drivers/staging/octeon/octeon-ethernet.h           |  22 +-
 drivers/staging/olpc_dcon/olpc_dcon.h              |   2 +-
 drivers/staging/rtl8188eu/include/osdep_service.h  |   2 +-
 drivers/staging/rtl8188eu/include/rtw_mlme_ext.h   |   8 +-
 drivers/staging/rtl8188eu/include/sta_info.h       |  18 +-
 drivers/staging/rtl8192e/rtl8192e/r8190P_rtl8256.h |  10 +-
 drivers/staging/rtl8192e/rtl8192e/r8192E_cmdpkt.h  |   4 +-
 .../staging/rtl8192e/rtl8192e/r8192E_firmware.h    |   2 +-
 drivers/staging/rtl8192e/rtl8192e/r8192E_phy.h     |  52 ++-
 drivers/staging/rtl8192e/rtl8192e/rtl_dm.h         |  22 +-
 drivers/staging/rtl8192e/rtllib.h                  | 347 ++++++++++-----------
 drivers/staging/rtl8192u/ieee80211/ieee80211.h     | 345 ++++++++++----------
 drivers/staging/rtl8192u/r8190_rtl8256.h           |  11 +-
 drivers/staging/rtl8192u/r8192U.h                  |   2 +-
 drivers/staging/rtl8192u/r8192U_dm.h               |  36 +--
 drivers/staging/rtl8192u/r8192U_wx.h               |   2 +-
 drivers/staging/rtl8192u/r819xU_cmdpkt.h           |   8 +-
 drivers/staging/rtl8192u/r819xU_phy.h              |  55 ++--
 drivers/staging/rtl8712/rtl871x_ioctl.h            |  28 +-
 drivers/staging/speakup/i18n.h                     |  12 +-
 drivers/staging/speakup/speakup.h                  |  68 ++--
 drivers/staging/wilc1000/coreconfigurator.h        |  30 +-
 drivers/staging/wilc1000/linux_wlan_common.h       |   4 +-
 drivers/staging/wilc1000/wilc_wfi_cfgoperations.h  |   2 +-
 drivers/staging/xgifb/vb_init.h                    |   4 +-
 drivers/staging/xgifb/vb_setmode.h                 |  34 +-
 52 files changed, 1007 insertions(+), 1042 deletions(-)

-- 
2.1.2


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 05/19] staging: iio: Remove unnecessary externs
  2015-08-10 21:51 [PATCH 00/19] staging: Remove unnecessary externs Joe Perches
@ 2015-08-10 21:51 ` Joe Perches
  2015-08-11 18:43   ` Lars-Peter Clausen
  0 siblings, 1 reply; 6+ messages in thread
From: Joe Perches @ 2015-08-10 21:51 UTC (permalink / raw)
  To: devel, Lars-Peter Clausen, Michael Hennerich
  Cc: Jonathan Cameron, Hartmut Knaack, Peter Meerwald,
	Greg Kroah-Hartman, linux-iio, linux-kernel

Using 'extern' is not necessary for function prototypes.

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/staging/iio/meter/ade7854.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/iio/meter/ade7854.h b/drivers/staging/iio/meter/ade7854.h
index 52ca541..52f4195 100644
--- a/drivers/staging/iio/meter/ade7854.h
+++ b/drivers/staging/iio/meter/ade7854.h
@@ -168,7 +168,7 @@ struct ade7854_state {
 
 };
 
-extern int ade7854_probe(struct iio_dev *indio_dev, struct device *dev);
-extern int ade7854_remove(struct iio_dev *indio_dev);
+int ade7854_probe(struct iio_dev *indio_dev, struct device *dev);
+int ade7854_remove(struct iio_dev *indio_dev);
 
 #endif
-- 
2.1.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 05/19] staging: iio: Remove unnecessary externs
  2015-08-10 21:51 ` [PATCH 05/19] staging: iio: " Joe Perches
@ 2015-08-11 18:43   ` Lars-Peter Clausen
  2015-08-15 19:57     ` Jonathan Cameron
  0 siblings, 1 reply; 6+ messages in thread
From: Lars-Peter Clausen @ 2015-08-11 18:43 UTC (permalink / raw)
  To: Joe Perches, devel, Michael Hennerich
  Cc: Jonathan Cameron, Hartmut Knaack, Peter Meerwald,
	Greg Kroah-Hartman, linux-iio, linux-kernel

On 08/10/2015 11:51 PM, Joe Perches wrote:
> Using 'extern' is not necessary for function prototypes.
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Acked-by: Lars-Peter Clausen <lars@metafoo.de>

Thanks.

> ---
>  drivers/staging/iio/meter/ade7854.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/iio/meter/ade7854.h b/drivers/staging/iio/meter/ade7854.h
> index 52ca541..52f4195 100644
> --- a/drivers/staging/iio/meter/ade7854.h
> +++ b/drivers/staging/iio/meter/ade7854.h
> @@ -168,7 +168,7 @@ struct ade7854_state {
>  
>  };
>  
> -extern int ade7854_probe(struct iio_dev *indio_dev, struct device *dev);
> -extern int ade7854_remove(struct iio_dev *indio_dev);
> +int ade7854_probe(struct iio_dev *indio_dev, struct device *dev);
> +int ade7854_remove(struct iio_dev *indio_dev);
>  
>  #endif
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 05/19] staging: iio: Remove unnecessary externs
  2015-08-11 18:43   ` Lars-Peter Clausen
@ 2015-08-15 19:57     ` Jonathan Cameron
  2015-08-15 20:05       ` Lars-Peter Clausen
  0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Cameron @ 2015-08-15 19:57 UTC (permalink / raw)
  To: Lars-Peter Clausen, Joe Perches, devel, Michael Hennerich
  Cc: Hartmut Knaack, Peter Meerwald, Greg Kroah-Hartman, linux-iio,
	linux-kernel

On 11/08/15 19:43, Lars-Peter Clausen wrote:
> On 08/10/2015 11:51 PM, Joe Perches wrote:
>> Using 'extern' is not necessary for function prototypes.
>>
>> Signed-off-by: Joe Perches <joe@perches.com>
> 
> Acked-by: Lars-Peter Clausen <lars@metafoo.de>
> 
Applied to the togreg branch of iio.git.  4.4 material now
probably.

Thanks,

Jonathan
> Thanks.
> 
>> ---
>>  drivers/staging/iio/meter/ade7854.h | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/staging/iio/meter/ade7854.h b/drivers/staging/iio/meter/ade7854.h
>> index 52ca541..52f4195 100644
>> --- a/drivers/staging/iio/meter/ade7854.h
>> +++ b/drivers/staging/iio/meter/ade7854.h
>> @@ -168,7 +168,7 @@ struct ade7854_state {
>>  
>>  };
>>  
>> -extern int ade7854_probe(struct iio_dev *indio_dev, struct device *dev);
>> -extern int ade7854_remove(struct iio_dev *indio_dev);
>> +int ade7854_probe(struct iio_dev *indio_dev, struct device *dev);
>> +int ade7854_remove(struct iio_dev *indio_dev);
>>  
>>  #endif
>>
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 05/19] staging: iio: Remove unnecessary externs
  2015-08-15 19:57     ` Jonathan Cameron
@ 2015-08-15 20:05       ` Lars-Peter Clausen
  2015-08-15 20:33         ` Jonathan Cameron
  0 siblings, 1 reply; 6+ messages in thread
From: Lars-Peter Clausen @ 2015-08-15 20:05 UTC (permalink / raw)
  To: Jonathan Cameron, Joe Perches, devel, Michael Hennerich
  Cc: Hartmut Knaack, Peter Meerwald, Greg Kroah-Hartman, linux-iio,
	linux-kernel

On 08/15/2015 09:57 PM, Jonathan Cameron wrote:
> On 11/08/15 19:43, Lars-Peter Clausen wrote:
>> On 08/10/2015 11:51 PM, Joe Perches wrote:
>>> Using 'extern' is not necessary for function prototypes.
>>>
>>> Signed-off-by: Joe Perches <joe@perches.com>
>>
>> Acked-by: Lars-Peter Clausen <lars@metafoo.de>
>>
> Applied to the togreg branch of iio.git.  4.4 material now
> probably.

Greg already picked the patch up earlier today on his own.


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 05/19] staging: iio: Remove unnecessary externs
  2015-08-15 20:05       ` Lars-Peter Clausen
@ 2015-08-15 20:33         ` Jonathan Cameron
  0 siblings, 0 replies; 6+ messages in thread
From: Jonathan Cameron @ 2015-08-15 20:33 UTC (permalink / raw)
  To: Lars-Peter Clausen, Joe Perches, devel, Michael Hennerich
  Cc: Hartmut Knaack, Peter Meerwald, Greg Kroah-Hartman, linux-iio,
	linux-kernel

On 15/08/15 21:05, Lars-Peter Clausen wrote:
> On 08/15/2015 09:57 PM, Jonathan Cameron wrote:
>> On 11/08/15 19:43, Lars-Peter Clausen wrote:
>>> On 08/10/2015 11:51 PM, Joe Perches wrote:
>>>> Using 'extern' is not necessary for function prototypes.
>>>>
>>>> Signed-off-by: Joe Perches <joe@perches.com>
>>>
>>> Acked-by: Lars-Peter Clausen <lars@metafoo.de>
>>>
>> Applied to the togreg branch of iio.git.  4.4 material now
>> probably.
> 
> Greg already picked the patch up earlier today on his own.
Thanks,
Dropped it from my branch as hadn't pushed out yet.

Jonathan
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-08-15 20:33 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-10 21:51 [PATCH 00/19] staging: Remove unnecessary externs Joe Perches
2015-08-10 21:51 ` [PATCH 05/19] staging: iio: " Joe Perches
2015-08-11 18:43   ` Lars-Peter Clausen
2015-08-15 19:57     ` Jonathan Cameron
2015-08-15 20:05       ` Lars-Peter Clausen
2015-08-15 20:33         ` Jonathan Cameron

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).