Re: [PATCH v3] mfd: cros-ec: copy the whole event in get_next_event_xfer

2018-10-09 Thread Benson Leung
Increase maximum mkbp event size") > > Acked-by: Enric Balletbo i Serra > Tested-by: Emil Renner Berthing > Signed-off-by: Emil Karlson Applied. Sent to Greg for rc8, hopefully. -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org

Re: [PATCH 2/2] platform/chrome: Add a new interrupt path for cros_ec_lpc

2018-10-09 Thread Benson Leung
&& ec_dev->irq <= 0) { > status = acpi_install_notify_handler(adev->handle, >ACPI_ALL_NOTIFY, >cros_ec_lpc_acpi_notify, > -- > 2.19.0.605.g01d371f741-goog > -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH 2/2] platform/chrome: Add a new interrupt path for cros_ec_lpc

2018-10-09 Thread Benson Leung
&& ec_dev->irq <= 0) { > status = acpi_install_notify_handler(adev->handle, >ACPI_ALL_NOTIFY, >cros_ec_lpc_acpi_notify, > -- > 2.19.0.605.g01d371f741-goog > -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

[GIT PULL] IB between mfd and platform/chrome for move of lpc driver

2018-10-09 Thread Benson Leung
insertions(+), 221 deletions(-) rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_mec.h (96%) rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_reg.h (94%) -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Proje

[GIT PULL] IB between mfd and platform/chrome for move of lpc driver

2018-10-09 Thread Benson Leung
insertions(+), 221 deletions(-) rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_mec.h (96%) rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_reg.h (94%) -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Proje

[GIT PULL] chrome-platform fixes for v4.19-rc8

2018-10-09 Thread Benson Leung
(-) -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

[GIT PULL] chrome-platform fixes for v4.19-rc8

2018-10-09 Thread Benson Leung
(-) -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH] mfd: cros-ec: copy the whole event in get_next_event_xfer

2018-10-09 Thread Benson Leung
his as urgent fix for the coming release I > am > happy with it. We can always send follow up patches to sync with the > downstream > version if is preferred. > > Neil, can you give us your Tested-by to have the make sure this doesn't break > with protocol v1, I don't have such hardware. > &

Re: [PATCH] mfd: cros-ec: copy the whole event in get_next_event_xfer

2018-10-09 Thread Benson Leung
his as urgent fix for the coming release I > am > happy with it. We can always send follow up patches to sync with the > downstream > version if is preferred. > > Neil, can you give us your Tested-by to have the make sure this doesn't break > with protocol v1, I don't have such hardware. > &

Re: [PATCH 2/2] mfd: cros_ec: Fix and improve kerneldoc comments.

2018-09-10 Thread Benson Leung
On Mon, Sep 10, 2018 at 5:12 PM Lee Jones wrote: > > On Fri, 07 Sep 2018, Benson Leung wrote: > > > Hi Enric, > > > > On Wed, Jul 18, 2018 at 06:09:56PM +0200, Enric Balletbo i Serra wrote: > > > cros-ec includes inside the MFD subsystem, specially the fil

Re: [PATCH 2/2] mfd: cros_ec: Fix and improve kerneldoc comments.

2018-09-10 Thread Benson Leung
On Mon, Sep 10, 2018 at 5:12 PM Lee Jones wrote: > > On Fri, 07 Sep 2018, Benson Leung wrote: > > > Hi Enric, > > > > On Wed, Jul 18, 2018 at 06:09:56PM +0200, Enric Balletbo i Serra wrote: > > > cros-ec includes inside the MFD subsystem, specially the fil

Re: [PATCH 2/2] mfd: cros_ec: Fix and improve kerneldoc comments.

2018-09-07 Thread Benson Leung
mentation. > This patch tries to improve the documentation and also fixes all the > issues reported by kerneldoc script. > > Signed-off-by: Enric Balletbo i Serra Applied, thanks. -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Projec

Re: [PATCH 2/2] mfd: cros_ec: Fix and improve kerneldoc comments.

2018-09-07 Thread Benson Leung
mentation. > This patch tries to improve the documentation and also fixes all the > issues reported by kerneldoc script. > > Signed-off-by: Enric Balletbo i Serra Applied, thanks. -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Projec

Re: [PATCH 1/2] platform/chrome: Move mfd/cros_ec_lpc* includes to drivers/platform.

2018-09-07 Thread Benson Leung
includes > are only used for the platform driver and are not really related to the > MFD subsystem, so move the includes from include/linux/mfd to > drivers/platform/chrome. > > Signed-off-by: Enric Balletbo i Serra Thanks. Applied to my working branch for v4.20. -- Benson Leung

Re: [PATCH 1/2] platform/chrome: Move mfd/cros_ec_lpc* includes to drivers/platform.

2018-09-07 Thread Benson Leung
includes > are only used for the platform driver and are not really related to the > MFD subsystem, so move the includes from include/linux/mfd to > drivers/platform/chrome. > > Signed-off-by: Enric Balletbo i Serra Thanks. Applied to my working branch for v4.20. -- Benson Leung

Re: [PATCH] ASoC: max98373: usleep_range() needs include/delay.h

2018-09-06 Thread Benson Leung
ce/src/third_party/kernel/v4.4/sound/soc/codecs/max98373.c:734:2: > error: implicit declaration of function 'usleep_range' > [-Werror,-Wimplicit-function-declaration] > usleep_range(1, 11000); > ^ > > Including delay.h "fixes" this. > > Signed-

Re: [PATCH] ASoC: max98373: usleep_range() needs include/delay.h

2018-09-06 Thread Benson Leung
ce/src/third_party/kernel/v4.4/sound/soc/codecs/max98373.c:734:2: > error: implicit declaration of function 'usleep_range' > [-Werror,-Wimplicit-function-declaration] > usleep_range(1, 11000); > ^ > > Including delay.h "fixes" this. > > Signed-

Re: [PATCH 0/2] mfd: platform/chrome: some more cleanups between these subsystems.

2018-09-06 Thread Benson Leung
h | 295 +++--- > > 8 files changed, 322 insertions(+), 221 deletions(-) > > rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_mec.h > > (96%) > > rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_reg.h >

Re: [PATCH 0/2] mfd: platform/chrome: some more cleanups between these subsystems.

2018-09-06 Thread Benson Leung
h | 295 +++--- > > 8 files changed, 322 insertions(+), 221 deletions(-) > > rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_mec.h > > (96%) > > rename {include/linux/mfd => drivers/platform/chrome}/cros_ec_lpc_reg.h >

Re: [GIT PULL] chrome-platform updates for v4.19

2018-08-21 Thread Benson Leung
On Tue, Aug 21, 2018 at 01:40:21PM -0700, Linus Torvalds wrote: > On Tue, Aug 21, 2018 at 11:54 AM Benson Leung wrote: > > > > Minor changes this time around for 4.19. > > > > cros_ec_i2c and cros_ec_spi drivers moved from mfd > > to chrome platform. > > A

Re: [GIT PULL] chrome-platform updates for v4.19

2018-08-21 Thread Benson Leung
On Tue, Aug 21, 2018 at 01:40:21PM -0700, Linus Torvalds wrote: > On Tue, Aug 21, 2018 at 11:54 AM Benson Leung wrote: > > > > Minor changes this time around for 4.19. > > > > cros_ec_i2c and cros_ec_spi drivers moved from mfd > > to chrome platform. > > A

[GIT PULL] chrome-platform updates for v4.19

2018-08-21 Thread Benson Leung
. cros_ec_i2c and cros_ec_spi drivers moved from mfd to chrome platform. Benson Leung (2): platform/chrome: chromeos_tbmc - fix SPDX identifier Merge tag 'ib-platform-chrome-mfd-move-cros-ec-transport-for-4.19' into working

[GIT PULL] chrome-platform updates for v4.19

2018-08-21 Thread Benson Leung
. cros_ec_i2c and cros_ec_spi drivers moved from mfd to chrome platform. Benson Leung (2): platform/chrome: chromeos_tbmc - fix SPDX identifier Merge tag 'ib-platform-chrome-mfd-move-cros-ec-transport-for-4.19' into working

[GIT PULL] Immutable branch (mfd, chrome) due for the v4.19 window

2018-07-08 Thread Benson Leung
2c.c (100%) rename drivers/{mfd => platform/chrome}/cros_ec_spi.c (100%) Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

[GIT PULL] Immutable branch (mfd, chrome) due for the v4.19 window

2018-07-08 Thread Benson Leung
2c.c (100%) rename drivers/{mfd => platform/chrome}/cros_ec_spi.c (100%) Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH 0/3] platform/chrome: mfd: Move cros-ec transport drivers to drivers/platform

