On Thu, 2007-08-23 at 00:34 +0530, Balbir Singh wrote:
quoted text
> Kay Sievers wrote:
> >> gargh, sorry, that's probably due to my screwed up attempt to fix Kay's
> >> screwed up
> >> gregkh-driver-driver-core-change-add_uevent_var-to-use-a-struct.patch.
> >>
> >> Kay sent an update patch but it didn't arrive in time.
> >>
> >> Greg, if you haven't yet merged that, please do so asap?
> >>
> >> So what _should_ this:
> >>
> >> --- a/arch/powerpc/kernel/vio.c~fix-4-gregkh-driver-driver-core-change-add_uevent_var-to-use-a-struct
> >> +++ a/arch/powerpc/kernel/vio.c
> >> @@ -373,7 +373,7 @@ static int vio_hotplug(struct device *de
> >>  	dn = dev->archdata.of_node;
> >>  	if (!dn)
> >>  		return -ENODEV;
> >> -	cp = of_get_property(dn, "compatible", &length);
> >> +	cp = of_get_property(dn, "compatible", &env->buflen);
> >>  	if (!cp)
> >>  		return -ENODEV;
> >>  
> >> _
> >>
> >> have done?
> > 
> > Does replacing "&length" with "NULL" work? That's what's in the updated
> > patch.

quoted text
> replacing &length with NULL does not work for me. I get a message saying that
> init terminated with signal 7.

Hi Balbir,
ugh, I can't see what's going wrong here.

Care to just "return 0" for the whole function, and try again? Just to
rule out that this is the cause of the problem.

Thanks,
Kay

-
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/