Fri, Oct 18, 2019 at 07:09:51PM CEST, j...@resnulli.us wrote: >From: Jiri Pirko <j...@mellanox.com> > >Currently, the name format is not required by the code, however it is >required during patch review. All params added until now are in-lined >with the following format: >1) lowercase characters, digits and underscored are allowed >2) underscore is neither at the beginning nor at the end and > there is no more than one in a row. > >Add checker to the code to require this format from drivers and warn if >they don't follow. > >Signed-off-by: Jiri Pirko <j...@mellanox.com> >--- >v1->v2: >- removed empty line after comment >- added check for empty string >- converted i and len to size_t and put on a single line >- s/valid_name/name_valid/ to be aligned with the rest
Will send v3 which would allow uppercase and will make it part of patchset that would check more devlink strings.
