All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomasz Figa <t.figa@samsung.com>
To: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
	Kukjin Kim <kgene.kim@samsung.com>
Cc: "Tomasz Figa" <tomasz.figa@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org,
	"Heiko Stübner" <heiko@sntech.de>,
	"Wim Van Sebroeck" <wim@iguana.be>,
	linux-watchdog@vger.kernel.org, jhbird.choi@samsung.com
Subject: Re: [PATCH 0/6] Samsung watchdog support clean-up
Date: Mon, 03 Jun 2013 14:49:28 +0200	[thread overview]
Message-ID: <1850418.EOBZxN7PB0@amdc1227> (raw)
In-Reply-To: <51A9ADF4.8010102@gmail.com>

Hi Sylwester, Kukjin,

On Saturday 01 of June 2013 10:16:52 Sylwester Nawrocki wrote:
> Hi Tomasz,
> 
> On 05/19/2013 12:37 AM, Tomasz Figa wrote:
> > This series is aiming at cleaning up a bit watchdog support on Samsung
> > SoCs. The patches tweak a bit all the watchdog handling code here and
> > there, with the goal of making it DT- and multiplatform-friendly.
> > 
> > See particular patches for more detailed descriptions.
> > 
> > On S3C6410-based Tiny6410 board (Mini6410-compatible):
> > 
> > Tested-by: Tomasz Figa<tomasz.figa@gmail.com>
> 
> I've tested this series on a s3c2440 based board. The system restart
> works fine.
> 
> Tested-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>

Thank you Sylwester for testing this series.

Kukjin, since it has been already tested on s3c24xx and s3c64xx and nobody 
seems to care about s5p64x0 and s5pc100 (although I don't see a reason why 
these patches should break them), could we have this series applied?

Best regards,
-- 
Tomasz Figa
Linux Kernel Developer
Samsung R&D Institute Poland
Samsung Electronics

> > Depends on:
> >   - [PATCH] ARM: s5p64x0: Use common uncompress.h part for plat-samsung
> > 
> > Tomasz Figa (6):
> >    watchdog: s3c2410_wdt: Use local register definitions
> >    ARM: SAMSUNG: uncompress: Use local definitions of watchdog registers
> >    ARM: SAMSUNG: Add watchdog reset driver
> >    ARM: SAMSUNG: Let platforms use the new watchdog reset driver
> >    ARM: SAMSUNG: Remove legacy watchdog reset code
> >    ARM: SAMSUNG: Remove unused plat/regs-watchdog.h header
> >   
> >   arch/arm/Kconfig                                   |  3 +
> >   arch/arm/mach-s3c24xx/Kconfig                      |  2 +
> >   arch/arm/mach-s3c24xx/s3c2410.c                    |  3 +-
> >   arch/arm/mach-s3c24xx/s3c244x.c                    |  3 +-
> >   arch/arm/mach-s3c64xx/common.c                     |  8 +-
> >   arch/arm/mach-s5p64x0/common.c                     |  4 +-
> >   arch/arm/mach-s5pc100/common.c                     |  3 +-
> >   arch/arm/plat-samsung/Kconfig                      |  6 ++
> >   arch/arm/plat-samsung/Makefile                     |  1 +
> >   arch/arm/plat-samsung/include/plat/regs-watchdog.h | 41 ---------
> >   arch/arm/plat-samsung/include/plat/uncompress.h    | 15 +++-
> >   .../arm/plat-samsung/include/plat/watchdog-reset.h | 38 ++-------
> >   arch/arm/plat-samsung/watchdog-reset.c             | 97
> >   ++++++++++++++++++++++ drivers/watchdog/s3c2410_wdt.c                  
> >     | 17 +++-
> >   14 files changed, 156 insertions(+), 85 deletions(-)
> >   delete mode 100644 arch/arm/plat-samsung/include/plat/regs-watchdog.h
> >   create mode 100644 arch/arm/plat-samsung/watchdog-reset.c>
> 
> --
> Regards,
> Sylwester
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc"
> in the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: t.figa@samsung.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/6] Samsung watchdog support clean-up
Date: Mon, 03 Jun 2013 14:49:28 +0200	[thread overview]
Message-ID: <1850418.EOBZxN7PB0@amdc1227> (raw)
In-Reply-To: <51A9ADF4.8010102@gmail.com>

Hi Sylwester, Kukjin,

On Saturday 01 of June 2013 10:16:52 Sylwester Nawrocki wrote:
> Hi Tomasz,
> 
> On 05/19/2013 12:37 AM, Tomasz Figa wrote:
> > This series is aiming at cleaning up a bit watchdog support on Samsung
> > SoCs. The patches tweak a bit all the watchdog handling code here and
> > there, with the goal of making it DT- and multiplatform-friendly.
> > 
> > See particular patches for more detailed descriptions.
> > 
> > On S3C6410-based Tiny6410 board (Mini6410-compatible):
> > 
> > Tested-by: Tomasz Figa<tomasz.figa@gmail.com>
> 
> I've tested this series on a s3c2440 based board. The system restart
> works fine.
> 
> Tested-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>

Thank you Sylwester for testing this series.

