Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Set conversion channels and rename ad7997_8_update_scan_mode()
rename since function is used by all chips with ALERT pin, not just ad7997/8
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Write default config on probe and reset alert status on probe
writing ALERT_EN and BUSY_ALERT to the chip config register clears
pending alerts, BUSY_ALERT is cleared when reading back the register
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Add helper function to read/write config register
16-bit on ad7997/ad7998, 8-bit elsewhere
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Make chan_spec const in ad799x_chip_config struct
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Only expose event interface when IRQ is available
an IRQ is necessary to handle the ALERT condition; without
IRQ, the IIO event interface serves no purpose
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Use BIT() and GENMASK()
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Save some lines in ad7997_8_update_scan_mode() exit handling
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Peter Meerwald [Sat, 6 Dec 2014 05:54:00 +0000 (05:54 +0000)]
iio:adc:ad799x: Drop I2C access helper functions
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 19:31:47 +0000 (12:31 -0700)]
Merge tag 'iio-for-3.17c' of git://git./linux/kernel/git/jic23/iio into staging-next
Jonathan writes:
3rd round of IIO new drivers, cleanups and functionality for the 3.17 cycle.
New drivers
* isl29125 digital color light sensor driver
* TAOS/AMS tcs3414 digital color sensor
Staging graduation
* ad7291 ADC driver.
New functionality
* st_sensors - device tree support and bindings
* mma8452 - device tree support
Cleanups
* Drop redundant variables in a number of drivers.
* Reorder a structure definition to ealy wiht a warning about static
not being at the beginning in the hid-sensors driver.
* Switch a few more drivers away from using explicit sampling_frequency
attribute to providing this through the core.
* Make hid_sensor_get_reporting_interval static as only used within a single
file.
* Drop a redundant check for negative values in an unsigned variable from
ad9832
* Drop some duplicate case labels in the event monitor example code.
* Use devm_ioremap_resource to simplify error handling.
* Use devm_kzalloc within the blackfin timer driver to simplify error
handling and removal.
* A number of cleanups of the ad7291 from Hartmut Knaack in response
to a patch moving it out of staging.
* Core support for the period info element about events. It has been
in the abi for a while, but not added until now to the newer handling
of information related to events.
* Add HAS_IOMEM dependency to mxs_lradc to avoid build issues when testing
enabled.
navin patidar [Sun, 13 Jul 2014 14:29:38 +0000 (19:59 +0530)]
staging: rtl8188eu: Rename _rtw_init_mlme_priv() to rtw_init_mlme_priv()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:37 +0000 (19:59 +0530)]
staging: rtl8188eu: Rename _rtw_free_mlme_priv() to rtw_free_mlme_priv()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:36 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function _rtw_enqueue_network()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:35 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function _rtw_dequeue_network()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:34 +0000 (19:59 +0530)]
staging: rtl8188eu: Make _rtw_free_network() static
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:33 +0000 (19:59 +0530)]
staging: rtl8188eu: Rename _rtw_find_network() to rtw_find_network()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:32 +0000 (19:59 +0530)]
staging: rtl8188eu: Rename _rtw_free_network_queue() to rtw_free_network_queue()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:31 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function rtw_get_timestampe_from_ie()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:30 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function _rtw_IOL_append_WW_cmd()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:29 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function rtw_set_802_11_remove_wep()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:28 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function rtw_set_802_11_remove_key()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:27 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function rtw_set_802_11_add_key()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:26 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function rtw_validate_ssid()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:25 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function rtw_set_scan_mode()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:24 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove function rtw_set_channel_plan()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
navin patidar [Sun, 13 Jul 2014 14:29:23 +0000 (19:59 +0530)]
staging: rtl8188eu: Remove unused function rtw_use_tkipkey_handler()
Signed-off-by: navin patidar <navin.patidar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:53 +0000 (10:42 +0100)]
staging: vt6656: wcnd.c clean up comments
Removing old functions and comments.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:52 +0000 (10:42 +0100)]
staging: vt6656: wcmd remove unneeded headers
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:51 +0000 (10:42 +0100)]
staging: vt6656: rename vResetCommandTimer to vnt_reset_command_timer
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:50 +0000 (10:42 +0100)]
staging: vt6656: Drop u8 pointer and rename bScheduleCommand.
Rename to vnt_schedule_command removing unused u8 pointer.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:49 +0000 (10:42 +0100)]
staging: vt6656: rename vRunCommand to vnt_run_command
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:48 +0000 (10:42 +0100)]
staging: vt6656: rename vCommandTimerWait to vnt_cmd_timer_wait
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:47 +0000 (10:42 +0100)]
staging: vt6656: Move and rename s_bCommandComplete to vnt_cmd_complete
Remove static declaration and moving for function visibility
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:46 +0000 (10:42 +0100)]
staging: vt6656: dead code remove typedef enum tagCMD_STATUS
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:45 +0000 (10:42 +0100)]
staging: vt6656: replace camel case from command variables
camel case changes
uCmdDequeueIdx -> cmd_dequeue_idx
uCmdEnqueueIdx -> cmd_enqueue_idx
cbFreeCmdQueue -> free_cmd_queue
bCmdRunning -> cmd_running
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:44 +0000 (10:42 +0100)]
staging: vt6656: remove unused variables bCmdClear and bNeedRadioOFF
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:43 +0000 (10:42 +0100)]
staging: vt6656: remove typedef struct tagCMD_ITEM
Since enum vnt_cmd is only member replace with array
enum vnt_cmd cmd_queue with size of CMD_Q_SIZE
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:42 +0000 (10:42 +0100)]
staging: vt6656: replace typedef enum tagCMD_STATE with enum vnt_cmd_state
Replacing variables.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sun, 13 Jul 2014 09:42:41 +0000 (10:42 +0100)]
staging: vt6656: replace typedef enum tagCMD_CODE with enum vnt_cmd
Replacing the variables.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:10 +0000 (09:32 +0200)]
staging: rtl8723au: Remove unncessary wrapper PHY_RFConfig8723A()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:09 +0000 (09:32 +0200)]
staging: rtl8723au: Remove obfuscating wrapper ODM_ConfigMACWithHeaderFile23a()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:08 +0000 (09:32 +0200)]
staging: rtl8723au: Remove unused ODM_MacStatusQuery23a()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:07 +0000 (09:32 +0200)]
staging: rtl8723au: Remove ugly wrapper ODM_ConfigBBWithHeaderFile23a()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:06 +0000 (09:32 +0200)]
staging: rtl8723au: Remove unused ODM_ReadAndConfig_PHY_REG_PG_8723A() and related code+data
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:05 +0000 (09:32 +0200)]
staging: rtl8723au: Remove pointless wrappers for calling odm_ConfigRFReg_8723A()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:04 +0000 (09:32 +0200)]
staging: rtl8723au: Remove zero value calculation added to address
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:03 +0000 (09:32 +0200)]
staging: rtl8723au: Eliminate wrapper ODM_ConfigRFWithHeaderFile23a()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:02 +0000 (09:32 +0200)]
staging: rtl8723au: Call to ODM_ConfigRFWithHeaderFile23a path is identical for RF_PATH_[AB]
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jes Sorensen [Sun, 13 Jul 2014 07:32:01 +0000 (09:32 +0200)]
staging: rtl8723au: Remove horrendous code obfuscation READ_AND_CONFIG*()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Jeremiah Mahler [Sun, 13 Jul 2014 01:26:39 +0000 (18:26 -0700)]
staging: dgnc: DGNC_VERIFY_BOARD macro in do while block
Enclose the body of the multi-statement DGNC_VERIFY_BOARD macro inside a
do - while block as per Documentation/CodingStyle. Fixes 1 error found
by checkpatch.pl.
Cc: Lidza Louina <lidza.louina@gmail.com>
Signed-off-by: Jeremiah Mahler <jmmahler@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Chen Gang [Sun, 13 Jul 2014 11:44:00 +0000 (12:44 +0100)]
staging:iio:adc:Kconfig: Let MXS_LRADC depend on HAS_IOMEM
MXS_LRADC need HAS_IOMEM, so let it depend on HAS_IOMEM
The related error (with allmodconfig under score):
MODPOST 1365 modules
ERROR: "devm_ioremap_resource" [drivers/staging/iio/adc/mxs-lradc.ko] undefined!
Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
Acked-by: Marek Vasut <marex@denx.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Martin Fuzzey [Fri, 7 Nov 2014 14:06:00 +0000 (14:06 +0000)]
iio: mma8452: add DT support
Allow the mma8452 to be described in the device tree.
Since no device specific binding attributes exist the trivial
I2C binding is sufficient to describe the compatible string.
Signed-off-by: Martin Fuzzey <mfuzzey@parkeon.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 04:26:09 +0000 (21:26 -0700)]
staging: lustre: remove proc function wrappers
ll_proc_dointvec and ll_proc_dolongvec didn't do anything except confuse
things due to a lack of a filep variable in the functions they were used
in. So remove them and just call the real proc functions. This enabled
the code to be unwound a bit and a few "empty" proc handlers were
removed and just passed directly to procfs instead.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 04:15:26 +0000 (21:15 -0700)]
staging: lustre: delete params_tree.h
It's not being used at all, so delete it.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 04:13:25 +0000 (21:13 -0700)]
staging: lustre: use CONFIG_PROC_FS
Don't rely on a "custom" LPROCFS define, it's not needed, just check for
the real thing. This will let us delete a whole .h file that is not
being used for anything other than one #define.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 03:56:47 +0000 (20:56 -0700)]
staging: lustre: don't use custom typedef for an inode
It's 'struct inode' so just use that.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 03:37:56 +0000 (20:37 -0700)]
staging: lustre: remove ALLOC_ATOMIC_TRY
Come on, is it so hard to type "GFP_ATOMIC"? Look, it's less
characters!!!
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 03:32:03 +0000 (20:32 -0700)]
staging: lustre: remove kp30.h
It's empty, so remove it and move the #include lines into the one file
that used to include this header.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 03:26:07 +0000 (20:26 -0700)]
staging: lustre: remove LPX64 define
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 03:06:04 +0000 (20:06 -0700)]
staging: lustre: remove LPU64 define
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:41:09 +0000 (18:41 -0700)]
staging: lustre: remove LPD64 define
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:31:44 +0000 (18:31 -0700)]
staging: lustre: remove LPX64i define
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:28:41 +0000 (18:28 -0700)]
staging: lustre: remove LPLU, LPLD, and LPLX defines
They are variable modifiers, come on, no need for a special define just
for them...
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:23:18 +0000 (18:23 -0700)]
staging: lustre: remove LPF64 #define
Just use the proper string for the variable type.
Also remove some variable types that are not used at all (LPO64 and
LPPID)
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:17:20 +0000 (18:17 -0700)]
staging: lustre: remove linux-prim.h
It's empty now, move the #include lines it was pulling in, into libcfs.h
and sort them to remove the duplicates.
Odds are they all aren't needed, but let's not break the build by
experimenting with removing them at this point in time.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:11:25 +0000 (18:11 -0700)]
staging: lustre: clean up some proc file declarations
This removes the DECLARE_PROC_HANDLER macro, and makes proc_call_handler
a static function (horrible name to be in a module). This makes the
linux-prim.h file empty now, so it can be removed soon.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:01:57 +0000 (18:01 -0700)]
staging: lustre: remove hpdd-discuss list from TODO file
It's a closed mailing list, what's the use of cc:ing patches there if
they never go through. So remove it from the TODO file.
A real MAINTAINERS entry is probably a good thing for the Lustre code,
maybe someday someone will send a patch to do that...
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sun, 13 Jul 2014 01:00:44 +0000 (18:00 -0700)]
staging: lustre: remove LL_PROC_PROTO macro
Spell out what is happening with the proc file functions, it helps to
unwind the call chain mess here.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 19:12:45 +0000 (20:12 +0100)]
staging: vt6656: baseband.c rename abyVT3184_VT3226D0 to vnt_vt3184_vt3226d0
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 19:12:44 +0000 (20:12 +0100)]
staging: vt6656: baseband.c remame abyVT3184_AL2230 to vnt_vt3184_al2230
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 19:12:43 +0000 (20:12 +0100)]
staging: vt6656: baseband.c rename abyVT3184_AGC to vnt_vt3184_agc
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 19:12:42 +0000 (20:12 +0100)]
staging: vt6656: baseband clean up VT3184 static variables
Remove white space comments and arrange 8 bytes wide
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 10:30:03 +0000 (11:30 +0100)]
staging: vt6656: rf.c cleanup comments
named comments are already referenced in revision history
in high-level comment.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 10:30:02 +0000 (11:30 +0100)]
staging: vt6656: vnt_rf_write_embedded add reg length and IFREGCTL_REGW
The rf register always have *_REG_LEN and IFREGCTL_REGW macros added.
*_REG_LEN is always 23(0x17) replace them with VNT_RF_REG_LEN.
Remove *_REG_LEN and IFREGCTL_REGW from tables and vnt_rf_set_txpower and
apply VNT_RF_REG_LEN and IFREGCTL_REGW in vnt_rf_write_embedded
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 10:30:01 +0000 (11:30 +0100)]
staging: vt6656: rf.c remove dead code RFaby11aChannelIndex
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 10:30:00 +0000 (11:30 +0100)]
staging: vt6656: rf.c clean up channel and power tables
Remove comments to channel number execpt where there is a
change to 5gHz channel 15
named comments are already referenced in revision history
in high-level comment.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Sat, 12 Jul 2014 09:36:25 +0000 (11:36 +0200)]
Staging: bcm: Transmit.c: Removed obvious comment
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Sat, 12 Jul 2014 09:36:24 +0000 (11:36 +0200)]
Staging: bcm: Transmit.c: Line length / Whitespace cleanup
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Matthias Beyer [Sat, 12 Jul 2014 09:36:23 +0000 (11:36 +0200)]
Staging: bcm: Transmit.c: Replaced member accessing with variable
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:48 +0000 (07:53 +0100)]
staging: vt6656: wcmd replace debug message with dev_dbg
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:47 +0000 (07:53 +0100)]
staging: vt6656: vResetCommandTimer remove camel case
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:46 +0000 (07:53 +0100)]
staging: vt6656: bScheduleCommand remove camel case
Camel case changes
pDevice -> priv
eCommand -> command
pbyItem0 -> item0
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:45 +0000 (07:53 +0100)]
staging: vt6656: s_bCommandComplete clean up and remove camel case
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:44 +0000 (07:53 +0100)]
staging: vt6656: s_bCommandComplete remove else from if
if returns out so remove else
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:43 +0000 (07:53 +0100)]
staging: vt6656: remove unused variables from struct tagCMD_ITEM
Also remove local variables
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:42 +0000 (07:53 +0100)]
staging: vt6656: vRunCommand remove camel case.
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:41 +0000 (07:53 +0100)]
staging: vt6656: wcmd remove unused commands
WLAN_CMD_CHANGE_BBSENSITIVITY and WLAN_CMD_MAC_DISPOWERSAVING
are already implemented and don't need the command timer.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:40 +0000 (07:53 +0100)]
staging: vt6656: wcmd Remove command WLAN_CMD_RADIO
Part of it has already been removed. checking isr1 & ISR_GPIO3
serves no purpose.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Malcolm Priestley [Sat, 12 Jul 2014 06:53:39 +0000 (07:53 +0100)]
staging: vt6656: vCommandTimerWait remove camel case.
camel case changes
pDevice -> priv
MSecond -> msecs
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 08:09:15 +0000 (01:09 -0700)]
staging: lustre: fid: fid_request.c: checkpatch cleanup: align arguments to parenthesis
whitespace changes only - git diff -w shows no difference
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 08:03:41 +0000 (01:03 -0700)]
staging: lustre: remove cfs_time_before()
Just call time_before() instead.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:54:26 +0000 (00:54 -0700)]
staging: lustre: cfs_time_sub() must return unsigned long
It looks like someone ment for it to return this value (there's a cast),
but the funtion says 'long' which causes problems when comparing values
from this function.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:45:51 +0000 (00:45 -0700)]
staging: lustre: remove cfs_duration_t typedef
Just use 'long' as that is what it was defined as.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:38:48 +0000 (00:38 -0700)]
staging: lustre: remove cfs_time_current_sec wrapper
Just call get_seconds() directly.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:31:18 +0000 (00:31 -0700)]
staging: lustre: remove cfs_time_beforeq wrapper
Use time_before_eq() call instead.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:27:46 +0000 (00:27 -0700)]
staging: lustre: remove cfs_time_t typedef
Just use unsigned long everywhere, like the rest of the kernel does.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:10:33 +0000 (00:10 -0700)]
staging: lustre: remove unused functions from linux-time.h
No one was using them, so delete them.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:04:53 +0000 (00:04 -0700)]
staging: lustre: remove cfs_fs_time_t
Use 'struct timespec' like the rest of the kernel does
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 07:01:03 +0000 (00:01 -0700)]
staging: lustre: remove linux-tcpip.h
There were just a few macros being used, remove them and have the places
where they were being used to make the proper kernel api call instead.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 06:52:29 +0000 (23:52 -0700)]
staging: lustre: linux-tcpip.h: remove unused macros
Some macros are never used, so remove them.
Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg Kroah-Hartman [Sat, 12 Jul 2014 06:49:54 +0000 (23:49 -0700)]
staging: lustre: fix makefile build breakage
Forgot to test with infiniband enabled.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>