linux-kernel - Re: [PATCH v2] Staging: nvec: Convert to_nvec

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <f9274ae0-2279-4a5b-5350-12ea5da3872a@inria.fr>
Date:   Mon, 20 Mar 2023 10:07:04 +0100 (CET)
From:   Julia Lawall <julia.lawall@...ia.fr>
To:     Sumitra Sharma <sumitraartsy@...il.com>
cc:     Dan Carpenter <error27@...il.com>, Marc Dietrich <marvin24@....de>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        ac100@...ts.launchpad.net, linux-tegra@...r.kernel.org,
        linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org,
        outreachy@...ts.linux.dev
Subject: Re: [PATCH v2] Staging: nvec: Convert to_nvec_led from a macro to
 an inline function



On Mon, 20 Mar 2023, Sumitra Sharma wrote:

> On Mon, Mar 20, 2023 at 08:39:49AM +0300, Dan Carpenter wrote:
> > On Sat, Mar 18, 2023 at 10:52:50AM -0700, Sumitra Sharma wrote:
> > > Convert to_nvec_led from a macro to a static inline function, to make the
> > > relevant types apparent in the definition and to benefit from the type
> > > checking performed by the compiler at call sites.
> > >
> > > Signed-off-by: Sumitra Sharma <sumitraartsy@...il.com>
> > > ---
> >
> > You need to compile test your changes.
> >
>
> Hi dan,
>
> I am facing problems in creating modules and in compiling them.

Note that unless the code is under an ifdef for a module, you don't
specifically need the driver to be configured as for a module either.
Since you don't have the device, you aren't going to be able to test it
anyway.  So 'y' is fine.

julia

>
> Any help in this would be appreciated. Here is the link to the thread https://lore.kernel.org/outreachy/alpine.DEB.2.22.394.2303191336090.2867@hadrien/T/#t
>
> Regards,
>
> Sumitra
>
> > regards,
> > dan carpenter
> >
>
>