linux-pwm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tomasz Figa <tomasz.figa@gmail.com>
To: "Sylwester Nawrocki" <sylvester.nawrocki@gmail.com>,
	"Heiko Stübner" <heiko@sntech.de>
Cc: 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 11:07:38 +0200	[thread overview]
Message-ID: <1808375.nuQYa2FdfG@flatron> (raw)
In-Reply-To: <51B909A5.2020409@gmail.com>

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 :)

Best regards,
Tomasz

  reply	other threads:[~2013-06-13  9:07 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 [this message]
2013-06-13 20:17             ` Heiko Stübner
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=1808375.nuQYa2FdfG@flatron \
    --to=tomasz.figa@gmail.com \
    --cc=arnd@arndb.de \
    --cc=broonie@kernel.org \
    --cc=heiko@sntech.de \
    --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 \
    /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).