Hans Verkuil
Cc: Mauro Carvalho Chehab
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/imx/imx-media-of.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/imx/imx-media-of.c
b/drivers/staging/media/imx/imx-media-of.c
index b026fe66467c..90e7e70
3bs/hal/hal_btcoex.c:1414:5: error: unused variable
'ret1' [-Werror=unused-variable]
This removes the declarations as well.
Fixes: 95b3b4238581 ("staging: rtl8723bs: remove ternary operators in assignmet
statments")
Signed-off-by: Arnd Bergmann
---
drivers/staging/rtl8723bs/h
he stale variable as well.
Fixes: f8af6a323368 ("staging: rtlwifi: Convert timers to use timer_setup()")
Signed-off-by: Arnd Bergmann
---
drivers/staging/rtlwifi/base.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/rtlwifi/base.c b/drivers/staging/rtlwifi/base.c
in
As of commit 8e1d6c336d74 ("greybus: loopback: drop bus aggregate
calculation"), nothing ever reads from kfifo_ts, so there is no
reason to write to it or even allocate it any more.
Signed-off-by: Arnd Bergmann
---
drivers/staging/greybus/loopback.c | 27 +++--
This driver is the only one using the deprecated timeval_to_ns()
helper. Changing it from do_gettimeofday() to ktime_get() makes
the code more efficient, more robust against concurrent
settimeofday(), more accurate and lets us get rid of that helper
in the future.
Signed-off-by: Arnd Bergmann
do_gettimeofday() is deprecated and slower than necessary for the purpose
of reading the seconds. This changes rtl_op_suspend/resume to use
ktime_get_real_seconds() instead, which is simpler and avoids confusion
about whether it is y2038-safe or not.
Signed-off-by: Arnd Bergmann
---
This
On Fri, Nov 10, 2017 at 11:42 PM, Deepa Dinamani wrote:
> The series is a preparation series for individual architectures
> to use 64 bit time_t syscalls in compat and 32 bit emulation modes.
>
> This is a follow up to the series Arnd Bergmann posted:
> https://sourceware.org/ml/
On Thu, Nov 16, 2017 at 10:04 AM, Thomas Gleixner wrote:
> On Wed, 15 Nov 2017, Deepa Dinamani wrote:
>> > I had on concern about x32, maybe we should check
>> > for "COMPAT_USE_64BIT_TIME" before zeroing out the tv_nsec
>> > bits.
>>
>> Thanks, I think you are right. I had the check conditional o
On Fri, Nov 17, 2017 at 9:58 AM, Thomas Gleixner wrote:
> On Fri, 17 Nov 2017, Arnd Bergmann wrote:
>> On Thu, Nov 16, 2017 at 10:04 AM, Thomas Gleixner wrote:
>> > On Wed, 15 Nov 2017, Deepa Dinamani wrote:
>> >> Would this work for everyone?
>> >
>
On Fri, Nov 17, 2017 at 10:54 AM, Thomas Gleixner wrote:
> On Fri, 17 Nov 2017, Arnd Bergmann wrote:
>> On Fri, Nov 17, 2017 at 9:58 AM, Thomas Gleixner wrote:
>>
>> No, syscall that existing 32-bit user space enters would be handled by
>> compat_sys_nanosleep() on bot
On Fri, Nov 17, 2017 at 11:40 AM, Thomas Gleixner wrote:
> On Fri, 17 Nov 2017, Arnd Bergmann wrote:
>> On Fri, Nov 17, 2017 at 10:54 AM, Thomas Gleixner wrote:
>> > On Fri, 17 Nov 2017, Arnd Bergmann wrote:
>> >> On Fri, Nov 17, 2017 at 9:58 AM, Thomas Gleixner
struct timeval is deprecated for in-kernel use, and converting
this function to use ktime_t makes it simpler as well.
Signed-off-by: Arnd Bergmann
---
.../vc04_services/bcm2835-camera/bcm2835-camera.c | 37 ++
.../vc04_services/bcm2835-camera/bcm2835-camera.h | 2 +-
2
igned-off-by: Arnd Bergmann
---
drivers/staging/media/atomisp/i2c/ov2680.h| 1 -
drivers/staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c | 15 ++-
drivers/staging/media/atomisp/i2c/ov5693/ov5693.h | 2 +-
3 files changed, 7 insertions(+), 11 deletions(-)
icit test for it.
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/imx/imx-media-csi.c | 8 ++--
drivers/staging/media/imx/imx-media-fim.c | 30 +-
drivers/staging/media/imx/imx-media.h | 2 +-
3 files changed, 20 insertions(+), 20 deletions(-)
dif
igned-off-by: Arnd Bergmann
---
v2: use min_t() as suggested by Andy Shevchenko
---
drivers/staging/media/atomisp/i2c/ov2680.h| 1 -
.../staging/media/atomisp/i2c/ov5693/atomisp-ov5693.c | 19 ---
drivers/staging/media/atomisp/i2c/ov5693/ov5693.h | 2 +-
3 files ch
On Mon, Nov 27, 2017 at 4:05 PM, Andy Shevchenko
wrote:
> On Mon, 2017-11-27 at 14:19 +0100, Arnd Bergmann wrote:
>> timespec overflows in 2038 on 32-bit architectures, and the
>> getnstimeofday() suffers from possible time jumps, so the
>> timestamps here are better done usi
On Mon, Nov 27, 2017 at 8:30 PM, Deepa Dinamani wrote:
> The series is a preparation series for individual architectures
> to use 64 bit time_t syscalls in compat and 32 bit emulation modes.
>
> This is a follow up to the series Arnd Bergmann posted:
> https://sourceware.org/ml/lib
On Mon, Nov 27, 2017 at 11:29 PM, Deepa Dinamani wrote:
>>> I decided against using LEGACY_TIME_SYSCALLS to conditionally compile
>>> legacy time syscalls such as sys_nanosleep because this will need to
>>> enclose compat_sys_nanosleep as well. So, defining it as
>>>
>>> config LEGACY_TIME_SYSCALL
On Wed, Nov 29, 2017 at 12:17 AM, Deepa Dinamani wrote:
> On Tue, Nov 28, 2017 at 6:17 AM, Arnd Bergmann wrote:
>> On Mon, Nov 27, 2017 at 11:29 PM, Deepa Dinamani
>> wrote:
> Right. There are three options:
>
> 1. Use two configs to identify which syscalls need n
fig symbol to avoid that problem,
but there is also seems to be little value in keeping the staging driver
around now.
Fixes: 4ba9c3afda41 ("gpio: mt7621: Add a driver for MT7621")
Signed-off-by: Arnd Bergmann
---
drivers/staging/Kconfig| 2 -
driver
s there are no
callers of this function.
Fixes: 9a69f5087ccc ("drivers/staging: Gasket driver framework + Apex driver")
Signed-off-by: Arnd Bergmann
---
drivers/staging/gasket/gasket_core.c | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/drivers/stag
d instance
of drivers/staging/rtl8723bs/hal/odm.h is garbled in a different way,
so I change it to be the same as well even though it is already
plain ASCII.
Signed-off-by: Arnd Bergmann
---
drivers/staging/rtl8188eu/include/odm.h | 2 +-
drivers/staging/rtl8723bs/hal/odm.h | 2 +-
2 files
spi, or mixed),
all such variables need to be dynamically allocated per instance and
stored in 'struct wilc' or one of the structures referenced by it.
Fixes: 9abc44ba4e2f ("staging: wilc1000: fix TODO to compile spi and sdio
components in single module")
Signed-off-by: Arnd
figure out
what is going on. The earlier partial workaround can be removed now,
as the new workaround is better.
Fixes: 890f27693f2a ("media: imx: work around false-positive warning")
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/imx/imx-media-csi.c | 5 -
1 file chang
On Tue, Aug 14, 2018 at 7:22 AM Ajay Singh wrote:
>
> Hi Arnd,
>
> On Mon, 13 Aug 2018 23:20:33 +0200
> Arnd Bergmann wrote:
>
> > The TODO item named "make spi and sdio components coexist in one
> > build" was apparently addressed a long time ago,
ef that
has the goto.
Fixes: dee863b571b0 ("hv: export current Hyper-V clocksource")
Signed-off-by: Arnd Bergmann
---
arch/x86/hyperv/hv_init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/hyperv/hv_init.c b/arch/x86/hyperv/hv_init.c
index db64baf0e500..8bef
On Wed, Feb 15, 2017 at 8:54 AM, maomao xu wrote:
> drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning:
> cast to restricted __le16
> drivers/staging/rtl8192u/ieee80211/ieee80211_crypt_tkip.c:177:16: warning:
> cast to restricted __le16
> drivers/staging/rtl8192u/ieee80211
quot;)
Signed-off-by: Arnd Bergmann
---
drivers/staging/octeon/ethernet-rx.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/octeon/ethernet-rx.c
b/drivers/staging/octeon/ethernet-rx.c
index 7f8cf875157c..65a285631994 100644
--- a/drivers/staging/octeon/ethernet-rx.c
+++ b/dr
, gcc can see that
we don't evaluate the cell numbers for an missing ranges property.
Signed-off-by: Arnd Bergmann
---
drivers/staging/fsl-mc/bus/fsl-mc-bus.c | 22 +-
1 file changed, 9 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/fsl-mc/bus/fsl-mc-bus.c
b
ranteed to be NULL when vchiq_complete_bulk()
gets called.
This adds a CONFIG_OF Kconfig dependency, which is also technically correct
but nonobvious, and thus seems like a good fit for the warning.
Signed-off-by: Arnd Bergmann
---
drivers/staging/vc04_services/Kconfig | 1 +
1 file changed, 1 insertion(+
While debugging another problem I noticed that g_cache_line_size gets set
to sizeof(CACHE_LINE_SIZE), which is sizeof(int) or 4, while presumably
CACHE_LINE_SIZE (e.g. 32) was meant.
This initializes it the way it was meant.
Signed-off-by: Arnd Bergmann
---
drivers/staging/vc04_services
quot;)
Signed-off-by: Arnd Bergmann
---
Originally sent on Feb 17, but got no reply
This is one of two build regressions found by kernelci.org that are not
yet fixed in linux-next. Please apply
drivers/staging/octeon/ethernet-rx.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/
#x27;
We could work around that in the code, but since we already have a hard
dependency on x86, adding the ACPI dependency seems to be the easiest
solution.
Fixes: a49d25364dfb ("staging/atomisp: Add support for the Intel IPU v2")
Signed-off-by: Arnd Bergmann
---
drivers/staging/m
as PCI config space accessors should always return something
when PCI is enabled.
Fixes: a49d25364dfb ("staging/atomisp: Add support for the Intel IPU v2")
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/atomisp/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
e function definition [-Werror=old-style-definition]
This adds a 'void' keywork to silence the warning.
Fixes: a49d25364dfb ("staging/atomisp: Add support for the Intel IPU v2")
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css_para
/atomisp: Add support for the Intel IPU v2")
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/atomisp/i2c/ov2680.c | 37 +++---
1 file changed, 19 insertions(+), 18 deletions(-)
diff --git a/drivers/staging/media/atomisp/i2c/ov2680.c
b/drivers/staging/me
;:
platform/clock/vlv2_plat_clock.c:209:2: error: implicit declaration of function
'iounmap' [-Werror=implicit-function-declaration]
This includes the required header file.
Fixes: a49d25364dfb ("staging/atomisp: Add support for the Intel IPU v2")
Signed-off-by: Arnd Bergmann
ut not used [-Werror=unused-but-set-parameter]
It turns out that not only the parameter is unused but the entire function has
no
caller. Let's just remove it.
Fixes: a49d25364dfb ("staging/atomisp: Add support for the Intel IPU v2")
Signed-off-by: Arnd Bergmann
---
.../staging/media/at
Without CONFIG_EFI, the driver fails to call efivar_entry_get:
drivers/staging/built-in.o: In function `gmin_get_config_var':
(.text+0x1e3b): undefined reference to `efivar_entry_get'
Fixes: a49d25364dfb ("staging/atomisp: Add support for the Intel IPU v2")
Signed-o
the Intel IPU v2")
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/atomisp/pci/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/atomisp/pci/Kconfig
b/drivers/staging/media/atomisp/pci/Kconfig
index e8f67835d03d..a72421431c7a 1006
has no member named 'entity'
drivers/staging/media/atomisp/i2c/ov2722.c: In function 'ov2722_remove':
drivers/staging/media/atomisp/i2c/ov2722.c:1253:31: error: 'struct v4l2_subdev'
has no member named 'entity'
Let's just require MEDIA_CONTROLLER for all of
_services: Create new BCM_VIDEOCORE setting
for VideoCore services.")
Signed-off-by: Arnd Bergmann
---
drivers/staging/vc04_services/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vc04_services/Kconfig
b/drivers/staging/vc04_services/Kconfig
index
On Mon, Mar 20, 2017 at 4:05 PM, Stephen Hemminger
wrote:
> On Mon, 20 Mar 2017 10:32:19 +0100
> Arnd Bergmann wrote:
>
>> -void ia_css_dequeue_param_buffers(/*unsigned int pipe_num*/)
>> +void ia_css_dequeue_param_buffers(/*unsigned int pipe_num*/ void)
>> {
&
On Wed, Mar 22, 2017 at 3:49 AM, Arushi Singhal
wrote:
> @@ -232,7 +232,7 @@ static int ad7150_write_event_config(struct iio_dev
> *indio_dev,
> if (ret < 0)
> goto error_ret;
>
> - cfg = ret & ~((0x03 << 5) | (0x1 << 7));
> + cfg = ret & ~((0x03 << 5) | (BIT(
ild in any configuration, so this tries a
different fix to make sure the symbol is defined.
Fixes: 9ca98bd07748 ("Revert "staging: media: atomisp: fill properly
hmm_bo_type_strings when ION is disabled"")
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/atomisp/pci/ato
re, as we'd probably catch the problem before even getting here,
and other checks for the same condition already use v4l2_err.
Signed-off-by: Arnd Bergmann
---
drivers/staging/media/atomisp/i2c/mt9m114.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/driv
drop us a note to
> help improve the system]
>
> url:
> https://github.com/0day-ci/linux/commits/Arnd-Bergmann/staging-atomisp-avoid-false-positive-maybe-uninitialized-warning/20170329-023715
> config: i386-allmodconfig (attached as .config)
> compiler: gcc-6 (Debian
t it will
also be builtin for that case. This looks like a rather old bug, but
I have never seen this in randconfig testing until today.
Signed-off-by: Arnd Bergmann
---
drivers/staging/comedi/Kconfig | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/
t it will
also be builtin for that case. This looks like a rather old bug, but
I have never seen this in randconfig testing until today.
Signed-off-by: Arnd Bergmann
---
drivers/staging/comedi/Kconfig | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
v2: fix typo from first version, forgot
re, as we'd probably catch the problem before even getting here,
and other checks for the same condition already use v4l2_err.
Signed-off-by: Arnd Bergmann
---
v2: fix new build regression found by 0day kbuild bot
---
drivers/staging/media/atomisp/i2c/mt9m114.c | 16
1 file ch
On Mon, Apr 2, 2018 at 4:26 PM, Robert Jarzmik wrote:
> +
> +static const struct dma_slave_map pxa_slave_map[] = {
> + /* PXA25x, PXA27x and PXA3xx common entries */
> + { "pxa-pcm-audio", "ac97_mic_mono", PDMA_FILTER_PARAM(LOWEST, 8) },
> + { "pxa-pcm-audio", "ac97_aux_mono_in",
On Mon, Apr 2, 2018 at 4:26 PM, Robert Jarzmik wrote:
>
> +static struct pxa_ssp_info pxa_ssp_infos[] = {
> + { .dma_chan_rx_name = "ssp1_rx", .dma_chan_tx_name = "ssp1_tx", },
> + { .dma_chan_rx_name = "ssp1_rx", .dma_chan_tx_name = "ssp1_tx", },
> + { .dma_chan_rx_name = "ssp2
On Mon, Apr 2, 2018 at 6:35 PM, kbuild test robot wrote:
>
>drivers/mtd/nand/marvell_nand.c:2621:17: sparse: undefined identifier
> 'pxad_filter_fn'
>>> drivers/mtd/nand/marvell_nand.c:2621:17: sparse: call with no type!
>In file included from drivers/mtd/nand/marvell_nand.c:21:0:
>d
On Mon, Apr 2, 2018 at 4:26 PM, Robert Jarzmik wrote:
> Hi,
>
> This serie is aimed at removing the dmaengine slave compat use, and transfer
> knowledge of the DMA requestors into architecture code.
>
> This was discussed/advised by Arnd a couple of years back, it's almost time.
>
> The serie is d
On Tue, Apr 3, 2018 at 5:18 PM, Robert Jarzmik wrote:
> Arnd Bergmann writes:
>
>>> + { "smc911x.0", "rx", PDMA_FILTER_PARAM(LOWEST, -1) },
>>> + { "smc911x.0", "tx", PDMA_FILTER_PARAM(LOWEST, -1) },
>>> +
On Thu, Apr 5, 2018 at 8:29 AM, Ulf Hansson wrote:
> On 4 April 2018 at 21:56, Boris Brezillon wrote:
>> On Wed, 04 Apr 2018 21:49:26 +0200
>> Robert Jarzmik wrote:
>>
>>> Ulf Hansson writes:
>>>
>>> > On 2 April 2018 at 16:26, Robert Jarzmik wrote:
>>> >> Hi,
>>> >>
>>> >> This serie is aimed
ches, and are thus marked
> RFC.
This all looks fine to me.
Acked-by: Arnd Bergmann
Arnd
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
& STAGING [=y] && TYPEC_TCPM [=y]
Fixes: ce08eaeb6388 ("staging: typec: rt1711h typec chip driver")
Signed-off-by: Arnd Bergmann
---
drivers/staging/typec/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/typec/Kconfig b/drivers/stagi
[=y]
This adds the necessary dependency.
Fixes: 9bdf43b3d40f ("staging: fsl-dpaa2/rtc: add rtc driver")
Signed-off-by: Arnd Bergmann
---
drivers/staging/fsl-dpaa2/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/fsl-dpaa2/Kconfig
b/
On Fri, May 25, 2018 at 5:52 PM, Greg Kroah-Hartman
wrote:
> On Fri, May 25, 2018 at 05:38:25PM +0200, Arnd Bergmann wrote:
>> Using 'select' instead of the normal 'depends on' causes
>> a build issue with CONFIG_I2C=m:
>>
>> WARNING: unm
`rdma_disconnect'
net/9p/trans_rdma.o: In function `rdma_destroy_trans':
trans_rdma.c:(.text+0x830): undefined reference to `ib_destroy_qp'
trans_rdma.c:(.text+0x858): undefined reference to `ib_dealloc_pd'
Fixes: 9533b292a7ac ("IB: remove redundant INFINIBAND kconfig dependen
^~
AT_HWCAP
This adds back an explicit include for the header.
Signed-off-by: Arnd Bergmann
---
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/lustre/lnet/klnds/o
get_monotonic_boottime() is deprecated because it uses the
old 'timespec' structure. This replaces one of the last callers
with a call to ktime_get_boottime, which also simplifies it
a bit by avoiding a double conversion.
Signed-off-by: Arnd Bergmann
---
drivers/staging/rtl872
The timespec structure suffers from the y2038 overflow and should not
be used. This changes handle_vsoc_cond_wait() to use ktime_t directly.
Signed-off-by: Arnd Bergmann
---
drivers/staging/android/vsoc.c | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/drivers
On Tue, Jun 19, 2018 at 9:18 AM, Quytelda Kahja wrote:
> I'm fairly sure this doesn't matter functionally (yet), but perhaps
> 'notify_timestamp' should be updated to an 's64' type since ktime_to_us()
> returns a signed type?
I think it's fine either way, ktime_t is only signed to deal with
diffe
turn, this broke ARM allyesconfig builds. Replacing the
omap4_ctrl_pad_writel call with WARN_ON(1) won't make the
situation any worse than it already is, but fixes the build
problem.
Signed-off-by: Arnd Bergmann
Fixes: efde234674d9 ("ARM: OMAP4+: control: remove support for legacy pad
read/wr
s annotation.
Signed-off-by: Arnd Bergmann
diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm750.c
index 3c7ea95dd9f9..dbbb2f879a29 100644
--- a/drivers/staging/sm750fb/sm750.c
+++ b/drivers/staging/sm750fb/sm750.c
@@ -1250,7 +1250,7 @@ err_enable:
return -ENODEV;
set of allocation
macros and open-code the normal kernel interface calls.
Signed-off-by: Arnd Bergmann
Fixes: 99d56ff7c1c ("staging/lustre: Always try kmalloc first for
OBD_ALLOC_LARGE")
diff --git a/drivers/staging/lustre/lustre/include/obd_support.h
b/drivers/staging/lustre/lustre/i
will break in the year 2038, so we
> have to replace that code with more appropriate types.
> Here struct timeval is replaced with ktime_t.
>
> Signed-off-by: Ksenija Stanojevic
>
Reviewed-by: Arnd Bergmann
___
f SPI or MMC
enabled, and are only able to pick an interface that works. It also
adds all the missing dependencies for networking infrastructure
(cfg80211, wext, and ipv4).
In order to make it readable, I also took the liberty of re-indenting
the Kconfig file to the normal conventions.
Signed-of
.
Some minor changes are included that are necessary to remove
other code.
Signed-off-by: Arnd Bergmann
diff --git a/drivers/staging/wilc1000/Makefile
b/drivers/staging/wilc1000/Makefile
index 84bd975ff3be..4eda4cc69486 100644
--- a/drivers/staging/wilc1000/Makefile
+++ b/drivers/staging
For code that is integrated into mainline Linux, checks for
the OS platform make no sense, because we know that we
are on Linux.
This removes all checks and the associated dead code.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/Makefile | 3 +--
drivers/staging/wilc1000
x27;m touching them anyway.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/host_interface.c | 32 ---
drivers/staging/wilc1000/host_interface.h | 24 -
drivers/staging/wilc1000/linux_mon.c | 2 +-
drivers/staging/wil
The driver version is meaningless, and in particular does not
have to be passed from the Makefile. This removes the macros,
but leaves the behavior of printing the 10.2 version untouched
for the moment.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/Makefile | 6 --
drivers
This avoids the remaining warnings that one gets on a normal
build: unused variables, unused labels, and invalid printk
format strings.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/linux_wlan.c | 8 ++--
drivers/staging/wilc1000/linux_wlan_spi.c | 3 ---
drivers/staging
The driver has a simple wrapper around msleep, as well as
a more advanced sleep function that is unused. This removes
the unused code and the options to turn the feature on or
off.
A follow-up should rework the code to use msleep directly.
Signed-off-by: Arnd Bergmann
---
drivers/staging
The EXPORT_SYMTAB symbol has not been used in Linux for a very
long time, the driver does not need to set it.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/Makefile
b/drivers
The driver has its own API for semaphores. This should
be replaced with mutexes and completions, but for the moment
we can start by removing the obviously unused parts.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_osconfig.h | 1 -
drivers/staging/wilc1000/wilc_oswrapper.h
The abstraction for time in this driver is completely
unused, so remove it.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_osconfig.h | 1 -
drivers/staging/wilc1000/wilc_oswrapper.h | 5 -
drivers/staging/wilc1000/wilc_platform.h | 12 --
drivers/staging/wilc1000
The driver has a simple wrapper around timer_list, and an
optional but unused feature to make the timer periodic.
This removes support for the periodic timer and simplifies
the code around timers.
A follow-up should replace the remaining wrapper with
open-coded timers.
Signed-off-by: Arnd
The wilc_thread code is a very thin wrapper around kthread,
so just remove it and use kthread directly.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/Makefile | 2 +-
drivers/staging/wilc1000/host_interface.c | 16 ++--
drivers/staging/wilc1000/wilc_osconfig.h | 3
with some normal kernel API.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_msgqueue.c | 9 -
drivers/staging/wilc1000/wilc_msgqueue.h | 30 --
drivers/staging/wilc1000/wilc_osconfig.h | 3 ---
drivers/staging/wilc1000/wilc_oswrapper.h
The driver provides wrappers for a lot of string operations.
Some of them are unused, while others should be replaced
with normal kernel functions.
This replaces the unused ones for now, and leaves the other
ones for a later cleanup.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000
The driver contains its own abstraction for memory allocation,
most of it unused. This removes the unused parts, but the
rest should also be removed later.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_memory.c| 5 --
drivers/staging/wilc1000/wilc_memory.h| 93
hangs after
receiving a signal, as the original code did by looping around
down_interruptible.
The semaphores still need to be turned into mutexes as a
follow-up step.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/coreconfigurator.c | 29 +---
drivers/staging/wilc1000
Hi Greg,
This turned out to be more work than the first version, but I
think I've split it up enough sensibly to allow review, and
I've added a few more patches at the end.
Arnd
Arnd Bergmann (16):
staging: wilc1000: remove linux version checks
staging: wilc1000: remov
All the remaining features from the OS abstraction layer
are not used at all in the driver, so we can just remove
the remaining references to them.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_errorsupport.h | 17
drivers/staging/wilc1000/wilc_event.h| 123
For code that is integrated into mainline Linux, checks for
the kernel version make no sense, because we know which version
we are compiling against.
This removes all checks and the associated dead code.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/linux_wlan.c | 88
old object
files and succeeding with that.
Arnd
Arnd Bergmann (12):
staging: wilc1000: remove time wrapper
staging: wilc1000: remove unused string functions
staging: wilc1000: simplify msgqueue code
staging: wilc1000: remove unused memory handling code
staging: wilc1000: simpli
The EXPORT_SYMTAB symbol has not been used in Linux for a very
long time, the driver does not need to set it.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wilc1000/Makefile
b/drivers
with some normal kernel API.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_msgqueue.c | 9 -
drivers/staging/wilc1000/wilc_msgqueue.h | 30 --
drivers/staging/wilc1000/wilc_osconfig.h | 3 ---
drivers/staging/wilc1000/wilc_oswrapper.h
The driver has a simple wrapper around timer_list, and an
optional but unused feature to make the timer periodic.
This removes support for the periodic timer and simplifies
the code around timers.
A follow-up should replace the remaining wrapper with
open-coded timers.
Signed-off-by: Arnd
The abstraction for time in this driver is completely
unused, so remove it.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/Makefile | 2 +-
drivers/staging/wilc1000/wilc_osconfig.h | 1 -
drivers/staging/wilc1000/wilc_oswrapper.h | 5 -
drivers/staging/wilc1000
x27;m touching them anyway.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/host_interface.c | 32 ---
drivers/staging/wilc1000/host_interface.h | 24 -
drivers/staging/wilc1000/linux_mon.c | 2 +-
drivers/staging/wil
The driver has a simple wrapper around msleep, as well as
a more advanced sleep function that is unused. This removes
the unused code and the options to turn the feature on or
off.
A follow-up should rework the code to use msleep directly.
Signed-off-by: Arnd Bergmann
---
drivers/staging
All the remaining features from the OS abstraction layer
are not used at all in the driver, so we can just remove
the remaining references to them.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_errorsupport.h | 17
drivers/staging/wilc1000/wilc_event.h| 123
This avoids the remaining warnings that one gets on a normal
build: unused variables, unused labels, and invalid printk
format strings.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/linux_wlan.c | 8 ++--
drivers/staging/wilc1000/linux_wlan_spi.c | 3 ---
drivers/staging
The driver contains its own abstraction for memory allocation,
most of it unused. This removes the unused parts, but the
rest should also be removed later.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_memory.c| 5 --
drivers/staging/wilc1000/wilc_memory.h| 93
The driver provides wrappers for a lot of string operations.
Some of them are unused, while others should be replaced
with normal kernel functions.
This replaces the unused ones for now, and leaves the other
ones for a later cleanup.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000
The driver has its own API for semaphores. This should
be replaced with mutexes and completions, but for the moment
we can start by removing the obviously unused parts.
Signed-off-by: Arnd Bergmann
---
drivers/staging/wilc1000/wilc_osconfig.h | 1 -
drivers/staging/wilc1000/wilc_oswrapper.h
101 - 200 of 643 matches
Mail list logo
