[PATCH] iio: declare struct to fix warning

2015-07-30 Thread Aaron Ma
When compile iio related driver the following warning shown: include/linux/iio/trigger.h:35:34: warning: 'struct iio_trigger' declared inside parameter list int (*set_trigger_state)(struct iio_trigger *trig, bool state); include/linux/iio/trigger.h:38:18: warning: 'struct iio_dev' declared

Re: RCU stall and the system boot hang with nfsroot

2016-01-04 Thread Aaron Ma
On Fri, Jan 1, 2016 at 3:49 AM, Paul E. McKenney <paul...@linux.vnet.ibm.com> wrote: > On Wed, Dec 30, 2015 at 09:41:45AM -0800, Paul E. McKenney wrote: >> On Wed, Dec 30, 2015 at 03:03:33PM +0800, Aaron Ma wrote: >> > On Wed, Dec 30, 2015 at 7:42 AM, Pau

Re: RCU stall and the system boot hang with nfsroot

2016-01-04 Thread Aaron Ma
On Tue, Jan 5, 2016 at 5:18 AM, Paul E. McKenney <paul...@linux.vnet.ibm.com> wrote: > On Mon, Jan 04, 2016 at 06:01:37PM +0800, Aaron Ma wrote: >> On Fri, Jan 1, 2016 at 3:49 AM, Paul E. McKenney >> <paul...@linux.vnet.ibm.com> wrote: >> > On Wed, Dec

Re: RCU stall and the system boot hang with nfsroot

2015-12-29 Thread Aaron Ma
On Wed, Dec 30, 2015 at 7:42 AM, Paul E. McKenney <paul...@linux.vnet.ibm.com> wrote: > On Tue, Dec 29, 2015 at 05:34:38PM +0800, Aaron Ma wrote: >> Add paul...@linux.vnet.ibm.com >> >> On Tue, Dec 29, 2015 at 5:32 PM, Aaron Ma <mapen...@gmail.com> wrote: >

Re: RCU stall and the system boot hang with nfsroot

2015-12-29 Thread Aaron Ma
Add paul...@linux.vnet.ibm.com On Tue, Dec 29, 2015 at 5:32 PM, Aaron Ma <mapen...@gmail.com> wrote: > Hi, Paul: > I found the linux-stable-4.1.15 with rt15 patches boot hang sometimes. > Hardware is Grantley-EP and WildcatPass. > No response by sysrq. > > Did y

[PATCH] Input: trackpoint: add new trackpoint firmware ID

2017-02-09 Thread Aaron Ma
Synaptics add new TP firmware ID: 0x2 and 0x3, for now both lower 2 bits are indicated as TP. Change the constant to bitwise values. Signed-off-by: Aaron Ma <aaron...@canonical.com> Cc: sta...@vger.kernel.org --- drivers/input/mouse/trackpoint.c | 3 ++- drivers/input/mouse/trackpoint

Re: [PATCH] Input: trackpoint: add new trackpoint firmware ID

2017-02-12 Thread Aaron Ma
On 02/13/2017 06:38 AM, Dmitry Torokhov wrote: > On Fri, Feb 10, 2017 at 11:02:21AM +0800, Aaron Ma wrote: >> Synaptics add new TP firmware ID: 0x2 and 0x3, for now both lower 2 bits >> are indicated as TP. Change the constant to bitwise values. > > Hmm, I thought IBM/Le

[PATCH] Input: synaptics - add Intertouch support on X1 Carbon 6th and X280

2017-10-16 Thread Aaron Ma
version 3 is using, patch: https://patchwork.kernel.org/patch/9989547/ enabled SMBus ver 3 and makes synaptics devices work fine on SMBus mode. Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/input/mouse/synaptics.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers

[PATCH] Input: ALPS - fix DualPoint flag for 74 03 28 devices

2017-11-14 Thread Aaron Ma
0f793, after restore the OTP reading order, it becomes to 0x10 as before and reports the right flag. Fixes: 4a646580f793 ("Input: ALPS - fix two-finger scroll breakage") Cc: <sta...@vger.kernel.org> Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/input/mouse/alp

Re: [PATCH] Support TrackStick of Thinkpad L570

2017-11-29 Thread Aaron Ma
Please add the patch version next time. The patch make trackstick work on L570. Tested-by: Aaron Ma <aaron...@canonical.com> On 11/29/2017 04:33 PM, Masaki Ota wrote: > From: Masaki Ota <masaki@jp.alps.com> > - The issue is that Thinkpad L570 TrackStick does not work

[PATCH] Input: elantech - add new icbody type 15

