This device has the same issues as the HP x360 wrt the MUTE LED and
the front speakers not working. This patch fixes the MUTE LED issue,
but doesn't touch the HDA verbs. The fix for the x360 does not work
on the Spectre.
Signed-off-by: Alexandru Gagniuc
---
sound/pci/hda/patch_realtek.c | 1 +
1
Some LED controllers have support for autonomously controlling
brightness over time, according to some preprogrammed pattern or
function.
This patch adds pattern trigger that LED device can configure the
pattern and trigger it.
Signed-off-by: Raphael Teysseyre
Signed-off-by: Baolin Wang
---
Cha
This patch implements the 'pattern_set'and 'pattern_clear'
interfaces to support SC27XX LED breathing mode.
Signed-off-by: Baolin Wang
---
Changes from v2:
- None.
Changes from v1:
- Remove pattern_get interface.
---
drivers/leds/leds-sc27xx-bltc.c | 94 +++
Hi Jacek,
On 3 August 2018 at 19:59, Jacek Anaszewski wrote:
> Hi Baolin,
>
> On 08/03/2018 10:05 AM, Baolin Wang wrote:
>> Hi Jacek,
>>
>> On 3 August 2018 at 05:21, Jacek Anaszewski
>> wrote:
>>> Hi Baolin,
>>>
>>> Thank you for addressing review remarks.
>>>
>>> I've played a bit with the in
On Fri, Aug 3, 2018 at 5:19 PM Hugh Dickins wrote:
>
> I thought that virt_to_page() only works on virtual addresses
> in the direct map
You're right that virt_to_page() does not work on any _actual_ virtual
mappings (ie no user pages, and no vmalloc() pages etc). It does not
follow page tables a
On 04/08/2018 17:29, Gabriele Mazzotta wrote:
> This change does not take into account that some BIOSes change
> MSR_IA32_MISC_ENABLE_TURBO_DISABLE depending on the power source.
> If the turbo is disabled when the system boots, policy.max_freq
> is set to pstate.max_pstate. However, if the BIOS la
From: Wang Xin
Within at24_loop_until_timeout the timestamp used for timeout checking
is recorded after the I2C transfer and sleep_range(). Under high CPU
load either the execution time for I2C transfer or sleep_range() could
actually be larger than the timeout value. Worst case the I2C transfer
On Thursday, August 2, 2018 11:30:00 PM CEST Linus Walleij wrote:
> On Wed, Aug 1, 2018 at 1:12 PM Florian Eckert wrote:
>
> > Add a new device driver "gpio-apu" which will now handle the GPIOs on
> > APU2 and APU3 devices from PC Engines.
> >
> > - APU2/APU3 -> front button reset support
> > - A
On Sat, 4 Aug 2018, Linus Torvalds wrote:
> On Fri, Aug 3, 2018 at 5:19 PM Hugh Dickins wrote:
> >
> > I thought that virt_to_page() only works on virtual addresses
> > in the direct map
>
> You're right that virt_to_page() does not work on any _actual_ virtual
> mappings (ie no user pages, and n
On 04.08.2018 01:51, Andrew Morton wrote:
> On Fri, 03 Aug 2018 18:36:14 +0300 Kirill Tkhai wrote:
>
>> The patch introduces a special value SHRINKER_REGISTERING to use instead
>> of list_empty() to detect a semi-registered shrinker.
>>
>> This should be clearer for a reader since "list is empty"
>> + sysctrl_DMC: system-controller@0 {
>> + compatible = "amlogic,meson-gx-dmc-sysctrl",
>> "syscon", "simple-mfd";
>
> we'd like to drop 'meson-' prefix, so better using "amlogic,gx-dmc-sysctrl",
> please take a look at the discussion here [1]
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Signed-off-by: Gustavo A. R. Silva
---
sound/pci/mixart/mixart.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/pci/mixart/mixart.c b/sound/pci/mixart/mixart.c
index a74f1ad..9
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Notice that in this particular case, I replaced the code comment with
a proper "fall through" annotation, which is what GCC is expecting
to find.
Signed-off-by: Gustavo A. R. Silva
---
s
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Addresses-Coverity-ID: 115156 ("Missing break in switch")
Signed-off-by: Gustavo A. R. Silva
---
sound/pci/echoaudio/echoaudio.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/soun
On Sat, Aug 4, 2018 at 8:43 PM, Mark Jonas wrote:
> -#define at24_loop_until_timeout(tout, op_time) \
> - for (tout = jiffies + msecs_to_jiffies(at24_write_timeout), \
> -op_time = 0; \
> -op_
On Sat, Aug 4, 2018 at 9:55 AM, Marcus Folkesson
wrote:
> On Fri, Aug 03, 2018 at 11:09:22PM +0100, Jonathan Cameron wrote:
>> On Thu, 2 Aug 2018 22:52:00 +0300
>> Andy Shevchenko wrote:
>>
>> > On Thu, Aug 2, 2018 at 10:15 PM, Marcus Folkesson
>> > wrote:
>> > > MCP3911 is a dual channel Analog
On Wed, Aug 01, 2018 at 11:32:27PM +0530, Manivannan Sadhasivam wrote:
> Add devicetree binding for Actions Semiconductor Owl I2C controller
>
> Signed-off-by: Manivannan Sadhasivam
> Reviewed-by: Rob Herring
Applied to for-next, thanks!
signature.asc
Description: PGP signature
On Wed, Aug 01, 2018 at 11:32:31PM +0530, Manivannan Sadhasivam wrote:
> Add Actions Semiconductor Owl family S900 I2C driver.
>
> Signed-off-by: Manivannan Sadhasivam
> Acked-by: Peter Rosin
Applied to for-next, thanks!
signature.asc
Description: PGP signature
On Sat, Aug 4, 2018 at 2:43 PM, Greg Kroah-Hartman
wrote:
> On Sat, Aug 04, 2018 at 12:54:22PM +0200, Hauke Mehrtens wrote:
>> On 08/03/2018 12:30 PM, Greg Kroah-Hartman wrote:
>> > On Fri, Aug 03, 2018 at 03:33:38PM +0800, Wu, Songjun wrote:
>> This patch makes it possible to use it with the leg
On Wed, Aug 01, 2018 at 11:32:28PM +0530, Manivannan Sadhasivam wrote:
> Add pinctrl definition for Actions Semiconductor S900 I2C controller.
> Pinctrl definitions are only available for I2C0, I2C1, and I2C2.
>
> Signed-off-by: Manivannan Sadhasivam
Note that I don't take DTS patches, so skippi
On Wed, Aug 01, 2018 at 11:32:32PM +0530, Manivannan Sadhasivam wrote:
> Add entry for Actions Semiconductor Owl I2C driver under ARM/ACTIONS
So, it goes upstream via arm-soc?
>
> Signed-off-by: Manivannan Sadhasivam
Acked-by: Wolfram Sang
signature.asc
Description: PGP signature
On Tue, Jul 24, 2018 at 01:36:15PM -0700, Jae Hyun Yoo wrote:
> This commit fixes this sparse warning:
> drivers/i2c/busses/i2c-aspeed.c:875:38: warning: incorrect type in assignment
> (different modifiers)
> drivers/i2c/busses/i2c-aspeed.c:875:38:expected unsigned int (
> *get_clk_reg_val )(
On Sat, Aug 04, 2018 at 07:04:56PM +0800, zhong jiang wrote:
> The err is not used after initalization. So just remove the variable.
>
> Signed-off-by: zhong jiang
I'll apply this patch, but how did you generate the diff? The
function name here is all wrong:
> diff --git a/fs/ext4/mballoc.c b/
On Fri, 9 Mar 2018 23:15:55 +
Sergei Trofimovich wrote:
I tried to explain in more detail breakage mechanics
of unwinder and gcc code generation quirks at:
https://trofi.github.io/posts/210-ptrace-and-accidental-boot-fix-on-ia64.html
Hopefully it gives better intuition of code change
ca
On Thu, Aug 2, 2018 at 3:58 PM, Dave Hansen wrote:
>
> From: Dave Hansen
>
> The kernel image is mapped into two places in the virtual address
> space (addresses without KASLR, of course):
>
> 1. The kernel direct map (0x8800)
> 2. The "high kernel map" (0x8100
Hi Baolin,
Thank you for the new version of the patch set.
On 08/04/2018 06:51 PM, Baolin Wang wrote:
> Some LED controllers have support for autonomously controlling
> brightness over time, according to some preprogrammed pattern or
> function.
>
> This patch adds pattern trigger that LED devic
Hi all,
This patchset aims to replace the use of integer values with a
proper true or false in return statements for functions returning
bool.
Thanks
Gustavo A. R. Silva (12):
ASoC: max9850: use true and false for boolean values
ASoC: rt5631: use true and false for boolean values
ASoC: tda
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/max9850.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/rt5631.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/so
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/tda7419.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/wm8990.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/cs4270.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/wm8996.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/so
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/da7219.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/twl6040.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/da7213.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/da9055.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/wm5100-tables.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
dif
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
sound/soc/codecs/wm8903.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
mm/kasan/kasan_init.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/mm/kasan
On Mon, Jul 16, 2018 at 03:03:31PM -0400, Rik van Riel wrote:
> The mm_struct always contains a cpumask bitmap, regardless of
> CONFIG_CPUMASK_OFFSTACK. That means the first step can be to
> simplify things, and simply have one bitmask at the end of the
> mm_struct for the mm_cpumask.
>
> This doe
On Fri, 27 Jul 2018, zhaowu...@wingtech.com wrote:
> >On Thu, 26 Jul 2018 15:03:23 +0800 "zhaowu...@wingtech.com"
> > wrote:
> >
> >> >On Thu 26-07-18 10:21:40, zhaowu...@wingtech.com wrote:
> >> >[...]
> >> >> Our project really needs a fix to this issue
> >> >
> >> >Could you be more specific wh
On Sat, Aug 04, 2018 at 09:42:05PM +0300, Kirill Tkhai wrote:
> This is exactly the thing the patch makes. Instead of inserting a shrinker
> pointer
> to idr, it inserts a fake value SHRINKER_REGISTERING there. The patch makes
> impossible
> to dereference a shrinker unless it's completely regist
One potential issue is with host1x clients where userspace processes can
submit jobs with invalid memory accesses (addresses not mapped to
IOMMU). If when such a failure happens, we disable the DMA for the whole
host1x client, unrelated userspace processes may see failures even
though there is
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
drivers/platform/x86/acer-wmi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
drivers/clk/qcom/clk-branch.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/driver
On Fri, Aug 03, 2018 at 08:16:21PM -0500, Benjamin Herrenschmidt wrote:
> On Fri, 2018-08-03 at 22:07 +0300, Michael S. Tsirkin wrote:
> > On Fri, Aug 03, 2018 at 10:58:36AM -0500, Benjamin Herrenschmidt wrote:
> > > On Fri, 2018-08-03 at 00:05 -0700, Christoph Hellwig wrote:
> > > > > 2- Make vi
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
drivers/input/touchscreen/raydium_i2c_ts.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
di
Return statements in functions returning bool should use true or false
instead of an integer value.
This code was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
drivers/misc/genwqe/card_base.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/driv
On Sunday, 5 August 2018 03:11:57 MSK Mikko Perttunen wrote:
> One potential issue is with host1x clients where userspace processes can
> submit jobs with invalid memory accesses (addresses not mapped to
> IOMMU). If when such a failure happens, we disable the DMA for the whole
> host1x client, unr
On 08/03/2018 09:47 PM, Masahiro Yamada wrote:
> 1/3 fixes the build failure reported by Randy.
> (I may have seen a similar report before, but I cannot recall it.)
>
> 2/3 and 3/3 clean-up the Makefile a bit more.
>
> I'd like to apply this series to kbuild tree.
> Ack from UML folks are appreci
Return statements in functions returning bool should use true or false
instead of an integer value.
This issue was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
drivers/s390/net/qeth_l3_main.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a
Return statements in functions returning bool should use true or false
instead of an integer value.
This issue was detected with the help of Coccinelle.
Signed-off-by: Gustavo A. R. Silva
---
drivers/soc/mediatek/mtk-pmic-wrap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On Fri, Aug 03, 2018 at 11:20:49AM +0300, Peter Ujfalusi wrote:
> Hi,
>
> On 2018-08-03 10:19, Huang Shijie wrote:
> > All the patches are using dmaenginem_async_device_register to simplify code
> > except the last one:
> > dmaengine: add COMPILE_TEST for the drivers
> >
> > I use the last one
On 08/02/2018 02:15 PM, Jerry Hoemann wrote:
When the pretimeout is specified as a module parameter, the
value should be reflected in hpwdt_dev.pretimeout. The default
(on) case is correct. But, when disabling pretimeout, the value
should be set to zero in hpwdt_dev.
Signed-off-by: Jerry Hoema
On 08/02/2018 02:15 PM, Jerry Hoemann wrote:
The hwpdt driver is overloaded for handling both the iLO
watchdog and the explicit "Generate NMI to System" virutal
button.
Claim the iLO NMI virtual button even if we are not claiming
the iLO watchdog pretimeout.
Signed-off-by: Jerry Hoemann
Gues
On 08/02/2018 02:15 PM, Jerry Hoemann wrote:
Print module parameters when the driver is loaded.
Signed-off-by: Jerry Hoemann
---
drivers/watchdog/hpwdt.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/watchdog/hpwdt.c b/drivers/watchdog/hpwdt.c
index 8a85ddd
On 08/02/2018 02:15 PM, Jerry Hoemann wrote:
Bump version number to reflect recent bug fixes.
Signed-off-by: Jerry Hoemann
As usual, my recommendation is to drop version numbers. The version number is
reflected by the Linux kernel version and SHA. Yet, I understand that people
feel quite stro
On Wed, 2018-08-01 at 19:54 +0200, Miguel Ojeda wrote:
> From the GCC manual:
>
> The nonstring variable attribute specifies that an object or member
> declaration with type array of char or pointer to char is intended to
> store character arrays that do not necessarily contain a terminating NUL
>
Hi Sebastian,
On 07/22/2018 05:17 PM, Sebastian Reichel wrote:
> Hi,
>
> On Wed, Jul 18, 2018 at 08:14:50AM +0200, Krzysztof Kozlowski wrote:
>> On 17 July 2018 at 23:47, Gustavo A. R. Silva wrote:
>>> It seems that a *break* is missing in order to avoid a fall-through.
>>> Otherwise, the calcul
In preparation to enabling -Wimplicit-fallthrough, mark switch cases
where we are expecting to fall through.
Addresses-Coverity-ID: 114713 ("Missing break in switch")
Signed-off-by: Gustavo A. R. Silva
---
kernel/power/hibernate.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/kernel/power/
2018-07-27 18:06 GMT+08:00 Will Deacon :
> On Thu, Jul 26, 2018 at 05:01:47PM -0400, Dongjiu Geng wrote:
>> In order to remove the additional check before calling the
>> ghes_notify_sea(), make stub definition when !CONFIG_ACPI_APEI_SEA.
>>
>> Signed-off-by: Dongjiu Geng
>> ---
>
> Acked-by: Will
Hi Jacek,
On 5 August 2018 at 05:41, Jacek Anaszewski wrote:
> Hi Baolin,
>
> Thank you for the new version of the patch set.
Thanks for your quick response.
>
> On 08/04/2018 06:51 PM, Baolin Wang wrote:
>> Some LED controllers have support for autonomously controlling
>> brightness over time,
On Sat, Aug 04, 2018 at 11:04:40PM +0200, Wolfram Sang wrote:
> On Wed, Aug 01, 2018 at 11:32:28PM +0530, Manivannan Sadhasivam wrote:
> > Add pinctrl definition for Actions Semiconductor S900 I2C controller.
> > Pinctrl definitions are only available for I2C0, I2C1, and I2C2.
> >
> > Signed-off-b
Hi Michal and David,
AFAICS, the null pointer is the task_struct of p in the dump_header.
Why don't we just add a if before printing the victim task's
informaiton.
diff --git a/mm/oom_kill.c b/mm/oom_kill.c
--- a/mm/oom_kill.c
+++ b/mm/oom_kill.c
@@ -438,14 +438,6 @@ static void dump_header(struct
Some LED controllers have support for autonomously controlling
brightness over time, according to some preprogrammed pattern or
function.
This patch adds pattern trigger that LED device can configure the
pattern and trigger it.
Signed-off-by: Raphael Teysseyre
Signed-off-by: Baolin Wang
---
Cha
On Sat, 2018-08-04 at 14:25 +0200, Mike Galbraith wrote:
>
> Besides, there are more interesting fish in the arm64 sea than kvm.
>
> virgin 4.16.18-rt12-rt
>
> [ 537.236131] ITS queue timeout (65440 65504 4640)
> [ 537.236150] ITS cmd its_build_inv_cmd failed
FWIW, I had thought that somethin
On 05.08.2018 03:03, Matthew Wilcox wrote:
> On Sat, Aug 04, 2018 at 09:42:05PM +0300, Kirill Tkhai wrote:
>> This is exactly the thing the patch makes. Instead of inserting a shrinker
>> pointer
>> to idr, it inserts a fake value SHRINKER_REGISTERING there. The patch makes
>> impossible
>> to de
> [ Goes around and rummages ]
>
> Oh, never mind, looking around reminded me why: we want to map the
> kernel text in the top 31 bits, so that we can use the faster
> -mcmodel=kernel because all symbols fit in sign-extended 32 bits.
>
> Maybe there was some other reason too, but I think that's i
On Sat, Aug 04, 2018 at 07:49:25AM -0700, Guenter Roeck wrote:
> On 08/04/2018 02:00 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.17.13 release.
> > There are 31 patches in this series, all will be posted as a response
> > to this one. If anyone has any
301 - 370 of 370 matches
Mail list logo