Re: [PATCH v8 1/5] Documentation: add sbsa-gwdt driver documentation

From: Timur Tabi
Date: Fri Oct 30 2015 - 14:53:34 EST


On 10/30/2015 01:35 PM, Fu Wei wrote:
I think maybe Mark was asking why WS1 is optional, not the WS1
My answer is for "why WS1 is optional"!

>interrupt. Maybe you can reword the documentation to make is clear
>that
I didn't say : "only the*interrupt* for WS1 is optional."

WS1 itself is not optional. The spec says that WS0 and WS1 are separate events, and doesn't saying anything about either being optional. The *interrupt* for WS1, however, is optional.

Besides, what does the driver do with the WS1 interrupt? If it's specified in the device tree, it appears to be ignored by the driver. And the ACPI table only allows for specifying ONE interrupt. So how would the driver register a handler for WS1 on an ACPI system?

--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/