From: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
To: Ian Molton <ian@mnementh.co.uk>
Cc: Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
linux-kernel@vger.kernel.org, Magnus Damm <damm@opensource.se>,
akpm@linux-foundation.org, Matt Fleming <matt@console-pimps.org>,
Philip Langdale <philipl@overt.org>,
Russell King - ARM Linux <linux@arm.linux.org.uk>
Subject: Re: [PATCH] tmio_mmc: Optionally support using platform clock
Date: Sat, 1 Aug 2009 18:10:52 +0400 [thread overview]
Message-ID: <bc64b4640908010710w1a0d6344s3e810433c28aadd5@mail.gmail.com> (raw)
In-Reply-To: <4A6F30AF.4000004@mnementh.co.uk>
Hi!
Sorry for the delay,
2009/7/28 Ian Molton <ian@mnementh.co.uk>:
> Guennadi Liakhovetski wrote:
>
>> Hi Ian
>
> Hi!
>
>> Thanks for the review.
>
> No problem - I hope you dont feel that I am picking on you in particular
> here, this problem is not isolated to your patches.
>
>> I understand your concerns. Of course, the _proper_ solution would be to
>> implement an architecture-independent clock API, something like what
>> clocklib was trying to do. So, yes, if clocklib were in place now, I
>> certainly would have used it.
>
> Which is of course a chicken/egg situation. I dealt with this a number of
> times during the e-series development, and every time, integration with
> mainline has gone more swiftly and with better quality, when the job was
> done properly to begin with.
>
>> I searched for those clocklib submission attempts (Dmitry added to CC).
>
> Also adding RMK to the CC:
>
>> Last one I can find (maybe I missed some) is from July 2008 - more than a
>> year ago. So, looks like our options currently are:
>>
>> 1. wait for new submissions of clocklib - if any are planned
>
> Dmitry: whats the current status of your clocklib work?
It wasn't refreshed since last submission or so, since for ARM we do have
rmk's clocks implementations and I other maintainers (except AVR32 IIRC)
didn't care at all.
>> 3. take over patches from Dmitry and bring them to a state acceptable for
>> mainline
>
> Take over / collaborate with. I'll happily help people push these patches.
>
>> I personally don't have free (as in beer) time to work on 2 or 3. Anyone?
>
> If I can find out what the current state of this stuff is, I'll help get it
> working - I can test / update all the TC6x and T7x MFD devices (and probably
> asic3 too)
I don't think I'll have time to work on/refresh clocklib patches. Feel
free to continue
where I've left them.
--
With best wishes
Dmitry
next prev parent reply other threads:[~2009-08-01 14:10 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-28 7:51 Guennadi Liakhovetski
2009-07-28 11:48 ` Matt Fleming
2009-07-28 13:39 ` Ian Molton
2009-07-28 13:46 ` Ian Molton
2009-07-28 15:45 ` Guennadi Liakhovetski
2009-07-28 17:09 ` Ian Molton
2009-08-01 14:10 ` Dmitry Eremin-Solenikov [this message]
2009-08-03 17:30 ` Ian Molton
2009-08-03 21:57 ` Dmitry Eremin-Solenikov
2009-07-29 15:20 ` pHilipp Zabel
2009-07-29 15:24 ` Magnus Damm
2009-07-29 15:33 ` pHilipp Zabel
2009-07-29 15:32 ` pHilipp Zabel
2009-07-29 23:40 ` Ian Molton
2009-07-29 21:12 ` pHilipp Zabel
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=bc64b4640908010710w1a0d6344s3e810433c28aadd5@mail.gmail.com \
--to=dbaryshkov@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=damm@opensource.se \
--cc=g.liakhovetski@gmx.de \
--cc=ian@mnementh.co.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=matt@console-pimps.org \
--cc=philipl@overt.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
all inboxes | Powered by JetHome®