Re: [leds:devel 39/59] drivers/leds/leds-lp5521.c:615:6: error: 'lp5521_led_attribute_group' undeclared

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 28, 2013 at 3:19 PM, Kim, Milo <Milo.Kim@xxxxxx> wrote:
>> -----Original Message-----
>> From: Bryan Wu [mailto:cooloney@xxxxxxxxx]
>> Sent: Tuesday, January 29, 2013 4:00 AM
>> To: kbuild test robot
>> Cc: Kim, Milo; linux-leds@xxxxxxxxxxxxxxx
>> Subject: Re: [leds:devel 39/59] drivers/leds/leds-lp5521.c:615:6: error:
>> 'lp5521_led_attribute_group' undeclared
>>
>> Milo,
>>
>> Do you have time to take a look at this?
>>
>> -Bryan
>>
>> On Wed, Jan 23, 2013 at 11:09 AM, kbuild test robot
>> <fengguang.wu@xxxxxxxxx> wrote:
>> > tree:   git://git.kernel.org/pub/scm/linux/kernel/git/cooloney/linux-
>> leds.git devel
>> > head:   7314a82b650c2cc48a43e269fff96a22b8af90e0
>> > commit: a2594118f274d12925ff7cbac06b8ec40bb001a7 [39/59] leds-lp55xx:
>> provide common LED current setting
>> > config: make ARCH=x86_64 allmodconfig
>> >
>> > Note: the leds/devel HEAD 7314a82 builds fine.
>> >       It only hurts bisectibility.
>> >
>> > All error/warnings:
>> >
>> >    drivers/leds/leds-lp5521.c: In function 'lp5521_unregister_sysfs':
>> >>> drivers/leds/leds-lp5521.c:615:6: error:
>> 'lp5521_led_attribute_group' undeclared (first use in this function)
>> >    drivers/leds/leds-lp5521.c:615:6: note: each undeclared identifier
>> is reported only once for each function it appears in
>> > --
>> >    drivers/leds/leds-lp5523.c: In function 'lp5523_unregister_sysfs':
>> >>> drivers/leds/leds-lp5523.c:670:6: error:
>> 'lp5523_led_attribute_group' undeclared (first use in this function)
>> >    drivers/leds/leds-lp5523.c:670:6: note: each undeclared identifier
>> is reported only once for each function it appears in
>> >
>> > vim +/lp5521_led_attribute_group +615 drivers/leds/leds-lp5521.c
>> >
>> >    609          int i;
>> >    610
>> >    611          sysfs_remove_group(&dev->kobj, &lp5521_group);
>> >    612
>> >    613          for (i = 0; i < chip->num_leds; i++)
>> >    614                  sysfs_remove_group(&chip->leds[i].cdev.dev-
>> >kobj,
>> >  > 615                                  &lp5521_led_attribute_group);
>> >    616  }
>> >    617
>> >    618  static void lp5521_unregister_leds(struct lp5521_chip *chip)
>> >
>> > ---
>> > 0-DAY kernel build testing backend              Open Source
>> Technology Center
>> > http://lists.01.org/mailman/listinfo/kbuild                 Intel
>> Corporation
>
> Hi Bryan,
>
> I think errors are shown in the middle of merging series of patch-set.
> In latest driver, no build error is found because these attributes were replaced
>  with attributes of the lp55xx common driver.
> Please see the patch below.
>
> http://git.kernel.org/?p=linux/kernel/git/cooloney/linux-leds.git;a=commitdiff;h=a2594118f274d12925ff7cbac06b8ec40bb001a7
>
> While I was generating a patch-set, the completed driver was built rather than
> compiling each patch unit one by one.
> I'll keep in mind the rule of generating patch-set.
> I apologize it.
>

Yes, I think we need obey this rule here. Breaking bisecting will
cause lots of pain for debugging.

So please help to fold some patch together or rebuild this patchset
(39 patches), then give me a git url. I can try to rebuilt my branch
later.

Thanks,
-BRyan

> Please let me know if I need to re-generate a patch-set.
>
> Best Regards,
> Milo
>
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux