Re: [PATCH] mtd: rawnand: constify elements of NAND_OP_PARSER(_PATTERN)
Hi Masahiro, Masahiro Yamada wrote on Tue, 9 Apr 2019 13:53:32 +0900: > Currently, drivers are able to constify a nand_op_parser array, > but not nand_op_parser_pattern and nand_op_parser_pattern_elem > since they are instantiated by using the NAND_OP_PARSER(_PATTERN). > > Add 'const' to them in order to move more driver data from .data to > .rodata section. > > Signed-off-by: Masahiro Yamada > --- > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git branch nand/next. Thanks, Miquèl
Re: [PATCH] mtd: rawnand: constify elements of NAND_OP_PARSER(_PATTERN)
On Tue, 9 Apr 2019 13:53:32 +0900 Masahiro Yamada wrote: > Currently, drivers are able to constify a nand_op_parser array, > but not nand_op_parser_pattern and nand_op_parser_pattern_elem > since they are instantiated by using the NAND_OP_PARSER(_PATTERN). > > Add 'const' to them in order to move more driver data from .data to > .rodata section. > > Signed-off-by: Masahiro Yamada Reviewed-by: Boris Brezillon > --- > > include/linux/mtd/rawnand.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/linux/mtd/rawnand.h b/include/linux/mtd/rawnand.h > index b7445a44a814..ebde52088e4d 100644 > --- a/include/linux/mtd/rawnand.h > +++ b/include/linux/mtd/rawnand.h > @@ -805,7 +805,7 @@ struct nand_op_parser_pattern { > #define NAND_OP_PARSER_PATTERN(_exec, ...) > \ > { > \ > .exec = _exec, > \ > - .elems = (struct nand_op_parser_pattern_elem[]) { __VA_ARGS__ > },\ > + .elems = (const struct nand_op_parser_pattern_elem[]) { > __VA_ARGS__ }, \ > .nelems = sizeof((struct nand_op_parser_pattern_elem[]) { > __VA_ARGS__ }) / \ > sizeof(struct nand_op_parser_pattern_elem), > \ > } > @@ -831,7 +831,7 @@ struct nand_op_parser { > > #define NAND_OP_PARSER(...) > \ > { > \ > - .patterns = (struct nand_op_parser_pattern[]) { __VA_ARGS__ }, > \ > + .patterns = (const struct nand_op_parser_pattern[]) { > __VA_ARGS__ },\ > .npatterns = sizeof((struct nand_op_parser_pattern[]) { > __VA_ARGS__ }) /\ >sizeof(struct nand_op_parser_pattern), > \ > }
[PATCH] mtd: rawnand: constify elements of NAND_OP_PARSER(_PATTERN)
Currently, drivers are able to constify a nand_op_parser array, but not nand_op_parser_pattern and nand_op_parser_pattern_elem since they are instantiated by using the NAND_OP_PARSER(_PATTERN). Add 'const' to them in order to move more driver data from .data to .rodata section. Signed-off-by: Masahiro Yamada --- include/linux/mtd/rawnand.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/mtd/rawnand.h b/include/linux/mtd/rawnand.h index b7445a44a814..ebde52088e4d 100644 --- a/include/linux/mtd/rawnand.h +++ b/include/linux/mtd/rawnand.h @@ -805,7 +805,7 @@ struct nand_op_parser_pattern { #define NAND_OP_PARSER_PATTERN(_exec, ...) \ { \ .exec = _exec, \ - .elems = (struct nand_op_parser_pattern_elem[]) { __VA_ARGS__ },\ + .elems = (const struct nand_op_parser_pattern_elem[]) { __VA_ARGS__ }, \ .nelems = sizeof((struct nand_op_parser_pattern_elem[]) { __VA_ARGS__ }) / \ sizeof(struct nand_op_parser_pattern_elem), \ } @@ -831,7 +831,7 @@ struct nand_op_parser { #define NAND_OP_PARSER(...) \ { \ - .patterns = (struct nand_op_parser_pattern[]) { __VA_ARGS__ }, \ + .patterns = (const struct nand_op_parser_pattern[]) { __VA_ARGS__ },\ .npatterns = sizeof((struct nand_op_parser_pattern[]) { __VA_ARGS__ }) /\ sizeof(struct nand_op_parser_pattern), \ } -- 2.17.1