From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no 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 192BEC48BE5 for ; Tue, 15 Jun 2021 08:42:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id F383F61413 for ; Tue, 15 Jun 2021 08:42:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231169AbhFOIoE (ORCPT ); Tue, 15 Jun 2021 04:44:04 -0400 Received: from mail.kernel.org ([198.145.29.99]:42010 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230454AbhFOIoA (ORCPT ); Tue, 15 Jun 2021 04:44:00 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 4FC6261421; Tue, 15 Jun 2021 08:41:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1623746516; bh=DOqkVmnBr80KyQKj5AB82+ytMFP18EFZJKipaflcsy0=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=Ka8rw4ixk0Wloh4el+pbnwNGpFhUGQ4CoZGE35yxnEcDn3sLFq3O9RVry4JkUc3sL bmYQlwsKwK2eIOJGGQpGOgRPKYQfxvuSvMPfiQ+H/O2Ai5dbFhzSiFUeGtZcOF3oO6 jfTSuQsCSgvn4brhCxNnbB2du6lGvUWyQJEkfUxTUMaEZWegX2Gzp9ZbSjI+kmlGOk aOFSGuEsdUd02va9aAy2F3BgpIs615TGqZfM6JN896vxt7dD5q4OdgSoGbCX/9ytKn 7ArzznK9rT8Ii6qZfb4zy/H3YF1LFXuqublKdEOUk1zZGFBOMwtRXgfNhJ/uExUZDL PoepvUPRFUwCQ== Received: by mail-wm1-f48.google.com with SMTP id l9so13630110wms.1; Tue, 15 Jun 2021 01:41:56 -0700 (PDT) X-Gm-Message-State: AOAM533UlchaLb4xNnwk37u4xXgwUwZdCofdx3UaUeyVMr7T3dfdxdwf dSftkXTTR3FGfIZG6y1MUDbTNT+5Dnyk5sz4hTQ= X-Google-Smtp-Source: ABdhPJxaQoNcNsZ00Pqi6S1MjwkYI6mlR4YjuOa9Sfkn6PYpJUxlyeUCejATypHdd5AYLoYMq78YjYfGyYxcHZdkwQM= X-Received: by 2002:a1c:c90f:: with SMTP id f15mr3915283wmb.142.1623746514904; Tue, 15 Jun 2021 01:41:54 -0700 (PDT) MIME-Version: 1.0 References: <20210614103409.3154127-1-arnd@kernel.org> <20210614103409.3154127-8-arnd@kernel.org> <2d988daa-cb25-9f0d-8309-99f4de2a1e58@xs4all.nl> In-Reply-To: <2d988daa-cb25-9f0d-8309-99f4de2a1e58@xs4all.nl> From: Arnd Bergmann Date: Tue, 15 Jun 2021 10:39:50 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 7/8] media: subdev: fix compat_ioctl32 To: Hans Verkuil Cc: Laurent Pinchart , Mauro Carvalho Chehab , "Lad, Prabhakar" , Eduardo Valentin , Sakari Ailus , Greg Kroah-Hartman , Vaibhav Gupta , Liu Shixin , Jacopo Mondi , Andy Shevchenko , Linux Kernel Mailing List , Linux Media Mailing List , linux-staging@lists.linux.dev Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 15, 2021 at 10:27 AM Hans Verkuil wrote: > On 14/06/2021 19:18, Laurent Pinchart wrote: > > > > Should we drop v4l2_subdev_core_ops.compat_ioctl32 and call > > v4l2_subdev_core_ops.ioctl from subdev_do_compat_ioctl32() ? New drivers > > should design custom ioctls in a way that doesn't require compat code. > > I agree, we can drop it completely. I agree about new drivers defining their ioctls in a compatible way, though Ideally I'd say subdev drivers should not use custom ioctls at all. There are two other drivers that define private subdev ioctls and that lack a working compat handler: - the omap3 isp driver would not work in compat mode, though I don't think there are any 64-bit SoCs using this hardware. The Sitara AM6 and J72 SoCs are the only chips in that family at the moment, but these don't seem to include any image processor. - The Intel Atom ISP driver has a broken compat ioctl handler that I remove in this series, but the v4l2_subdev_core_ops->compat_ioctl32 callback would be the correct place to hook these up if they do not get fixed. > I'll skip this patch, but I'll take the other 7 patches and make a v3 PR with > updated Reviewed-by tags from Laurent. Thanks! Arnd