mbox series

[v2,0/3] ACPI / watchdog: Fix two reported issues

Message ID 20200212145941.32914-1-mika.westerberg@linux.intel.com (mailing list archive)
Headers show
Series ACPI / watchdog: Fix two reported issues | expand

Message

Mika Westerberg Feb. 12, 2020, 2:59 p.m. UTC
Hi,

This is an updated version of the patch series posted here:

  https://lore.kernel.org/linux-watchdog/20200212110540.83559-1-mika.westerberg@linux.intel.com/

This should fix the two issues reported by Jean Delvare. I also addressed
all the review comments and fixed the bugs in the previous version.

This time I actually tested these on Intel Joule board and the watchdog
still works as expected.

I'm now including ACPICA maintainers as well to get some feedback regarding
the first patch of the series.

Mika Westerberg (3):
  ACPICA: Introduce ACPI_ACCESS_BYTE_WIDTH() macro
  ACPI / watchdog: Fix gas->access_width usage
  ACPI / watchdog: Set default timeout in probe

 drivers/acpi/acpi_watchdog.c |  3 +--
 drivers/watchdog/wdat_wdt.c  | 25 ++++++++++++++++++++++++-
 include/acpi/actypes.h       |  3 ++-
 3 files changed, 27 insertions(+), 4 deletions(-)

Comments

Rafael J. Wysocki Feb. 13, 2020, 10:53 p.m. UTC | #1
On Wed, Feb 12, 2020 at 3:59 PM Mika Westerberg
<mika.westerberg@linux.intel.com> wrote:
>
> Hi,
>
> This is an updated version of the patch series posted here:
>
>   https://lore.kernel.org/linux-watchdog/20200212110540.83559-1-mika.westerberg@linux.intel.com/
>
> This should fix the two issues reported by Jean Delvare. I also addressed
> all the review comments and fixed the bugs in the previous version.
>
> This time I actually tested these on Intel Joule board and the watchdog
> still works as expected.
>
> I'm now including ACPICA maintainers as well to get some feedback regarding
> the first patch of the series.
>
> Mika Westerberg (3):
>   ACPICA: Introduce ACPI_ACCESS_BYTE_WIDTH() macro
>   ACPI / watchdog: Fix gas->access_width usage
>   ACPI / watchdog: Set default timeout in probe

Applied as fixes for 5.6, thanks!