mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Michael Riesch <michael.riesch@wolfvision.net>
To: Sascha Hauer <s.hauer@pengutronix.de>,
	Barebox List <barebox@lists.infradead.org>
Subject: Re: [PATCH 14/18] overlay: Add filters to choose which overlays to apply
Date: Mon, 28 Jun 2021 20:58:45 +0200	[thread overview]
Message-ID: <a8d6812c-9bfa-e629-ecae-0005bb12d3fd@wolfvision.net> (raw)
In-Reply-To: <20210624085223.14616-15-s.hauer@pengutronix.de>

Hello Sascha,

On 6/24/21 10:52 AM, Sascha Hauer wrote:
> [...]
> +
> +static int of_overlay_global_fixup(struct device_node *root, void *data)
> +{
> +	char *dir;
> +	int ret;
> +
> +	if (*of_overlay_dir == '/')
> +		return of_overlay_apply_dir(root, of_overlay_dir, true);
> +
> +	dir = concat_path_file(of_overlay_basedir, of_overlay_dir);
> +
> +	ret = of_overlay_apply_dir(root, of_overlay_dir, true);

shouldn't this be

	ret = of_overlay_apply_dir(root, dir, true);

in order to use the concatenated path?

Regards,
Michael

> +
> +	free(dir);
> +
> +	return ret;
> +}
> +
> +/**
> + * of_overlay_register_filter - register a new overlay filter
> + * @filter: The new filter
> + *
> + * Register a new overlay filter. A filter can either match on
> + * the filename or on the content of an overlay, but not on both.
> + * If that's desired two filters have to be registered.
> + *
> + * @return: 0 for success, negative error code otherwise
> + */
> +int of_overlay_register_filter(struct of_overlay_filter *filter)
> +{
> +	if (filter->filter_filename && filter->filter_content)
> +		return -EINVAL;
> +
> +	list_add_tail(&filter->list, &of_overlay_filters);
> +
> +	return 0;
> +}
> +
> +/**
> + * of_overlay_filter_filename - A filter that matches on the filename of
> + *                                an overlay
> + * @f: The filter
> + * @filename: The filename of the overlay
> + *
> + * This filter matches when the filename matches one of the patterns given
> + * in global.of.overlay.filepattern. global.of.overlay.filepattern shall
> + * contain a space separated list of wildcard patterns.
> + *
> + * @return: True when the overlay shall be applied, false otherwise.
> + */
> +static bool of_overlay_filter_filename(struct of_overlay_filter *f,
> +				       const char *filename)
> +{
> +	char *p, *path, *n;
> +	int ret;
> +	bool apply;
> +
> +	p = path = strdup(of_overlay_filepattern);
> +
> +	while ((n = strsep_unescaped(&p, " "))) {
> +		if (!*n)
> +			continue;
> +
> +		ret = fnmatch(n, filename, 0);
> +
> +		if (!ret) {
> +			apply = true;
> +			goto out;
> +		}
> +	}
> +
> +	apply = false;
> +out:
> +	free(path);
> +
> +	return apply;
> +}
> +
> +static struct of_overlay_filter of_overlay_filepattern_filter = {
> +	.name = "filepattern",
> +	.filter_filename = of_overlay_filter_filename,
> +};
> +
> +/**
> + * of_overlay_filter_compatible - A filter that matches on the compatible of
> + *                                an overlay
> + * @f: The filter
> + * @ovl: The overlay
> + *
> + * This filter matches when the compatible of an overlay matches to one
> + * of the compatibles given in global.of.overlay.compatible. When the
> + * overlay doesn't contain a compatible entry it is considered matching.
> + * Also when no compatibles are given in global.of.overlay.compatible
> + * all overlays will match.
> + *
> + * @return: True when the overlay shall be applied, false otherwise.
> + */
> +static bool of_overlay_filter_compatible(struct of_overlay_filter *f,
> +					struct device_node *ovl)
> +{
> +	char *p, *n, *compatibles;
> +	bool res = false;
> +
> +	if (!of_overlay_compatible || !*of_overlay_compatible)
> +		return true;
> +	if (!of_find_property(ovl, "compatible", NULL))
> +		return true;
> +
> +	p = compatibles = xstrdup(of_overlay_compatible);
> +
> +	while ((n = strsep_unescaped(&p, " "))) {
> +		if (!*n)
> +			continue;
> +
> +		if (of_device_is_compatible(ovl, n)) {
> +			res = true;
> +			break;
> +		}
> +	}
> +
> +	free(compatibles);
> +
> +	return res;
> +}
> +
> +static struct of_overlay_filter of_overlay_compatible_filter = {
> +	.name = "compatible",
> +	.filter_content = of_overlay_filter_compatible,
> +};
> +
> +static int of_overlay_init(void)
> +{
> +	of_overlay_filepattern = strdup("*");
> +	of_overlay_filter = strdup("filepattern compatible");
> +	of_overlay_set_basedir("/");
> +
> +	globalvar_add_simple_string("of.overlay.compatible", &of_overlay_compatible);
> +	globalvar_add_simple_string("of.overlay.filepattern", &of_overlay_filepattern);
> +	globalvar_add_simple_string("of.overlay.filter", &of_overlay_filter);
> +	globalvar_add_simple_string("of.overlay.dir", &of_overlay_dir);
> +
> +	of_overlay_register_filter(&of_overlay_filepattern_filter);
> +	of_overlay_register_filter(&of_overlay_compatible_filter);
> +
> +	of_register_fixup(of_overlay_global_fixup, NULL);
> +
> +	return 0;
> +}
> +device_initcall(of_overlay_init);
> +
> +BAREBOX_MAGICVAR(global.of.overlay.compatible, "space separated list of compatibles an overlay must match");
> +BAREBOX_MAGICVAR(global.of.overlay.filepattern, "space separated list of filepatterns an overlay must match");
> +BAREBOX_MAGICVAR(global.of.overlay.dir, "Directory to look for dt overlays");
> +BAREBOX_MAGICVAR(global.of.overlay.filter, "space separated list of filters");
> diff --git a/include/of.h b/include/of.h
> index 59c1250fb2..b698220b1e 100644
> --- a/include/of.h
> +++ b/include/of.h
> @@ -1022,12 +1022,20 @@ static inline struct device_node *of_find_root_node(struct device_node *node)
>  	return node;
>  }
>  
> +struct of_overlay_filter {
> +	bool (*filter_filename)(struct of_overlay_filter *, const char *filename);
> +	bool (*filter_content)(struct of_overlay_filter *, struct device_node *);
> +	char *name;
> +	struct list_head list;
> +};
> +
>  #ifdef CONFIG_OF_OVERLAY
>  struct device_node *of_resolve_phandles(struct device_node *root,
>  					const struct device_node *overlay);
>  int of_overlay_apply_tree(struct device_node *root,
>  			  struct device_node *overlay);
> -int of_overlay_apply_file(struct device_node *root, const char *filename);
> +int of_overlay_apply_file(struct device_node *root, const char *filename,
> +			  bool filter);
>  int of_register_overlay(struct device_node *overlay);
>  int of_process_overlay(struct device_node *root,
>  		    struct device_node *overlay,
> @@ -1038,6 +1046,8 @@ int of_process_overlay(struct device_node *root,
>  int of_overlay_pre_load_firmware(struct device_node *root, struct device_node *overlay);
>  int of_overlay_load_firmware(void);
>  void of_overlay_load_firmware_clear(void);
> +void of_overlay_set_basedir(const char *path);
> +int of_overlay_register_filter(struct of_overlay_filter *);
>  #else
>  static inline struct device_node *of_resolve_phandles(struct device_node *root,
>  					const struct device_node *overlay)
> @@ -1052,7 +1062,7 @@ static inline int of_overlay_apply_tree(struct device_node *root,
>  }
>  
>  static inline int of_overlay_apply_file(struct device_node *root,
> -					const char *filename)
> +					const char *filename, bool filter)
>  {
>  	return -ENOSYS;
>  }
> @@ -1086,6 +1096,10 @@ static inline void of_overlay_load_firmware_clear(void)
>  {
>  }
>  
> +static inline void of_overlay_set_basedir(const char *path)
> +{
> +}
> +
>  #endif
>  
>  #endif /* __OF_H */
> 

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


  reply	other threads:[~2021-06-28 19:00 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-24  8:52 [PATCH v3 00/18] Apply device tree overlays to kernel tree Sascha Hauer
2021-06-24  8:52 ` [PATCH 01/18] of: of_copy_node(): Copy phandles as well Sascha Hauer
2021-06-24  8:52 ` [PATCH 02/18] of: Add function to duplicate a device tree Sascha Hauer
2021-06-24  8:52 ` [PATCH 03/18] fdt: Check blob size during unflattening Sascha Hauer
2021-06-24  8:52 ` [PATCH 04/18] firmware: make device_node argument non const Sascha Hauer
2021-06-24  8:52 ` [PATCH 05/18] libbb: Add find_path function Sascha Hauer
2021-06-24  8:52 ` [PATCH 06/18] firmware: consolidate ifdefs Sascha Hauer
2021-06-24  8:52 ` [PATCH 07/18] firmware: Add search path Sascha Hauer
2021-06-24  8:52 ` [PATCH 08/18] firmware: Fix device_node matching Sascha Hauer
2021-06-24  8:52 ` [PATCH 09/18] firmware: recognize by reproducible name Sascha Hauer
2021-06-24  8:52 ` [PATCH 10/18] blspec: Set firmware searchpath Sascha Hauer
2021-06-24  8:52 ` [PATCH 11/18] overlay: Add of_overlay_apply_file() Sascha Hauer
2021-06-24  8:52 ` [PATCH 12/18] firmware: Load from global search path Sascha Hauer
2021-06-24  8:52 ` [PATCH 13/18] blspec: Rework firmware load Sascha Hauer
2021-06-24  8:52 ` [PATCH 14/18] overlay: Add filters to choose which overlays to apply Sascha Hauer
2021-06-28 18:58   ` Michael Riesch [this message]
2021-06-28 20:13     ` Sascha Hauer
2021-06-24  8:52 ` [PATCH 15/18] blspec: Apply overlays from rootfs Sascha Hauer
2021-06-24  8:52 ` [PATCH 16/18] doc: devicetree: Refer to internal device tree also as live tree Sascha Hauer
2021-06-24  8:52 ` [PATCH 17/18] Documentation: Add documentation for device tree overlays Sascha Hauer
2021-06-24  8:52 ` [PATCH 18/18] of_firmware: Fix handling of firmware-name property Sascha Hauer

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=a8d6812c-9bfa-e629-ecae-0005bb12d3fd@wolfvision.net \
    --to=michael.riesch@wolfvision.net \
    --cc=barebox@lists.infradead.org \
    --cc=s.hauer@pengutronix.de \
    /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