lkml.org 
[lkml]   [2013]   [Mar]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2] sp5100_tco: Remove code that may cause a boot failure
From
Hi Paul,

Thank you for your advice!
The patch reflecting what was pointed out is posted later.

2013/3/3 Paul Menzel <paulepanter@users.sourceforge.net>:
> I still do not understand it, as your patches were for SB8x0 support,
> right?

There are two methods of accessing the watchdog registers.

1. Re-programming a resource address obtained by allocate_resource()
to chipset.
2. Use the direct memory-mapped IO access.

The method 1 can be used by all the chipsets (SP5100, SB7x0, SB8x0 or
later). However, experience shows that only PC with the SB8x0 (or
later) chipsets can use the method 2.

This patch removes the method 1, because the critical problem was found.
That's why the watchdog timer was able to be used on SP5100 and SB7x0
chipsets until now.


> Is this patch exactly reverting these two patches?

Yes. This patch was made on the linux-next brunch (next-20130301).

This patch is dependent on the following patches. So, This patch isn't
applicable to Linux 3.8 kernel which doesn't apply the following patches.

commit adbdcc030be5f7bb54d229c04efdd0c6b7ed3ab7
watchdog: sp5100_tco: Write back the original value to reserved
bits, instead of zero

commit a6e26b1369b34760ca5e1f16039c9018bf68ce7e
watchdog: sp5100_tco: Fix wrong indirect I/O access for getting
value of reserved bits


> As Linux 3.8 has been released, please add

The above-mentioned two patches may be soon applied to stable kernel (3.8.n).

https://lkml.org/lkml/2013/3/1/546

I thought that I should submit this patch to the stable after these
two patches are merged into stable 3.8.n.


Regards,
Takahisa


\
 
 \ /
  Last update: 2013-03-03 17:21    [W:0.065 / U:0.396 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site