[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2024041825-sprinkled-popcorn-8a85@gregkh>
Date: Thu, 18 Apr 2024 15:05:34 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: FUJITA Tomonori <fujita.tomonori@...il.com>
Cc: netdev@...r.kernel.org, andrew@...n.ch, rust-for-linux@...r.kernel.org,
tmgross@...ch.edu, mcgrof@...nel.org, russ.weight@...ux.dev
Subject: Re: [PATCH net-next v1 3/4] rust: net::phy support Firmware API
On Thu, Apr 18, 2024 at 09:51:08PM +0900, FUJITA Tomonori wrote:
> Hi,
>
> On Mon, 15 Apr 2024 13:10:59 +0200
> Greg KH <gregkh@...uxfoundation.org> wrote:
>
> > On Mon, Apr 15, 2024 at 07:47:00PM +0900, FUJITA Tomonori wrote:
> >> This patch adds support to the following basic Firmware API:
> >>
> >> - request_firmware
> >> - release_firmware
> >>
> >> Signed-off-by: FUJITA Tomonori <fujita.tomonori@...il.com>
> >> CC: Luis Chamberlain <mcgrof@...nel.org>
> >> CC: Russ Weight <russ.weight@...ux.dev>
> >> ---
> >> drivers/net/phy/Kconfig | 1 +
> >> rust/bindings/bindings_helper.h | 1 +
> >> rust/kernel/net/phy.rs | 45 +++++++++++++++++++++++++++++++++
> >
> > Please do not bury this in the phy.rs file, put it in drivers/base/ next
> > to the firmware functions it is calling.
>
> Sure. I had a version of creating rust/kernel/firmware.rs but I wanted
> to know if a temporary solution could be accepted.
>
> With the build system for Rust, we can't put it in drivers/base/ yet.
What is the status of fixing that?
Powered by blists - more mailing lists