2017-11-19 Thread Aaron Ma
The touchpad of Lenovo Thinkpad L480 reports it's version as 15. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/input/mouse/elantech.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/input/mouse/elantech.c b/drivers/input

Re: [PATCH] Support TrackStick of Thinkpad L570

2017-11-20 Thread Aaron Ma
Tested-by: Aaron Ma <aaron...@canonical.com> On 11/20/2017 03:55 PM, Masaki Ota wrote: > From: Masaki Ota <masaki@jp.alps.com> > - The issue is that Thinkpad L570 TrackStick does not work. Because the main > interface of Thinkpad L570 device is SMBus, so ALPS ove

Re: [PATCH] Input: ALPS - fix DualPoint flag for 74 03 28 devices

2017-11-16 Thread Aaron Ma
jp.alps.com> > Cc: linux-in...@vger.kernel.org; linux-kernel@vger.kernel.org; > dmitry.torok...@gmail.com; Aaron Ma <aaron...@canonical.com> > Subject: Re: [PATCH] Input: ALPS - fix DualPoint flag for 74 03 28 devices > > On Wednesday 15 November 2017 14:34:04 Aaron Ma wrote: >&

Re: [PATCH v2 0/9] Input: support for latest Lenovo thinkpads (series 80)

2018-05-16 Thread Aaron Ma
On 05/16/2018 03:09 PM, Teika Kazura wrote: > From: Aaron Ma <aaron...@canonical.com> > Date: Tue, 17 Apr 2018 19:42:27 +0800 > >> Could you apply my patch too? >> >> It add LEN0096 that Benjamin's patch doesn't include. >> >> +"LEN0096&qu

[PATCH] Input: elan: enable middle button of touchpads on ThinkPad P52

2018-06-12 Thread Aaron Ma
PNPID is better way to identify the type of touchpads. Enable middle button support on 2 types of touchpads on Lenovo P52. Cc: sta...@vger.kernel.org Cc: KT Liao Signed-off-by: Aaron Ma --- drivers/input/mouse/elantech.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git

Re: [PATCH] Input: elan: enable middle button of touchpads on ThinkPad P52

2018-06-19 Thread Aaron Ma
On 06/19/2018 08:21 PM, Benjamin Tissoires wrote: > On Tue, Jun 12, 2018 at 9:10 AM Aaron Ma wrote: >> PNPID is better way to identify the type of touchpads. >> Enable middle button support on 2 types of touchpads on Lenovo P52. >> >> Cc: sta...@vger.kernel.org &g

[PATCH v2 1/2] HID: i2c-hid: fix size check and type usage

2018-01-07 Thread Aaron Ma
When convert char array with signed int, if the inbuf[x] is negative then upper bits will be set to 1. Fix this by using u8 instead of char. ret_size has to be at least 3, hid_input_report use it after minus 2 bytes. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma <aaron...@canonical.

[PATCH v2 2/2] HID: core: Fix size as type u32

2018-01-07 Thread Aaron Ma
When size is negative, calling memset will make segment fault. Declare the size as type u32 to keep memset safe. size in struct hid_report is unsigned, fix return type of hid_report_len to u32. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/h

Re: [PATCH] Input: trackpoint - force 3 buttons if 0 button is reported

2018-01-12 Thread Aaron Ma
Only this laptop had been confirmed is ALPS: 0102 – FF02 trackpoint, it return 0 of extended button. I saw the other device that returned 0, but don't have it now, so the ID can not be checked. ThinkPad always have 3 buttons installed, I suggest always set 0x33 for button info when the button

Re: [PATCH] Input: trackpoint - force 3 buttons if 0 button is reported

2018-01-12 Thread Aaron Ma
Will your patch go to stable kernel? If yes, that's fine.

[PATCH] Input: trackpoint - force 3 buttons if 0 button is reported

2018-01-11 Thread Aaron Ma
Lenovo introduced trackpoint compatible sticks with minimum PS/2 commands. Some of these sticks with 3 buttons always return 0 when reading extended button info, set it as 3 buttons to enable middle button. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma <aaron...@canonical.com> --- d

Re: [PATCH v2 2/2] HID: core: Fix size as type u32

2018-02-02 Thread Aaron Ma
Hi: Could anyone review and apply these 2 patch? Regards, Aaron

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-02-02 Thread Aaron Ma
Hi Could anyone review an apply this single patch? The 2nd patch had been sent as v2. Regards, Aaron

Re: [PATCH 1/2] HID: core: i2c-hid: fix size check and type usage

2018-02-03 Thread Aaron Ma
Hi Marcus: This patch is replaced by v2 patches you just reviewed. Thanks, Aaron