2018-07-02 Thread Benson Leung
or the others to pull. What do you all think? Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH 0/3] platform/chrome: mfd: Move cros-ec transport drivers to drivers/platform

2018-07-02 Thread Benson Leung
or the others to pull. What do you all think? Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH 2/3] arm/arm64: configs: Remove the MFD_ prefix for MFD_CROS_EC_I2C/SPI symbols.

2018-07-02 Thread Benson Leung
e you are pushing this through arm-soc or Russell. > In such case, for Exynos boards: > Acked-by: Krzysztof Kozlowski > > Best regards, > Krzysztof I was planning on creating an immutable branch for this series and merging it through platform/chrome. Lee, Russell, Dmitry, what do

Re: [PATCH 2/3] arm/arm64: configs: Remove the MFD_ prefix for MFD_CROS_EC_I2C/SPI symbols.

2018-07-02 Thread Benson Leung
e you are pushing this through arm-soc or Russell. > In such case, for Exynos boards: > Acked-by: Krzysztof Kozlowski > > Best regards, > Krzysztof I was planning on creating an immutable branch for this series and merging it through platform/chrome. Lee, Russell, Dmitry, what do

Re: [PATCH] mfd: cros_ec: remove unused __remove function

2018-06-08 Thread Benson Leung
s: 3aa2177e4787 ("mfd: cros_ec: Use devm_kzalloc for private data") Gwendal, in PATCH v2 of https://patchwork.kernel.org/patch/10439449/ you mentioned that you readded the __remove to avoid a warning when built as a module. Can you explain what's going on? Benson -- Benson Leung St

