linux-kernel - Re: [PATCH v2] sp5100_tco: Add SB8x0 chipset support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1354094368.4467.4.camel@mattotaupa>
Date:	Wed, 28 Nov 2012 10:19:28 +0100
From:	Paul Menzel <paulepanter@...rs.sourceforge.net>
To:	Takahisa Tanaka <mc74hc00@...il.com>
Cc:	linux-watchdog@...r.kernel.org, Wim Van Sebroeck <wim@...ana.be>,
	Arkadiusz Miskiewicz <arekm@...en.pl>,
	Bjorn Helgaas <bhelgaas@...gle.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Jonathan Nieder <jrnieder@...il.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] sp5100_tco: Add SB8x0 chipset support

Dear Takahisa,


as replied to ticket #43176 in Bugzilla thanks a lot for your work.


Am Freitag, den 16.11.2012, 00:23 +0900 schrieb Takahisa Tanaka:
> The current sp5100_tco driver only supports SP5100/SB7x0 chipset, doesn't
> support SB8x0 chipset, because current sp5100_tco driver doesn't know that the
> offset address for watchdog timer was changed from SB8x0 chipset.
> 
> The offset address of SP5100 and SB7x0 chipsets are as follows, quote from the
> AMD SB700/710/750 Register Reference Guide(Page 164) and the AMD SP5100

In English you leave a space before parentheses: … Guide (page 164) …

> Register Reference Guide(Page 166).

Dito.

>   WatchDogTimerControl 69h
>   WatchDogTimerBase0   6Ch
>   WatchDogTimerBase1   6Dh
>   WatchDogTimerBase2   6Eh
>   WatchDogTimerBase3   6Fh
> 
> In contrast, the offset address of SB8x0 chipset is as follows, quote from
> AMD SB800-Series Southbridges Register Reference Guide(Page 147).

Dito.

>   WatchDogTimerEn      48h
>   WatchDogTimerConfig  4Ch
> 
> So, In the case of SB8x0 chipset, sp5100_tco reads meaningless MMIO
> address(for example, 0xbafe00) from wrong offset address, and the following

Dito.

> message is logged.
> 
>    SP5100 TCO timer: mmio address 0xbafe00 already in use
> 
> With this patch, sp5100_tco driver supports SB8x0 chipset, and can avoid
> iomem resource conflict.
> 
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=43176
> Tested-by: Arkadiusz Miskiewicz <arekm@...en.pl>
> Signed-off-by: Takahisa Tanaka <mc74hc00@...il.com>
> 
> ---
> v1 -> v2
>  -Fix typo in module description and source code.
>  -Fix a bug that can't correctly determine the watchdog fired.
>  -Improve syslog messages.
>  -Truncate the commit log, because previous commit log is too long.

There are no too long commit messages. ;-)

> ---
>  drivers/watchdog/sp5100_tco.c | 321 +++++++++++++++++++++++++++++++++++-------
>  drivers/watchdog/sp5100_tco.h |  46 ++++--
>  2 files changed, 306 insertions(+), 61 deletions(-)

[…]

Tested-by: Paul Menzel <paulepanter@...rs.sourceforge.net>


Thanks,

Paul

Download attachment "signature.asc" of type "application/pgp-signature" (199 bytes)