2008-07-28 20:21:10

by Adrian Bunk

[permalink] [raw]
Subject: drivers/sh/maple/maple.c compile error

Commit 306cfd630a4d121cf4e08b894d8b4c4cf106e57e
(maple: tidy maple_driver code by removing redundant connect/disconnect)
causes the following compile error:

<-- snip -->

...
CC drivers/sh/maple/maple.o
/home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'attach_matching_maple_driver':
/home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:259: error: 'struct maple_driver' has no member named 'connect'
/home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'maple_detach_driver':
/home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:272: error: 'struct maple_driver' has no member named 'disconnect'
/home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:273: error: 'struct maple_driver' has no member named 'disconnect'
make[4]: *** [drivers/sh/maple/maple.o] Error 1

<-- snip -->

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed


2008-07-28 20:45:35

by Paul Mundt

[permalink] [raw]
Subject: Re: drivers/sh/maple/maple.c compile error

On Mon, Jul 28, 2008 at 11:20:12PM +0300, Adrian Bunk wrote:
> Commit 306cfd630a4d121cf4e08b894d8b4c4cf106e57e
> (maple: tidy maple_driver code by removing redundant connect/disconnect)
> causes the following compile error:
>
> <-- snip -->
>
> ...
> CC drivers/sh/maple/maple.o
> /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'attach_matching_maple_driver':
> /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:259: error: 'struct maple_driver' has no member named 'connect'
> /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'maple_detach_driver':
> /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:272: error: 'struct maple_driver' has no member named 'disconnect'
> /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:273: error: 'struct maple_driver' has no member named 'disconnect'
> make[4]: *** [drivers/sh/maple/maple.o] Error 1
>
> <-- snip -->
>
Adrian was supposed to send follow-up patches for this, which never
happened. I thought I had dropped this already, but in this case I'll
just revert it.

2008-07-28 20:52:23

by Adrian McMenamin

[permalink] [raw]
Subject: Re: drivers/sh/maple/maple.c compile error

On Tue, 2008-07-29 at 05:44 +0900, Paul Mundt wrote:
> On Mon, Jul 28, 2008 at 11:20:12PM +0300, Adrian Bunk wrote:
> > Commit 306cfd630a4d121cf4e08b894d8b4c4cf106e57e
> > (maple: tidy maple_driver code by removing redundant connect/disconnect)
> > causes the following compile error:
> >
> > <-- snip -->
> >
> > ...
> > CC drivers/sh/maple/maple.o
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'attach_matching_maple_driver':
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:259: error: 'struct maple_driver' has no member named 'connect'
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'maple_detach_driver':
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:272: error: 'struct maple_driver' has no member named 'disconnect'
> > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:273: error: 'struct maple_driver' has no member named 'disconnect'
> > make[4]: *** [drivers/sh/maple/maple.o] Error 1
> >
> > <-- snip -->
> >
> Adrian was supposed to send follow-up patches for this, which never
> happened. I thought I had dropped this already, but in this case I'll
> just revert it.

Actually I wasn't asked about this, but to submit a fix/replacement for
a patch that didn't apply.

I will submit a patch for the whole thing then - as I cannot actually
match the commit Adrian has quoted to anything.

I think the problem is that I am trying to get a change in the bus and
in the drivers (which have different maintainers) to go in together and
that has never quite worked. The code runs fine on my box and has done
for months now.

Adrian

2008-07-28 21:02:15

by Paul Mundt

[permalink] [raw]
Subject: Re: drivers/sh/maple/maple.c compile error

