Re: [PATCH V8 06/14] rust: Add bare minimal bindings for clk framework

Viresh Kumar
2025-02-07T09:24:51Z

On 07-02-25, 00:11, Danilo Krummrich wrote:
> On Thu, Feb 06, 2025 at 12:05:59PM -0800, Stephen Boyd wrote:
> > Quoting Danilo Krummrich (2025-02-06 03:52:41)
> > > On Thu, Feb 06, 2025 at 12:49:14PM +0100, Danilo Krummrich wrote:
> > > > On Thu, Feb 06, 2025 at 02:58:27PM +0530, Viresh Kumar wrote:

> > > > > +/// A simple implementation of `struct clk` from the C code.
> > > > > +#[repr(transparent)]
> > > > > +pub struct Clk(*mut bindings::clk);
> > > >
> > > > Guess this should be Opaque<bindings::clk>.
> > >
> > > Sorry, I meant NonNull<bindings::clk>.
> >
> > NULL is a valid clk. It's like "don't care" in the common clk framework
>
> Thanks for clarifying!

> Guess this should be Opaque<bindings::clk>.

So it should be this now ?

--
viresh