On 18/12/2015 13:10, Mason wrote:
> On 18/12/2015 12:22, Mauro Carvalho Chehab wrote:
>
>> Patch applied.
>
> Great! Thanks.
>
> Using the latest media_build master + my writel_relaxed work-around,
> compilation proceeds much further, then dies on device tree stuff:
> (same error with vanilla and custom kernel)
>
> Will look into it. Any idea? :-(
>
> By the way, if I was not clear, I'm cross-compiling for an ARM platform.
>
> CC [M] /tmp/sandbox/media_build/v4l/v4l2-of.o
> /tmp/sandbox/media_build/v4l/v4l2-of.c: In function 'v4l2_of_parse_csi_bus':
> /tmp/sandbox/media_build/v4l/v4l2-of.c:38:4: error: implicit declaration of function 'of_prop_next_u32' [-Werror=implicit-function-declaration]
> lane = of_prop_next_u32(prop, lane, &v);
> ^
of_prop_next_u32() was introduced by commit c541adc637066
$ git describe --contains c541adc637066
v3.5-rc1~176^2~34
So it seems something needs to be done for kernels older than 3.5
I'll hack around it by adding
static inline const __be32 *of_prop_next_u32(struct property *prop,
const __be32 *cur, u32 *pu)
{
return NULL;
}
What's the correct fix?
> /tmp/sandbox/media_build/v4l/v4l2-of.c: In function 'v4l2_of_parse_link':
> /tmp/sandbox/media_build/v4l/v4l2-of.c:287:24: warning: passing argument 1 of 'of_parse_phandle' discards 'const' qualifier from pointer target type
> np = of_parse_phandle(node, "remote-endpoint", 0);
> ^
Commit b8fbdc42c5c5d made the first parameter const.
$ git describe --contains b8fbdc42c5c5d
v3.8-rc1~105^2~13
I suppose I can live with the warning for now.
Regards.
--
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
