From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <SRS0=gC0j=VU=vger.kernel.org=linux-next-owner@kernel.org>
X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on
	aws-us-west-2-korg-lkml-1.web.codeaurora.org
X-Spam-Level: 
X-Spam-Status: No, score=-6.7 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS,
	INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,
	UNPARSEABLE_RELAY,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0
Received: from mail.kernel.org (mail.kernel.org [198.145.29.99])
	by smtp.lore.kernel.org (Postfix) with ESMTP id D669DC7618B
	for <linux-next@archiver.kernel.org>; Tue, 23 Jul 2019 15:14:46 +0000 (UTC)
Received: from vger.kernel.org (vger.kernel.org [209.132.180.67])
	by mail.kernel.org (Postfix) with ESMTP id B15E5218F0
	for <linux-next@archiver.kernel.org>; Tue, 23 Jul 2019 15:14:46 +0000 (UTC)
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S2390790AbfGWPOq (ORCPT <rfc822;linux-next@archiver.kernel.org>);
        Tue, 23 Jul 2019 11:14:46 -0400
Received: from bhuna.collabora.co.uk ([46.235.227.227]:48994 "EHLO
        bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1729004AbfGWPOq (ORCPT
        <rfc822;linux-next@vger.kernel.org>); Tue, 23 Jul 2019 11:14:46 -0400
Received: from [127.0.0.1] (localhost [127.0.0.1])
        (Authenticated sender: ezequiel)
        with ESMTPSA id A43222861F8
Message-ID: <13b14eb9475de71b408d1eed3c5e69fe089deeb0.camel@collabora.com>
Subject: Re: linux-next: build failure after merge of the v4l-dvb tree
From:   Ezequiel Garcia <ezequiel@collabora.com>
To:     Stephen Rothwell <sfr@canb.auug.org.au>,
        Mauro Carvalho Chehab <mchehab@kernel.org>
Cc:     Linux Next Mailing List <linux-next@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Pawel Osciak <posciak@chromium.org>,
        Hans Verkuil <hverkuil-cisco@xs4all.nl>,
        Tomasz Figa <tfiga@chromium.org>,
        Boris Brezillon <boris.brezillon@collabora.com>,
        Nicolas Dufresne <nicolas.dufresne@collabora.com>
Date:   Tue, 23 Jul 2019 12:14:20 -0300
In-Reply-To: <20190723101341.751596b2@canb.auug.org.au>
References: <20190723101341.751596b2@canb.auug.org.au>
Organization: Collabora
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.30.5-1.1 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-next-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-next.vger.kernel.org>
X-Mailing-List: linux-next@vger.kernel.org

On Tue, 2019-07-23 at 10:13 +1000, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the v4l-dvb tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
> 
> In file included from <command-line>:
> include/media/vp8-ctrls.h:25:2: error: unknown type name '__s8'
>   __s8 quant_update[4];
>   ^~~~
> include/media/vp8-ctrls.h:26:2: error: unknown type name '__s8'
>   __s8 lf_update[4];
>   ^~~~
> include/media/vp8-ctrls.h:27:2: error: unknown type name '__u8'
>   __u8 segment_probs[3];
>   ^~~~
> include/media/vp8-ctrls.h:28:2: error: unknown type name '__u8'
>   __u8 padding;
>   ^~~~
> include/media/vp8-ctrls.h:29:2: error: unknown type name '__u32'
>   __u32 flags;
>   ^~~~~
> include/media/vp8-ctrls.h:36:2: error: unknown type name '__s8'
>   __s8 ref_frm_delta[4];
>   ^~~~
> include/media/vp8-ctrls.h:37:2: error: unknown type name '__s8'
>   __s8 mb_mode_delta[4];
>   ^~~~
> include/media/vp8-ctrls.h:38:2: error: unknown type name '__u8'
>   __u8 sharpness_level;
>   ^~~~
> include/media/vp8-ctrls.h:39:2: error: unknown type name '__u8'
>   __u8 level;
>   ^~~~
> include/media/vp8-ctrls.h:40:2: error: unknown type name '__u16'
>   __u16 padding;
>   ^~~~~
> include/media/vp8-ctrls.h:41:2: error: unknown type name '__u32'
>   __u32 flags;
>   ^~~~~
> include/media/vp8-ctrls.h:45:2: error: unknown type name '__u8'
>   __u8 y_ac_qi;
>   ^~~~
> include/media/vp8-ctrls.h:46:2: error: unknown type name '__s8'
>   __s8 y_dc_delta;
>   ^~~~
> include/media/vp8-ctrls.h:47:2: error: unknown type name '__s8'
>   __s8 y2_dc_delta;
>   ^~~~
> include/media/vp8-ctrls.h:48:2: error: unknown type name '__s8'
>   __s8 y2_ac_delta;
>   ^~~~
> include/media/vp8-ctrls.h:49:2: error: unknown type name '__s8'
>   __s8 uv_dc_delta;
>   ^~~~
> include/media/vp8-ctrls.h:50:2: error: unknown type name '__s8'
>   __s8 uv_ac_delta;
>   ^~~~
> include/media/vp8-ctrls.h:51:2: error: unknown type name '__u16'
>   __u16 padding;
>   ^~~~~
> include/media/vp8-ctrls.h:55:2: error: unknown type name '__u8'
>   __u8 coeff_probs[4][8][3][11];
>   ^~~~
> include/media/vp8-ctrls.h:56:2: error: unknown type name '__u8'
>   __u8 y_mode_probs[4];
>   ^~~~
> include/media/vp8-ctrls.h:57:2: error: unknown type name '__u8'
>   __u8 uv_mode_probs[3];
>   ^~~~
> include/media/vp8-ctrls.h:58:2: error: unknown type name '__u8'
>   __u8 mv_probs[2][19];
>   ^~~~
> include/media/vp8-ctrls.h:59:2: error: unknown type name '__u8'
>   __u8 padding[3];
>   ^~~~
> include/media/vp8-ctrls.h:63:2: error: unknown type name '__u8'
>   __u8 range;
>   ^~~~
> include/media/vp8-ctrls.h:64:2: error: unknown type name '__u8'
>   __u8 value;
>   ^~~~
> include/media/vp8-ctrls.h:65:2: error: unknown type name '__u8'
>   __u8 bit_count;
>   ^~~~
> include/media/vp8-ctrls.h:66:2: error: unknown type name '__u8'
>   __u8 padding;
>   ^~~~
> include/media/vp8-ctrls.h:86:2: error: unknown type name '__u16'
>   __u16 width;
>   ^~~~~
> include/media/vp8-ctrls.h:87:2: error: unknown type name '__u16'
>   __u16 height;
>   ^~~~~
> include/media/vp8-ctrls.h:89:2: error: unknown type name '__u8'
>   __u8 horizontal_scale;
>   ^~~~
> include/media/vp8-ctrls.h:90:2: error: unknown type name '__u8'
>   __u8 vertical_scale;
>   ^~~~
> include/media/vp8-ctrls.h:92:2: error: unknown type name '__u8'
>   __u8 version;
>   ^~~~
> include/media/vp8-ctrls.h:93:2: error: unknown type name '__u8'
>   __u8 prob_skip_false;
>   ^~~~
> include/media/vp8-ctrls.h:94:2: error: unknown type name '__u8'
>   __u8 prob_intra;
>   ^~~~
> include/media/vp8-ctrls.h:95:2: error: unknown type name '__u8'
>   __u8 prob_last;
>   ^~~~
> include/media/vp8-ctrls.h:96:2: error: unknown type name '__u8'
>   __u8 prob_gf;
>   ^~~~
> include/media/vp8-ctrls.h:97:2: error: unknown type name '__u8'
>   __u8 num_dct_parts;
>   ^~~~
> include/media/vp8-ctrls.h:99:2: error: unknown type name '__u32'
>   __u32 first_part_size;
>   ^~~~~
> include/media/vp8-ctrls.h:100:2: error: unknown type name '__u32'
>   __u32 first_part_header_bits;
>   ^~~~~
> include/media/vp8-ctrls.h:101:2: error: unknown type name '__u32'
>   __u32 dct_part_sizes[8];
>   ^~~~~
> include/media/vp8-ctrls.h:103:2: error: unknown type name '__u64'
>   __u64 last_frame_ts;
>   ^~~~~
> include/media/vp8-ctrls.h:104:2: error: unknown type name '__u64'
>   __u64 golden_frame_ts;
>   ^~~~~
> include/media/vp8-ctrls.h:105:2: error: unknown type name '__u64'
>   __u64 alt_frame_ts;
>   ^~~~~
> include/media/vp8-ctrls.h:107:2: error: unknown type name '__u64'
>   __u64 flags;
>   ^~~~~
> 
> Caused by commit
> 
>   a57d6acaf352 ("media: uapi: Add VP8 stateless decoder API")
> 
> I have added the following fix patch for today.
> 

Looks good, thanks for the fix. I wonder why this
got triggered with VP8 only, and not with MPEG-2.

Acked-by: Ezequiel Garcia <ezequiel@collabora.com>

> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Date: Tue, 23 Jul 2019 10:07:40 +1000
> Subject: [PATCH] media: uapi: new file needs types.h
> 
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
>  include/media/vp8-ctrls.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/include/media/vp8-ctrls.h b/include/media/vp8-ctrls.h
> index 6cc2eeea4c90..53cba826e482 100644
> --- a/include/media/vp8-ctrls.h
> +++ b/include/media/vp8-ctrls.h
> @@ -11,6 +11,8 @@
>  #ifndef _VP8_CTRLS_H_
>  #define _VP8_CTRLS_H_
>  
> +#include <linux/types.h>
> +
>  #define V4L2_PIX_FMT_VP8_FRAME v4l2_fourcc('V', 'P', '8', 'F')
>  
>  #define V4L2_CID_MPEG_VIDEO_VP8_FRAME_HEADER (V4L2_CID_MPEG_BASE + 2000)
> -- 
> 2.20.1
>