Re: [PATCH RESEND 08/16] backlight: ti-lmu-backlight: add LM3532 driver

[Kim, Milo]

Ouch! The notifier.h must be included. I didn't notice this error in ARM 
architecture. Let me fix this error in next patch-set.
Thanks a lot for catching this!

Best regards,
Milo

diff --git a/include/linux/mfd/ti-lmu.h b/include/linux/mfd/ti-lmu.h
index eeb6b9e..44268c7 100644
--- a/include/linux/mfd/ti-lmu.h
+++ b/include/linux/mfd/ti-lmu.h
@@ -14,6 +14,7 @@
 #define __MFD_TI_LMU_H__

 #include <linux/gpio.h>
+#include <linux/notifier.h>
 #include <linux/regmap.h>

 /* Notifier event */

On 11/2/2015 2:37 PM, kbuild test robot wrote:
> Hi Milo,
>
> [auto build test ERROR on ljones-mfd/for-mfd-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
>
> url:    https://github.com/0day-ci/linux/commits/Milo-Kim/Support-TI-LMU-devices/20151102-130804
> config: mips-allyesconfig (attached as .config)
> reproduce:
>          wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>          chmod +x ~/bin/make.cross
>          # save the attached .config to linux build tree
>          make.cross ARCH=mips
>
> All errors (new ones prefixed by >>):
>
>     In file included from drivers/video/backlight/lm3532_bl.c:14:0:
> > > include/linux/mfd/ti-lmu.h:74:32: error: field 'notifier' has incomplete type
>       struct blocking_notifier_head notifier;
>                                     ^
>
> vim +/notifier +74 include/linux/mfd/ti-lmu.h
>
> 51194f72 Milo Kim 2015-11-02  68   * @notifier:	Notifier for reporting hwmon event
> 51194f72 Milo Kim 2015-11-02  69   */
> 51194f72 Milo Kim 2015-11-02  70  struct ti_lmu {
> 51194f72 Milo Kim 2015-11-02  71  	struct device *dev;
> 51194f72 Milo Kim 2015-11-02  72  	struct regmap *regmap;
> 51194f72 Milo Kim 2015-11-02  73  	int en_gpio;
> 51194f72 Milo Kim 2015-11-02 @74  	struct blocking_notifier_head notifier;
> 51194f72 Milo Kim 2015-11-02  75  };
> 51194f72 Milo Kim 2015-11-02  76
> 51194f72 Milo Kim 2015-11-02  77  int ti_lmu_read_byte(struct ti_lmu *lmu, u8 reg, u8 *read);
>
> :::::: The code at line 74 was first introduced by commit
> :::::: 51194f72221b7b83a8f6a5ba9bf0c49a10233ca2 mfd: add TI LMU driver
>
> :::::: TO: Milo Kim <milo.kim@xxxxxx>
> :::::: CC: 0day robot <fengguang.wu@xxxxxxxxx>
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/