On Sun, Nov 26, 2017 at 01:45:55AM +0300, Mike wrote:
>
> This is my task from Little Penguin :)
What does that mean?
>
> Mike (10):
> trailing whitespace fixed
> spaces before tabs are fixed
> trailing */ on a separate line fixed
> if-else code style fixed
> space prohibited before t
On Sun, Nov 26, 2017 at 01:45:56AM +0300, Mike wrote:
> Signed-off-by: Mike
> ---
> drivers/staging/irda/drivers/ali-ircc.c | 1002
> +++
> 1 file changed, 501 insertions(+), 501 deletions(-)
>
Hi,
This is the friendly patch-bot of Greg Kroah-Hartman. You have sen
On 1 December 2017 at 16:33, Kees Cook wrote:
> On Fri, Dec 1, 2017 at 7:34 AM, Greg Kroah-Hartman
> wrote:
>
>> And isn't there a specific %p modifier you should use for a kernel
>> pointer. I've lost the thread here for what should, or should not, be
>> done for kernel pointers these days base
On Fri, Dec 01, 2017 at 11:21:44AM -0800, Paul E. McKenney wrote:
> If cond_resched() returns false, then it has already invoked
> rcu_all_qs(). This is also invoked (now redundantly) by
> rcu_note_voluntary_context_switch(). This commit therefore changes
> cond_resched_rcu_qs() to invoke rcu_not
On Tue, Nov 28, 2017 at 06:42:17PM +, Will Deacon wrote:
> Hi all,
>
> The following two patches do the following:
>
> 1. Fix boot breakage reported on s390 caused by a8a217c22116
> 2. Kill off the break_lock field entirely, since it's not actually
> that useful
>
> I didn't go the
The TCP/IP transition from TCP_LISTEN to TCP_SYN_RECV and some other
transitions are not traced with tcp_set_state tracepoint.
In order to trace the whole tcp lifespans, two helpers are introduced,
void __tcp_set_state(struct sock *sk, int state)
void __sk_state_store(struct sock *sk, int newstate
From: root
Defines used in cases contain already shifted bits, so currentValue must not be
shifted.
Signed-off-by: Marcus Wolf
---
drivers/staging/pi433/rf69.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c
ind
> Il giorno 30 nov 2017, alle ore 22:21, Jens Axboe ha
> scritto:
>
> On 11/28/2017 02:37 AM, Paolo Valente wrote:
>> Commit a33801e8b473 ("block, bfq: move debug blkio stats behind
>> CONFIG_DEBUG_BLK_CGROUP") introduced two batches of confusing ifdefs:
>> one reported in [1], plus a similar o
On Sat, Dec 02, 2017 at 11:58:12AM +0200, Marcus Wolf wrote:
> From: root
I think something went wrong here :)
Also, you should never need to do kernel development as root...
greg k-h
Hi Greg,
yes, of course. Just got the wrong console to enter the commit command -
but when I recognized, it was already too late :-/ The patch already was
generated in user console.
I am wondering myself, where this From: comes from. I used 'git
send-email -1 --annotate'
How to prevent the
Hi!
next from 2017-12-01 was tested.
MMC is detected, but then I see wtl4030_bci: .. battery temperature
out of range, and machine halts. This is just before userspace would
be started.
4.15-rc1 seems to work ok on the box.
Any ideas?
mbox_msg->len is of type size_t and %d is incorrect format. Instead
use %zu for handling size_t correctly.
Signed-off-by: Nishanth Menon
---
drivers/firmware/ti_sci.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c
ind
Switch to SPDX licensing and drop the GPL text which comes redundant.
Signed-off-by: Nishanth Menon
---
Jassi,
I am not sure if you are planning on doing mailbox enmasse, if you are,
then feel free to drop this to take the overall changes.
Regards,
Nishanth Menon
drivers/mailbox/ti-msgmgr.c |
message->len is of type size_t and %d is incorrect format usage.
Instead use %zu for handling size_t correctly.
Signed-off-by: Nishanth Menon
---
drivers/mailbox/ti-msgmgr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mailbox/ti-msgmgr.c b/drivers/mailbox/ti-msgmg
Defines used in cases contain already shifted bits, so currentValue must not be
shifted.
signed_of_by: Marcus Wolf
---
drivers/staging/pi433/rf69.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c
index e69a215..12
On Fri, 1 Dec 2017, Andy Lutomirski wrote:
> This is obviously a hack. Either the patch should be adjusted back to
> the version I sent or trap_init should forcibly initialize all PMDs
> by something like __set_fixmap(..., __mkpte(0)); or however it's spelled.
I split it because the whole thing
On Fri, 1 Dec 2017, Andy Lutomirski wrote:
> We were manually configuring cpu_entry_area in the usermode tables.
> This was error-prone and wasted memory. (Not much memory, but
> still.) Instead, just reference the same pagetables.
>
> This avoids needing to keep the KPTI code and the normal
>
On Sat, 2 Dec 2017, Thomas Gleixner wrote:
> On Fri, 1 Dec 2017, Andy Lutomirski wrote:
>
> > This is obviously a hack. Either the patch should be adjusted back to
> > the version I sent or trap_init should forcibly initialize all PMDs
> > by something like __set_fixmap(..., __mkpte(0)); or howe
Defines used in cases contain already shifted bits, so currentValue must not be
shifted.
Signed-off-by: Marcus Wolf
---
drivers/staging/pi433/rf69.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/pi433/rf69.c b/drivers/staging/pi433/rf69.c
index e69a215..1
Hi Russel
The Grygorii has raised one Additional point (about netif_carrier_off) I just
didn't want to start before finishing the previous one.
On ifconfig-down the mac_config() called but with LINK=0.
The config has no any knowledge what is intention -- up or down and should be
done under disab
On Saturday 02 December 2017 11:14:56 Pavel Machek wrote:
> Hi!
>
> next from 2017-12-01 was tested.
>
> MMC is detected, but then I see wtl4030_bci: .. battery temperature
> out of range, and machine halts. This is just before userspace would
> be started.
>
> 4.15-rc1 seems to work ok on the b
On Mon, 27 Nov 2017 10:09:47 +0100
Fabrice Gasnier wrote:
> On 09/25/2017 07:26 PM, Jonathan Cameron wrote:
> > On Mon, 25 Sep 2017 11:16:04 +0100
> > Jonathan Cameron wrote:
> >
> >> On Mon, 25 Sep 2017 11:54:06 +0200
> >> Fabrice Gasnier wrote:
> >>
> >>> On 09/10/2017 06:00 PM, Jonathan
The function tilcdc_mode_hvtotal is local to the source and does not need
to be in global scope, so make it static.
drivers/gpu/drm/tilcdc/tilcdc_crtc.c:297:6: warning: no previous prototype for
'tilcdc_mode_hvtotal' [-Wmissing-prototypes]
uint tilcdc_mode_hvtotal(const struct drm_display_mode *
The Gemini pin controller can set drive strength for a few
select groups of pins (not individually). Implement this
for GMAC0 and 1 (ethernet ports), IDE and PCI.
Cc: devicet...@vger.kernel.org
Signed-off-by: Linus Walleij
---
The DT binding part is just using generic bindings, so should be
prett
On Tue, 28 Nov 2017 21:11:29 +0800
Yixun Lan wrote:
> On 11/11/17 08:37, Jonathan Cameron wrote:
> > On Tue, 7 Nov 2017 22:36:00 +0100
> > Martin Blumenstingl wrote:
> >
> >> Hi Yixun,
> >>
> >> On Tue, Nov 7, 2017 at 3:09 PM, Yixun Lan wrote:
> >>> patch [1/4]:
> >>> Fix wrong SARADC/SA
On Mon, 27 Nov 2017 11:54:49 +0100
Bartosz Golaszewski wrote:
> As discussed with Marc Zyngier: irq_sim_init() and its devres variant
> should return the base of the allocated interrupt range on success
> rather than 0. This will be modified later - first, change the way
> users handle the return
Since dev_dbg already depends on define DEBUG, there was no sense, to enclose
dev_dbg lines with #ifdef DEBUG.
Instead of removing #ifdef DEBUG, I introduced define DEBUG_FUNC_ENTRY. So now
it is
possible to switch of the function entry debug lines even while debug is
switched on.
Signed-off-by:
On Mon, 27 Nov 2017 12:51:48 +0100
Arnd Bergmann wrote:
> getnstimeofday() suffers from the overflow in y2038 on 32-bit
> architectures and requires a conversion into the nanosecond format that
> we want here.
>
> This changes ssp_parse_dataframe() to use ktime_get_real_ns() directly,
> which do
On Fri, 1 Dec 2017 12:10:58 +0100
Javier Martinez Canillas wrote:
> The driver doesn't have a struct of_device_id table but supported devices
> are registered via Device Trees. This is working on the assumption that a
> I2C device registered via OF will always match a legacy I2C device ID and
>
On Tue, Nov 28, 2017 at 7:23 PM, Colin King wrote:
> From: Colin Ian King
>
> Don't populate the read-only arrays edge_det_values, rise_values and
> fall_values on the stack but instead make them static and constify them.
> Makes the object code smaller by over 240 bytes:
>
> Before:
>text
On Wed, Nov 29, 2017 at 12:36 PM, Richard Fitzgerald
wrote:
> On 29/11/17 10:18, Linus Walleij wrote:
>>
>> On Thu, Nov 23, 2017 at 6:13 PM, Richard Fitzgerald
>> wrote:
>>
>>> +config MFD_MADERA_I2C
>>> + bool "Cirrus Logic Madera codecs with I2C"
>>> + select MFD_MADERA
>>> +
On Wed, Nov 29, 2017 at 2:25 PM, Mika Westerberg
wrote:
> When a GPIO is requested using gpiod_get_* APIs the intel pinctrl driver
> switches the pin to GPIO mode and makes sure interrupts are routed to
> the GPIO hardware instead of IOAPIC. However, if the GPIO is used
> directly through irqchip
On Wed, Nov 29, 2017 at 3:25 PM, Benjamin Gaignard
wrote:
> Uniformize STMicroelectronics copyrights header
>
> Signed-off-by: Benjamin Gaignard
Patch applied.
Yours,
Linus Walleij
Hello, my name is Harry and this is my first post here, hope I'm doing
this the right way, sorry if not ...
I'm not a subscriber to the full list yet so I understand I shall ask
you to be personally CCed.
I am following this as I do experience the same (or sort-a same) issue
with 4.14.2.
M
On Wed, 29 Nov 2017 22:31:12 +
Jeremy Cline wrote:
> Some BOSC0200 acpi_device-s describe two accelerometers in a single ACPI
> device. Check for a companion device and handle a second i2c_client
> if it is present.
+ Mika and Wolfram - please cc them on anything odd and i2c / ACPI related.
To increase the readability of the register accesses, the abstraction of the
helpers was increased from simple read and write to set bit, reset bit and
read modify write bit. In addition - according to the proposal from Walter
Harms from 20.07.2017 - instead of marcros inline functions were used.
On Thu, 30 Nov 2017 21:13:34 +0530
Arvind Yadav wrote:
> The platform_get_irq() function returns negative if an error occurs.
> zero or positive number on success. platform_get_irq() error checking
> for zero is not correct.
>
> Signed-off-by: Arvind Yadav
Applied to the togreg branch of iio.gi
On Sat, Dec 02, 2017 at 09:56:26AM +0100, Peter Zijlstra wrote:
> On Fri, Dec 01, 2017 at 11:21:44AM -0800, Paul E. McKenney wrote:
> > If cond_resched() returns false, then it has already invoked
> > rcu_all_qs(). This is also invoked (now redundantly) by
> > rcu_note_voluntary_context_switch().
I'm happy to ack your patch but you have to send it correctly.
You are missing a changelog (although it ought to be small due
to the trivial change) as well as you SoB tag. Please consult
Documentation/process/submitting-patches.rst.
You also need to Cc akpm as he routes such areas to Linus.
Tha
On Wed, Nov 29, 2017 at 6:37 PM, Florian Fainelli wrote:
> On 11/29/2017 09:02 AM, Tony Lindgren wrote:
>> Hmm well typically a device driver that loses it's context just does
>> save and restore of the registers in runtime PM suspend/resume
>> as needed. In this case it would mean duplicating th
On 2017-12-01 13:32:22 [-0500], Steven Rostedt wrote:
>
> Daniel Wagner reported a crash on the beaglebone black. This is a
> single CPU architecture, and does not have a functional:
> arch_send_call_function_single_ipi() and can crash if that is called.
>
> As it only has one CPU, it shouldn't b
Hi,
just a heads up to everyone that there are multiple regressions in
v4.15-rc1. For starters hostapd (=AP mode) doesn't work because of:
net: netlink: Update attr validation to require exact length for some types
https://git.kernel.org/linus/28033ae4e0f5
Jouni fixed this already in hostapd but
On Thu, 30 Nov 2017 22:43:42 +0100
Andreas Klinger wrote:
> Add buffer to device data struct and add trigger function
>
> Data format is quite simple:
> voltage - channel 0 32 Bit
> voltage - channel 1 32 Bit
> timestamp 64 Bit
>
> Using both channels at the sa
On Thu, 30 Nov 2017 22:44:45 +0100
Andreas Klinger wrote:
> return value in hx711_reset() should indicate status of dout
> otherwise the calling function is reporting an error als false positive
>
> wait up to 1 second until the device is in normal state
> by changing the channel on every trigge
git commit df04abfd181a ("fs/proc/kcore.c: Add bounce buffer for ktext
data") added a bounce buffer to avoid hardened usercopy
checks. Copying to the bounce buffer was implemented with a simple
memcpy() assuming that it is always valid to read from kernel memory
iff the kern_addr_valid() check pass
> On Dec 2, 2017, at 2:47 AM, Thomas Gleixner wrote:
>
>> On Sat, 2 Dec 2017, Thomas Gleixner wrote:
>>
>>> On Fri, 1 Dec 2017, Andy Lutomirski wrote:
>>>
>>> This is obviously a hack. Either the patch should be adjusted back to
>>> the version I sent or trap_init should forcibly initialize
To deal with checkpatch warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Jiaxun Yang
---
drivers/platform/x86/ideapad-laptop.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/platform/x86/ideapad-laptop.c
b/drivers/pl
To deal with checkpatch warning:
WARNING: Prefer using '"%s...", __func__' to using 'write_ec_cmd',
this function's name, in a string
Signed-off-by: Jiaxun Yang
---
drivers/platform/x86/ideapad-laptop.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/platform/x86/idea
To deal with checkpatch warnings:
WARNING: Prefer kstrto to single variable sscanf
WARNING: Missing a blank line after declarations
WARNING: Block comments use a trailing */ on a separate line
Signed-off-by: Jiaxun Yang
---
drivers/platform/x86/ideapad-laptop.c | 20 ++--
1 fil
Since Roger Jargoyhen reported that this device doesn't
have
Hardware rfkill switch, this patch add it to no_hw_rfkill_list to prevent radio
always
be blocked
Signed-off-by: Jiaxun Yang
---
drivers/platform/x86/ideapad-laptop.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/driver
On Sat, Dec 02, 2017 at 04:22:20AM -0800, Paul E. McKenney wrote:
> Because there are a few key places within RCU and rcutorture that need it.
> Without it, there are scenarios where the new cond_resched() never gets
> activated, and thus doesn't take effect.
Ah, I missed that interaction.
> The
On Sat, Dec 02, 2017 at 01:53:31PM +0100, Sebastian Andrzej Siewior wrote:
> On 2017-12-01 13:32:22 [-0500], Steven Rostedt wrote:
> >
> > Daniel Wagner reported a crash on the beaglebone black. This is a
> > single CPU architecture, and does not have a functional:
> > arch_send_call_function_sing
2017-11-30 21:20 UTC+01:00, Paul E. McKenney :
> Commit 5c4991e24c69 ("sched/isolation: Split out new
> CONFIG_CPU_ISOLATION=y config from CONFIG_NO_HZ_FULL") can result in RCU
> CPU stall warnings when running rcutorture with CONFIG_NO_HZ_FULL_ALL=y
> and CONFIG_CPU_ISOLATION=n. These warnings ar
"Gustavo A. R. Silva" wrote:
> Make use of the swap macro and remove unnecessary variable temp.
> This makes the code easier to read and maintain.
>
> This code was detected with the help of Coccinelle.
>
> Signed-off-by: Gustavo A. R. Silva
> Signed-off-by: Kalle Valo
Patch applied to ath-n
On Sat, 2 Dec 2017, Andy Lutomirski wrote:
>
>
> > On Dec 2, 2017, at 2:47 AM, Thomas Gleixner wrote:
> >
> >> On Sat, 2 Dec 2017, Thomas Gleixner wrote:
> >>
> >>> On Fri, 1 Dec 2017, Andy Lutomirski wrote:
> >>>
> >>> This is obviously a hack. Either the patch should be adjusted back to
>
On Fri, Dec 01, 2017 at 09:50:33AM -0600, Yazen Ghannam wrote:
> From: Yazen Ghannam
>
> Scalable MCA systems have various types of banks. The bank's type can
> determine how we handle errors from it. For example, if a bank represents
> a UMC then we will need to convert its address from a normal
On Fri, Dec 01, 2017 at 05:29:27PM +0100, Bartosz Golaszewski wrote:
> 2017-12-01 16:35 GMT+01:00 Sakari Ailus :
> > Hi Sven,
> >
> > On Fri, Dec 01, 2017 at 10:20:41AM -0500, Sven Van Asbroeck wrote:
> >> Thank you, it fixes the issue on the multi-address eeprom that I have
> >> access to.
> >>
>
On Fri, 1 Dec 2017 09:20:07 -0800
Davidlohr Bueso wrote:
>
> Hmm yeah that's pretty fishy, also shm_atime = 0, no?
>
Yeah, definitely, other data structure fields can also be
inconsistent, and applying not only to shmem, but also to
other ipc mechanisms.
Thank you for noting the typo, 'll se
Hi,
On Fri, Dec 01, 2017 at 01:37:12PM -0500, Sven Van Asbroeck wrote:
> From: Sakari Ailus
>
> The at24 driver creates dummy I2C devices to access offsets in the chip
> that are outside the area supported using a single I2C address. It is not
> meaningful to use runtime PM to such devices; the
On Sat, Dec 02, 2017 at 11:08:45AM +, Yan Markman wrote:
> Hi Russel
>
> The Grygorii has raised one Additional point (about netif_carrier_off)
> I just didn't want to start before finishing the previous one.
>
> On ifconfig-down the mac_config() called but with LINK=0.
> The config has no any
On Fri, Dec 01, 2017 at 06:16:26PM -0800, john.hubb...@gmail.com wrote:
> MAP_FIXED has been widely used for a very long time, yet the man
> page still claims that "the use of this option is discouraged".
I think we should continue to discourage the use of this option, but
I'm going to include som
The define FIFO_SIZE was moved to rf69_registers.h. Although it is not a
register,
it is a value, that is given by hardware (like the registers).
The define FIFO_THRESHOLD was moved to pi433_if.c, since it is a value, that is
freely choosen by the interface implementation. The better the response
On Thu, Nov 30, 2017 at 10:29:44PM -0800, Andy Lutomirski wrote:
> diff --git a/arch/x86/entry/entry_64.S b/arch/x86/entry/entry_64.S
> index caf74a1bb3de..28f4e7553c26 100644
> --- a/arch/x86/entry/entry_64.S
> +++ b/arch/x86/entry/entry_64.S
> @@ -180,14 +180,24 @@ ENTRY(entry_SYSCALL_64_trampoli
On Thu, Nov 30, 2017 at 4:55 AM, Andrew Jeffery wrote:
> General support for state persistence is added to gpiolib with the
> introduction of a new pinconf parameter to propagate the request to
> hardware. The existing persistence support for sleep is adapted to
> include hardware support if the
rf69_set_dc_cut_off_frequency_intern is used by rf69.c internally only.
Therefore removed the function from header and declared it staic in
the implemtation.
Signed-off-by: Marcus Wolf
---
drivers/staging/pi433/rf69.c |2 +-
drivers/staging/pi433/rf69.h |1 -
2 files changed, 1 insertion(
On Thu, Nov 30, 2017 at 4:55 AM, Andrew Jeffery wrote:
> Use the new pinconf parameter for state persistence to expose the
> associated capability of the Aspeed GPIO controller.
>
> Signed-off-by: Andrew Jeffery
> Reviewed-by: Joel Stanley
Patch applied.
Yours,
Linus Walleij
On Thu, Nov 30, 2017 at 9:46 AM, Benjamin Gaignard
wrote:
> Uniformize STMicroelectronics copyrights header
> Add SPDX identifier
>
> Signed-off-by: Benjamin Gaignard
> Acked-by: Alexandre TORGUE
> CC: Maxime Coquelin
Pulled out the v1 I applied and applied this instead.
Yours,
Linus Walleij
Hi Arvind,
thanks for the patch!
On Thu, Nov 30, 2017 at 3:12 PM, Arvind Yadav wrote:
> The platform_get_irq() function returns negative if an error occurs.
> zero or positive number on success. platform_get_irq() error checking
> for zero is not correct.
>
> Signed-off-by: Arvind Yadav
(...)
On Tue, Nov 28, 2017 at 8:49 AM, Masahiro Yamada
wrote:
> The driver has been in the tree for a while, but its binding document
> is missing. Hence, here it is.
>
> Signed-off-by: Masahiro Yamada
Patch applied with Rob's ACK.
Yours,
Linus Walleij
2017-12-02 12:43 GMT+01:00 Jonathan Cameron :
> On Mon, 27 Nov 2017 11:54:49 +0100
> Bartosz Golaszewski wrote:
>
>> As discussed with Marc Zyngier: irq_sim_init() and its devres variant
>> should return the base of the allocated interrupt range on success
>> rather than 0. This will be modified l
As described in the title, this patch fixes id_ds
inconsistency when ctl_stat executes concurrently
with some ds-changing function, e.g. shmat, msgsnd or
whatever.
For instance, if shmctl(IPC_STAT) is running concurrently
with shmat, following data structure can be returned:
{... shm_lpid = 0, shm
On Fri, Dec 01, 2017 at 02:44:51PM +0100, Quentin Schulz wrote:
> On AXP813/818, GPIO0 and GPIO1 can be used as LDO as (respectively)
> ldo_io0 and ldo_io1.
>
> Let's add the pinctrl properties to the said regulators.
>
> Signed-off-by: Quentin Schulz
Acked-by: Maxime Ripard
Maxime
--
Maxim
On Fri, Dec 01, 2017 at 02:44:47PM +0100, Quentin Schulz wrote:
> To prepare for patches that will add support for a new PMIC that has a
> different GPIO adc muxing value, add an adc_mux within axp20x_pctl
> structure and use it.
>
> Signed-off-by: Quentin Schulz
Acked-by: Maxime Ripard
Maxime
On Fri, Dec 01, 2017 at 02:36:04PM +0530, Jagan Teki wrote:
> Enable RTL8723BS WiFi chip on a64-olinuxino board:
> - WiFi SDIO interface is connected to MMC1
> - WiFi REG_ON pin connected to gpio PL2: attach to mmc-pwrseq
> - WiFi HOST_WAKE pin connected to gpio PL3
>
> To make rtl8723bs chip to w
On Fri, Dec 01, 2017 at 02:44:43PM +0100, Quentin Schulz wrote:
> +static void axp20x_gpio_set(struct gpio_chip *chip, unsigned offset,
> + int value)
> +{
checkpatch output:
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
> +static int axp20x_pmx_set_mux(struct p
Hi
On Fri, Dec 01, 2017 at 02:36:05PM +0530, Jagan Teki wrote:
> Since current tree support AXP803 regulators,
> replace fixed regulator vcc3v3 with AXP803 dcdc1 regulator.
>
> Tested mmc0 on sopine baseboard.
>
> Signed-off-by: Jagan Teki
Is it supposed to be a fix?
Maxime
--
Maxime Ripard
On Wed, 29 Nov 2017 00:16:56 +0900
Masanari Iida wrote:
> This patch fix following warning during 'make xmldocs'
>
> Documentation/driver-api/dmaengine/client.rst:188:
> WARNING: Title underline too short.
>
> Further APIs:
>
Applied to the docs tree, thanks.
jon
Revieved-by: Marcus Wolf
Am 11.10.2017 um 21:24 schrieb Marcin Ciupak:
This patch fixes the following checkpatch.pl error:
ERROR: code indent should use tabs where possible
in rf69.c file as requested by TODO file.
Additionally some style warnings remain valid here and could be fixed by
anot
On Thu, 30 Nov 2017 17:47:25 +0700
Nguyễn Thái Ngọc Duy wrote:
> Signed-off-by: Nguyễn Thái Ngọc Duy
> ---
> Documentation/process/kernel-docs.rst | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
So I'm reluctant to take patches without a changelog. One would hel
On Tue, 28 Nov 2017 06:16:21 +0100
Jonathan Neuschäfer wrote:
> In code blocks, :c:func:`...` annotations don't result in
> cross-references. Instead, they are rendered verbatim. Remove these
> broken annotations, and mark function calls with parentheses() again.
>
> Fixes: 76d40fae1351 ("gener
Leftover changes from patch [PATCH 2/5] staging: pi433: rf69.c style fix -
spaces required around
from Marcin Ciupak (11.10.2017)
Signed-of-by: Marcus Wolf
---
drivers/staging/pi433/rf69.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/pi433/rf69.c b/driv
On Sat, Dec 02, 2017 at 12:32:20PM +0200, Marcus Wolf wrote:
> Defines used in cases contain already shifted bits, so currentValue must not
> be shifted.
Please properly wrap your changelog text.
And you need a blank line before your signed-off-by line.
> signed_of_by: Marcus Wolf
"Signed-off
On Sat, Dec 02, 2017 at 12:51:54PM +0200, Marcus Wolf wrote:
> Defines used in cases contain already shifted bits, so currentValue must not
> be shifted.
> Signed-off-by: Marcus Wolf
Odd extra ' ' in your subject :(
And again, linewrap please.
thanks,
greg k-h
On Sat, Dec 02, 2017 at 01:45:50PM +0200, Marcus Wolf wrote:
> Since dev_dbg already depends on define DEBUG, there was no sense, to enclose
> dev_dbg lines with #ifdef DEBUG.
> Instead of removing #ifdef DEBUG, I introduced define DEBUG_FUNC_ENTRY. So
> now it is
> possible to switch of the funct
On Sat, Dec 02, 2017 at 02:21:32PM +0200, Marcus Wolf wrote:
> To increase the readability of the register accesses, the abstraction of the
> helpers was increased from simple read and write to set bit, reset bit and
> read modify write bit. In addition - according to the proposal from Walter
> Har
--
GREETINGS BELOVED
I AM BORTE ,I WAS DIAGNOSE WITH OVARIAN CANCER,WHICH DOCTORS HAVE
CONFIRMED THAT I HAVE ONLY FEW WEEKS TO LIVE, SO I HAVE DECIDED TO
DONATE EVERYTHING I HAVE TO THE ORPHANAGE AND THE POOR WIDOWS THROUGH
YOU AND YOUR HELP .PLEASE KINDLY REPLY ME ONLY ON MY EMAIL ADDRES
From: Ursula Braun
[ Upstream commit 4e1061f4a2bba1669c7297455c73ddafbebf2b12 ]
Commit 3e034725c0d8 ("net/smc: common functions for RMBs and send buffers")
merged handling of SMC receive and send buffers. It introduced sk_buf_size
as merged start value for size determination. But since sk_buf_si
From: Randy Dunlap
[ Upstream commit 1f3c790bd5989fcfec9e53ad8fa09f5b740c958f ]
line-range is supposed to treat "1-" as "1-endoffile", so
handle the special case by setting last_lineno to UINT_MAX.
Fixes this error:
dynamic_debug:ddebug_parse_query: last-line:0 < 1st-line:1
dynamic_debug:d
From: Eric Dumazet
[ Upstream commit 89ad2fa3f043a1e8daae193bcb5fe34d5f8caf28 ]
pcpu_freelist_pop() needs the same lockdep awareness than
pcpu_freelist_populate() to avoid a false positive.
[ INFO: SOFTIRQ-safe -> SOFTIRQ-unsafe lock order detected ]
switchto-defaul/12508 [HC0[0]:SC0[6]:HE0:
From: Arvind Yadav
[ Upstream commit bde533f2ea607cbbbe76ef8738b36243939a7bc2 ]
atm_dev_register() can fail here and passed parameters to free irq
which is not initialised. Initialization of 'dev->irq' happened after
the 'goto out_free_irq'. So using 'irq' insted of 'dev->irq' in
free_irq().
Si
From: Majd Dibbiny
[ Upstream commit 31fde034a8bd964a5c7c1a5663fc87a913158db2 ]
The UMR's QP is created by calling mlx5_ib_create_qp directly, and
therefore the send CQ and the recv CQ on the ibqp weren't assigned.
Assign them right after calling the mlx5_ib_create_qp to assure
that any access
On Wed, Nov 29, 2017 at 03:00:56PM +0100, Benjamin Gaignard wrote:
> 2017-11-28 14:32 GMT+01:00 Greg KH :
> > On Mon, Nov 06, 2017 at 04:59:45PM +0100, Benjamin Gaignard wrote:
> >> Instead a getting only one common device "/dev/ion" for
> >> all the heaps this patch allow to create one device
> >>
From: David Howells
[ Upstream commit f4b3526d83c40dd8bf5948b9d7a1b2c340f0dcc8 ]
The handler for the CB.ProbeUuid operation in the cache manager is
implemented, but isn't listed in the switch-statement of operation
selection, so won't be used. Fix this by adding it.
Signed-off-by: David Howell
From: Eric Dumazet
[ Upstream commit 89ad2fa3f043a1e8daae193bcb5fe34d5f8caf28 ]
pcpu_freelist_pop() needs the same lockdep awareness than
pcpu_freelist_populate() to avoid a false positive.
[ INFO: SOFTIRQ-safe -> SOFTIRQ-unsafe lock order detected ]
switchto-defaul/12508 [HC0[0]:SC0[6]:HE0:
From: Leon Romanovsky
[ Upstream commit 7d7d065a5eec7e218174d5c64a9f53f99ffdb119 ]
Chelsio cxgb4 HW is big-endian, hence there is need to properly
annotate r2 and stag fields as __be32 and not __u32 to fix the
following sparse warnings.
drivers/infiniband/hw/cxgb4/qp.c:614:16:
warning: in
From: Leon Romanovsky
[ Upstream commit 7d7d065a5eec7e218174d5c64a9f53f99ffdb119 ]
Chelsio cxgb4 HW is big-endian, hence there is need to properly
annotate r2 and stag fields as __be32 and not __u32 to fix the
following sparse warnings.
drivers/infiniband/hw/cxgb4/qp.c:614:16:
warning: in
From: Mark Bloch
[ Upstream commit 5f22a1d87c5315a98981ecf93cd8de226cffe6ca ]
Maximal message should be used as a limit to the max message payload allowed,
without the headers. The ConnectX-3 check is done against this value includes
the headers. When the payload is 4K this will cause the NIC to
From: Keefe Liu
[ Upstream commit ca29fd7cce5a6444d57fb86517589a1a31c759e1 ]
When process the outbound packet of ipv6, we should assign the master
device to output device other than input device.
Signed-off-by: Keefe Liu
Acked-by: Mahesh Bandewar
Signed-off-by: David S. Miller
Signed-off-by:
From: Majd Dibbiny
[ Upstream commit 31fde034a8bd964a5c7c1a5663fc87a913158db2 ]
The UMR's QP is created by calling mlx5_ib_create_qp directly, and
therefore the send CQ and the recv CQ on the ibqp weren't assigned.
Assign them right after calling the mlx5_ib_create_qp to assure
that any access
From: Arvind Yadav
[ Upstream commit bde533f2ea607cbbbe76ef8738b36243939a7bc2 ]
atm_dev_register() can fail here and passed parameters to free irq
which is not initialised. Initialization of 'dev->irq' happened after
the 'goto out_free_irq'. So using 'irq' insted of 'dev->irq' in
free_irq().
Si
1 - 100 of 336 matches
Mail list logo