Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752124AbbKBED1 (ORCPT ); Sun, 1 Nov 2015 23:03:27 -0500 Received: from mail-ob0-f179.google.com ([209.85.214.179]:33396 "EHLO mail-ob0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752002AbbKBEDS (ORCPT ); Sun, 1 Nov 2015 23:03:18 -0500 MIME-Version: 1.0 In-Reply-To: <20151030190514.GD30791@leverpostej> References: <1445961999-9506-1-git-send-email-fu.wei@linaro.org> <1445961999-9506-2-git-send-email-fu.wei@linaro.org> <20151027162257.GJ3091@leverpostej> <5633BCA4.1050400@codeaurora.org> <20151030190514.GD30791@leverpostej> Date: Mon, 2 Nov 2015 12:03:17 +0800 Message-ID: Subject: Re: [PATCH v8 1/5] Documentation: add sbsa-gwdt driver documentation From: Fu Wei To: Mark Rutland Cc: Timur Tabi , Linaro ACPI Mailman List , linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, LKML , linux-doc@vger.kernel.org, Wei Fu , Arnd Bergmann , Guenter Roeck , Vipul Gandhi , Wim Van Sebroeck , Jon Masters , Leo Duran , Jon Corbet , Catalin Marinas , Will Deacon , Rafael Wysocki , Dave Young , Pratyush Anand , Suravee Suthikulpanit , Rob Herring Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2533 Lines: 75 Hi Mark, Great thanks for your feedback. On 31 October 2015 at 03:05, Mark Rutland wrote: > On Fri, Oct 30, 2015 at 01:53:24PM -0500, Timur Tabi wrote: >> 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. > > This is a moot point. The distintion between the signal and the > interrupt doens't matter here. > > I was only asking why the interrupt was optional, and it seems per the > spec it's expected to be handed to an agent at a higher exception level. yes, that is the good point. Thanks I have thought about it My thought is : In virtualization system, Linux kernel with KVM support as a Hypervisor, and guest are using a one of SBSA watchdog. WS0 is handled by guest OS, and WS1 will be handled by Hypervisor. And in datasheet of Foundation model, we can see: IRQ ID SPI offset Device 60 28 EL2 Generic Watchdog WS1 So maybe we need WS1 interrupt info, Maybe not. So I say : WS1 info in FDT binding info is optional. *BUT*, (1) I don't see any hardware need to handle WS1 for now, because AMD seattle is the only real hardware with SBSA watchdog I can test now. (2) In GTDT, there is not data about WS1 (3) I don't handle WS1 in this driver. > > That implies that the OS should only care about WS0, assuming that I've > understood correctly. yes, after getting your and Timur's email. I have thought about this in the weekend, Maybe we can forget about WS1 in the FDT binding info temporary until we need to handle WS1 in Linux on any hardware(or model). > > Thanks, > Mark. -- Best regards, Fu Wei Software Engineer Red Hat Software (Beijing) Co.,Ltd.Shanghai Branch Ph: +86 21 61221326(direct) Ph: +86 186 2020 4684 (mobile) Room 1512, Regus One Corporate Avenue,Level 15, One Corporate Avenue,222 Hubin Road,Huangpu District, Shanghai,China 200021 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/