Thread (25 messages) flat view 25 messages, 4 authors, 2017-10-23

Re: [PATCHv3 1/7] switch dereference_function_descriptor() to `unsigned long'

From: Petr Mladek <pmladek@suse.com>
Date: 2017-10-20 13:25:49
Also in: lkml

On Thu 2017-10-19 15:50:04, Sergey Senozhatsky wrote:
On (10/04/17 10:24), Petr Mladek wrote:
[..]
quoted
To make it clear. All these comments are not a big deal and I do
not want to invalidate all the acked-by and tested-by just because
of them.

But please, consider removing this change if we need to do
another iteration of this patchset. IMHO, there is no real win
and it would just pollute the git history.
I tend to rather keep it. would that cause any problems on your side?
it saves a ton of ugly casts later in the patch set and lets us to drop
some casts from the modules code/etc. the patch is here because otherwise
I had to add a bunch of new casts.
OK, let's keep it.

Best Regards,
Petr
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help