This is a patch to add support for
NXP rtc pca8565
Signed-off-by: Venkat Prashanth B U
---
---
drivers/rtc/Kconfig | 9 ++
drivers/rtc/Makefile | 1 +
drivers/rtc/rtc-pca8565.c | 221 ++
3 files changed, 231 insertions(+)
diff --git a/d
'devm_ioremap_resource()' returns an error pointer in case of error, not
NULL. So test it with IS_ERR.
Signed-off-by: Christophe JAILLET
---
'return -EIO;' could also be turned into 'return PTR_ERR(pcubase);' in
order to propagate the error value.
---
arch/arm/mach-zx/zx296702-pm-domain.c | 2 +-
Hi Alex,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 2a26d99b251b8625d27aed14e97fc10707a3a81f
commit: ebb5e78cc63417a35254a791de66e1cc84f963cc MIPS: Initial implementation
of a VDSO
date: 12 months ago
config:
'clk_register_fixed_rate()' returns an error pointer in case of error, not
NULL. So test it with IS_ERR.
Signed-off-by: Christophe JAILLET
---
arch/mips/ath79/clock.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/mips/ath79/clock.c b/arch/mips/ath79/clock.c
index cc3a1
Hi Guenter,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 2a26d99b251b8625d27aed14e97fc10707a3a81f
commit: 398c7500a1f5f74e207bd2edca1b1721b3cc1f1e MIPS: VDSO: Fix build error
with binutils 2.24 and earlier
date
'sun4i_layers_init()' returns an error pointer in case of error, not
NULL. So test it with IS_ERR.
Signed-off-by: Christophe JAILLET
---
drivers/gpu/drm/sun4i/sun4i_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/sun4i/sun4i_drv.c
b/drivers/gpu/drm/sun4
On 2016/10/27 23:18, Greg Kroah-Hartman wrote:
> On Mon, Oct 24, 2016 at 11:59:20AM +0800, Kefeng Wang wrote:
>> Hi Greg, any more comments, thanks.
>
> Never wait, just resend if you have comments and you know you have to
> fix them up...
>
Hi Greg, as I mentioned in previous mail, compatible
Hi Dave,
few fixes for 4.9. I tagged this on the plane over a slow mosh
connection while travelling to Plumbers so I might have done something
wrong, please check more carefully than usually. For example I had to
redo the signed tag because of some whitespace damage.
Please let me know if there a
Hi James,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 2a26d99b251b8625d27aed14e97fc10707a3a81f
commit: 034827c727f7f3946a18355b63995b402c226c82 MIPS: Fix -mabi=64 build of
vdso.lds
date: 3 weeks ago
config: mi
On Sat 29-10-16 13:09:25, Linus Torvalds wrote:
> On Sat, Oct 29, 2016 at 12:17 PM, Al Viro wrote:
> >
> > AFAICS, the possibility of dropping the last reference to struct file
> > before ->write_iter() has returned is fundamentally broken. I might be
> > missing something subtle here, but...
>
Any news?
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."
On Sat 29-10-16 16:10:27, Wei Yongjun wrote:
> From: Wei Yongjun
>
> Fixes the following sparse warning:
>
> drivers/block/brd.c:411:15: warning:
> symbol 'rd_size' was not declared. Should it be static?
It should not. It is used in arch/arm/.
With the current kernel release, wpa_supplicant results in authentication
failure
with a Cube i9 tablet (a Surface Pro like device):
Successfully initialized wpa_supplicant
wlp0s20f0u7i2: SME: Trying to authenticate with 10:fe:ed:62:7a:78
(SSID='localre' freq=2417 MHz)
wlp0s20f0u7i2: SME: Trying
This fix enables the same sequence of init behaviour as the alternative
working driver for the wireless rtl8723bu IC at
https://github.com/lwfinger/rtl8723bu
For exampe rtl8xxxu_init_device is now called each time
userspace wpa_supplicant is executed instead of just once when
modprobe is executed.
Code tests show data returned by rtl8xxxu_read8(priv, REG_CR), used to set
macpower, is never 0xea. It is only ever 0x01 (first time after modprobe)
using wpa_supplicant and 0x00 thereafter using wpa_supplicant. These results
occurs with 'Fix for authentication failure' [PATCH 1/2] in place.
Whate
root_task_group defined in sched/core.c is enclosed by
CONFIG_CGROUP_SCHED, so the export declaration should
also be enclosed.
Signed-off-by: Haishuang Yan
---
include/linux/init_task.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/linux/init_task.h b/include/linu
On Sat, Oct 29, 2016 at 05:07:44PM -0600, Shuah Khan wrote:
> On 10/29/2016 07:48 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.8.6 release.
> > There are 125 patches in this series, all will be posted as a response
> > to this one. If anyone has any issu
On Sat, Oct 29, 2016 at 05:44:38PM -0700, Guenter Roeck wrote:
> On 10/29/2016 06:48 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.8.6 release.
> > There are 125 patches in this series, all will be posted as a response
> > to this one. If anyone has any i
FYI: I added this report to the list of regressions for Linux 4.9. I'll
watch this thread for further updates on this issue to document progress
in my weekly reports. Please let me know via regressi...@leemhuis.info
in case the discussion moves to a different place (bugzilla or another
mail thread
Trigger an IRQ first with a test IRQ handler to find out if it really
works. Disable the IRQ if not.
This prevents hang when incorrect IRQ was specified by user.
Signed-off-by: Ondrej Zary
---
drivers/scsi/g_NCR5380.c | 31 +--
1 file changed, 29 insertions(+), 2 de
Use standard probe_irq_on() and probe_irq_off() functions instead of own
implementation.
This prevents warning messages like this in the kernel log:
genirq: Flags mismatch irq 1. (NCR-probe) vs. 0080 (i8042)
Move the IRQ trigger code to a separate function so it can be used for
other
On 27.10.2016 10:10, Kalle Valo wrote:
> (Adding Thorsten because of a serious regression and Steven because he
> tried to fix something in the same commit)
Many thx.
I added this report to the list of regressions for Linux 4.9. I'll
watch this thread for further updates on this issue to document
On Sun 30-10-16 10:44:37, Jan Kara wrote:
> On Sat 29-10-16 13:09:25, Linus Torvalds wrote:
> > On Sat, Oct 29, 2016 at 12:17 PM, Al Viro wrote:
> > >
> > > AFAICS, the possibility of dropping the last reference to struct file
> > > before ->write_iter() has returned is fundamentally broken. I mi
Hi Venkat,
On 30 October 2016 at 09:03, VENKAT PRASHANTH B U
wrote:
> This is a patch to add support for
> NXP rtc pca8565
>From a quick look the PCA8565 register layout looks very similar to
PCF8563 which already has a driver (rtc-pcf8563).
Can you use that driver instead of creating a new one
FYI: I added this report to the list of regressions for Linux 4.9. I'll
watch this thread for further updates on this issue to document progress
in my weekly reports. Please let me know via regressi...@leemhuis.info
in case the discussion moves to a different place (bugzilla or another
mail thread
Hi !
in your commit f5b586909581 ("rtlwifi: btcoexist: Modify driver to support
BT coexistence in rtl8723be") you introduced a if/else where both branches
are the same but the comment in the else branch suggests that this might be
unintended.
from code review only I can´t say what the int
Hi Venkat,
On 19 October 2016 at 19:41, VENKAT PRASHANTH B U
wrote:
> This is a patch to add support for
> NXP rtc pca21125
The register layout PCA21125 looks similar to PCF8563 which already
has a driver.
Would it be possible to support the PCA21125 device with the rtc-pcf8563 driver?
You will
Hi
On Sat, 29 Oct 2016 21:37:04 +0200
Julia Lawall wrote:
> Use DEVICE_ATTR_RW for read-write attributes. This simplifies the
> source code, improves readbility, and reduces the chance of
> inconsistencies.
>
...
>
> - DEVICE_ATTR(x, \(0644\|S_IRUGO|S_IWUSR\), x_show, x_store);
> + DEVICE_ATT
From: Borislav Petkov
... in order to fix this randconfig build warning:
drivers/idle/i7300_idle.c: In function ‘i7300_idle_stop’:
./include/asm-generic/bug.h:117:24: warning: ‘got_ctl’ is used uninitialized
in this function [-Wuninitialized]
int __ret_warn_once = !!(condition); \
Hi Rich,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 2a26d99b251b8625d27aed14e97fc10707a3a81f
commit: b4214e41b7152b1964a3421a40251d202ae2d2c0 sh: add SMP support for J2
date: 3 months ago
config: sh-j2_defconf
On Sun, 30 Oct 2016, Jarkko Nikula wrote:
> Hi
>
> On Sat, 29 Oct 2016 21:37:04 +0200
> Julia Lawall wrote:
>
> > Use DEVICE_ATTR_RW for read-write attributes. This simplifies the
> > source code, improves readbility, and reduces the chance of
> > inconsistencies.
> >
> ...
> >
> > - DEVICE_AT
Hi,
BTW, memory allocation in 'sun4i_layers_init()' looks spurious,
especially the use of 'layer' in the for loop.
Just my 2 cents.
I also forgot to say that we could propagate the error code returned by
sun4i_layers_init instead of returning -EINVAL unconditionally
CJ
Le 30/10/2016 à
John Heenan writes:
> Code tests show data returned by rtl8xxxu_read8(priv, REG_CR), used to set
> macpower, is never 0xea. It is only ever 0x01 (first time after modprobe)
> using wpa_supplicant and 0x00 thereafter using wpa_supplicant. These results
> occurs with 'Fix for authentication failure'
Hi Mark,
On Sat, Oct 29, 2016 at 8:40 PM, Mark Brown wrote:
> On Fri, Oct 28, 2016 at 09:41:44PM +0200, Axel Haslam wrote:
>
>> i think today each time an event occurs a notification is sent with the
>> corresponding flag(s) set.
>
> Right, so I think the problem here is actually that you called
Below is the list of build error/warning regressions/improvements in
v4.9-rc3[1] compared to v4.8[2].
Summarized:
- build errors: +176/-3
- build warnings: +1671/-1232
JFYI, when comparing v4.9-rc3[1] to v4.9-rc2[3], the summaries are:
- build errors: +175/-38
- build warnings: +356/-420
JFYI: I added this report to the list of regressions for Linux 4.9. I'll
watch this thread for further updates on this issue to document progress
in my weekly reports. Please let me know via regressi...@leemhuis.info
in case the discussion moves to a different place (bugzilla or another
mail thread
Hi Faisal !
your commit 4097351a47c5 ("i40iw: virtual channel handling files")
adds the following lines in i40iw_vchnl_recv_pf()
+ if (vchnl_msg->iw_op_ver != I40IW_VCHNL_OP_GET_VER_V0)
+ vchnl_pf_send_get_ver_resp(dev, vf_id, vchnl_msg);
+ else
On 10/30/2016 11:57 AM, Jens Axboe wrote:
> On 10/29/2016 10:10 AM, Wei Yongjun wrote:
>> From: Wei Yongjun
>>
>> Fixes the following sparse warning:
>>
>> drivers/block/brd.c:411:15: warning:
>> symbol 'rd_size' was not declared. Should it be static?
>
> We just tried this, it's used in arm:
>
>
The vchiq_copy_from_user function is not portable
and is consider "bad practice." Replace this function
with a callback based mechanism that is passed downward
on the stack. When it is actually time to copy the data,
the callback is called to copy the data into the message.
This callback is prov
On Sun, Oct 30, 2016 at 1:08 PM, Geert Uytterhoeven
wrote:
> Below is the list of build error/warning regressions/improvements in
> v4.9-rc3[1] compared to v4.8[2].
>
> Summarized:
> - build errors: +176/-3
+ /home/kisskb/slave/src/arch/arc/include/asm/atomic.h: Error: bad
instruction `llock
On Fri, Oct 28, 2016 at 02:25 PM GMT, Tom Herbert wrote:
> On Fri, Oct 28, 2016 at 1:32 AM, Jakub Sitnicki wrote:
>> On Thu, Oct 27, 2016 at 10:35 PM GMT, Tom Herbert wrote:
>>> On Mon, Oct 24, 2016 at 2:28 AM, Jakub Sitnicki wrote:
Same as for the transmit path, let's do our best to ensure
On 23/10/16 23:39, Peter Rosin wrote:
> From: Jonathan Cameron
>
> A large number of attributes can only take a limited range of values.
> Currently in IIO this is handled by directly registering additional
> *_available attributes thus providing this information to userspace.
>
> It is desirabl
On Sun, 2016-10-30 at 05:41 +0100, Andrey Konovalov wrote:
> Sorry, the warning is still there.
>
> I'm not sure adding sched_annotate_sleep() does anything, since it's
> defined as (in case CONFIG_DEBUG_ATOMIC_SLEEP is not set):
> # define sched_annotate_sleep() do { } while (0)
Thanks again for
On 23/10/16 23:39, Peter Rosin wrote:
> Specifically a helper for reading the available maximum raw value of a
> channel and a helper for forwarding read_avail requests for raw values
> from one iio driver to an iio channel that is consumed.
>
> These rather specific helpers are in turn built with
Hi! Here is my second regression report for Linux 4.9. It lists 14
regressions I'm aware of. 4 of them are new; 3 got fixed since last weeks
report.
As always: Are you aware of any other regressions? Then please let me
know (simply CC regressi...@leemhuis.info). And please tell me if there
is any
On 23/10/16 23:39, Peter Rosin wrote:
> Example:
>
> $ cat '/sys/bus/iio/devices/iio:device0/out_resistance_raw_available'
> [0 1 256]
>
> Meaning: min 0, step 1 and max 256.
>
> Signed-off-by: Peter Rosin
Looks good. On comment inline, but nothing to change.
This series is likely to just be w
On 23/10/16 23:39, Peter Rosin wrote:
> Signed-off-by: Peter Rosin
I'm happy with this, but it's odd enough I'd like some devicetree maintainer
feedback ideally.
Thanks,
Jonathan
> ---
> .../devicetree/bindings/iio/dac/dpot-dac.txt | 41
> ++
> MAINTAINERS
On 23/10/16 23:39, Peter Rosin wrote:
> It is assumed that the dpot is used as a voltage divider between the
> current dpot wiper setting and the maximum resistance of the dpot. The
> divided voltage is provided by a vref regulator.
>
> .--.
>.---. | |
>
On 23/10/16 23:39, Peter Rosin wrote:
> Signed-off-by: Peter Rosin
I'm happy with this, but again it's odd enough I'd like some input from
a device tree bindings maintainer.
Thanks,
Jonathan
> ---
> .../bindings/iio/adc/envelope-detector.txt | 54
> ++
> MAINTAINERS
Alexandre Bailon writes:
> From: Petr Kulhavy
>
> This adds the function musb_get_mode() to get the DT property "dr_mode"
>
> Signed-off-by: Petr Kulhavy
> Acked-by: Sergei Shtylyov
> Signed-off-by: Alexandre Bailon
> Tested-by: David Lechner
> ---
> drivers/usb/musb/musb_core.c | 19 ++
On Sat, 29 Oct 2016 16:38:43 -0400
Paul Gortmaker wrote:
> The Makefile currently controlling compilation of this code is obj-y,
> meaning that it currently is not being built as a module by anyone.
>
> Lets remove the couple traces of modular usage, so that when reading
> the driver there is no
On 23/10/16 23:39, Peter Rosin wrote:
> The DAC is used to find the peak level of an alternating voltage input
> signal by a binary search using the output of a comparator wired to
> an interrupt pin. Like so:
> _
> | \
> input +-->---|
These were previously set in dw_hdmi_connector_get_modes but this
isn't called when the EDID is overridden. This can be seen in
drm_helper_probe_single_connector_modes. Using the
drm_kms_helper.edid_firmware parameter therefore always resulted in
silence, even when providing the very same EDID that
Thanks for your reply.
The code was tested on a Cube i9 which has an internal rtl8723bu.
No other devices were tested.
I am happy to accept in an ideal context hard coding macpower is
undesirable, the comment is undesirable and it is wrong to assume the
issue is not unique to the rtl8723bu.
You
On Sat, Oct 29, 2016 at 06:32:38PM -0700, Eric Wheeler wrote:
> On Thu, 27 Oct 2016, Jens Axboe wrote:
>
> > On 10/27/2016 05:27 PM, Eric Wheeler wrote:
> > > Hi Jens,
> > >
> > > Please pull this v4.9-rc2 based series of bcache updates for v4.9-rc3:
> > > (You may disregard the previous -rc1-base
On 25/10/16 17:25, Fabrice Gasnier wrote:
> Add per channel "smpr" IIO extended attribute, to allow sampling
> time configuration.
>
> Signed-off-by: Fabrice Gasnier
First thing is that any attributes need to be documented in
Documentation/ABI/testing/sysfs-bus-iio-*
Secondly this feels rather l
On 25/10/16 17:25, Fabrice Gasnier wrote:
> Add trigger polarity extended attribute.
>
> Signed-off-by: Fabrice Gasnier
ABI always needs documenting. Add docs also makes it easy to review
without diving into the code.
This will also need a convincing argument for why it doesn't belong in the
de
On 25/10/16 17:25, Fabrice Gasnier wrote:
> STM32 ADC may use EXTi signals routed internally as trigger source
> for conversions. Configure them as interrupt to configure this path
> in HW to adc.
> Note: interrupt handler isn't required here, and corresponding interrupt
> can be kept masked at ext
On Sun, Oct 30, 2016 at 04:49:30PM +0800, Kefeng Wang wrote:
>
>
> On 2016/10/27 23:18, Greg Kroah-Hartman wrote:
> > On Mon, Oct 24, 2016 at 11:59:20AM +0800, Kefeng Wang wrote:
> >> Hi Greg, any more comments, thanks.
> >
> > Never wait, just resend if you have comments and you know you have t
On Mon, Oct 03, 2016 at 02:13:45AM +0300, Serge Semin wrote:
> Hello linux folks,
>
> This driver primarily is developed to give an access to EEPROM of IDT
> PCIe-switches. Such switches provide a simple SMBus interface to perform
> IO-operations from/to EEPROM, which is located at private (al
Hi,
when trying to (ab)use the firmware loading interface in a local kernel module
(in order to load the EEPROM content into a PCIE card), I noticed that the
manual firmware loading interface (the one from
/sys/class/firmware//{loading,data}) was broken.
After instrumenting the kernel I noticed a
From: Yves-Alexis Perez
wait_for_completion_interruptible_timeout() return value is either
-ERESTARTSYS (in case it was interrupted), 0 (in case the timeout expired)
or the number of jiffies left until timeout. The return value is stored in
a long, but in _request_firmware_load() it's silently ca
On 10/30/16 04:42, Borislav Petkov wrote:
> From: Borislav Petkov
>
> ... in order to fix this randconfig build warning:
>
> drivers/idle/i7300_idle.c: In function ‘i7300_idle_stop’:
> ./include/asm-generic/bug.h:117:24: warning: ‘got_ctl’ is used
> uninitialized in this function [-Wuniniti
On Thu, Oct 27, 2016 at 06:11:56PM -0400, James Simmons wrote:
> From: John L. Hammond
>
> Add a new regular file ioctl LL_IOC_FUTIMES_3 similar to futimes() but
> which allows setting of all three inode timestamps. Use this ioctl
> during HSM restore to ensure that the volatile file has the same
On 10/30/2016 08:00 AM, Kent Overstreet wrote:
On Sat, Oct 29, 2016 at 06:32:38PM -0700, Eric Wheeler wrote:
On Thu, 27 Oct 2016, Jens Axboe wrote:
On 10/27/2016 05:27 PM, Eric Wheeler wrote:
Hi Jens,
Please pull this v4.9-rc2 based series of bcache updates for v4.9-rc3:
(You may disregard t
On Sat, Oct 29, 2016 at 09:36:57PM +0200, Julia Lawall wrote:
> Use DEVICE_ATTR_RO for read only attributes. This simplifies the source
> code, improves readbility, and reduces the chance of inconsistencies.
>
> The semantic patch that makes this change is as follows:
> (http://coccinelle.lip6.fr
On 10/29/2016 10:28 PM, Christophe JAILLET wrote:
'blk_mq_alloc_request()' returns an error pointer in case of error, not
NULL. So test it with IS_ERR.
Fixes: fd8383fd88a2 ("nbd: convert to blkmq")
Applied, thanks.
--
Jens Axboe
On Thu, Oct 27, 2016 at 06:11:46PM -0400, James Simmons wrote:
> From: Gregoire Pichon
>
> This patch is the main client part of a new feature that supports
> multiple modify metadata RPCs in parallel. Its goal is to improve
> metadata operations performance of a single client, while maintening
>
On Sun, Oct 30, 2016 at 02:56:06AM +0530, Manoj Sawai wrote:
> Fixed a coding style error involving use of spaces instead of tabs.
>
> Signed-off-by: Manoj Sawai
> ---
> drivers/staging/ks7010/ks7010_sdio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Doesn't apply to my tree at all,
> But what would you expect printf error checking to tell a user?
I wonder that the error detection and corresponding exception handling
was not corrected for the affected source files of build-time tools so far.
https://www.securecoding.cert.org/confluence/display/c/EXP12-C.+Do+not+ignore+values+
> On 23/10/16 23:39, Peter Rosin wrote:
> > Specifically a helper for reading the available maximum raw value of a
> > channel and a helper for forwarding read_avail requests for raw values
> > from one iio driver to an iio channel that is consumed.
> >
> > These rather specific helpers are in tu
On 25/10/16 17:25, Fabrice Gasnier wrote:
> This patch adds support for STMicroelectronics STM32 MCU's analog to
> digital converter.
>
> Signed-off-by: Fabrice Gasnier
Hi Fabrice,
Sometimes I hate SoC ADCs. For some reason the hardware designers seem to
try and throw everything and the kitchen
On Sun, 30 Oct 2016, Jonathan Cameron wrote:
> On 23/10/16 23:39, Peter Rosin wrote:
> > It is assumed that the dpot is used as a voltage divider between the
> > current dpot wiper setting and the maximum resistance of the dpot. The
> > divided voltage is provided by a vref regulator.
> >
> >
On 10/27/2016 03:19 PM, Sean Young wrote:
On Wed, Oct 26, 2016 at 01:16:16PM -0500, Nathan Zimmer wrote:
On 10/25/2016 03:41 PM, Sean Young wrote:
On Mon, Oct 24, 2016 at 04:49:25PM -0500, Nathan Zimmer wrote:
[1.565062] serial8250: ttyS1 at I/O 0x2f8 (irq = 3) is a 16550A
The isa probe
On 10/28/2016 09:29 PM, Colin Ian King wrote:
On 28/10/16 20:21, Manfred Spraul wrote:
Hi Colin,
On 10/28/2016 08:11 PM, Colin King wrote:
[...]
--- a/ipc/sem.c
+++ b/ipc/sem.c
@@ -1839,7 +1839,7 @@ SYSCALL_DEFINE4(semtimedop, int, semid, struct
sembuf __user *, tsops,
max = 0;
On 25/10/16 17:25, Fabrice Gasnier wrote:
> Add optional DMA support to STM32 ADC.
> Use dma cyclic mode with at least two periods.
>
> Signed-off-by: Fabrice Gasnier
Few little bits inline, but looks superficially good (my dma knowledge isn't
really up to reviewing this.) Lars can you take a loo
'dma_buf_map_attachment()' can not return NULL, so there is no need to
check for it.
Also add a space in order to improve layout.
Signed-off-by: Christophe JAILLET
---
drivers/gpu/drm/armada/armada_gem.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/ar
free_pardevice() is called by parport_unregister_device() and already frees
pp->pdev->name, don't try to do it again.
This bug causes kernel crashes.
I found and verified this with KASAN and some added pr_emerg()s:
[ 60.316568] pp_release: pp->pdev->name == 88039cb264c0
[ 60.316692] free
On Sat, Oct 29, 2016 at 4:28 AM, Nicolas Iooss
wrote:
> In order to test that the name of a resource begins with "pmem", call
> strncmp() with 4 as length instead of 3 to match the whole prefix.
>
> Fixes: 16660eaea0cc ("libnvdimm, namespace: update label implementation
> for multi-pmem")
> Signed
'dma_buf_map_attachment()' can not return NULL, so there is no need to
check for it.
Signed-off-by: Christophe JAILLET
---
drivers/gpu/drm/tegra/gem.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gpu/drm/tegra/gem.c
index 95e622e31931..19bf9cdf1f1
On Sun, Oct 30, 2016 at 11:52:31AM +0100, Jan Kara wrote:
> Hum, the additional refcount patch oopses on me when running generic/323,
> I'll have to board to my flight to US shortly so I won't be able to send it
> soon - maybe when I'm transferring in Denver ;).
I've got a version that works and t
On Sun, Oct 30, 2016 at 5:08 AM, Thorsten Leemhuis
wrote:
> JFYI: I added this report to the list of regressions for Linux 4.9. I'll
> watch this thread for further updates on this issue to document progress
> in my weekly reports. Please let me know via regressi...@leemhuis.info
> in case the dis
On 2016-10-30 14:44, Jonathan Cameron wrote:
> On 23/10/16 23:39, Peter Rosin wrote:
>> The DAC is used to find the peak level of an alternating voltage input
>> signal by a binary search using the output of a comparator wired to
>> an interrupt pin. Like so:
>> _
>>
On 10/29/2016 04:16 PM, Sean Young wrote:
On Fri, Oct 28, 2016 at 02:42:25PM -0500, Nathan Zimmer wrote:
On Thu, Oct 27, 2016 at 09:19:16PM +0100, Sean Young wrote:
On Wed, Oct 26, 2016 at 01:16:16PM -0500, Nathan Zimmer wrote:
On 10/25/2016 03:41 PM, Sean Young wrote:
On Mon, Oct 24, 2016
[Re: [PATCH 3/3] s390: char: make slp_ctl explicitly non-modular] On 30/10/2016
(Sun 14:43) Martin Schwidefsky wrote:
[...]
> That idea behind the patch makes sense to me. But the built robot
> reports a new warning which should be fixed. If that is done:
>
> Acked-by: Martin Schwidefsky
The
Some toolchains fail to build mips images with the following build error.
arch/mips/vdso/gettimeofday.c:1:0: error: '-march=r3000' requires '-mfp32'
This is seen, for example, with the 'mipsel-linux-gnu-gcc (Debian 6.1.1-9)
6.1.1 20160705' toolchain as used by the 0Day build robot when building
d
If CONFIG_ETRAX_AXISFLASHMAP is not configured, the flash rescue image
object file is empty. With recent versions of binutils, this results
in the following build error.
cris-linux-objcopy: error:
the input file 'arch/cris/boot/rescue/rescue.o' has no sections
This is seen, for example, w
From: Rafael J. Wysocki
Make the device suspend/resume part of the core system
suspend/resume code use device links to ensure that supplier
and consumer devices will be suspended and resumed in the right
order in case of async suspend/resume.
The idea, roughly, is to use dpm_wait() to wait for a
On Mon, Oct 24, 2016 at 6:09 AM, Matt Fleming wrote:
>
> From d2c17f46686076677da3bf04caa2f69d654f8d8a Mon Sep 17 00:00:00 2001
> From: Matt Fleming
> Date: Thu, 20 Oct 2016 22:17:21 +0100
> Subject: [PATCH] x86/efi: Prevent mixed mode boot corruption with
> CONFIG_VMAP_STACK
>
> Booting an EFI
From: Rafael J. Wysocki
Modify the runtime PM framework to use device links to ensure that
supplier devices will not be suspended if any of their consumer
devices are active.
The idea is to reference count suppliers on the consumer's resume
and drop references to them on its suspend. The inform
From: Rafael J. Wysocki
Subject:
If the device has no links to suppliers that should be used for
runtime PM (links with DEVICE_LINK_PM_RUNTIME set), there is no
reason to walk the list of suppliers for that device during
runtime suspend and resume.
Add a simple mechanism to detect that case and
Hi,
Let me quote from the previous intro messages for this series first:
> > Time for another update. :-)
> >
> > Fewer changes this time, mostly to address issues found by Lukas and
> > Marek.
> >
> > The most significant one is to make device_link_add() cope with the case
> > when
> > the con
From: Rafael J. Wysocki
Currently, there is a problem with taking functional dependencies
between devices into account.
What I mean by a "functional dependency" is when the driver of device
B needs device A to be functional and (generally) its driver to be
present in order to work properly. Thi
From: Rafael J. Wysocki
Add an internal wrapper around __device_release_driver() that will
acquire device locks and do the necessary checks before calling it.
The next patch will make use of it.
Signed-off-by: Rafael J. Wysocki
Tested-by: Marek Szyprowski
---
drivers/base/dd.c | 30 +++
On Sun, Oct 30, 2016 at 8:44 AM, Jann Horn wrote:
> free_pardevice() is called by parport_unregister_device() and already frees
> pp->pdev->name, don't try to do it again.
>
> This bug causes kernel crashes.
>
> I found and verified this with KASAN and some added pr_emerg()s:
>
> [ 60.316568] pp
On 25/10/16 20:20, Paul Kocialkowski wrote:
> This adds devicetree support for the si7020 iio driver. Since it works
> well without requiring any additional property, its compatible string is
> added to the trivial i2c devices bindings list.
>
> Signed-off-by: Paul Kocialkowski
Applied to the tog
On Sun, Oct 30, 2016 at 07:54:44AM -0700, Randy Dunlap wrote:
> depends on PCI
> (no CONFIG_ prefix)
Whoops!
Thanks. FWIW, it didn't fire even with "depends on CONFIG_PCI" anymore.
Didn't investigate why though...
---
From: Borislav Petkov
Date: Sun, 30 Oct 2016 14:39:28 +0100
Subject: [P
On Sunday, October 30, 2016 05:22:13 PM Rafael J. Wysocki wrote:
> Hi,
>
> Let me quote from the previous intro messages for this series first:
>
> > > Time for another update. :-)
> > >
> > > Fewer changes this time, mostly to address issues found by Lukas and
> > > Marek.
> > >
> > > The most
Hi Alex,
As you're well-aware, your commit
8539b37acef73949861a16808b60cb8b5b9b3bab (drm/nouveau/gr: use
NVIDIA-provided external firmwares) broke tons of existing setups for
people who were using extracted firmware files (stored in the
"nouveau" firmware directory) as a result of nouveau's ctxsw
Otherwise we might dereference an already freed file and/or inode
when aio_complete is called before we return from the read_iter or
write_iter method.
Signed-off-by: Christoph Hellwig
---
fs/aio.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/fs/aio.c b/fs/aio.c
index 1157e13..0aa71d3 1
1 - 100 of 387 matches
Mail list logo