Re: [PATCH] mfd: cros_ec: remove unused __remove function

2018-06-08 Thread Benson Leung
s: 3aa2177e4787 ("mfd: cros_ec: Use devm_kzalloc for private data") Gwendal, in PATCH v2 of https://patchwork.kernel.org/patch/10439449/ you mentioned that you readded the __remove to avoid a warning when built as a module. Can you explain what's going on? Benson -- Benson Leung St

Re: [PATCH] ACPI LID: increment wakeup count only when notified.

2018-06-06 Thread Benson Leung
gt;input->users; > > @@ -426,7 +424,6 @@ static void acpi_button_notify(struct acpi_device > > *device, u32 event) > > } else { > > int keycode; > > > > - acpi_pm_wakeup_event(>dev); > >

Re: [PATCH] ACPI LID: increment wakeup count only when notified.

2018-06-06 Thread Benson Leung
gt;input->users; > > @@ -426,7 +424,6 @@ static void acpi_button_notify(struct acpi_device > > *device, u32 event) > > } else { > > int keycode; > > > > - acpi_pm_wakeup_event(>dev); > >

Re: [PATCH v2 2/8] platform/chrome: pstore: Switch to SPDX identifier.

2018-06-05 Thread Benson Leung
// Copyright (C) 2013 Google, Inc. > > #include > #include > -- > 2.17.1 > Thanks! -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH v2 2/8] platform/chrome: pstore: Switch to SPDX identifier.

2018-06-05 Thread Benson Leung
// Copyright (C) 2013 Google, Inc. > > #include > #include > -- > 2.17.1 > Thanks! -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH v2 1/8] platform/chrome: cros_ec: Switch to SPDX identifier.

2018-06-05 Thread Benson Leung
e > diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c > b/drivers/platform/chrome/cros_kbd_led_backlight.c > index ca3e4da852b4..bd4aa9331806 100644 > --- a/drivers/platform/chrome/cros_kbd_led_backlight.c > +++ b/drivers/platform/chrome/cros_kbd_led_backlight.c >

Re: [PATCH v2 1/8] platform/chrome: cros_ec: Switch to SPDX identifier.

2018-06-05 Thread Benson Leung
e > diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c > b/drivers/platform/chrome/cros_kbd_led_backlight.c > index ca3e4da852b4..bd4aa9331806 100644 > --- a/drivers/platform/chrome/cros_kbd_led_backlight.c > +++ b/drivers/platform/chrome/cros_kbd_led_backlight.c >

Re: [PATCH v2 7/8] extcon: usbc-cros-ec: Switch to SPDX identifier.

2018-06-05 Thread Benson Leung
On Tue, Jun 05, 2018 at 07:54:28PM +0200, Enric Balletbo i Serra wrote: > Adopt the SPDX license identifier headers to ease license compliance > management. > > Signed-off-by: Enric Balletbo i Serra Reviewed-by: Benson Leung > --- > > Changes in v2: > - [8/9] Fi

Re: [PATCH v2 7/8] extcon: usbc-cros-ec: Switch to SPDX identifier.

2018-06-05 Thread Benson Leung
On Tue, Jun 05, 2018 at 07:54:28PM +0200, Enric Balletbo i Serra wrote: > Adopt the SPDX license identifier headers to ease license compliance > management. > > Signed-off-by: Enric Balletbo i Serra Reviewed-by: Benson Leung > --- > > Changes in v2: > - [8/9] Fi

Re: [PATCH] ACPI LID: increment wakeup count only when notified.

2018-06-04 Thread Benson Leung
> source for the last wake will be thrown off by this. Instead increment > the wakeup count only when there is a FIXED_HARDWARE/NOTFIY_STATUS event. > > Signed-off-by: Ravi Chandra Sadineni Reviewed-by: Benson Leung -- Benson Leung Staff Software Engineer Chrome OS Kernel Google I

Re: [PATCH] ACPI LID: increment wakeup count only when notified.

2018-06-04 Thread Benson Leung
> source for the last wake will be thrown off by this. Instead increment > the wakeup count only when there is a FIXED_HARDWARE/NOTFIY_STATUS event. > > Signed-off-by: Ravi Chandra Sadineni Reviewed-by: Benson Leung -- Benson Leung Staff Software Engineer Chrome OS Kernel Google I

[GIT PULL] chrome-platform updates for v4.18

2018-06-03 Thread Benson Leung
driver, which is useful for x86 convertible Chromebooks. Other misc. cleanup. Thanks, Benson Arnd Bergmann (1): platform: chrome: Add input dependency for tablet switch driver Benson Leung (2): platform/chrome

[GIT PULL] chrome-platform updates for v4.18

2018-06-03 Thread Benson Leung
driver, which is useful for x86 convertible Chromebooks. Other misc. cleanup. Thanks, Benson Arnd Bergmann (1): platform: chrome: Add input dependency for tablet switch driver Benson Leung (2): platform/chrome

Re: [PATCH] mfd: cros_ec_dev: Don't advertise junk features on failure

2018-05-31 Thread Benson Leung
ction to check EC > features.") > Cc: Benson Leung > Cc: Enric Balletbo i Serra > Cc: Gwendal Grignou > Cc: Andy Shevchenko > Signed-off-by: Stephen Boyd Acked-by: Benson Leung > --- > drivers/mfd/cros_ec_dev.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 delet

Re: [PATCH] mfd: cros_ec_dev: Don't advertise junk features on failure

2018-05-31 Thread Benson Leung
ction to check EC > features.") > Cc: Benson Leung > Cc: Enric Balletbo i Serra > Cc: Gwendal Grignou > Cc: Andy Shevchenko > Signed-off-by: Stephen Boyd Acked-by: Benson Leung > --- > drivers/mfd/cros_ec_dev.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 delet

Re: [PATCH v2] platform/chrome: Use to_cros_ec_dev more broadly

2018-05-30 Thread Benson Leung
he new macro in cros_ec.h through my tree. Looks to merge clean with mfd's changes this time. Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH v2] platform/chrome: Use to_cros_ec_dev more broadly

2018-05-30 Thread Benson Leung
he new macro in cros_ec.h through my tree. Looks to merge clean with mfd's changes this time. Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH] platform/chrome: chromeos_laptop: fix touchpad button mapping on Celes

2018-05-29 Thread Benson Leung
ton being stuck in pressed state > after the first click. > > Reported-by: Sultan Alsawaf > Signed-off-by: Dmitry Torokhov Applied for v4.18. Thanks. -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org sig

Re: [PATCH] platform/chrome: chromeos_laptop: fix touchpad button mapping on Celes

2018-05-29 Thread Benson Leung
ton being stuck in pressed state > after the first click. > > Reported-by: Sultan Alsawaf > Signed-off-by: Dmitry Torokhov Applied for v4.18. Thanks. -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org sig

Re: [PATCH] platform: chrome: Add input dependency for tablet switch driver

2018-05-28 Thread Benson Leung
efined! > > This adds the corresponding Kconfig dependency > > Fixes: b418f74170d7 ("platform: chrome: Add Tablet Switch ACPI driver") > Signed-off-by: Arnd Bergmann Thanks for the catch! Applied for 4.18. Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Go

Re: [PATCH] platform: chrome: Add input dependency for tablet switch driver

2018-05-28 Thread Benson Leung
efined! > > This adds the corresponding Kconfig dependency > > Fixes: b418f74170d7 ("platform: chrome: Add Tablet Switch ACPI driver") > Signed-off-by: Arnd Bergmann Thanks for the catch! Applied for 4.18. Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Go

Re: [PATCH v2 3/5] mfd: cros-ec: Introduce CEC commands and events definitions.

2018-05-23 Thread Benson Leung
; + uint8_t address; > + }; > +} __packed; > + > +/* Read various CEC parameters */ > +#define EC_CMD_CEC_GET 0x00BB > + > +struct ec_params_cec_get { > + uint8_t cmd; /* enum cec_command */ > +} __packed; > + > +struct ec_response_cec_get { &

