Re: smiapp-core.c error if !defined(CONFIG_OF)

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

 



Hi Hans,

On Tue, Jan 13, 2015 at 09:54:05AM +0100, Hans Verkuil wrote:
> Hi Sakari,
> 
> The daily build fails because of this error:
> 
> media_build/v4l/smiapp-core.c: In function 'smiapp_get_pdata':
> media_build/v4l/smiapp-core.c:3061:3: error: implicit declaration of function 'of_read_number' [-Werror=implicit-function-declaration]
>    pdata->op_sys_clock[i] = of_read_number(val + i * 2, 2);
>    ^
> 
> Some digging showed that of_read_number is only available if CONFIG_OF
> is defined. As far as I can see that is actually a bug in linux/of.h, as
> I see no reason why it should be under CONFIG_OF.

Well, it could be defined I guess --- it shouldn't have any use if OF isn't
in use. I'll submit a patch for that.

The problem in the smiapp driver is better fixed by applying "smiapp: Use
of_property_read_u64_array() to read a 64-bit number array" instead. Could
you try that? I'll submit this as a fix then. of_read_number() was just a
workaround for missing of_property_read_u64_array().

-- 
Kind regards,

Sakari Ailus
e-mail: sakari.ailus@xxxxxx	XMPP: sailus@xxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux