From patchwork Thu Sep 17 15:28:17 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 313129 Delivered-To: patch@linaro.org Received: by 2002:a92:5ad1:0:0:0:0:0 with SMTP id b78csp550968ilg; Thu, 17 Sep 2020 08:49:29 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwPFpo8y2V946YOYB740kjGzS/wyb6Bvp1sefwtGIbi3uBCRgAxEHTUfZ0gqufbrJGoWAFY X-Received: by 2002:a05:6402:164d:: with SMTP id s13mr33237787edx.222.1600357769358; Thu, 17 Sep 2020 08:49:29 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1600357769; cv=none; d=google.com; s=arc-20160816; b=qip2022w0Q7iPZk62+Ta6zmZJMg42SSAGeNAk6NurHQRnHvW5/nX24o3rdyu5Ns4+m Gr7T6VHRKgRgpo2W1m86v/pgev6I+GZu66wgk+UBuJl1I5oZGN8r5FW0C8XXa76mB5tB SZN0f2ePUMMUu5ODi8jr9DSFWaQDVXRdUPF7TlM5xwfCXZ1mY9I9QqBH2E+uk1Sln7FG rQSFyuut2cQFCs1yqFae2F5JjiYSbyD4NH0WazunEMWuO+ZGXmeaJXUtFpJ6Nef0LJgt LNmZ0RuV6mY9NvemtK/em9VUJ+ZGjhBClbQuweDPRzzyNZOAVZaE6/HX5evCylVCYVCj X9NA== 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; bh=I/R7p+SukKDUOFvguxhhk/LAjYrYB1CkSAGwI1qOpqI=; b=PPqSl+TjxGkSoK6IUqUD5m9fz9N5SsLxoB7jkabnYHZtQYWJyYNJ/GasDGE7UiGzPo 3UigQjDSk76bTZcBAwgyOt1uY9OTKlMy+PFXj6RDbsxqsmFCG8A7mdkBXC5xsob3jl/T LjEIj1d3DE/m7GeD5fh1QOxntpQy/Qqj+LRSXsT/aBJMy2Avez/BDcSkKnH9Ygqs2TKY wxjs4c9i952InJX8JvgJvtMhSl4ynX7weBHN/Jp0Fysa6OQz/U0b/WFVnMJ7JEpvxBBd 3q3RKsH+kSlB6y3SzQe1lXZRIwH1Zzm9QN7/Ss+p6SpmfJxga0aRD4dpJbYvtIwVUg3y G1aw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-media-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-media-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p10si227562ejy.70.2020.09.17.08.49.29; Thu, 17 Sep 2020 08:49:29 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-media-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-media-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-media-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728262AbgIQPtY (ORCPT + 4 others); Thu, 17 Sep 2020 11:49:24 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:39401 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728161AbgIQPrT (ORCPT ); Thu, 17 Sep 2020 11:47:19 -0400 Received: from threadripper.lan ([149.172.98.151]) by mrelayeu.kundenserver.de (mreue011 [212.227.15.129]) with ESMTPA (Nemesis) id 1MFsER-1kH4gn01sd-00HQ4A; Thu, 17 Sep 2020 17:28:26 +0200 From: Arnd Bergmann To: linux-media@vger.kernel.org, mchehab@kernel.org, hverkuil@xs4all.nl Cc: hch@lst.de, linux-kernel@vger.kernel.org, Arnd Bergmann Subject: [PATCH 2/8] media: v4l2: remove unneeded compat ioctl handlers Date: Thu, 17 Sep 2020 17:28:17 +0200 Message-Id: <20200917152823.1241599-3-arnd@arndb.de> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20200917152823.1241599-1-arnd@arndb.de> References: <20200917152823.1241599-1-arnd@arndb.de> MIME-Version: 1.0 X-Provags-ID: V03:K1:++SSgZ3DNeodfdLMxxhVdMY6F5zK7tmiDsaTaRq8Tv7DlFh0gI1 bSE1SOX4Mq/pQJNnY081OagYNaYRDgtdgzvyX6j3+W5TTZVlTNC+wIXK/6nmp38R9MtZ2r+ zfPuySC/KQv/lnf4LVP2rI3XMD0qnAz71YWxkNnottxLKpYWSrAISp3ynQ2/H1x7z/8t8fV JQCdNrjlY22DKDkHBeMbA== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V03:K0:onJUFuCvh7o=:5RoNHygoStyY931DmdqFm4 duaiwinvc2Vc78k4u3lbIyCUnSIEy+EYHiaN887S27rLZxkCwE8rw/OuoVn7HT+r1Y9qo/KiM F9f2rdgW9x42AnuwcxtHfboCUQwRUZnIS7IhwU6sSPeze1kRAUGPoGkkwtjzYgNa/Ii9NRO+k hQNuNKfuLw3aQh+i1TBWPRmPGdYkIG4+t3I0oREuSw98Duoh3TvJ0W+Flo5Mt2GaacvjBbiiW hZ4xsVT0YtXY8LEaelpOpZK6W02s3AThMigWVjFaglq4sFbbCJ/usER6IHugm73yTK1ll1A6/ LSye6FsPtag/KDz6XMp5O2mptZrX84ui2/dCfuBB/DjHzk8jRJzh5ECiSNUOZTJfyNGwVYofk 6x6mai33gaJl+DtdjtFQ6dg6Y7vBaJ8mRbAYQlJ/IAe66m2JbA6uOeP4P2UxShQUrJkBRHMO1 57nOokeYn4eln70iZ5sEd6zIE+xdXX8phwazbrX85uYNdFDPWS2h+GxDUc9U3SMctR1iyx2FM Z6u4KPOcC4PBXh/VJesY4HB2g0qGjoWDxNuLhlYgqIxMoE5Yn+rShnDEM+R3Wwh5DbsHz112L GvQ5zzgYyPwEjOMb+oG7vZbrEEmBK2jP2mzG09NHhe5WxrM5G7qavM87QW0EieK4cWeJ1Ixoj 9GeFbP0msdXbh5L5PGhRYDkBYvSebt6zvF85ZFVnU7+i6IDTbIzB2SbpO4ORsebv92VQGn1eN n1kaeBywXEwdaOW32FdV5Z37DSI3QJr2FdY0Au5gSe091HUppGWEPWodt42G5UqGgFoXtruCT kXMOqXVG90KDNYZ9SYFGReFvb0cxk+5tvs9CZlg6E34zSvjXBUlU7hR1uYH7Rbhwxh5HlWU Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org These seven commands are all compatible and do not need any conversion handlers. The existing ones just copy 32-bit integers around, and those are always compatible. Signed-off-by: Arnd Bergmann --- drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 42 ------------------- 1 file changed, 42 deletions(-) -- 2.27.0 diff --git a/drivers/media/v4l2-core/v4l2-compat-ioctl32.c b/drivers/media/v4l2-core/v4l2-compat-ioctl32.c index 8f2cb03be48b..f04e29f3aecc 100644 --- a/drivers/media/v4l2-core/v4l2-compat-ioctl32.c +++ b/drivers/media/v4l2-core/v4l2-compat-ioctl32.c @@ -1412,14 +1412,6 @@ static int put_v4l2_edid32(struct v4l2_edid __user *p64, #define VIDIOC_PREPARE_BUF32 _IOWR('V', 93, struct v4l2_buffer32) #define VIDIOC_PREPARE_BUF32_TIME32 _IOWR('V', 93, struct v4l2_buffer32_time32) -#define VIDIOC_OVERLAY32 _IOW ('V', 14, s32) -#define VIDIOC_STREAMON32 _IOW ('V', 18, s32) -#define VIDIOC_STREAMOFF32 _IOW ('V', 19, s32) -#define VIDIOC_G_INPUT32 _IOR ('V', 38, s32) -#define VIDIOC_S_INPUT32 _IOWR('V', 39, s32) -#define VIDIOC_G_OUTPUT32 _IOR ('V', 46, s32) -#define VIDIOC_S_OUTPUT32 _IOWR('V', 47, s32) - unsigned int v4l2_compat_translate_cmd(unsigned int cmd) { switch (cmd) { @@ -1550,13 +1542,6 @@ static long do_video_ioctl(struct file *file, unsigned int cmd, unsigned long ar case VIDIOC_DQEVENT32: ncmd = VIDIOC_DQEVENT; break; case VIDIOC_DQEVENT32_TIME32: ncmd = VIDIOC_DQEVENT_TIME32; break; #endif - case VIDIOC_OVERLAY32: ncmd = VIDIOC_OVERLAY; break; - case VIDIOC_STREAMON32: ncmd = VIDIOC_STREAMON; break; - case VIDIOC_STREAMOFF32: ncmd = VIDIOC_STREAMOFF; break; - case VIDIOC_G_INPUT32: ncmd = VIDIOC_G_INPUT; break; - case VIDIOC_S_INPUT32: ncmd = VIDIOC_S_INPUT; break; - case VIDIOC_G_OUTPUT32: ncmd = VIDIOC_G_OUTPUT; break; - case VIDIOC_S_OUTPUT32: ncmd = VIDIOC_S_OUTPUT; break; case VIDIOC_CREATE_BUFS32: ncmd = VIDIOC_CREATE_BUFS; break; case VIDIOC_PREPARE_BUF32: ncmd = VIDIOC_PREPARE_BUF; break; case VIDIOC_PREPARE_BUF32_TIME32: ncmd = VIDIOC_PREPARE_BUF_TIME32; break; @@ -1571,24 +1556,6 @@ static long do_video_ioctl(struct file *file, unsigned int cmd, unsigned long ar * argument into it. */ switch (cmd) { - case VIDIOC_OVERLAY32: - case VIDIOC_STREAMON32: - case VIDIOC_STREAMOFF32: - case VIDIOC_S_INPUT32: - case VIDIOC_S_OUTPUT32: - err = alloc_userspace(sizeof(unsigned int), 0, &new_p64); - if (!err && assign_in_user((unsigned int __user *)new_p64, - (compat_uint_t __user *)p32)) - err = -EFAULT; - compatible_arg = 0; - break; - - case VIDIOC_G_INPUT32: - case VIDIOC_G_OUTPUT32: - err = alloc_userspace(sizeof(unsigned int), 0, &new_p64); - compatible_arg = 0; - break; - case VIDIOC_G_EDID32: case VIDIOC_S_EDID32: err = alloc_userspace(sizeof(struct v4l2_edid), 0, &new_p64); @@ -1761,15 +1728,6 @@ static long do_video_ioctl(struct file *file, unsigned int cmd, unsigned long ar * the original 32 bits structure. */ switch (cmd) { - case VIDIOC_S_INPUT32: - case VIDIOC_S_OUTPUT32: - case VIDIOC_G_INPUT32: - case VIDIOC_G_OUTPUT32: - if (assign_in_user((compat_uint_t __user *)p32, - ((unsigned int __user *)new_p64))) - err = -EFAULT; - break; - case VIDIOC_G_FBUF32: err = put_v4l2_framebuffer32(new_p64, p32); break;