Change in flashrom[master]: layout: Tidy up forward declarations

[Nico Huber (Code Review)]

Attention is currently required from: Edward O'Callaghan, Daniel Campello, Anastasia Klimchuk, Peter Marheine.
Hello Edward O'Callaghan, Daniel Campello, Anastasia Klimchuk, Peter Marheine,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/flashrom/+/54288

to review the following change.

Change subject: layout: Tidy up forward declarations
......................................................................

layout: Tidy up forward declarations

* Parameter names like `flashctx` for a `struct flashrom_flashctx`
  don't add any value.
* `const` qualification of parameters is meaningless in forward
  declarations. Arguments are always passed by copy and an API
  user does not need to know what callees do with their copy.

Change-Id: Iadcc1670ff86578a400dec9e804d6dda93e0fcf0
Signed-off-by: Nico Huber 
---
M layout.h
1 file changed, 7 insertions(+), 7 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/88/54288/1

diff --git a/layout.h b/layout.h
index 0cb8879..713241f 100644
--- a/layout.h
+++ b/layout.h
@@ -51,21 +51,21 @@

struct flashrom_flashctx;
 const struct flashrom_layout *get_default_layout(const struct flashrom_flashctx *);
-const struct flashrom_layout *get_layout(const struct flashrom_flashctx *const flashctx);
+const struct flashrom_layout *get_layout(const struct flashrom_flashctx *);

int layout_from_file(struct flashrom_layout **, const char *name);

-int register_include_arg(struct layout_include_args **args, const char *arg);
-int process_include_args(struct flashrom_layout *l, const struct layout_include_args *const args);
-void cleanup_include_args(struct layout_include_args **args);
+int register_include_arg(struct layout_include_args **, const char *arg);
+int process_include_args(struct flashrom_layout *, const struct layout_include_args *);
+void cleanup_include_args(struct layout_include_args **);

-int get_region_range(struct flashrom_layout *const l, const char *name,
+int get_region_range(struct flashrom_layout *, const char *name,
    	     unsigned int *start, unsigned int *len);
 const struct romentry *layout_next_included_region(const struct flashrom_layout *, chipoff_t);
 const struct romentry *layout_next_included(const struct flashrom_layout *, const struct romentry *);
 const struct romentry *layout_next(const struct flashrom_layout *, const struct romentry *);
-int included_regions_overlap(const struct flashrom_layout *const flashrom_layout);
-void prepare_layout_for_extraction(struct flashrom_flashctx *flash);
+int included_regions_overlap(const struct flashrom_layout *);
+void prepare_layout_for_extraction(struct flashrom_flashctx *);
 int layout_sanity_checks(const struct flashrom_flashctx *);

#endif /* !__LAYOUT_H__ */