Re: [PATCH v2 3/5] mfd: cros-ec: Introduce CEC commands and events definitions.

2018-05-23 Thread Benson Leung
}; > +} __packed; > + > +/* Read various CEC parameters */ > +#define EC_CMD_CEC_GET 0x00BB > + > +struct ec_params_cec_get { > + uint8_t cmd; /* enum cec_command */ > +} __packed; > + > +struct ec_response_cec_get { > + union { > + uint8_t enable;

Re: [PATCH v3] platform: chrome: Add Tablet Switch ACPI driver

2018-05-23 Thread Benson Leung
return ret; > + } > + return 0; > +} > + > +static const struct acpi_device_id chromeos_tbmc_acpi_device_ids[] = { > + { ACPI_DRV_NAME, 0 }, > + { } > +}; > +MODULE_DEVICE_TABLE(acpi, chromeos_tbmc_acpi_device_ids); > + > +static const SIMPLE_D

Re: [PATCH v3] platform: chrome: Add Tablet Switch ACPI driver

2018-05-23 Thread Benson Leung
return 0; > +} > + > +static const struct acpi_device_id chromeos_tbmc_acpi_device_ids[] = { > + { ACPI_DRV_NAME, 0 }, > + { } > +}; > +MODULE_DEVICE_TABLE(acpi, chromeos_tbmc_acpi_device_ids); > + > +static const SIMPLE_DEV_PM_OPS(chromeos_tbmc_pm_ops, NULL,

Re: [PATCH 2/2] Input: atmel_mxt_ts - require device properties present when probing

2018-05-23 Thread Benson Leung
k...@gmail.com> Looks good with the chromeos_laptop patch that preceeds this. Reviewed-by: Benson Leung <ble...@chromium.org> > --- > drivers/input/touchscreen/atmel_mxt_ts.c | 152 ++- > 1 file changed, 12 insertions(+), 140 deletions(-) > > diff --g

Re: [PATCH 2/2] Input: atmel_mxt_ts - require device properties present when probing

2018-05-23 Thread Benson Leung
good with the chromeos_laptop patch that preceeds this. Reviewed-by: Benson Leung > --- > drivers/input/touchscreen/atmel_mxt_ts.c | 152 ++- > 1 file changed, 12 insertions(+), 140 deletions(-) > > diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c &g

[GIT PULL] IB chrome-platform/input for atmel_mxt_ts & chromeos_laptop for device properties

2018-05-23 Thread Benson Leung
(1): platform/chrome: chromeos_laptop - supply properties for ACPI devices drivers/platform/chrome/chromeos_laptop.c | 307 +++--- 1 file changed, 278 insertions(+), 29 deletions(-) Thanks! -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble

[GIT PULL] IB chrome-platform/input for atmel_mxt_ts & chromeos_laptop for device properties

2018-05-23 Thread Benson Leung
(1): platform/chrome: chromeos_laptop - supply properties for ACPI devices drivers/platform/chrome/chromeos_laptop.c | 307 +++--- 1 file changed, 278 insertions(+), 29 deletions(-) Thanks! -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble

Re: [PATCH 1/2] platform/chrome: chromeos_laptop - supply properties for ACPI devices

2018-05-23 Thread Benson Leung
return ERR_PTR(-ENOMEM); > + > + error = chromeos_laptop_prepare_i2c_peripherals(cros_laptop, src); > + if (!error) > + error = chromeos_laptop_prepare_acpi_peripherals(cros_laptop, > + src); > + > +

Re: [PATCH 1/2] platform/chrome: chromeos_laptop - supply properties for ACPI devices

2018-05-23 Thread Benson Leung
+ error = chromeos_laptop_prepare_i2c_peripherals(cros_laptop, src); > + if (!error) > + error = chromeos_laptop_prepare_acpi_peripherals(cros_laptop, > + src); > + > + if (error) { > +

Re: [v3] platform: chrome: Add Tablet Switch ACPI driver

2018-05-23 Thread Benson Leung
On Wed, May 23, 2018 at 11:19 AM Guenter Roeck <li...@roeck-us.net> wrote: > What happened with this driver ? It is not upstream, yet the comments > on patchwork seem to suggest that it was accepted. My bad. I dropped this last year. I'll pick this now. -- Benson Leung Staff Softw

Re: [v3] platform: chrome: Add Tablet Switch ACPI driver

2018-05-23 Thread Benson Leung
On Wed, May 23, 2018 at 11:19 AM Guenter Roeck wrote: > What happened with this driver ? It is not upstream, yet the comments > on patchwork seem to suggest that it was accepted. My bad. I dropped this last year. I'll pick this now. -- Benson Leung Staff Software Engineer Chrome OS

Re: [PATCH] platform/chrome: cros_ec_lpc: do not try DMI match when ACPI device found

2018-05-22 Thread Benson Leung
rokhov <dmitry.torok...@gmail.com> Nice fix. Applied. Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH] platform/chrome: cros_ec_lpc: do not try DMI match when ACPI device found

2018-05-22 Thread Benson Leung
itry Torokhov Nice fix. Applied. Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [GIT PULL] chrome-platform updates for v4.17

2018-04-13 Thread Benson Leung
based form. > > I'll take this for now, but next time I see this kind of "late in the > merge window, absolutely zero exposure anywhere else", I'll just skip > the whole pull request. Don't do it. Sorry about that, Linus. Won't happen again. -- Benson Leung Staff Softw

Re: [GIT PULL] chrome-platform updates for v4.17

2018-04-13 Thread Benson Leung
based form. > > I'll take this for now, but next time I see this kind of "late in the > merge window, absolutely zero exposure anywhere else", I'll just skip > the whole pull request. Don't do it. Sorry about that, Linus. Won't happen again. -- Benson Leung Staff Softw

[GIT PULL] chrome-platform updates for v4.17

2018-04-12 Thread Benson Leung
Benson Leung (2): Merge remote-tracking branch 'chrome-platform-stop-being-a-platform-driver-plus-atmel_mxt_ts-for-v4.17' into working-branch-for-4.17 Merge remote-tracking branch 'origin/ib-chrome-platform-cros-ec-sysfs-debugfs-for-v4.17' into working

[GIT PULL] chrome-platform updates for v4.17

2018-04-12 Thread Benson Leung
Benson Leung (2): Merge remote-tracking branch 'chrome-platform-stop-being-a-platform-driver-plus-atmel_mxt_ts-for-v4.17' into working-branch-for-4.17 Merge remote-tracking branch 'origin/ib-chrome-platform-cros-ec-sysfs-debugfs-for-v4.17' into working

Re: [PATCH v2 00/14] chrome_laptop: stop being a platform driver

2018-04-10 Thread Benson Leung
1 - > drivers/input/touchscreen/atmel_mxt_ts.c | 231 +++--- > drivers/platform/chrome/chromeos_laptop.c | 896 +++-- > include/linux/platform_data/atmel_mxt_ts.h | 31 - > 4 files changed, 579 insertions(+), 580 deletions(-) > delete mode 100644 include/linux/platform_data/atmel_mxt_ts.

Re: [PATCH v2 00/14] chrome_laptop: stop being a platform driver

2018-04-10 Thread Benson Leung
1 - > drivers/input/touchscreen/atmel_mxt_ts.c | 231 +++--- > drivers/platform/chrome/chromeos_laptop.c | 896 +++-- > include/linux/platform_data/atmel_mxt_ts.h | 31 - > 4 files changed, 579 insertions(+), 580 deletions(-) > delete mode 100644 include/linux/platform_data/atmel_mxt_ts.

[GIT PULL] Immutable branch between MFD and Platform due for the v4.17 merge window

2018-04-10 Thread Benson Leung
changed, 194 insertions(+), 59 deletions(-) -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

[GIT PULL] Immutable branch between MFD and Platform due for the v4.17 merge window

2018-04-10 Thread Benson Leung
changed, 194 insertions(+), 59 deletions(-) -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH v2 01/14] Input: atmel_mxt_ts - do not pass suspend mode in platform data