Kukjin, since it has been already tested on s3c24xx and s3c64xx and nobody 
seems to care about s5p64x0 and s5pc100 (although I don't see a reason why 
these patches should break them), could we have this series applied?

Best regards,
-- 
Tomasz Figa
Linux Kernel Developer
Samsung R&D Institute Poland
Samsung Electronics

> > Depends on:
> >   - [PATCH] ARM: s5p64x0: Use common uncompress.h part for plat-samsung
> > 
> > Tomasz Figa (6):
> >    watchdog: s3c2410_wdt: Use local register definitions
> >    ARM: SAMSUNG: uncompress: Use local definitions of watchdog registers
> >    ARM: SAMSUNG: Add watchdog reset driver
> >    ARM: SAMSUNG: Let platforms use the new watchdog reset driver
> >    ARM: SAMSUNG: Remove legacy watchdog reset code
> >    ARM: SAMSUNG: Remove unused plat/regs-watchdog.h header
> >   
> >   arch/arm/Kconfig                                   |  3 +
> >   arch/arm/mach-s3c24xx/Kconfig                      |  2 +
> >   arch/arm/mach-s3c24xx/s3c2410.c                    |  3 +-
> >   arch/arm/mach-s3c24xx/s3c244x.c                    |  3 +-
> >   arch/arm/mach-s3c64xx/common.c                     |  8 +-
> >   arch/arm/mach-s5p64x0/common.c                     |  4 +-
> >   arch/arm/mach-s5pc100/common.c                     |  3 +-
> >   arch/arm/plat-samsung/Kconfig                      |  6 ++
> >   arch/arm/plat-samsung/Makefile                     |  1 +
> >   arch/arm/plat-samsung/include/plat/regs-watchdog.h | 41 ---------
> >   arch/arm/plat-samsung/include/plat/uncompress.h    | 15 +++-
> >   .../arm/plat-samsung/include/plat/watchdog-reset.h | 38 ++-------
> >   arch/arm/plat-samsung/watchdog-reset.c             | 97
> >   ++++++++++++++++++++++ drivers/watchdog/s3c2410_wdt.c                  
> >     | 17 +++-
> >   14 files changed, 156 insertions(+), 85 deletions(-)
> >   delete mode 100644 arch/arm/plat-samsung/include/plat/regs-watchdog.h
> >   create mode 100644 arch/arm/plat-samsung/watchdog-reset.c>
> 
> --
> Regards,
> Sylwester
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc"
> in the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2013-06-03 12:49 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-18 22:37 [PATCH 0/6] Samsung watchdog support clean-up Tomasz Figa
2013-05-18 22:37 ` Tomasz Figa
2013-05-18 22:37 ` [PATCH 1/6] watchdog: s3c2410_wdt: Use local register definitions Tomasz Figa
2013-05-18 22:37   ` Tomasz Figa
2013-05-18 22:37 ` [PATCH 2/6] ARM: SAMSUNG: uncompress: Use local definitions of watchdog registers Tomasz Figa
2013-05-18 22:37   ` Tomasz Figa
2013-05-18 22:50   ` [PATCH v2 " Tomasz Figa
2013-05-18 22:50     ` Tomasz Figa
2013-05-18 22:37 ` [PATCH 3/6] ARM: SAMSUNG: Add watchdog reset driver Tomasz Figa
2013-05-18 22:37   ` Tomasz Figa
2013-05-18 22:37 ` [PATCH 4/6] ARM: SAMSUNG: Let platforms use the new " Tomasz Figa
2013-05-18 22:37   ` Tomasz Figa
2013-05-18 22:37 ` [PATCH 5/6] ARM: SAMSUNG: Remove legacy watchdog reset code Tomasz Figa
2013-05-18 22:37   ` Tomasz Figa
2013-05-18 22:37 ` [PATCH 6/6] ARM: SAMSUNG: Remove unused plat/regs-watchdog.h header Tomasz Figa
2013-05-18 22:37   ` Tomasz Figa
2013-06-01  8:16 ` [PATCH 0/6] Samsung watchdog support clean-up Sylwester Nawrocki
2013-06-01  8:16   ` Sylwester Nawrocki
2013-06-03 12:49   ` Tomasz Figa [this message]
2013-06-03 12:49     ` Tomasz Figa
2013-06-04  9:37     ` Kukjin Kim
2013-06-04  9:37       ` Kukjin Kim
2013-06-04  9:45       ` Tomasz Figa
2013-06-04  9:45         ` Tomasz Figa
2013-06-10 10:00         ` Kukjin Kim
2013-06-10 10:00           ` Kukjin Kim
2013-06-13 19:11           ` Tomasz Figa
2013-06-13 19:11             ` Tomasz Figa
2013-06-13 23:22             ` Kukjin Kim
2013-06-13 23:22               ` Kukjin Kim
2013-06-13 23:25               ` Tomasz Figa
2013-06-13 23:25                 ` Tomasz Figa

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1850418.EOBZxN7PB0@amdc1227 \
    --to=t.figa@samsung.com \
    --cc=heiko@sntech.de \
    --cc=jhbird.choi@samsung.com \
    --cc=kgene.kim@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=sylvester.nawrocki@gmail.com \
    --cc=tomasz.figa@gmail.com \
    --cc=wim@iguana.be \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.