On Mon, Jul 28, 2008 at 09:51:48PM +0100, Adrian McMenamin wrote:
> On Tue, 2008-07-29 at 05:44 +0900, Paul Mundt wrote:
> > On Mon, Jul 28, 2008 at 11:20:12PM +0300, Adrian Bunk wrote:
> > > Commit 306cfd630a4d121cf4e08b894d8b4c4cf106e57e
> > > (maple: tidy maple_driver code by removing redundant connect/disconnect)
> > > causes the following compile error:
> > >
> > > <-- snip -->
> > >
> > > ...
> > > CC drivers/sh/maple/maple.o
> > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'attach_matching_maple_driver':
> > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:259: error: 'struct maple_driver' has no member named 'connect'
> > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'maple_detach_driver':
> > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:272: error: 'struct maple_driver' has no member named 'disconnect'
> > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:273: error: 'struct maple_driver' has no member named 'disconnect'
> > > make[4]: *** [drivers/sh/maple/maple.o] Error 1
> > >
> > > <-- snip -->
> > >
> > Adrian was supposed to send follow-up patches for this, which never
> > happened. I thought I had dropped this already, but in this case I'll
> > just revert it.
>
> Actually I wasn't asked about this, but to submit a fix/replacement for
> a patch that didn't apply.
>
> I will submit a patch for the whole thing then - as I cannot actually
> match the commit Adrian has quoted to anything.
>
> I think the problem is that I am trying to get a change in the bus and
> in the drivers (which have different maintainers) to go in together and
> that has never quite worked. The code runs fine on my box and has done
> for months now.
>
This seems to point to a serious problem in your working environment, as
you are neither able to view commits that are referenced or realize there
is a problem when patches fail to apply anywhere outside of your home
directory.

The problem is that the change you made to maple.c is incomplete, it
basically should have followed 2 other patches, one removing the logic
entirely from maple.c, and one removing it from the input driver, before
killing off the function pointers completely. You were asked to fix up
and resend the input patch (which seems not to have happened due to a
problem in your work environment), but I'm unable to find anything
bordering on a maple patch that actually cleans up
attach_matching_maple_driver() and maple_detach_driver()
connect/disconnect references?

In any event, I'm going to revert this. Once you get your environment
fixed and are able to send a coherent series, we can revisit this. Dmitry
has already indicated that taking the input bits through my tree is fine,
so any process problems there are the least of the problem.

2008-07-28 21:19:04

by Adrian McMenamin

[permalink] [raw]
Subject: Re: drivers/sh/maple/maple.c compile error

On Tue, 2008-07-29 at 06:01 +0900, Paul Mundt wrote:
> On Mon, Jul 28, 2008 at 09:51:48PM +0100, Adrian McMenamin wrote:
> > On Tue, 2008-07-29 at 05:44 +0900, Paul Mundt wrote:
> > > On Mon, Jul 28, 2008 at 11:20:12PM +0300, Adrian Bunk wrote:
> > > > Commit 306cfd630a4d121cf4e08b894d8b4c4cf106e57e
> > > > (maple: tidy maple_driver code by removing redundant connect/disconnect)
> > > > causes the following compile error:
> > > >
> > > > <-- snip -->
> > > >
> > > > ...
> > > > CC drivers/sh/maple/maple.o
> > > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'attach_matching_maple_driver':
> > > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:259: error: 'struct maple_driver' has no member named 'connect'
> > > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c: In function 'maple_detach_driver':
> > > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:272: error: 'struct maple_driver' has no member named 'disconnect'
> > > > /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/sh/maple/maple.c:273: error: 'struct maple_driver' has no member named 'disconnect'
> > > > make[4]: *** [drivers/sh/maple/maple.o] Error 1
> > > >
> > > > <-- snip -->
> > > >
> > > Adrian was supposed to send follow-up patches for this, which never
> > > happened. I thought I had dropped this already, but in this case I'll
> > > just revert it.
> >
> > Actually I wasn't asked about this, but to submit a fix/replacement for
> > a patch that didn't apply.
> >
> > I will submit a patch for the whole thing then - as I cannot actually
> > match the commit Adrian has quoted to anything.
> >
> > I think the problem is that I am trying to get a change in the bus and
> > in the drivers (which have different maintainers) to go in together and
> > that has never quite worked. The code runs fine on my box and has done
> > for months now.
> >
> This seems to point to a serious problem in your working environment, as
> you are neither able to view commits that are referenced or realize there
> is a problem when patches fail to apply anywhere outside of your home
> directory.
>

I searched in Linus's tree on kernel.org after I couldn't find it in
mine. I can see the commit (306cfd630a4d121cf4e08b894d8b4c4cf106e57e)
referred to in both trees but it doesn't include any changes to the
maple stuff.