2018-04-09 Thread Benson Leung
35,6 +3149,9 @@ static int mxt_probe(struct i2c_client *client, const > struct i2c_device_id *id) > init_completion(>reset_completion); > init_completion(>crc_completion); > > + data->suspend_mode = dmi_check_system(chromebook_T9_suspend_dmi) ? > +

Re: [PATCH v2 01/14] Input: atmel_mxt_ts - do not pass suspend mode in platform data

2018-04-09 Thread Benson Leung
(struct i2c_client *client, const > struct i2c_device_id *id) > init_completion(>reset_completion); > init_completion(>crc_completion); > > + data->suspend_mode = dmi_check_system(chromebook_T9_suspend_dmi) ? > + MXT_SUSPEND_T9_CTRL : MXT_SUS

Re: [PATCH v2] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-24 Thread Benson Leung
ary and incorrect whitespace changes. Thanks Alan. Fixed in v3. Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [PATCH v2] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-24 Thread Benson Leung
ary and incorrect whitespace changes. Thanks Alan. Fixed in v3. Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

[PATCH v3] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-24 Thread Benson Leung
Print bcdDevice which is used by vendors to identify different versions of the same product (or different versions of firmware). Adding this to the logs will be useful for support purposes. Match the %2x.%02x formatting that's used by lsusb -v for this same value. Signed-off-by: Benson Leung

[PATCH v3] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-24 Thread Benson Leung
Print bcdDevice which is used by vendors to identify different versions of the same product (or different versions of firmware). Adding this to the logs will be useful for support purposes. Match the %2x.%02x formatting that's used by lsusb -v for this same value. Signed-off-by: Benson Leung

Re: [PATCH] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-23 Thread Benson Leung
s this (bcdDevice: 0.03 for example) and in my experience > > it has generally matched up with how hardware manufacturers refer > > to their firmware. > > Nevermind, %x should show bcd properly. looks fine. I like the way lsusb -v does it better. Take a look at v2. Thanks, Benson --

Re: [PATCH] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-23 Thread Benson Leung
s this (bcdDevice: 0.03 for example) and in my experience > > it has generally matched up with how hardware manufacturers refer > > to their firmware. > > Nevermind, %x should show bcd properly. looks fine. I like the way lsusb -v does it better. Take a look at v2. Thanks, Benson --

[PATCH v2] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-23 Thread Benson Leung
Print bcdDevice which is used by vendors to identify different versions of the same product (or different versions of firmware). Adding this to the logs will be useful for support purposes. Match the %2x.%02x formatting that's used by lsusb -v for this same value. Signed-off-by: Benson Leung

[PATCH v2] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-23 Thread Benson Leung
Print bcdDevice which is used by vendors to identify different versions of the same product (or different versions of firmware). Adding this to the logs will be useful for support purposes. Match the %2x.%02x formatting that's used by lsusb -v for this same value. Signed-off-by: Benson Leung

