From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Message-ID: <557D7EEF.7010501@codeaurora.org> Date: Sun, 14 Jun 2015 08:17:35 -0500 From: Timur Tabi MIME-Version: 1.0 To: Fu Wei CC: Suravee Suthikulpanit , Linaro ACPI Mailman List , linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, Wei Fu , G Gregory , Al Stone , Hanjun Guo , Ashwin Chaugule , Arnd Bergmann , Guenter Roeck , Vipul Gandhi , Wim Van Sebroeck , Jon Masters , Leo Duran , Jon Corbet , Mark Rutland , Catalin Marinas , Will Deacon , rjw@rjwysocki.net Subject: Re: [PATCH non-pretimeout 3/7] ARM64: add SBSA Generic Watchdog device node in amd-seattle-soc.dtsi References: <=fu.wei@linaro.org> <1433958452-23721-1-git-send-email-fu.wei@linaro.org> <1433958452-23721-4-git-send-email-fu.wei@linaro.org> <557B470B.9030908@codeaurora.org> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-ID: Fu Wei wrote: >> >I think the sizes are wrong. They should be 0x1000 instead of 0x10000. > This has been proved by test, it works well on Seattle > Foundation model has same value. So I don't think it is wrong > > otherwise someone has the data sheet of Seattle B0, and it shows that is wrong. The registers block is only 0x1000 bytes long. So you're wasting 63KB of virtual address space for each block. -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation.