[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a41e68b927b0a1bc864673583a54d736.sboyd@kernel.org>
Date: Tue, 02 Jul 2024 15:34:06 -0700
From: Stephen Boyd <sboyd@...nel.org>
To: David Gow <davidgow@...gle.com>
Cc: Michael Turquette <mturquette@...libre.com>, linux-kernel@...r.kernel.org, linux-clk@...r.kernel.org, patches@...ts.linux.dev, kunit-dev@...glegroups.com, linux-kselftest@...r.kernel.org, devicetree@...r.kernel.org, Brendan Higgins <brendan.higgins@...ux.dev>, Rae Moar <rmoar@...gle.com>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Rafael J . Wysocki <rafael@...nel.org>, Rob Herring <robh@...nel.org>, Saravana Kannan <saravanak@...gle.com>, Daniel Latypov <dlatypov@...gle.com>, Christian Marangi <ansuelsmth@...il.com>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, Conor Dooley <conor+dt@...nel.org>, Maxime Ripard <maxime@...no.tech>
Subject: Re: [PATCH v5 11/11] clk: Add KUnit tests for clks registered with struct clk_parent_data
Quoting Stephen Boyd (2024-07-02 15:25:47)
> Quoting David Gow (2024-06-13 00:56:08)
> >
> > We should use an action wrapper here (KUNIT_DEFINE_ACTION_WRAPPER()),
> > as casting function pointers to kunit_action_t* breaks control-flow
> > integrity.
>
> Got it, thanks. Maybe there should be an of_node_put_kunit_exit() helper that
> does that and can be used anywhere.
>
Heh I already have that with of_node_put_kunit(). I could add exit to it
for clarity? I don't really care either way.
Powered by blists - more mailing lists