Messages in this thread | | | Date | Mon, 23 Feb 2015 13:55:23 +0000 | From | Matt Fleming <> | Subject | Re: [Patch v2 3/3] firmware: dmi_scan: use direct access to static vars |
| |
On Wed, 18 Feb, at 01:33:21PM, Ivan Khoronzhuk wrote: > There is no reason to pass static vars to function that can use > only them. > > The dmi_table() can use only dmi_len and dmi_num static vars, so use > them directly. In this case we can freely change their type in one > place and slightly decrease redundancy. > > Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> > --- > drivers/firmware/dmi_scan.c | 17 +++++++++-------- > 1 file changed, 9 insertions(+), 8 deletions(-)
Applied to the EFI 'next' branch, thanks Ivan.
-- Matt Fleming, Intel Open Source Technology Center
| |