Received: by 2002:a05:6602:2086:0:0:0:0 with SMTP id a6csp4421854ioa; Wed, 27 Apr 2022 03:36:28 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyQSetU3RHpYLcAWi8lpCKLN7Gqt+ScWDpgiKRAPR1gsFkXOE7gg0mq1E4TzPpH5wW/o9q+ X-Received: by 2002:a63:82c1:0:b0:3ab:1add:62ea with SMTP id w184-20020a6382c1000000b003ab1add62eamr15360824pgd.535.1651055787885; Wed, 27 Apr 2022 03:36:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1651055787; cv=none; d=google.com; s=arc-20160816; b=pclNXMHj1vvTn7XLrB9CqELcEi/WxVuNaPGKyRFWvfDVgCQkmIg52+H2wwAkVDpKSB Lw66uzJ3j9l1PperQrxzca05+1e1cIhPVBg30CIc2MDLJrOzjyNZxjBEO2NBe06K3ql7 rR0kpOA1+Vde45Z/4X+P4j3vh/k6g1+D+NBV6AA4sHcYEv54VTX1gT2i/FO1rCNaT/xI 9vwh8j0ZY9uJM5DucapahMepyW8VFl4E7f2t5xy2wwo/0D3ZbzKFZg64+RFsCNi+pVZL Z5VK36apf5s2XAcMuBm4gp36ISgU+lgrwqhN/8fdRAqQX4EIOJ0C6CMI3t35X/G6E0UC Xf/Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=eguoGjxtfi6rV5tK2QC5CBa8IuulTl4jBLJYsSf3WG4=; b=kxESLEdsUK6/5uqB9Hxfj1z+9Vth0SXoQk2VDbKM6ZRMAqmjVGZLgaeEEc2ejm21R6 kOObPYmvRvysVETxVTPtgIjedox8hzQarAJrNFOhl6vvn0wAdBPCEoOxgmbpuke0Q5UC cT4yAwJ6CuAsggkDDt1YbN1CrKqYjmt7v/FrLyeE19ahv6sl77RlqrjofSaWzBbV6mH7 29zl1WYh2aeUJ92M/hoQOG+mD2jaQV06TPUPq2q2MZW1EizpXCVa8eO9rP0E+rjYq6gL zFFhp1IMRKLe0sra90lK0G57fm/B1y2R7G2ZI0DbftumgALjNoCQ1nKdfK0lMy1tdgwI moFw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=Od0WtrC9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=collabora.com Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [2620:137:e000::1:18]) by mx.google.com with ESMTPS id n128-20020a632786000000b003aa8cf89631si1078093pgn.407.2022.04.27.03.36.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Apr 2022 03:36:27 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=Od0WtrC9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=collabora.com Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 3F2E63D4207; Wed, 27 Apr 2022 02:49:36 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1343672AbiDZNCO (ORCPT + 99 others); Tue, 26 Apr 2022 09:02:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47128 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1350321AbiDZNBg (ORCPT ); Tue, 26 Apr 2022 09:01:36 -0400 Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [46.235.227.227]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 02D7317FBFA; Tue, 26 Apr 2022 05:58:26 -0700 (PDT) Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: nicolas) with ESMTPSA id C07D41F43992 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1650977905; bh=Zdyv7lj5Zwk1trozSl5/ToeVKlMaPGRdeC6c5HT1zS8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Od0WtrC9/3w+VoxRpi3zn10sJ0Br+Gyxb8aoVslheETwwKNg/iGAv2aeShXCkitp/ Mu4Hk9C93KBfpJpPhx+gFja3MN2vX+NB0+0n75rpbpgc71IN++FRPYYTja1zrDaY/n 3kUPquAsCscegcf3N7IYdBYQloUa83CIBpVs5ZxT7WEvn2yM/NBLYZKjrNrE7pWiTJ ZHOiflQlrvYlhRoSitzL1RDhm9rQ++Mj2oFRrdWePsCXJcFPMU9H15LspCuU6jmBLa vpwvQbkapiOUjG2v5o0kcEh5tZLdyUH6uhfMZ5GWU0SHZQBPMBsPa4/rAIxwk0kvQo xpd9x7iSfaPsw== From: Nicolas Dufresne To: Ezequiel Garcia , Mauro Carvalho Chehab , Greg Kroah-Hartman Cc: nicolas@ndufresne.ca, linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: [PATCH v4 15/24] media: rkvdec: Move H264 SPS validation in rkvdec-h264 Date: Tue, 26 Apr 2022 08:57:41 -0400 Message-Id: <20220426125751.108293-16-nicolas.dufresne@collabora.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220426125751.108293-1-nicolas.dufresne@collabora.com> References: <20220426125751.108293-1-nicolas.dufresne@collabora.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,UNPARSEABLE_RELAY autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org No functional change, this moves H264 specific validation into the H264 specific code. This is in preparation of improving this validation and reusing it when VIDIOC_STREAMON is called. Signed-off-by: Nicolas Dufresne Reviewed-by: Ezequiel Garcia --- drivers/staging/media/rkvdec/rkvdec-h264.c | 23 ++++++++++++++++++++++ drivers/staging/media/rkvdec/rkvdec.c | 23 ++++++---------------- drivers/staging/media/rkvdec/rkvdec.h | 1 + 3 files changed, 30 insertions(+), 17 deletions(-) diff --git a/drivers/staging/media/rkvdec/rkvdec-h264.c b/drivers/staging/media/rkvdec/rkvdec-h264.c index 8d44a884a52e..0dcbcb1bac80 100644 --- a/drivers/staging/media/rkvdec/rkvdec-h264.c +++ b/drivers/staging/media/rkvdec/rkvdec-h264.c @@ -1137,9 +1137,32 @@ static int rkvdec_h264_run(struct rkvdec_ctx *ctx) return 0; } +static int rkvdec_h264_try_ctrl(struct rkvdec_ctx *ctx, struct v4l2_ctrl *ctrl) +{ + if (ctrl->id == V4L2_CID_STATELESS_H264_SPS) { + const struct v4l2_ctrl_h264_sps *sps = ctrl->p_new.p_h264_sps; + /* + * TODO: The hardware supports 10-bit and 4:2:2 profiles, + * but it's currently broken in the driver. + * Reject them for now, until it's fixed. + */ + if (sps->chroma_format_idc > 1) + /* Only 4:0:0 and 4:2:0 are supported */ + return -EINVAL; + if (sps->bit_depth_luma_minus8 != sps->bit_depth_chroma_minus8) + /* Luma and chroma bit depth mismatch */ + return -EINVAL; + if (sps->bit_depth_luma_minus8 != 0) + /* Only 8-bit is supported */ + return -EINVAL; + } + return 0; +} + const struct rkvdec_coded_fmt_ops rkvdec_h264_fmt_ops = { .adjust_fmt = rkvdec_h264_adjust_fmt, .start = rkvdec_h264_start, .stop = rkvdec_h264_stop, .run = rkvdec_h264_run, + .try_ctrl = rkvdec_h264_try_ctrl, }; diff --git a/drivers/staging/media/rkvdec/rkvdec.c b/drivers/staging/media/rkvdec/rkvdec.c index 2df8cf4883e2..e3d44d5b35f3 100644 --- a/drivers/staging/media/rkvdec/rkvdec.c +++ b/drivers/staging/media/rkvdec/rkvdec.c @@ -29,23 +29,12 @@ static int rkvdec_try_ctrl(struct v4l2_ctrl *ctrl) { - if (ctrl->id == V4L2_CID_STATELESS_H264_SPS) { - const struct v4l2_ctrl_h264_sps *sps = ctrl->p_new.p_h264_sps; - /* - * TODO: The hardware supports 10-bit and 4:2:2 profiles, - * but it's currently broken in the driver. - * Reject them for now, until it's fixed. - */ - if (sps->chroma_format_idc > 1) - /* Only 4:0:0 and 4:2:0 are supported */ - return -EINVAL; - if (sps->bit_depth_luma_minus8 != sps->bit_depth_chroma_minus8) - /* Luma and chroma bit depth mismatch */ - return -EINVAL; - if (sps->bit_depth_luma_minus8 != 0) - /* Only 8-bit is supported */ - return -EINVAL; - } + struct rkvdec_ctx *ctx = container_of(ctrl->handler, struct rkvdec_ctx, ctrl_hdl); + const struct rkvdec_coded_fmt_desc *desc = ctx->coded_fmt_desc; + + if (desc->ops->try_ctrl) + return desc->ops->try_ctrl(ctx, ctrl); + return 0; } diff --git a/drivers/staging/media/rkvdec/rkvdec.h b/drivers/staging/media/rkvdec/rkvdec.h index 2f4ea1786b93..9df0fba799a4 100644 --- a/drivers/staging/media/rkvdec/rkvdec.h +++ b/drivers/staging/media/rkvdec/rkvdec.h @@ -72,6 +72,7 @@ struct rkvdec_coded_fmt_ops { void (*done)(struct rkvdec_ctx *ctx, struct vb2_v4l2_buffer *src_buf, struct vb2_v4l2_buffer *dst_buf, enum vb2_buffer_state result); + int (*try_ctrl)(struct rkvdec_ctx *ctx, struct v4l2_ctrl *ctrl); }; struct rkvdec_coded_fmt_desc { -- 2.34.1