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 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5CAADC27C53 for ; Fri, 7 Jun 2024 17:55:58 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id DAAE610ECE0; Fri, 7 Jun 2024 17:55:57 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (1024-bit key; unprotected) header.d=redhat.com header.i=@redhat.com header.b="ZsnGQM33"; dkim-atps=neutral Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by gabe.freedesktop.org (Postfix) with ESMTPS id 283D510ECE1 for ; Fri, 7 Jun 2024 17:55:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1717782955; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=c2USRNsUh4hVMRo8qpq7zv19zfSRCqrdM7qMgjktFAk=; b=ZsnGQM33b3kf3iI0jBaLoOhssVfT0GlFPdSBFZlo1qG5x0b+OKFG3a24yVBn6who4pDbIu ykSRf23I2qnewzqa3oGOOSP0kUhp/vVXYcTyrRo7wfI8KzrVEH6u4fCDL8N6Robdsa0aok zqOaqHtPk05IfRm07sFomxIqcXM0MTU= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-91-d24t7weBNLWfHN8StyMh9A-1; Fri, 07 Jun 2024 13:55:54 -0400 X-MC-Unique: d24t7weBNLWfHN8StyMh9A-1 Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-35dbf83bb20so1579072f8f.2 for ; Fri, 07 Jun 2024 10:55:53 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1717782953; x=1718387753; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=c2USRNsUh4hVMRo8qpq7zv19zfSRCqrdM7qMgjktFAk=; b=EGOnoHdH+W63d8HwOkm7msSXjD7zi3iJowuXm5XZR9JWqifhd4P7yvr5IXfIuoC4/k /iHkBTr2DYYk6Y112IV9cLnzeYdH3vcFiInbp/3dmTu88uj3IkxfB4YT64+tzmG7OCI8 LA1r1S73BWxXbmbsOFxs3OHe5K2GzQcPgpBOL2KGqctDq2DZtvCxIWydxyEZvjeHCJel sWiNXtcCNay4YNpsj9P6pJKc62nPQyWfCvPD6t0K+Ljdhy8WJT1W4LBRsDDTFT6U754P PbpWm1FTuMKPlSUWi5kLQ+zGsJihyPMWPuDfi/ZZ6V2YOBCLhU3F2gu7LRjPbrM/ZN31 NNiw== X-Forwarded-Encrypted: i=1; AJvYcCWWJnMwrklecGZ4jFYN2nX9v5T6lODCD5irjGo6j4px20iDJ9J1VdVYfOSPiSVjRyu4qp4l773SJAnUr8Cz2WtA2EfLZukBGRNYkUXC9g== X-Gm-Message-State: AOJu0YzwWgMBlnagzoafuNScXPqT/jAVi8ZsV6GlqXiZWmhi6se7QGGB niezN2TJXZ4MlrB2pluD/nWXfXXcj8/UGi/0bWyaNrRP4cSYK9qHCknYi+FkMD827zobXMY2eQ3 vopYZsKhz7ObZnuY+gCn7gfsf8Ih22OTcUTVXGIQbWVMjIWyw73lQZXugyMXE3GU= X-Received: by 2002:a05:6000:c6:b0:35d:cd60:cc59 with SMTP id ffacd0b85a97d-35efed54c85mr2247710f8f.35.1717782952925; Fri, 07 Jun 2024 10:55:52 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHd0q63Q+ll6ZV0ldUT81ypvw/xe2C9I7iCd+hxmTDRSsRX+Yo5EMV9qMgBA5eGvpIRAqCJCQ== X-Received: by 2002:a05:6000:c6:b0:35d:cd60:cc59 with SMTP id ffacd0b85a97d-35efed54c85mr2247698f8f.35.1717782952564; Fri, 07 Jun 2024 10:55:52 -0700 (PDT) Received: from cassiopeiae ([2a02:810d:4b3f:ee94:642:1aff:fe31:a19f]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-35ef5e983d8sm4420813f8f.84.2024.06.07.10.55.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 07 Jun 2024 10:55:51 -0700 (PDT) Date: Fri, 7 Jun 2024 19:55:49 +0200 From: Danilo Krummrich To: Greg KH Cc: FUJITA Tomonori , wedsonaf@gmail.com, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, airlied@gmail.com, daniel@ffwll.ch, ojeda@kernel.org, alex.gaynor@gmail.com, boqun.feng@gmail.com, gary@garyguo.net, bjorn3_gh@protonmail.com, benno.lossin@proton.me, a.hindborg@samsung.com, aliceryhl@google.com, lina@asahilina.net, pstanner@redhat.com, ajanulgu@redhat.com, lyude@redhat.com, rust-for-linux@vger.kernel.org, dri-devel@lists.freedesktop.org, nouveau@lists.freedesktop.org, mcgrof@kernel.org, russ.weight@linux.dev Subject: Re: [RFC PATCH 7/8] rust: add firmware abstractions Message-ID: References: <20240531.165032.1895479176481879387.fujita.tomonori@gmail.com> <20240607.211132.1411950625632247437.fujita.tomonori@gmail.com> <2024060708-residue-hardy-af36@gregkh> <2024060745-palatable-dragging-32d1@gregkh> MIME-Version: 1.0 In-Reply-To: <2024060745-palatable-dragging-32d1@gregkh> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-BeenThere: nouveau@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Nouveau development list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: nouveau-bounces@lists.freedesktop.org Sender: "Nouveau" On Fri, Jun 07, 2024 at 05:41:11PM +0200, Greg KH wrote: > On Fri, Jun 07, 2024 at 03:33:39PM +0200, Danilo Krummrich wrote: > > On Fri, Jun 07, 2024 at 02:36:50PM +0200, Greg KH wrote: > > > Anyway, that's all hand-wavy right now, sorry, to get back to the point > > > here, again, let's take this, which will allow the firmware bindings to > > > be resubmitted and hopefully accepted, and we can move forward from > > > there to "real" things like a USB or PCI or even platform device and > > > driver binding stuff. > > > > In order to continue I propose to send out the following series: > > > > 1) minimal device and firmware abstractions only > > Sounds good. Just a heads-up, I'll probably send this one quite a bit earlier than the other two to make sure to unblock Fujita on their PHY driver. > > But after this, I don't want to take ANY driver core rust code that is > not able to live in the "normal" part of the Linux kernel tree. It's > just unsustainable to have it all in one directory sorry. If this > deadline makes that kbuild work actually happen faster, all the more > reason to have it. If that kbuild work is somehow stalled due to other > issues, please let me know what that is. > > > 2) v2 of all other device / driver, devres and PCI driver abstractions > > I will be glad to review this. Glad to hear that! > > > 3) v2 of basic DRM driver abstractions and Nova > > I love it how one of the most complex driver subsystems we have (drm) is > attempting to be the "first real" driver use for the rust apis. Bold > move :) I'd argue that as one of the most complex driver subsystems we have a lot of need for the advantages Rust provides to us. :) > > > The v2 series would contain static driver allocation (in case of DRM even const) > > and quite a few more simplifications around `driver::Registration` and > > `device::Data`. > > > > Does that make sense? > > Yes, but note, I'm going to probably push back on the "driver::" stuff. That's OK, I'm happy to convince you of its usefulness. :) > But will do so with more constructive criticism as I want this to work > very much and I agree that I think we are both talking past each other > in different ways. Mostly probably due to my total lack of Rust > experience, my apologies, thanks for your patience with me for this. Very much appreciated! Please don't hesitate to ask for more explanation on certain things if they're unclear. I'm happy trying my best to provide useful answers. - Danilo > > thanks, > > greg k-h >