* Was this really supposed to go in?
@ 2006-05-20 15:36 Pierre Ossman
2006-05-20 15:45 ` Russell King
0 siblings, 1 reply; 3+ messages in thread
From: Pierre Ossman @ 2006-05-20 15:36 UTC (permalink / raw)
To: linux-kernel, ppisa4lists, Russell King
Commit 2c171bf13423dc5293188cea7f6c2da1720926e2 in Linus' tree seems
strange. It includes more changes than Pavel's original patch, but with
the same commit message. Also, I think the extra changes are broken as
we then would have two parameters that have that contain the same
information, yet the do not have the same ranges.
gitweb of the commit:
http://www.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=2c171bf13423dc5293188cea7f6c2da1720926e2
Rgds
Pierre
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Was this really supposed to go in?
2006-05-20 15:36 Was this really supposed to go in? Pierre Ossman
@ 2006-05-20 15:45 ` Russell King
2006-05-20 15:49 ` Pierre Ossman
0 siblings, 1 reply; 3+ messages in thread
From: Russell King @ 2006-05-20 15:45 UTC (permalink / raw)
To: Pierre Ossman; +Cc: linux-kernel, ppisa4lists
On Sat, May 20, 2006 at 05:36:40PM +0200, Pierre Ossman wrote:
> Commit 2c171bf13423dc5293188cea7f6c2da1720926e2 in Linus' tree seems
> strange. It includes more changes than Pavel's original patch, but with
> the same commit message.
They shouldn't have gone in - they were a change I was working on a few
days ago which I left in the git tree uncommitted. Applying Pavel's
patch then committed them.
> Also, I think the extra changes are broken as we then would have two
> parameters that have that contain the same information, yet the do not
> have the same ranges.
It's part of transitioning the data transfers over to taking the byte
size instead of the log2 byte size.
Well, I can't do anything about it now - I'm going away for a couple
of weeks from tomorrow morning.
--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of: 2.6 Serial core
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Was this really supposed to go in?
2006-05-20 15:45 ` Russell King
@ 2006-05-20 15:49 ` Pierre Ossman
0 siblings, 0 replies; 3+ messages in thread
From: Pierre Ossman @ 2006-05-20 15:49 UTC (permalink / raw)
To: Pierre Ossman, linux-kernel, ppisa4lists
Russell King wrote:
> It's part of transitioning the data transfers over to taking the byte
> size instead of the log2 byte size.
>
As long as it's a complete move and not having both, then I'm all for it.
> Well, I can't do anything about it now - I'm going away for a couple
> of weeks from tomorrow morning.
>
>
The patch doesn't really break anything as the only two users make sure
the parameters are in sync, so it's no rush. Just make sure it's on your
list when you get back. ;)
Rgds
Pierre
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-05-20 15:49 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-05-20 15:36 Was this really supposed to go in? Pierre Ossman
2006-05-20 15:45 ` Russell King
2006-05-20 15:49 ` Pierre Ossman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox