This header requires the definition of struct radix_tree_root, but
does not need to know anything about other radix tree stuff.
Include instead of to
reduce the header dependency.
Signed-off-by: Masahiro Yamada
---
include/linux/blk-cgroup.h | 2 +-
1 file
The header include/linux/mlx5/driver.h requires the definition of
struct radix_tree_root, but does not need to know anything about
other radix tree stuff.
Include instead of to
reduce the number of included header files.
Also, add to include/linux/mlx5/gp.h where radix
tree accessors are
The motivation of this series is to cut down unnecessary header
dependency in terms of radix tree.
Sub-systems or drivers that use radix-tree for data management
typically embed struct radix_tree_root in their data structures,
like this:
struct foo {
...
struct radix_tree_root
The headers
- include/linux/mlx4/device.h
- drivers/net/ethernet/mellanox/mlx4/mlx4.h
require the definition of struct radix_tree_root, but do not need to
know anything about other radix tree stuff.
Include instead of to
reduce the header dependency.
While we are here, let's add missing
From: Masahiro Yamada
Date: Mon, 9 Oct 2017 01:10:11 +0900
> The headers
> - include/linux/mlx4/device.h
> - drivers/net/ethernet/mellanox/mlx4/mlx4.h
> require the definition of struct radix_tree_root, but do not need to
> know anything about other radix tree
On Mon, 2017-10-09 at 02:29 +0900, Masahiro Yamada wrote:
> The idea is simple; include necessary headers explicitly.
Try that for kernel.h
There's a reason aggregation of #includes is useful.
On Thu, Oct 5, 2017 at 5:31 PM, Hans de Goede wrote:
> Add an entry to make myself the maintainer of the PEAQ WMI hotkeys driver.
Pushed to my review queue, thanks!
>
> Signed-off-by: Hans de Goede
> ---
> MAINTAINERS | 6 ++
> 1 file changed, 6
On Sun, Oct 8, 2017 at 6:48 PM, Andy Shevchenko
wrote:
> On Sat, Oct 7, 2017 at 7:59 AM, Mario Limonciello
> wrote:
>> This splits up the dell-smbios driver into two drivers:
>> * dell-smbios
>> * dell-smbios-smm
>>
>> dell-smbios can
On Sun, 2016-05-15 at 19:56 -0700, Guenter Roeck wrote:
[...]
> For 3.4 and 3.2 kernels to build with binutils v2.24, it would be necessary to
> apply patch c02263063362 ("MIPS: Refactor 'clear_page' and 'copy_page'
> functions").
> It applies cleanly to 3.4, but has a Makefile conflict in 3.2.
On Sun, Oct 08, 2017 at 02:37:41PM +0200, Thorsten Leemhuis wrote:
> Hi! Find below my second regression report for Linux 4.14. It lists 8
> regressions I'm currently aware of. One regression was fixed since last
> weeks report. One was in there that shouldn't have been there.
>
> == Current
On Mon, Oct 09, 2017 at 01:10:01AM +0900, Masahiro Yamada wrote:
<...>
>
> By splitting out the radix_tree_root definition,
> we can reduce the header file dependency.
>
> Reducing the header dependency will help for speeding the kernel
> build, suppressing unnecessary recompile of objects during
Hi Andy,
On 10/8/2017 11:38 AM, Andy Shevchenko wrote:
On Sun, Oct 8, 2017 at 1:19 AM,
wrote:
From: Kuppuswamy Sathyanarayanan
Currently, update_no_reboot_bit() function implemented in this driver
uses
Hello, Boqun.
On Sun, Oct 08, 2017 at 05:02:23PM +0800, Boqun Feng wrote:
> Josef reported a HARDIRQ-safe -> HARDIRQ-unsafe lock order detected by
> lockdep:
>
> | [ 1270.472259] WARNING: HARDIRQ-safe -> HARDIRQ-unsafe lock order detected
> | [ 1270.472783] 4.14.0-rc1-xfstests-12888-g76833e8
Fix BUG() calls to use BUG_ON(conditional) macros.
This was found using make coccicheck M=net/core on linux next
tag next-20170929.
Signed-off-by: Tim Hansen
---
net/core/skbuff.c | 15 ++-
1 file changed, 6 insertions(+), 9 deletions(-)
diff --git
According to sst-haswell-ipc.c, the driver may sleep under a spinlock,
and the function call path is:
hsw_irq_thread (acquire the spinlock)
hsw_process_notification
hsw_log_message
mutex_lock --> may sleep
This bug is found by my static analysis tool and my code review.
Thanks,
According to stmmac_main.c, the driver may sleep under a spinlock,
and the function call path is:
stmmac_suspend (acquire the spinlock)
stmmac_disable_all_queues
napi_disable
might_sleep --> may sleep
msleep --> may sleep
This bug is found by my static analysis tool and my code
On Thu, Sep 28, 2017 at 10:25:40AM -0500, Brandon Streiff wrote:
> +void mv88e6xxx_port_txtstamp(struct dsa_switch *ds, int port,
> + struct sk_buff *clone, unsigned int type)
> +{
> + struct mv88e6xxx_chip *chip = ds->priv;
> + struct mv88e6xxx_port_hwtstamp *ps =
On Fri, Sep 29, 2017 at 05:43:23AM -0400, Richard Cochran wrote:
> I happy to see this series. I just finished porting an out-of-tree
> PHC driver for the Marvell mv88e635x, and I want to mainline it, but I
> also have a few uglies.
This series looks really good. I won't even post my mine, as
On Sat, Aug 19, 2017 at 2:23 AM, Rajat Jain wrote:
> On Fri, Aug 18, 2017 at 10:47 AM, Rajneesh Bhardwaj
> wrote:
>> On Fri, Aug 18, 2017 at 08:17:32PM +0300, Andy Shevchenko wrote:
>>> On Fri, Aug 18, 2017 at 5:58 PM, Rajneesh Bhardwaj
>>>
On Sun, Oct 8, 2017 at 1:19 AM,
wrote:
> From: Kuppuswamy Sathyanarayanan
>
> Currently, update_no_reboot_bit() function implemented in this driver
> uses mutex_lock() to protect its register updates. But
On Sun, 8 Oct 2017 01:42:15 -0700
Joel Fernandes wrote:
> Hi Steve,
>
> > "Joel Fernandes (Google)" wrote:
> [..]
> > Also could you let me know what is the correct behavior of the filters
> > after a module being traced is unloaded, are the filters
Hi Steve,
On Sun, Oct 8, 2017 at 11:42 AM, Steven Rostedt wrote:
> On Sun, 8 Oct 2017 01:42:15 -0700
[..]
>> > "Joel Fernandes (Google)" wrote:
>> [..]
>> > Also could you let me know what is the correct behavior of the filters
>> > after a module
On Tue, 2016-06-14 at 23:35 +0100, Sudip Mukherjee wrote:
> On Tuesday 14 June 2016 11:16 PM, Ben Hutchings wrote:
> > On Tue, 2016-06-14 at 22:56 +0100, Sudip Mukherjee wrote:
> > > On Sunday 12 June 2016 10:34 PM, Ben Hutchings wrote:
> > > > This is the start of the stable review cycle for the
Variable reg is unsigned so checking whether it is less than zero is not
necessary.
Signed-off-by: Christos Gkekas
---
drivers/leds/leds-tca6507.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/leds/leds-tca6507.c b/drivers/leds/leds-tca6507.c
On Mon, Oct 09, 2017 at 02:29:15AM +0900, Masahiro Yamada wrote:
> 2017-10-09 2:00 GMT+09:00 David Miller :
> > From: Masahiro Yamada
> > Date: Mon, 9 Oct 2017 01:10:11 +0900
> >
> >> The headers
> >> - include/linux/mlx4/device.h
> >> -
On Sun, Oct 08, 2017 at 06:06:19PM +0530, Anand Moon wrote:
> Hi Krzysztof,
>
> On 6 October 2017 at 12:08, Krzysztof Kozlowski wrote:
> > On Fri, Oct 6, 2017 at 6:36 AM, Anand Moon wrote:
> >> update the usbdrd link control and phy contol clks.
> >
> >
On Sun, Oct 08, 2017 at 03:03:32AM +0100, Al Viro wrote:
> On Sun, Oct 08, 2017 at 01:56:08AM +0100, Al Viro wrote:
>
> > What's more, we need to be careful about resize vs. drain. Right now it's
> > on list_lrus_mutex, but if we drop that around actual resize of an
> > individual
> > list_lru,
Hello,
Thanks for checking the patch.
I missed the #else part of he CONFIG_OF #ifdef previously.
I made the code properly depend on CONFIG_OF now.
I am not familiar with this code base. When time allows I would like to
contribute by refactoring code in this area.
Let me know if you have
Signed-off-by: Artur Lorincz
---
include/linux/of.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/linux/of.h b/include/linux/of.h
index cfc34117fc92..250ab4d55fac 100644
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -538,6 +538,11 @@ const char
After the parsing memory leak fixes.
Signed-off-by: Ulf Magnusson
---
scripts/kconfig/zconf.tab.c_shipped | 562 +++-
1 file changed, 294 insertions(+), 268 deletions(-)
diff --git a/scripts/kconfig/zconf.tab.c_shipped
The following strings would leak before this change:
- option env="LEAKED"
- option defconfig_list="LEAKED"
These come in the form of T_WORD tokens and are always allocated on the
heap in zconf.l. Free them.
Summary from Valgrind on 'menuconfig' (ARCH=x86) before the fix:
This is just for completeness to get rid of the last memory leak
currently generated during parsing for ARCH=x86. The symbol
DVB_NETUP_UNIDVB in drivers/media/pci/netup_unidvb/Kconfig currently has
two help strings, and we leak the first one.
Summary from Valgrind on 'menuconfig' (ARCH=x86)
Hello,
This patchset plugs all memory leaks that occur in the parser (zconf.y) while
parsing the x86 Kconfigs (and likely the other ARCHes too). I noticed that
Kconfig is pretty leaky while working on the fix for 'm' before MODULES
(http://www.spinics.net/lists/linux-kbuild/msg15606.html).
The
From: Haiyang Zhang
Date: Fri, 6 Oct 2017 08:33:56 -0700
> From: Haiyang Zhang
>
> The patch set simplifies the existing hash level switching code for
> UDP. It also adds the support for changing TCP hash level. So users
> can switch
When propagating dependencies from parents after parsing, an expression
node is allocated if the parent symbol is a 'choice'. This node was
never freed.
Outline of leak:
if (sym && sym_is_choice(sym)) {
...
*Allocate (in this case only)*
Hello,
This patchset plugs all memory leaks that are due to expressions not being
freed, when parsing the x86 Kconfigs (and likely the other ARCHes too). 336 KB
are leaked in total.
Together with the memory leaks plugged in the parser in
https://lkml.org/lkml/2017/10/8/126, this plugs all memory
Only the E_NOT operand and not the E_NOT node itself was freed, due to
accidentally returning too early in expr_free(). Outline of leak:
switch (e->type) {
...
case E_NOT:
expr_free(e->left.expr);
return;
...
}
*Never
expr_trans_compare() always allocates and returns a new expression,
giving the following leak outline:
...
*Allocate*
basedep = expr_trans_compare(basedep, E_UNEQUAL, _no);
...
for (menu = parent->next; menu; menu = menu->next) {
...
Variable active_slots is unsigned so checking whether it is less than
zero is not necessary.
Signed-off-by: Christos Gkekas
---
sound/soc/bcm/cygnus-ssp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/bcm/cygnus-ssp.c
On 10/08/2017 11:12 AM, Ben Hutchings wrote:
On Sun, 2016-05-15 at 19:56 -0700, Guenter Roeck wrote:
[...]
For 3.4 and 3.2 kernels to build with binutils v2.24, it would be necessary to
apply patch c02263063362 ("MIPS: Refactor 'clear_page' and 'copy_page'
functions").
It applies cleanly to
On 10/08/2017 11:49 AM, Guenter Roeck wrote:
On 10/08/2017 11:12 AM, Ben Hutchings wrote:
On Sun, 2016-05-15 at 19:56 -0700, Guenter Roeck wrote:
[...]
For 3.4 and 3.2 kernels to build with binutils v2.24, it would be necessary to
apply patch c02263063362 ("MIPS: Refactor 'clear_page' and
Many macros in dwmac-ipq806x are unused and should be removed.
Moreover gmac->id is an unsigned variable and therefore checking
whether it is less than zero is redundant.
Signed-off-by: Christos Gkekas
---
drivers/net/ethernet/stmicro/stmmac/dwmac-ipq806x.c | 14
This header contains references to spinlock_t and lockdep_is_held(),
both of which are defined in
Include instead of to reduce
the header dependency.
Signed-off-by: Masahiro Yamada
---
include/linux/radix-tree.h | 2 +-
1 file changed, 1 insertion(+), 1
The idea is similar to the separation of
and .
The users of radix tree typically embed struct radix_tree_root in
their data structures, but those headers do not need to know anything
about the other radix tree stuff like radix_tree_node, accessors.
Including pulls in lots of bloat while struct
This header requires the definition of struct radix_tree_root, but
does not need to know anything about other radix tree stuff.
Include instead of to
reduce the header dependency.
Signed-off-by: Masahiro Yamada
---
include/linux/iocontext.h | 2 +-
1 file
This header requires the definition of struct radix_tree_root, but
does not need to know anything about other radix tree stuff.
Include instead of to
reduce the header dependency.
Signed-off-by: Masahiro Yamada
---
drivers/sh/intc/internals.h | 2 +-
1 file
Prior to this fix, zconf.y did not free symbol names from zconf.l in
these contexts:
- After T_CONFIG ('config LEAKED')
- After T_MENUCONFIG ('menuconfig LEAKED')
- After T_SELECT ('select LEAKED')
- After T_IMPLY ('imply LEAKED')
- After T_DEFAULT in a
If a 'mainmenu' entry appeared in the Kconfig files, two things would
leak:
- The 'struct property' allocated for the default "Linux Kernel
Configuration" prompt.
- The string for the T_WORD/T_WORD_QUOTE prompt after the
T_MAINMENU token, allocated on the heap
The 'source_stmt' nonterminal takes a 'prompt', which consists of either
a T_WORD or a T_WORD_QUOTE, both of which are always allocated on the
heap in zconf.l and need to have their associated strings freed. Free
them.
The existing code already makes sure to always copy the string, but add
a
2017-10-09 2:00 GMT+09:00 David Miller :
> From: Masahiro Yamada
> Date: Mon, 9 Oct 2017 01:10:11 +0900
>
>> The headers
>> - include/linux/mlx4/device.h
>> - drivers/net/ethernet/mellanox/mlx4/mlx4.h
>> require the definition of struct
It's tricky to figure out what it does (and how) without staring at the
code for a long time. Document it to make it more transparent.
No functional changes. Only comments added.
Signed-off-by: Ulf Magnusson
---
scripts/kconfig/menu.c | 38
Variable byte_offset is unsigned so checking whether it is greater or
equal to zero is redundant.
Signed-off-by: Christos Gkekas
---
drivers/input/touchscreen/atmel_mxt_ts.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hi Artur,
[auto build test ERROR on v4.14-rc3]
[also build test ERROR on next-20170929]
[cannot apply to glikely/devicetree/next]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
This semantic patch detects duplicate arrays declared using BQ27XXX_DATA
within a single structure. It is currently specific to the file
drivers/power/supply/bq27xxx_battery.c. Nevertheless, having the script in
the kernel will allow others to check their code if the data structures
change in
Rename battery.cocci as check_bq27xxx_data.cocci and adjust the warning
messages to indicate that some structures are the same, in response to
feedback from Liam Breck.
Although check_bq27xxx_data.cocci says is requires Coccinelle 1.0.7, which
is not yet released, it actually works with the
Drop bettery.cocci as a more specific filename was preferred.
Signed-off-by: Julia Lawall
---
scripts/coccinelle/api/battery.cocci | 161 ---
1 file changed, 161 deletions(-)
diff --git a/scripts/coccinelle/api/battery.cocci
On 10/8/17 5:56 AM, Kirill A. Shutemov wrote:
On Sat, Oct 07, 2017 at 04:22:10AM +0800, Yang Shi wrote:
When passing "huge=always" option for mounting tmpfs, THP is supposed to
be allocated all the time when it can fit, but when the available space is
smaller than the size of THP (2MB on
CONFIG_SLABINFO is removed, but slabinfo dump is not applicable to slob,
protect slbinfo stats from !CONFIG_SLOB to avoid the below compile
error reported by 0-DAY kernel test:
mm/slab_common.o: In function `dump_unreclaimable_slab':
>> mm/slab_common.c:1298: undefined reference to
On Sun, Oct 08, 2017 at 07:44:18PM +0100, Christos Gkekas wrote:
> Variable byte_offset is unsigned so checking whether it is greater or
> equal to zero is redundant.
>
> Signed-off-by: Christos Gkekas
Yep - looks sensible to me.
Signed-off-by: Nick Dyer
Mistakenly sent the patch previously with a missing semicolon.
Apologies.
Fix BUG() calls to use BUG_ON(conditional) macros.
This was found using make coccicheck M=net/core on linux next
tag next-20170929
Signed-off-by: Tim Hansen
---
net/core/skbuff.c | 15
On Sun, 8 Oct 2017, Sebastian Reichel wrote:
> Hi,
>
> On Sun, Oct 08, 2017 at 09:18:39PM +0200, Julia Lawall wrote:
> > Rename battery.cocci as check_bq27xxx_data.cocci and adjust the warning
> > messages to indicate that some structures are the same, in response to
> > feedback from Liam
The new stack validator in objdump doesn't like directly assigning r11
to rsp, warning with something like:
warning: objtool: chacha20_4block_xor_ssse3()+0xa: unsupported stack pointer
realignment
warning: objtool: chacha20_8block_xor_avx2()+0x6: unsupported stack pointer
realignment
This
On 10/7/17 9:20 AM, Borislav Petkov wrote:
> On Fri, Oct 06, 2017 at 08:06:00PM -0500, Brijesh Singh wrote:
>> Add a include file which defines the ioctl and command id used for
>> issuing SEV platform management specific commands.
>>
>> Cc: Paolo Bonzini
>> Cc: "Radim
Hi,
On Sun, Oct 08, 2017 at 10:44:50PM +0200, Julia Lawall wrote:
> On Sun, 8 Oct 2017, Sebastian Reichel wrote:
> > Hi,
> >
> > On Sun, Oct 08, 2017 at 09:18:39PM +0200, Julia Lawall wrote:
> > > Rename battery.cocci as check_bq27xxx_data.cocci and adjust the warning
> > > messages to indicate
Filters are supposed to be saved for init functions, however module init memory
is freed after module init which also frees the ftrace dyn records. However, the
filters are still left as is, this patch uses the mod_map infrastructure
added by Steven to clear the hashes of the saved init functions
menu_end_entry() is empty and completely unused as far as I can tell:
$ git log -G menu_end_entry --oneline
a02f057 [PATCH] kconfig: improve error handling in the parser
1da177e Linux-2.6.12-rc2
Last one is the initial Git commit, where menu_end_entry() is empty as
well.
Variable size_presence_reg·is unsigned so checking whether it is less
than zero is redundant.
Signed-off-by: Christos Gkekas
---
drivers/input/rmi4/rmi_driver.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/rmi4/rmi_driver.c
On Donnerstag, 5. Oktober 2017 05:35:29 CEST Namhyung Kim wrote:
> On Sun, Oct 01, 2017 at 04:30:50PM +0200, Milian Wolff wrote:
> > The inline_node structs are maintained by the new dso->inlines
> > tree. This in turn keeps ownership of the fake symbols and
> > srcline string representing an
On Donnerstag, 5. Oktober 2017 05:43:38 CEST Namhyung Kim wrote:
> On Sun, Oct 01, 2017 at 04:30:57PM +0200, Milian Wolff wrote:
> > When no inlined frames could be found for a given address,
> > we did not store this information anywhere. That means we
> > potentially do the costly inliner lookup
Hi,
On Sun, Oct 08, 2017 at 09:18:39PM +0200, Julia Lawall wrote:
> Rename battery.cocci as check_bq27xxx_data.cocci and adjust the warning
> messages to indicate that some structures are the same, in response to
> feedback from Liam Breck.
>
> Although check_bq27xxx_data.cocci says is requires
Hi Krzysztof,
On 8 October 2017 at 21:20, Krzysztof Kozlowski wrote:
> On Sun, Oct 08, 2017 at 06:11:12PM +0530, Anand Moon wrote:
>> Hi Krzysztof,
>>
>> On 6 October 2017 at 12:12, Krzysztof Kozlowski wrote:
>> > On Fri, Oct 6, 2017 at 6:36 AM, Anand Moon
On 10/5/17 3:56 PM, Borislav Petkov wrote:
> On Wed, Oct 04, 2017 at 08:13:52AM -0500, Brijesh Singh wrote:
>> Define Secure Encrypted Virtualization (SEV) key management command id
>> and structure. The command definition is available in SEV KM [1] spec
>> 0.14.
>>
>> [1]
On Sun, Oct 08, 2017 at 06:47:46PM +0300, Vladimir Davydov wrote:
> On Sun, Oct 08, 2017 at 03:03:32AM +0100, Al Viro wrote:
> > On Sun, Oct 08, 2017 at 01:56:08AM +0100, Al Viro wrote:
> >
> > > What's more, we need to be careful about resize vs. drain. Right now it's
> > > on list_lrus_mutex,
> -Original Message-
> From: Andy Shevchenko [mailto:andy.shevche...@gmail.com]
> Sent: Sunday, October 8, 2017 1:13 PM
> To: Limonciello, Mario
> Cc: dvh...@infradead.org; LKML ; Platform Driver
>
Variable spp->cpp_index is unsigned so checking whether it is less than
zero is redundant.
Signed-off-by: Christos Gkekas
---
drivers/scsi/eata.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/scsi/eata.c b/drivers/scsi/eata.c
index
On Sat, Oct 07, 2017 at 09:20:46AM +0800, Jia-Ju Bai wrote:
> The kernel may sleep under a spinlock, and the function call path is:
> ext2_remount
> parse_options
> match_int
> match_number (lib/parser.c)
> kmalloc(GFP_KERNEL) --> may sleep
>
> To fix it, GFP_KERNEL is
s/objdump/objtool/g obviously.
V2:
- Added Reviewed-by and Acked-by as indicated by Philipp
- octp_params -> ocotp_params Philipp Zabel
- Added whitespace to aggregate initaliser - Philipp Zabel
- Dropped (void*) cast to data - Philipp Zabel
- Made use of of_device_get_match_data - Philipp Zabel
- Got rid of params->banked -
This patch adds logic to correctly setup the write timing parameters
when blowing an OTP fuse for the i.MX7S/D.
Fixes: 0642bac7da42 ("nvmem: imx-ocotp: add write support")
Signed-off-by: Bryan O'Donoghue
---
drivers/nvmem/imx-ocotp.c | 65
Variable val is unsigned so checking whether it is less than zero is
redundant.
Signed-off-by: Christos Gkekas
---
drivers/net/wireless/ath/ath10k/spectral.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/net/wireless/ath/ath10k/spectral.c
On Donnerstag, 5. Oktober 2017 03:56:13 CEST Namhyung Kim wrote:
> Hi Milian,
>
> On Sun, Oct 01, 2017 at 04:30:48PM +0200, Milian Wolff wrote:
> > This is a requirement to create real callchain entries for inlined
> > frames.
> >
> > Since the list of inlines usually contains the target symbol
On Sun, 2017-10-08 at 12:01 -0700, Guenter Roeck wrote:
> On 10/08/2017 11:49 AM, Guenter Roeck wrote:
> > On 10/08/2017 11:12 AM, Ben Hutchings wrote:
> > > On Sun, 2016-05-15 at 19:56 -0700, Guenter Roeck wrote:
> > > [...]
> > > > For 3.4 and 3.2 kernels to build with binutils v2.24, it would
Hi Rob, Greg,
On 04/10/17 10:38, Rob Herring wrote:
> On Fri, Sep 22, 2017 at 03:48:13AM +1200, Chris Packham wrote:
>> Document the device tree bindings for the uio-prv-genirq driver. Provide
>> some examples on how it can be used.
>>
>> Signed-off-by: Chris Packham
On Fri, Oct 06, 2017 at 02:34:37PM +0100, Will Deacon wrote:
> Hi all,
>
> This is version two of the patches I posted yesterday:
>
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2017-October/534666.html
>
> I'd normally leave it longer before posting again, but Peter had a good
>
Use kmem_cache_free instead of kfree for freeing the memory previously
allocated with kmem_cache_zalloc/kmem_cache_alloc/kmem_cache_node.
Signed-off-by: Himanshu Jha
---
drivers/block/skd_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Variable val holds the default brightness and is unsigned, therefore
checking whether it is less than zero is redundant.
Signed-off-by: Christos Gkekas
---
drivers/video/backlight/tps65217_bl.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
Reviewed-by: Jim Mattson
On Sat, Oct 7, 2017 at 7:35 PM, Wanpeng Li wrote:
> From: Wanpeng Li
>
> SDM section 2.6 mentioned:
>
> After reset, all bits (except bit 0) in XCR0 are cleared to zero; XCR0[0] is
> set to 1.
>
> This
On Wed, Oct 04, 2017 at 09:35:31PM +0200, Emil Renner Berthing wrote:
> The Samsung Chromebook Plus (rk3399-gru-kevin) has two of
> these controllers. One for the touchscreen and one for
> the touchpad. However the touchpad doesn't have any
> associated gpio buttons, so it shows up as another
>
David & Maureen picked you for $850.000.00 Donation, Kindly reply for details
and claim.
From: Kuppuswamy Sathyanarayanan
Currently, update_no_reboot_bit() function implemented in this driver
uses mutex_lock() to protect its register updates. But this function is
called with in atomic context in iTCO_wdt_start() and iTCO_wdt_stop()
On Fri, 6 Oct 2017, Paul Burton wrote:
> From: Paul Burton
>
> MIPS will soon not be a part of Imagination Technologies, and as such
> many @imgtec.com email addresses will no longer be valid. This patch
> updates the addresses for all those who:
>
> - Have 10 or more
Hi Krzysztof,
On 8 October 2017 at 21:17, Krzysztof Kozlowski wrote:
> On Sun, Oct 08, 2017 at 06:06:19PM +0530, Anand Moon wrote:
>> Hi Krzysztof,
>>
>> On 6 October 2017 at 12:08, Krzysztof Kozlowski wrote:
>> > On Fri, Oct 6, 2017 at 6:36 AM, Anand Moon
Variable val is unsigned so checking whether it is less than zero is
redundant.
Signed-off-by: Christos Gkekas
---
drivers/net/wireless/ath/ath9k/debug.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/net/wireless/ath/ath9k/debug.c
On Fri, Oct 06, 2017 at 07:57:01AM -0700, Matthew Wilcox wrote:
> On Fri, Oct 06, 2017 at 01:09:42PM +1100, Dave Chinner wrote:
> > On Thu, Oct 05, 2017 at 12:16:19PM -0400, J. Bruce Fields wrote:
> > > This kind of restriction sounds more like a permanent feature of the
> > >
On Sun, Oct 8, 2017 at 7:11 PM, Ulf Magnusson wrote:
> After the parsing memory leak fixes.
>
> Signed-off-by: Ulf Magnusson
> ---
> scripts/kconfig/zconf.tab.c_shipped | 562
> +++-
> 1 file changed, 294 insertions(+),
Variable did_old is unsigned so checking whether it is greater or equal
to zero is not necessary.
Signed-off-by: Christos Gkekas
---
drivers/iommu/intel-iommu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/iommu/intel-iommu.c
If it were me, I'd apply De Morgan to that expression, but the logic looks fine.
Reviewed-by: Jim Mattson
On Sat, Oct 7, 2017 at 7:35 PM, Wanpeng Li wrote:
> From: Wanpeng Li
>
> SDM mentioned:
>
> "If either the “unrestricted
It will be useful in later patches to know the register access mode and
bit-shift to apply to a given input offset.
Fixes: 0642bac7da42 ("nvmem: imx-ocotp: add write support")
Signed-off-by: Bryan O'Donoghue
---
drivers/nvmem/imx-ocotp.c | 32
The i.MX7S/D has a different set of timing requirements, as a pre-cursor to
adding the i.MX7 timing parameters, move the i.MX6 stuff to a dedicated
function.
Fixes: 0642bac7da42 ("nvmem: imx-ocotp: add write support")
Signed-off-by: Bryan O'Donoghue
Reviewed-by:
The i.MX7S/D takes the bank address in the CTRLn.ADDR field and the data
value in one of the DATAx {0, 1, 2, 3} register fields. The current write
routine is based on writing the CTRLn.ADDR field and writing a single DATA
register only.
Fixes: 0642bac7da42 ("nvmem: imx-ocotp: add write support")
101 - 200 of 574 matches
Mail list logo