Thread (28 messages) 28 messages, 7 authors, 2017-11-30
STALE3125d

[PATCH] rtc: Allow rtc drivers to specify the tv_nsec value for ntp

From: Alexandre Belloni <hidden>
Date: 2017-11-30 19:39:21

On 28/11/2017 at 10:20:25 +0000, Russell King - ARM Linux wrote:
On Mon, Nov 27, 2017 at 08:31:35PM +0100, Alexandre Belloni wrote:
quoted
On 27/11/2017 at 18:53:52 +0000, Russell King - ARM Linux wrote:
quoted
You are, yet again, wrong.

I am in a position to make the comment because it was me who identified
the problem, put in the hours to work on, develop and extensively test
Jason's patch.  So, it's partly my time that you seem to be wasting,
and that gives me every right to complain at this point.

You, on the other hand, were copied with every single email, and did
nothing to discuss the issue except for the "easy" bits when I posted
a relatively smaller patch - but you ignored the bigger issue.
And this is exactly what you do with other people patches/time when you
don't like their changes.
You simply ignore the patch series until they go away.
That is not intentional.
So please, don't assume that I replied late intentionally.
quoted
I would really expect people merging code in any subsystem to wait for
the ack of the maintainer of that subsystem.

I didn't complain about any missing email addresses, I said the RTC ML
was not copied but that is didn't matter.
A mailing list is an email address.
Yes and again, this was a non issue, simply that the patch was not in
the RTC patchwork but that is completely irrelevant to the matter.
Now you're making crap up.  I've not made any comment about this being
something that needs fixing urgently.  In fact, as I've said several
times already, I really don't care what you do with the mainline kernel,
because I have a fix here locally that I intend to use and maintain
into the future.  For me, the issue is fixed and resolved, and I intend
to spend no further time developing any fixes for it.

My comments are about the _two months_ its taken to get to the stage of
finding out that you don't like the approach.
Again, sorry, this was not intentional.
The result of this is, most likely, one of:
1. you'll revert the patch (which, incidentally, has no real effect until
   my other patches get merged) and everyone will either be stuck with a
   kernel that sets their RTC time wrong when they have NTP installed
Ok, let's not waste everything. I think the main issue is that I still
don't get how this can actually improve the situation with regards to
userspace. Can you share your series so I could maybe understand? I'm
especially interested in how you will handle the pcf8523 (if you did
that of course).


-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help