linux-pwm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: Tomasz Figa <tomasz.figa@gmail.com>
Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
	Olof Johansson <olof@lixom.net>,
	linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-pwm@vger.kernel.org,
	Kukjin Kim <kgene.kim@samsung.com>,
	thierry.reding@gmail.com, Arnd Bergmann <arnd@arndb.de>,
	Mark Brown <broonie@kernel.org>,
	Thomas Abraham <thomas.abraham@linaro.org>
Subject: Re: [PATCH 00/15] Final Samsung PWM support cleanup
Date: Thu, 13 Jun 2013 22:17:43 +0200	[thread overview]
Message-ID: <201306132217.44265.heiko@sntech.de> (raw)
In-Reply-To: <1808375.nuQYa2FdfG@flatron>

Am Donnerstag, 13. Juni 2013, 11:07:38 schrieb Tomasz Figa:
> Hi Heiko, Sylwester,
> 
> On Thursday 13 of June 2013 01:52:05 Sylwester Nawrocki wrote:
> > On 06/13/2013 01:38 AM, Heiko St�bner wrote:
> > > Am Donnerstag, 13. Juni 2013, 01:13:47 schrieb Tomasz Figa:
> > >> Hi Olof,
> > >> 
> > >> On Wednesday 12 of June 2013 16:00:23 Olof Johansson wrote:
> > >>> On Wed, Jun 12, 2013 at 11:48:56PM +0200, Tomasz Figa wrote:
> > >>>>> Needs testing on other platforms as I could only compile test for
> > >>>>> them.
> > >>> 
> > >>> How/who can help here? WHo has and still cares for other affected
> > >>> platforms?
> > >>> 
> > >>>>> 66 files changed, 627 insertions(+), 1580 deletions(-)
> > >> 
> > >> I will try to test this on Exynos 4 at work, if I somehow manage to
> > >> find any way to use PWM on our boards (other than already-tested PWM
> > >> clocksource).
> > >> 
> > >> Unfortunately, except S3C6410, I don't have access to other
> > >> platforms, or at least I won't in any reasonable time.
> > >> 
> > >> If I remember correctly, in case of my watchdog support cleanup,
> > >> Kukjin
> > >> has been able to test it on S5P64x0.
> > >> 
> > >> Heiko and Sylwester used to have some s3c24xx-based boards, not sure
> > >> if
> > >> they still do and are interested in their support in mainline.
> > >> 
> > >> So all left is S5PC100 (for which I couldn't find anyone who cares)
> > >> and
> > >> Exynos5.
> > >> 
> > >>> Nice cleanup.
> > >> 
> > >> Thanks.
> > >> 
> > >>>> Any comments for this series?
> > >>> 
> > >>> Yes, above. :) In general looks good -- testing coverage is a bit of
> > >>> a
> > >>> concern.
> > >> 
> > >> Yeah. Unfortunately all I could do was testing on Tiny6410 and
> > >> compile
> > >> testing on others. I hope we find some really helpful people over
> > >> here who could waste a bit of their precious time for this museum.
> > >> ;)
> > > 
> > > I'll try to give it a testrun tomorrow. And my s3c2416 will always be
> > > the first-love-architecture for me ;-)
> > 
> > I'll likely test it tomorrow on a Mini2440 board, I was going to do it
> > earlier
> > but got distracted by plenty other things that needed attention too. At
> > least
> > that's one of the reasons I got the board - testing the cross-platform
> > cleanups
> > and ensuring they are not breaking the older platforms.
> 
> Thank you both for your support. Looking forward for testing results :)


on a s3c2416-based machine for dt and non-dt case
[with the limit that my machine does not has anything connected to the pwm 
outputs, so I only get to test the base-pwm and timer part]

Tested-by: Heiko Stuebner <heiko@sntech.de>

  reply	other threads:[~2013-06-13 20:17 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-05 21:18 [PATCH 00/15] Final Samsung PWM support cleanup Tomasz Figa
2013-06-05 21:18 ` [PATCH 01/15] ARM: SAMSUNG: Unify base address definitions of timer block Tomasz Figa
2013-06-05 21:18 ` [PATCH 02/15] ARM: SAMSUNG: Add new PWM platform device Tomasz Figa
2013-06-05 21:18 ` [PATCH 03/15] ARM: SAMSUNG: Set PWM platform data Tomasz Figa
2013-06-05 21:18 ` [PATCH 04/15] ARM: SAMSUNG: Move all platforms to new clocksource driver Tomasz Figa
2013-06-05 21:18 ` [PATCH 05/15] ARM: SAMSUNG: Remove old samsung-time driver Tomasz Figa
2013-06-05 21:18 ` [PATCH 06/15] ARM: SAMSUNG: Remove unused PWM timer IRQ chip code Tomasz Figa
2013-06-05 21:18 ` [PATCH 07/15] pwm: samsung: Rename to pwm-samsung-legacy Tomasz Figa
2013-06-05 21:18 ` [PATCH 08/15] pwm: Add new pwm-samsung driver Tomasz Figa
2013-06-13 20:14   ` Heiko Stübner
2013-06-13 20:18     ` Tomasz Figa
2013-06-16 17:19   ` Tomasz Figa
2013-06-16 20:45     ` Kukjin Kim
2013-06-17 20:29   ` Thierry Reding
2013-06-17 20:50     ` Tomasz Figa
2013-06-18 22:17       ` Thierry Reding
2013-06-18 22:45         ` Tomasz Figa
2013-06-19  9:43           ` Thierry Reding
2013-06-19 10:23             ` Tomasz Figa
2013-06-24 17:52               ` Tomasz Figa
2013-06-24 19:50                 ` Thierry Reding
2013-06-24 20:03                   ` Tomasz Figa
2013-06-24 20:28                     ` Thierry Reding
2013-06-24 20:55                     ` Arnd Bergmann
2013-06-18 17:59     ` Tomasz Figa
2013-06-18 18:06       ` Kukjin Kim
2013-06-18 18:13         ` Tomasz Figa
2013-06-18 21:33           ` Thierry Reding
2013-06-18 21:35             ` Tomasz Figa
2013-06-18 19:41     ` Tomasz Figa
2013-06-18 21:40       ` Thierry Reding
2013-06-18 21:42         ` Tomasz Figa
2013-06-18 22:20           ` Thierry Reding
2013-06-05 21:18 ` [PATCH 09/15] ARM: SAMSUNG: Rework private data handling in dev-backlight Tomasz Figa
2013-06-05 21:18 ` [PATCH 10/15] ARM: SAMSUNG: Modify board files to use new PWM platform device Tomasz Figa
2013-06-05 21:18 ` [PATCH 11/15] pwm: Remove superseded pwm-samsung-legacy driver Tomasz Figa
2013-06-05 21:18 ` [PATCH 12/15] ARM: SAMSUNG: Remove old PWM timer platform devices Tomasz Figa
2013-06-05 21:18 ` [PATCH 13/15] ARM: SAMSUNG: Remove pwm-clock infrastructure Tomasz Figa
2013-06-05 21:18 ` [PATCH 14/15] ARM: SAMSUNG: Remove remaining uses of plat/regs-timer.h header Tomasz Figa
2013-06-13 20:24   ` Heiko Stübner
2013-06-13 20:38     ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 15/15] ARM: SAMSUNG: Remove " Tomasz Figa
2013-06-12 21:48 ` [PATCH 00/15] Final Samsung PWM support cleanup Tomasz Figa
2013-06-12 23:00   ` Olof Johansson
2013-06-12 23:13     ` Tomasz Figa
2013-06-12 23:38       ` Heiko Stübner
2013-06-12 23:52         ` Sylwester Nawrocki
2013-06-13  9:07           ` Tomasz Figa
2013-06-13 20:17             ` Heiko Stübner [this message]
2013-06-13 22:27               ` Heiko Stübner
2013-06-13  0:25     ` Kukjin Kim

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=201306132217.44265.heiko@sntech.de \
    --to=heiko@sntech.de \
    --cc=arnd@arndb.de \
    --cc=broonie@kernel.org \
    --cc=kgene.kim@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pwm@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=olof@lixom.net \
    --cc=sylvester.nawrocki@gmail.com \
    --cc=thierry.reding@gmail.com \
    --cc=thomas.abraham@linaro.org \
    --cc=tomasz.figa@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).