From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Sakari Ailus <sakari.ailus@linux.intel.com>
Cc: Petr Mladek <pmladek@suse.com>,
linux-kernel@vger.kernel.org, rafael@kernel.org,
linux-acpi@vger.kernel.org, devicetree@vger.kernel.org,
Rob Herring <robh@kernel.org>
Subject: Re: [PATCH v3 10/10] lib/test_printf: Add tests for %pfw printk modifier
Date: Fri, 30 Aug 2019 16:04:30 +0300 [thread overview]
Message-ID: <20190830130430.GK2680@smile.fi.intel.com> (raw)
In-Reply-To: <20190829101043.24963-11-sakari.ailus@linux.intel.com>
On Thu, Aug 29, 2019 at 01:10:43PM +0300, Sakari Ailus wrote:
> Add a test for the %pfw printk modifier using software nodes.
>
Having tests is always a good thing!
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
> lib/test_printf.c | 37 +++++++++++++++++++++++++++++++++++++
> 1 file changed, 37 insertions(+)
>
> diff --git a/lib/test_printf.c b/lib/test_printf.c
> index 944eb50f38625..9c6d716979fb1 100644
> --- a/lib/test_printf.c
> +++ b/lib/test_printf.c
> @@ -22,6 +22,8 @@
> #include <linux/gfp.h>
> #include <linux/mm.h>
>
> +#include <linux/property.h>
> +
> #include "../tools/testing/selftests/kselftest_module.h"
>
> #define BUF_SIZE 256
> @@ -588,6 +590,40 @@ flags(void)
> kfree(cmp_buffer);
> }
>
> +static void __init fwnode_pointer(void)
> +{
> + const struct software_node softnodes[] = {
> + { .name = "first", },
> + { .name = "second", .parent = &softnodes[0], },
> + { .name = "third", .parent = &softnodes[1], },
> + { NULL /* Guardian */ },
> + };
> + const char * const full_name = "/second/third";
> + const char * const full_name_second = "/second";
> + const char * const second_name = "second";
> + const char * const third_name = "third";
> + int rval;
> +
> + rval = software_node_register_nodes(softnodes);
> + if (rval) {
> + pr_warn("cannot register softnodes; rval %d\n", rval);
> + return;
> + }
> +
> + test(full_name_second, "%pfw",
> + software_node_fwnode(&softnodes[ARRAY_SIZE(softnodes) - 3]));
> + test(full_name, "%pfw",
> + software_node_fwnode(&softnodes[ARRAY_SIZE(softnodes) - 2]));
> + test(full_name, "%pfwf",
> + software_node_fwnode(&softnodes[ARRAY_SIZE(softnodes) - 2]));
> + test(second_name, "%pfwP",
> + software_node_fwnode(&softnodes[ARRAY_SIZE(softnodes) - 3]));
> + test(third_name, "%pfwP",
> + software_node_fwnode(&softnodes[ARRAY_SIZE(softnodes) - 2]));
> +
> + software_node_unregister_nodes(softnodes);
> +}
> +
> static void __init
> test_pointer(void)
> {
> @@ -610,6 +646,7 @@ test_pointer(void)
> bitmap();
> netdev_features();
> flags();
> + fwnode_pointer();
> }
>
> static void __init selftest(void)
> --
> 2.20.1
>
--
With Best Regards,
Andy Shevchenko
prev parent reply other threads:[~2019-08-30 13:04 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-29 10:10 [PATCH v3 00/10] Device property improvements, add %pfw format specifier Sakari Ailus
2019-08-29 10:10 ` [PATCH v3 01/10] software node: Get reference to parent swnode in get_parent op Sakari Ailus
2019-08-30 12:22 ` Andy Shevchenko
2019-09-02 6:46 ` Sakari Ailus
2019-08-30 12:26 ` Heikki Krogerus
2019-08-30 12:33 ` Andy Shevchenko
2019-08-29 10:10 ` [PATCH v3 02/10] software node: Make argument to to_software_node const Sakari Ailus
2019-08-30 12:22 ` Andy Shevchenko
2019-08-30 12:27 ` Heikki Krogerus
2019-08-29 10:10 ` [PATCH v3 03/10] device property: Add functions for accessing node's parents Sakari Ailus
2019-08-29 10:44 ` [PATCH v4 " Sakari Ailus
2019-08-30 12:42 ` Andy Shevchenko
2019-09-02 7:05 ` Sakari Ailus
2019-08-30 12:34 ` [PATCH v3 " Andy Shevchenko
2019-09-02 7:30 ` Sakari Ailus
2019-08-29 10:10 ` [PATCH v3 04/10] device property: Add fwnode_get_name for returning the name of a node Sakari Ailus
2019-08-30 12:37 ` Andy Shevchenko
2019-08-29 10:10 ` [PATCH v3 05/10] device property: Add a function to obtain a node's prefix Sakari Ailus
2019-08-30 12:46 ` Andy Shevchenko
2019-09-02 7:29 ` Sakari Ailus
2019-08-29 10:10 ` [PATCH v3 06/10] lib/vsprintf: Remove support for %pF and %pf in favour of %pS and %ps Sakari Ailus
2019-08-30 12:48 ` Andy Shevchenko
2019-09-02 7:06 ` Sakari Ailus
2019-08-30 12:55 ` Andy Shevchenko
2019-09-02 7:07 ` Sakari Ailus
2019-08-29 10:10 ` [PATCH v3 07/10] lib/vsprintf: Make use of fwnode API to obtain node names and separators Sakari Ailus
2019-08-30 12:53 ` Andy Shevchenko
2019-09-02 7:11 ` Sakari Ailus
2019-09-02 9:48 ` Andy Shevchenko
2019-08-29 10:10 ` [PATCH v3 08/10] lib/vsprintf: OF nodes are first and foremost, struct device_nodes Sakari Ailus
2019-08-30 12:57 ` Andy Shevchenko
2019-09-02 7:28 ` Sakari Ailus
2019-08-29 10:10 ` [PATCH v3 09/10] lib/vsprintf: Add %pfw conversion specifier for printing fwnode names Sakari Ailus
2019-08-30 13:03 ` Andy Shevchenko
2019-09-02 7:22 ` Sakari Ailus
2019-08-29 10:10 ` [PATCH v3 10/10] lib/test_printf: Add tests for %pfw printk modifier Sakari Ailus
2019-08-30 13:04 ` Andy Shevchenko [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190830130430.GK2680@smile.fi.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pmladek@suse.com \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=sakari.ailus@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).