[PATCH] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-23 Thread Benson Leung
Print bcdDevice which is used by vendors to identify different versions of the same product (or different versions of firmware). Adding this to the logs will be useful for support purposes. Signed-off-by: Benson Leung <ble...@chromium.org> --- drivers/usb/core/hub.c | 16 +

[PATCH] USB: announce bcdDevice as well as idVendor, idProduct.

2018-03-23 Thread Benson Leung
Print bcdDevice which is used by vendors to identify different versions of the same product (or different versions of firmware). Adding this to the logs will be useful for support purposes. Signed-off-by: Benson Leung --- drivers/usb/core/hub.c | 16 +--- 1 file changed, 9

Re: [PATCH v2 05/14] platform/chrome: chromeos_laptop - stop setting suspend mode for Atmel devices

2018-03-22 Thread Benson Leung
_board_info atmel_224s_tp_device = { > > static struct mxt_platform_data atmel_1664s_platform_data = { > .irqflags = IRQF_TRIGGER_FALLING, > - .suspend_mode = MXT_SUSPEND_T9_CTRL, > }; > > static struct i2c_board_info atmel_1664s_device = { > -- >

Re: [PATCH v2 05/14] platform/chrome: chromeos_laptop - stop setting suspend mode for Atmel devices

2018-03-22 Thread Benson Leung
= { > > static struct mxt_platform_data atmel_1664s_platform_data = { > .irqflags = IRQF_TRIGGER_FALLING, > - .suspend_mode = MXT_SUSPEND_T9_CTRL, > }; > > static struct i2c_board_info atmel_1664s_device = { > -- > 2.16.2.804.g6dcf76e118-goog >

Re: [PATCH 07/14] platform/chrome: chromeos_laptop - factor out getting IRQ from DMI

2018-03-19 Thread Benson Leung
if (!dev_data) { > - pr_err("failed to get data from dmi for %s\n", name); > + info->irq = chromeos_laptop_get_irq_from_dmi(name); > + if (info->irq < 0) > return NULL; > - } > - info->irq = dev_data->instance; > } > > adapter = i2c_get_adapter(bus); > -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: OpenPGP digital signature

Re: [PATCH 07/14] platform/chrome: chromeos_laptop - factor out getting IRQ from DMI

2018-03-19 Thread Benson Leung
pr_err("failed to get data from dmi for %s\n", name); > + info->irq = chromeos_laptop_get_irq_from_dmi(name); > + if (info->irq < 0) > return NULL; > - } > - info->irq = dev_data->instance; > } > > adapter = i2c_get_adapter(bus); > -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: OpenPGP digital signature

Re: [PATCH 06/14] platform/chrome: chromeos_laptop - introduce pr_fmt()

2018-03-19 Thread Benson Leung
atform/chrome/chromeos_laptop.c > +++ b/drivers/platform/chrome/chromeos_laptop.c > @@ -4,6 +4,8 @@ > // Copyright (C) 2012 Google, Inc. > // Author: Benson Leung <ble...@chromium.org> > > +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > + > #incl

Re: [PATCH 06/14] platform/chrome: chromeos_laptop - introduce pr_fmt()

2018-03-19 Thread Benson Leung
c > +++ b/drivers/platform/chrome/chromeos_laptop.c > @@ -4,6 +4,8 @@ > // Copyright (C) 2012 Google, Inc. > // Author: Benson Leung > > +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > + > #include > #include > #include > @@ -139,15 +1

Re: [PATCH 04/14] platform/chrome: chromeos_laptop - add SPDX identifier

2018-03-19 Thread Benson Leung
omeos_laptop.c > index d8599736a41a2..54a13c70e1d8f 100644 > --- a/drivers/platform/chrome/chromeos_laptop.c > +++ b/drivers/platform/chrome/chromeos_laptop.c > @@ -1,25 +1,8 @@ > -/* > - * chromeos_laptop.c - Driver to instantiate Chromebook i2c/smbus devices. &

Re: [PATCH 04/14] platform/chrome: chromeos_laptop - add SPDX identifier

2018-03-19 Thread Benson Leung
599736a41a2..54a13c70e1d8f 100644 > --- a/drivers/platform/chrome/chromeos_laptop.c > +++ b/drivers/platform/chrome/chromeos_laptop.c > @@ -1,25 +1,8 @@ > -/* > - * chromeos_laptop.c - Driver to instantiate Chromebook i2c/smbus devices. > - * > - * Author : Ben

Re: [RESEND][PATCH] platform/chrome: cros_ec_lpc: Add support for Google devices using custom coreboot firmware

2018-03-19 Thread Benson Leung
ff-by: Vittorio Gambaletta <linuxb...@vittgam.net> Applied for v4.17. Thanks! Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: PGP signature

Re: [RESEND][PATCH] platform/chrome: cros_ec_lpc: Add support for Google devices using custom coreboot firmware

2018-03-19 Thread Benson Leung
ot;Google_*" as BIOS version string, > instead the system vendor string would still be "GOOGLE", so this > seems to be a reasonable match for every Chromebook/box running > a custom firmware. > > Signed-off-by: Salvatore Bellizzi > Signed-off-by: Vittorio Gambaletta

Re: [PATCH] platform/chrome: cros_ec_lpc: wake up from s2idle on Chrome EC

2018-03-19 Thread Benson Leung
2idle can happen. > > Suggested-by: Rafael J. Wysocki <rafael.j.wyso...@intel.com> > Signed-off-by: Wenkai Du <wenkai...@intel.com> LGTM. Applied to my for v4.17 branch. Thank you! Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@goog

Re: [PATCH] platform/chrome: cros_ec_lpc: wake up from s2idle on Chrome EC

2018-03-19 Thread Benson Leung
2idle can happen. > > Suggested-by: Rafael J. Wysocki > Signed-off-by: Wenkai Du LGTM. Applied to my for v4.17 branch. Thank you! Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.as

Re: [PATCH 1/3] powercap: intel_rapl: Add support for Cannon Lake

2018-03-19 Thread Benson Leung
ANNONLAKE_MOBILE, rapl_defaults_core), > > RAPL_CPU(INTEL_FAM6_ATOM_SILVERMONT1, rapl_defaults_byt), > RAPL_CPU(INTEL_FAM6_ATOM_AIRMONT, rapl_defaults_cht), > -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: OpenPGP digital signature

Re: [PATCH 1/3] powercap: intel_rapl: Add support for Cannon Lake

2018-03-19 Thread Benson Leung
ore), > > RAPL_CPU(INTEL_FAM6_ATOM_SILVERMONT1, rapl_defaults_byt), > RAPL_CPU(INTEL_FAM6_ATOM_AIRMONT, rapl_defaults_cht), > -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. ble...@google.com Chromium OS Project ble...@chromium.org signature.asc Description: OpenPGP digital signature

Re: [PATCH 3/3] perf/x86/intel: Enable C-state residency events for Cannon Lake

2018-03-19 Thread Benson Leung
> > Signed-off-by: Harry Pan <harry@intel.com> Reviewed-by: Benson Leung <ble...@chromium.org> > --- > arch/x86/events/intel/cstate.c | 44 > +- > 1 file changed, 31 insertions(+), 13 deletions(-) > > diff --git a/arc

<    1   2   3   4   5   6   7   8   >