> The problem is that the change you made to maple.c is incomplete, it
> basically should have followed 2 other patches, one removing the logic
> entirely from maple.c, and one removing it from the input driver, before
> killing off the function pointers completely. You were asked to fix up
> and resend the input patch (which seems not to have happened due to a
> problem in your work environment), but I'm unable to find anything
> bordering on a maple patch that actually cleans up
> attach_matching_maple_driver() and maple_detach_driver()
> connect/disconnect references?
>

http://lkml.org/lkml/2008/6/15/121 - removes the references from
keyboard
http://lkml.org/lkml/2008/6/15/122 - removes the references from the
headers

> In any event, I'm going to revert this. Once you get your environment
> fixed and are able to send a coherent series, we can revisit this. Dmitry
> has already indicated that taking the input bits through my tree is fine,
> so any process problems there are the least of the problem.

I don't think there is anything wrong with my environment. But I am
happy to submit a new set of patches

2008-07-28 21:25:27

by Paul Mundt

[permalink] [raw]
Subject: Re: drivers/sh/maple/maple.c compile error

On Mon, Jul 28, 2008 at 10:13:15PM +0100, Adrian McMenamin wrote:
> On Tue, 2008-07-29 at 06:01 +0900, Paul Mundt wrote:
> > On Mon, Jul 28, 2008 at 09:51:48PM +0100, Adrian McMenamin wrote:
> > This seems to point to a serious problem in your working environment, as
> > you are neither able to view commits that are referenced or realize there
> > is a problem when patches fail to apply anywhere outside of your home
> > directory.
>
> I searched in Linus's tree on kernel.org after I couldn't find it in
> mine. I can see the commit (???306cfd630a4d121cf4e08b894d8b4c4cf106e57e)
> referred to in both trees but it doesn't include any changes to the
> maple stuff.
>
What do you think it contains? This isn't exactly difficult:

http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=306cfd630a4d121cf4e08b894d8b4c4cf106e57e;hp=4cec1a37ba7d9dce6ed5d8259b95272100a98b1f

Surprise, your header patch.

> > The problem is that the change you made to maple.c is incomplete, it
> > basically should have followed 2 other patches, one removing the logic
> > entirely from maple.c, and one removing it from the input driver, before
> > killing off the function pointers completely. You were asked to fix up
> > and resend the input patch (which seems not to have happened due to a
> > problem in your work environment), but I'm unable to find anything
> > bordering on a maple patch that actually cleans up
> > attach_matching_maple_driver() and maple_detach_driver()
> > connect/disconnect references?
> >
>
> http://lkml.org/lkml/2008/6/15/121 - removes the references from
> keyboard
> http://lkml.org/lkml/2008/6/15/122 - removes the references from the
> headers
>
And what about the changes required to maple.c? Did you even bother
reading what I was talking about in that paragraph? Or even bother
looking at the build failure? Your changes are _incomplete_, even if I
had fixed up and merged the input driver changes (which you weren't able
to see as being a problem, despite the fact the underlying code being
patch had changed completely). Until you can figure this out and get your
environment resembling something sane, please stop sending me any email
or patches, thanks.

2008-07-28 21:25:40

by Adrian McMenamin

[permalink] [raw]
Subject: Re: drivers/sh/maple/maple.c compile error

On Mon, 2008-07-28 at 22:13 +0100, Adrian McMenamin wrote:
> On Tue, 2008-07-29 at 06:01 +0900, Paul Mundt wrote:
> > On Mon, Jul 28, 2008 at 09:51:48PM +0100, Adrian McMenamin wrote:
> > > On Tue, 2008-07-29 at 05:44 +0900, Paul Mundt wrote:
> > > > On Mon, Jul 28, 2008 at 11:20:12PM +0300, Adrian Bunk wrote:
> > > > > Commit 306cfd630a4d121cf4e08b894d8b4c4cf106e57e
> > > > > (maple: tidy maple_driver code by removing redundant connect/disconnect)
> > > > > causes the following compile error:

> >
>
> http://lkml.org/lkml/2008/6/15/121 - removes the references from
> keyboard
> http://lkml.org/lkml/2008/6/15/122 - removes the references from the
> headers
>

Ah I see what the issue is now. My apologies, it seems I made a patch
and for some reason never submitted it. But will do so now.