Re: [PATCH v2 2/2] HID: core: Fix size as type u32

2018-02-03 Thread Aaron Ma
On 02/03/2018 03:55 PM, Marcus Folkesson wrote: > Hi Aaron, > > On Mon, Jan 08, 2018 at 10:41:41AM +0800, Aaron Ma wrote: >> When size is negative, calling memset will make segment fault. >> Declare the size as type u32 to keep memset safe. >> >> size in struct h

[PATCH] HID: Fix hid_report_len usage

2018-02-03 Thread Aaron Ma
Follow the change of return type u32 of hid_report_len, fix all the types of variables those get the return value of hid_report_len to u32, and all other code already uses u32. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/hid/hid-input.c

[PATCH] platform/x86: ideapad-laptop: Increase timeout to wait for EC answer

2018-02-11 Thread Aaron Ma
Lenovo E41-20 needs more time than 100ms to read VPC, the funtion keys always failed responding. Increase timeout to get the value from VPC, then the funtion keys like mic mute key work well. Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/platform/x86/ideapad-laptop.c | 2

[PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-01-02 Thread Aaron Ma
as a quirk. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/hid/hid-ids.h | 3 +++ drivers/hid/i2c-hid/i2c-hid.c | 13 + 2 files changed, 16 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 5da3d6

[PATCH 1/2] HID: core: i2c-hid: fix size check and type usage

2018-01-02 Thread Aaron Ma
Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/hid/hid-core.c| 4 ++-- drivers/hid/i2c-hid/i2c-hid.c | 10 +- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 0c3f608131cf..992547771d96

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-01-04 Thread Aaron Ma
Hi Benjamin: Thanks for reviewing my patches. This issue only happened on this RayD 2386:3118 touchscreen. No other devices found, so I think it should be in quirk.

Re: [PATCH 1/2] HID: core: i2c-hid: fix size check and type usage

2018-01-04 Thread Aaron Ma
I will follow your advice and send V2. Thanks, Aaron

[PATCH] ACPI / EC: Use ec_no_wakeup on ThinkPad X1 Yoga 3rd

2018-07-31 Thread Aaron Ma
Like on X1C6, on X1Y3 EC interrupts constantly wake up system from s2idle, the power consumption is extremely high. So make ec_no_wakeup be true as default to keep system in s2idle mode and reduce power consumption. Power button works when ec_no_wakeup=true. Signed-off-by: Aaron Ma --- drivers

[PATCH] Input: elantech - enable middle button of touchpad on ThinkPad P72

2018-09-07 Thread Aaron Ma
Adding 2 new touchpad IDs to support middle button support. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/input/mouse/elantech.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c index 44f57cf6675b

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-03 Thread Aaron Ma
Hi Jiri: This patch is pending for long time. Could you merge this single patch to upstream? Regards, Aaron

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-09 Thread Aaron Ma
Hi Jiri: Re-send this patch. Thanks, Aaron On 01/03/2018 01:30 AM, Aaron Ma wrote: > When Rayd touchscreen resumed from S3, it issues too many errors like: > i2c_hid i2c-RAYD0001:00: i2c_hid_get_input: incomplete report (58/5442) > > And all the report data are corrupted,

[PATCH] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-09 Thread Aaron Ma
as a quirk. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/hid/hid-ids.h | 3 +++ drivers/hid/i2c-hid/i2c-hid.c | 13 + 2 files changed, 16 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 5a3a7e

Re: [PATCH v2 0/9] Input: support for latest Lenovo thinkpads (series 80)

2018-04-17 Thread Aaron Ma
Hi Dmitry and Benjamin: Could you apply my patch too? It add LEN0096 that Benjamin's patch doesn't include. + "LEN0096", /* X280 */ Regards, Aaron

[PATCH v2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-18 Thread Aaron Ma
<dan.carpen...@oracle.com> Signed-off-by: Aaron Ma <aaron...@canonical.com> --- drivers/hid/hid-ids.h | 3 +++ drivers/hid/i2c-hid/i2c-hid.c | 13 + 2 files changed, 16 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 5a3a7ead3012..

Re: [PATCH v2 0/9] Input: support for latest Lenovo thinkpads (series 80)

2018-05-16 Thread Aaron Ma
On 05/16/2018 03:09 PM, Teika Kazura wrote: > From: Aaron Ma > Date: Tue, 17 Apr 2018 19:42:27 +0800 > >> Could you apply my patch too? >> >> It add LEN0096 that Benjamin's patch doesn't include. >> >> +"LEN0096", /* X280 */ > Aaron, i

[PATCH] Input: synaptics - add Intertouch support on X1 Carbon 6th and X280

2017-10-16 Thread Aaron Ma
version 3 is using, patch: https://patchwork.kernel.org/patch/9989547/ enabled SMBus ver 3 and makes synaptics devices work fine on SMBus mode. Signed-off-by: Aaron Ma --- drivers/input/mouse/synaptics.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/input/mouse/synaptics.c b

[PATCH v2 1/2] HID: i2c-hid: fix size check and type usage

2018-01-07 Thread Aaron Ma
When convert char array with signed int, if the inbuf[x] is negative then upper bits will be set to 1. Fix this by using u8 instead of char. ret_size has to be at least 3, hid_input_report use it after minus 2 bytes. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/hid/i2c-hid

[PATCH v2 2/2] HID: core: Fix size as type u32

2018-01-07 Thread Aaron Ma
When size is negative, calling memset will make segment fault. Declare the size as type u32 to keep memset safe. size in struct hid_report is unsigned, fix return type of hid_report_len to u32. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/hid/hid-core.c | 10

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-01-04 Thread Aaron Ma
Hi Benjamin: Thanks for reviewing my patches. This issue only happened on this RayD 2386:3118 touchscreen. No other devices found, so I think it should be in quirk.

Re: [PATCH 1/2] HID: core: i2c-hid: fix size check and type usage

2018-01-04 Thread Aaron Ma
I will follow your advice and send V2. Thanks, Aaron

[PATCH] Input: elantech - add new icbody type 15

2017-11-19 Thread Aaron Ma
The touchpad of Lenovo Thinkpad L480 reports it's version as 15. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/input/mouse/elantech.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c index

Re: [PATCH] Support TrackStick of Thinkpad L570

2017-11-20 Thread Aaron Ma
Tested-by: Aaron Ma On 11/20/2017 03:55 PM, Masaki Ota wrote: > From: Masaki Ota > - The issue is that Thinkpad L570 TrackStick does not work. Because the main > interface of Thinkpad L570 device is SMBus, so ALPS overlooked PS2 interface > Firmware setting of TrackStick

Re: [PATCH v2 0/9] Input: support for latest Lenovo thinkpads (series 80)

2018-04-17 Thread Aaron Ma
Hi Dmitry and Benjamin: Could you apply my patch too? It add LEN0096 that Benjamin's patch doesn't include. + "LEN0096", /* X280 */ Regards, Aaron

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-09 Thread Aaron Ma
Hi Jiri: Re-send this patch. Thanks, Aaron On 01/03/2018 01:30 AM, Aaron Ma wrote: > When Rayd touchscreen resumed from S3, it issues too many errors like: > i2c_hid i2c-RAYD0001:00: i2c_hid_get_input: incomplete report (58/5442) > > And all the report data are corrupted,

[PATCH] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-09 Thread Aaron Ma
as a quirk. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/hid/hid-ids.h | 3 +++ drivers/hid/i2c-hid/i2c-hid.c | 13 + 2 files changed, 16 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 5a3a7ead3012..0b5cc910f62e 100644 --- a/drivers

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-03 Thread Aaron Ma
Hi Jiri: This patch is pending for long time. Could you merge this single patch to upstream? Regards, Aaron

[PATCH v2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-04-18 Thread Aaron Ma
Signed-off-by: Aaron Ma --- drivers/hid/hid-ids.h | 3 +++ drivers/hid/i2c-hid/i2c-hid.c | 13 + 2 files changed, 16 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 5a3a7ead3012..0b5cc910f62e 100644 --- a/drivers/hid/hid-ids.h +++ b/drivers/hid

Re: [PATCH v2 2/2] HID: core: Fix size as type u32

2018-02-02 Thread Aaron Ma
Hi: Could anyone review and apply these 2 patch? Regards, Aaron

Re: [PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-02-02 Thread Aaron Ma
Hi Could anyone review an apply this single patch? The 2nd patch had been sent as v2. Regards, Aaron

Re: [PATCH 1/2] HID: core: i2c-hid: fix size check and type usage

2018-02-03 Thread Aaron Ma
Hi Marcus: This patch is replaced by v2 patches you just reviewed. Thanks, Aaron

Re: [PATCH v2 2/2] HID: core: Fix size as type u32

2018-02-03 Thread Aaron Ma
On 02/03/2018 03:55 PM, Marcus Folkesson wrote: > Hi Aaron, > > On Mon, Jan 08, 2018 at 10:41:41AM +0800, Aaron Ma wrote: >> When size is negative, calling memset will make segment fault. >> Declare the size as type u32 to keep memset safe. >> >> size in struct h

[PATCH] HID: Fix hid_report_len usage

2018-02-03 Thread Aaron Ma
Follow the change of return type u32 of hid_report_len, fix all the types of variables those get the return value of hid_report_len to u32, and all other code already uses u32. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/hid/hid-input.c | 3 ++- drivers/hid/hid

[PATCH] Input: trackpoint - force 3 buttons if 0 button is reported

2018-01-11 Thread Aaron Ma
Lenovo introduced trackpoint compatible sticks with minimum PS/2 commands. Some of these sticks with 3 buttons always return 0 when reading extended button info, set it as 3 buttons to enable middle button. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/input/mouse/trackpoint.c

Re: [PATCH] Input: trackpoint - force 3 buttons if 0 button is reported

2018-01-12 Thread Aaron Ma
Only this laptop had been confirmed is ALPS: 0102 – FF02 trackpoint, it return 0 of extended button. I saw the other device that returned 0, but don't have it now, so the ID can not be checked. ThinkPad always have 3 buttons installed, I suggest always set 0x33 for button info when the button

Re: [PATCH] Input: trackpoint - force 3 buttons if 0 button is reported

2018-01-12 Thread Aaron Ma
Will your patch go to stable kernel? If yes, that's fine.

[PATCH 2/2] HID: i2c-hid: Fix resume issue on Raydium touchscreen device

2018-01-02 Thread Aaron Ma
as a quirk. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/hid/hid-ids.h | 3 +++ drivers/hid/i2c-hid/i2c-hid.c | 13 + 2 files changed, 16 insertions(+) diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index 5da3d6256d25..753cc10aa699 100644 --- a/drivers

[PATCH 1/2] HID: core: i2c-hid: fix size check and type usage

2018-01-02 Thread Aaron Ma
Signed-off-by: Aaron Ma --- drivers/hid/hid-core.c| 4 ++-- drivers/hid/i2c-hid/i2c-hid.c | 10 +- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 0c3f608131cf..992547771d96 100644 --- a/drivers/hid/hid-core.c +++ b

Re: [PATCH] Support TrackStick of Thinkpad L570

2017-11-29 Thread Aaron Ma
Please add the patch version next time. The patch make trackstick work on L570. Tested-by: Aaron Ma On 11/29/2017 04:33 PM, Masaki Ota wrote: > From: Masaki Ota > - The issue is that Thinkpad L570 TrackStick does not work. Because the main > interface of Thinkpad L570 device is

[PATCH v2] platform/x86: thinkpad_acpi: re-initialize ACPI buffer size when reuse

2020-10-02 Thread Aaron Ma
_BCL") Signed-off-by: Aaron Ma --- drivers/platform/x86/thinkpad_acpi.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c index 9c4df41687a3..477d63c49c04 100644 --- a/drivers/platform/x86/thinkpad_ac

[PATCH 1/2] drm/i915/dpcd_bl: uncheck PWM_PIN_CAP when detect eDP backlight capabilities

2020-10-09 Thread Aaron Ma
BOE panel with ID 2270 claims both PWM_PIN_CAP and AUX_SET_CAP backlight control bits, but default chip backlight failed to control brightness. Check AUX_SET_CAP and proceed to check quirks or VBT backlight type. DPCD can control the brightness of this pannel. Signed-off-by: Aaron Ma

[PATCH 2/2] drm/i915: Force DPCD backlight mode for BOE 2270 panel

2020-10-09 Thread Aaron Ma
BOE 2270 panel failed to control backlight brightness. Add it in edid quirks to force using DPCD backlight control. Then the brightness can be controlled. Signed-off-by: Aaron Ma --- drivers/gpu/drm/drm_dp_helper.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm

Re: [PATCH 1/2] Input: synaptics-rmi4 - clear irqs before set irqs

2019-06-13 Thread Aaron Ma
On 6/12/19 1:35 AM, Dmitry Torokhov wrote: > On Tue, Jun 11, 2019 at 12:55:58AM +0800, Aaron Ma wrote: >> On 6/10/19 12:55 AM, Dmitry Torokhov wrote: >>> Hi Aaron, >>> >>> On Wed, Feb 20, 2019 at 05:41:59PM +0100, Aaron Ma wrote: >>>> rmi4

[PATCH] platform/x86: thinkpad_acpi: re-initialize acpi buffer size when reuse

2020-09-29 Thread Aaron Ma
Evaluating acpi _BCL could be failed, then acpi buffer size will be set to 0. When reuse this acpi buffer, AE_BUFFER_OVERFLOW will be triggered. Re-initialize buffer size will make acpi evaluate successfully. Signed-off-by: Aaron Ma --- drivers/platform/x86/thinkpad_acpi.c | 4 +++- 1 file

[PATCH] xhci-pci: Set AMD Renoir USB controller to D3 when shutdown

2021-02-03 Thread Aaron Ma
power on and reboot. Signed-off-by: Aaron Ma --- drivers/usb/host/xhci-pci.c | 8 drivers/usb/host/xhci.h | 1 + 2 files changed, 9 insertions(+) diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c index 84da8406d5b4..a31be1ba927f 100644 --- a/drivers/usb/host/xhci

[PATCH] Input: ALPS - fix DualPoint flag for 74 03 28 devices

2017-11-14 Thread Aaron Ma
0f793, after restore the OTP reading order, it becomes to 0x10 as before and reports the right flag. Fixes: 4a646580f793 ("Input: ALPS - fix two-finger scroll breakage") Cc: Signed-off-by: Aaron Ma --- drivers/input/mouse/alps.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(

Re: [PATCH] Input: ALPS - fix DualPoint flag for 74 03 28 devices

2017-11-16 Thread Aaron Ma
er.kernel.org; linux-kernel@vger.kernel.org; > dmitry.torok...@gmail.com; Aaron Ma > Subject: Re: [PATCH] Input: ALPS - fix DualPoint flag for 74 03 28 devices > > On Wednesday 15 November 2017 14:34:04 Aaron Ma wrote: >> There is a regression of commit 4a646580f793 (&

[PATCH] platform/x86: ideapad-laptop: Increase timeout to wait for EC answer

2018-02-11 Thread Aaron Ma
Lenovo E41-20 needs more time than 100ms to read VPC, the funtion keys always failed responding. Increase timeout to get the value from VPC, then the funtion keys like mic mute key work well. Signed-off-by: Aaron Ma --- drivers/platform/x86/ideapad-laptop.c | 2 +- 1 file changed, 1 insertion

Re: [PATCH 2/2] Input: synaptics - remove X240 from the topbuttonpad list

2019-05-22 Thread Aaron Ma
On 5/21/19 2:49 PM, Benjamin Tissoires wrote: > A quick google image search showed that the X240 had 2 versions: one > with the top software buttons, one without. > > And this definitively rings a bell. I am sure we asked Lenovo and > Synaptics to change the PnPID when they would do such a

Re: [PATCH 1/2] Input: synaptics-rmi4 - clear irqs before set irqs

2019-06-10 Thread Aaron Ma
On 6/10/19 12:55 AM, Dmitry Torokhov wrote: > Hi Aaron, > > On Wed, Feb 20, 2019 at 05:41:59PM +0100, Aaron Ma wrote: >> rmi4 got spam data after S3 resume on some ThinkPads. >> Then TrackPoint lost when be detected by psmouse. >> Clear irqs status before set irq

[PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623

2019-10-22 Thread Aaron Ma
be shown, and audio output works fine. Signed-off-by: Aaron Ma --- sound/pci/hda/patch_realtek.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index b000b36ac3c6..c34d8b435f58 100644 --- a/sound/pci/hda/patch_realtek.c

Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623

2019-10-23 Thread Aaron Ma
On 10/23/19 4:44 PM, Kailang wrote: > > >> -Original Message- >> From: Takashi Iwai >> Sent: Wednesday, October 23, 2019 12:08 AM >> To: Aaron Ma >> Cc: pe...@perex.cz; Kailang ; >> hui.w...@canonical.com; alsa-de...@alsa-project.org; >

Re: [PATCH 1/2] Input: synaptics-rmi4 - clear irqs before set irqs

2019-06-07 Thread Aaron Ma
Hi Dmitry: Will you apply them? Thanks, Aaron On 6/4/19 1:19 PM, Christopher Heiny wrote: > Given that, I'm willing to accept the patch as is. > > Cheers, > Chris

Re: [PATCH 1/2] Input: synaptics-rmi4 - clear irqs before set irqs

2019-06-03 Thread Aaron Ma
Hi Christopher: Have got time to review these 2 patches? Users reported it works fine since I sent out this patch. Thanks, Aaron On 4/3/19 9:58 PM, Aaron Ma wrote: > Sure, take your time, if you have any questions let me know please. > > Thanks, > Aaron

[PATCH] platform/x86: thinkpad_acpi: not loading brightness_init when _BCL invalid

2020-07-02 Thread Aaron Ma
When _BCL invalid, disable thinkpad_acpi backlight brightness control. brightness_enable is already checked at the beginning, Always print notice when enabled brightness control. Signed-off-by: Aaron Ma --- drivers/platform/x86/thinkpad_acpi.c | 7 +-- 1 file changed, 5 insertions(+), 2

Re: [PATCH] platform/x86: thinkpad_acpi: not loading brightness_init when _BCL invalid

2020-07-02 Thread Aaron Ma
On 7/2/20 5:30 PM, Andy Shevchenko wrote: > On Thu, Jul 2, 2020 at 11:55 AM Aaron Ma wrote: >> >> When _BCL invalid, disable thinkpad_acpi backlight brightness control. >> >> brightness_enable is already checked at the beginning, > >> Always print notice when

[v2][PATCH] platform/x86: thinkpad_acpi: not loading brightness_init when _BCL invalid

2020-07-02 Thread Aaron Ma
When _BCL invalid, disable thinkpad_acpi backlight brightness control. brightness_enable is already checked at the beginning. Most new thinkpads are using GPU driver to control brightness now, print notice when enabled brightness control even when brightness_enable = 1. Signed-off-by: Aaron Ma

[PATCH] drm/amd/display: add dmcub check on RENOIR

2020-07-08 Thread Aaron Ma
RENOIR loads dmub fw not dmcu, check dmcu only will prevent loading iram, it breaks backlight control. Bug: https://bugzilla.kernel.org/show_bug.cgi?id=208277 Signed-off-by: Aaron Ma --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[v3][PATCH] platform/x86: thinkpad_acpi: not loading brightness_init when _BCL invalid

2020-07-09 Thread Aaron Ma
When _BCL invalid, disable thinkpad_acpi backlight brightness control. brightness_enable is already checked at the beginning. Most new thinkpads are using GPU driver to control brightness now, print notice when enabled brightness control even when brightness_enable = 1. Signed-off-by: Aaron Ma

[v3][PATCH] e1000e: continue to init phy even when failed to disable ULP

2020-06-18 Thread Aaron Ma
test result good too. Fixes: 0c80cdbf3320 ("e1000e: Warn if disabling ULP failed") Signed-off-by: Aaron Ma --- drivers/net/ethernet/intel/e1000e/ich8lan.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c b/drivers/net/ethernet/intel/e1000

[v4][PATCH] e1000e: continue to init phy even when failed to disable ULP

2020-06-18 Thread Aaron Ma
test result good too. Fixes: 0c80cdbf3320 ("e1000e: Warn if disabling ULP failed") Signed-off-by: Aaron Ma --- drivers/net/ethernet/intel/e1000e/ich8lan.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c b/drivers/net/ethernet/intel/e1000

[PATCH] rtw88: 8822ce: add support for device ID 0xc82f

2020-06-12 Thread Aaron Ma
New device ID 0xc82f found on Lenovo ThinkCenter. Tested it with c822 driver, works good. PCI id: 03:00.0 Network controller [0280]: Realtek Semiconductor Co., Ltd. Device [10ec:c82f] Subsystem: Lenovo Device [17aa:c02f] Signed-off-by: Aaron Ma --- drivers/net/wireless/realtek/rtw88

Re: [PATCH] e1000e: continue to init phy even when failed to disable ULP

2020-06-16 Thread Aaron Ma
On 6/16/20 7:23 PM, Kai-Heng Feng wrote: > > >> On Jun 16, 2020, at 18:05, Aaron Ma wrote: >> >> After commit "e1000e: disable s0ix entry and exit flows for ME systems", >> some ThinkPads always failed to disable ulp by ME. >> commit "e100

[v2][PATCH] e1000e: continue to init phy even when failed to disable ULP

2020-06-17 Thread Aaron Ma
ult good too. Fixes: 0c80cdbf33 ("e1000e: Warn if disabling ULP failed") Signed-off-by: Aaron Ma --- drivers/net/ethernet/intel/e1000e/ich8lan.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c b/drivers/net/ethernet/intel/e1000

[PATCH] e1000e: continue to init phy even when failed to disable ULP

2020-06-16 Thread Aaron Ma
rn to e_dbg, in case it confuses. Signed-off-by: Aaron Ma --- drivers/net/ethernet/intel/e1000e/ich8lan.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel/e1000e/ich8lan.c b/drivers/net/ethernet/intel/e1000e/ich8lan.c index f999cca37a8a..63405819eb83

Re: [Intel-wired-lan] [PATCH] e1000e: continue to init phy even when failed to disable ULP

2020-06-16 Thread Aaron Ma
On 6/16/20 6:20 PM, Paul Menzel wrote: > Dear Aaron, > > > Thank you for your patch. > > (Rant: Some more fallout from the other patch, which nobody reverted.) > Would you like a revert? Thanks, Aaron > Am 16.06.20 um 12:05 schrieb Aaron Ma: >> After comm

Re: [PATCH] Input: elan: enable middle button of touchpads on ThinkPad P52

2018-06-19 Thread Aaron Ma
On 06/19/2018 08:21 PM, Benjamin Tissoires wrote: > On Tue, Jun 12, 2018 at 9:10 AM Aaron Ma wrote: >> PNPID is better way to identify the type of touchpads. >> Enable middle button support on 2 types of touchpads on Lenovo P52. >> >> Cc: sta...@vger.kernel.org &g

[PATCH] ACPI / EC: Use ec_no_wakeup on ThinkPad X1 Yoga 3rd

2018-07-31 Thread Aaron Ma
Like on X1C6, on X1Y3 EC interrupts constantly wake up system from s2idle, the power consumption is extremely high. So make ec_no_wakeup be true as default to keep system in s2idle mode and reduce power consumption. Power button works when ec_no_wakeup=true. Signed-off-by: Aaron Ma --- drivers

[PATCH] nvme: determine the number of IO queues

2019-04-17 Thread Aaron Ma
-by: Aaron Ma --- drivers/nvme/host/core.c | 29 ++--- 1 file changed, 18 insertions(+), 11 deletions(-) diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c index 2c43e12b70af..fb7f05c310c8 100644 --- a/drivers/nvme/host/core.c +++ b/drivers/nvme/host/core.c

Re: [PATCH] nvme: determine the number of IO queues

2019-04-18 Thread Aaron Ma
On 4/18/19 1:33 AM, Maxim Levitsky wrote: > On Wed, 2019-04-17 at 20:32 +0300, Maxim Levitsky wrote: >> On Wed, 2019-04-17 at 22:12 +0800, Aaron Ma wrote: >>> Some controllers support limited IO queues, when over set >>> the number, it will return invalid field error. &g

Re: [PATCH] nvme: determine the number of IO queues

2019-04-18 Thread Aaron Ma
On 4/18/19 5:30 AM, Edmund Nadolski (Microsoft) wrote: > On 4/17/19 7:12 AM, Aaron Ma wrote: >> Some controllers support limited IO queues, when over set >> the number, it will return invalid field error. >> Then NVME will be removed by driver. >> >>

Re: [PATCH] nvme: determine the number of IO queues

2019-04-18 Thread Aaron Ma
On 4/18/19 8:13 PM, Minwoo Im wrote: >> Yes the IO queues number is 0's based, but driver would return error and >> remove the nvme device as dead. > > IMHO, if a controller indicates an error with this set_feature command, > then > we need to figure out why the controller was returning the

Re: [PATCH] nvme: determine the number of IO queues

2019-04-18 Thread Aaron Ma
On 4/18/19 9:33 PM, Minwoo Im wrote: > If the controller returns error for that command, how can we assure that > the controller would support a single I/O queue ? Make sense, I will keep *count = 0 in V2. Thanks, Aaron

Re: [PATCH] nvme: determine the number of IO queues

2019-04-18 Thread Aaron Ma
On 4/18/19 9:48 PM, Keith Busch wrote: > It does change the default behavior. If I have a degraded controller that > can't do IO in a machine with 1000's of CPUs, I have to iterate this > non-standard behavior 1000's of times before the drive is servicable > again. We currenlty figure that out

Re: [PATCH] nvme: determine the number of IO queues

2019-04-25 Thread Aaron Ma
On 4/25/19 10:39 PM, Christoph Hellwig wrote: > Honestly, unless this is a device shiping in a max market consumer > product already I don't think we should work around this crap at all, > given that this device has obviously never been tested at all. It > really needs a firmware fix instead of

[PATCH 1/2] Input: elantech - enable middle button support on 2 ThinkPads

2019-05-19 Thread Aaron Ma
Adding 2 new touchpad PNPIDs to enable middle button support. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/input/mouse/elantech.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c index a7f8b1614559

[PATCH 2/2] Input: synaptics - remove X240 from the topbuttonpad list

2019-05-19 Thread Aaron Ma
Lenovo ThinkPad X240 does not have the top software button. When this wrong ID in top button list, smbus mode will fail to probe, so keep it working at PS2 mode. Cc: sta...@vger.kernel.org Signed-off-by: Aaron Ma --- drivers/input/mouse/synaptics.c | 1 - 1 file changed, 1 deletion(-) diff

  1   2   >