diff for duplicates of <20180718080913.GB4641@dell> diff --git a/a/1.txt b/N1/1.txt index 054b158..a6c9694 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -38,7 +38,7 @@ should remain? If it was reviewed too, perhaps an AB too? > - */ > - if (num_steps) { > + if ((of_property_read_u32(node, "num-interpolated-steps", -> + &num_steps) = 0) && num_steps) { +> + &num_steps) == 0) && num_steps) { This is pretty ugly, and isn't it suffering from over-bracketing? My suggestion would be to break out the invocation of diff --git a/a/content_digest b/N1/content_digest index 1240c54..0a854b8 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,7 +1,7 @@ "ref\020180716210241.9457-1-daniel.thompson@linaro.org\0" "From\0Lee Jones <lee.jones@linaro.org>\0" "Subject\0Re: [PATCH] backlight: pwm_bl: Fix uninitialized variable\0" - "Date\0Wed, 18 Jul 2018 08:09:13 +0000\0" + "Date\0Wed, 18 Jul 2018 09:09:13 +0100\0" "To\0Daniel Thompson <daniel.thompson@linaro.org>\0" "Cc\0Jingoo Han <jingoohan1@gmail.com>" Thierry Reding <thierry.reding@gmail.com> @@ -54,7 +54,7 @@ "> -\t\t */\n" "> -\t\tif (num_steps) {\n" "> +\t\tif ((of_property_read_u32(node, \"num-interpolated-steps\",\n" - "> +\t\t\t\t\t &num_steps) = 0) && num_steps) {\n" + "> +\t\t\t\t\t &num_steps) == 0) && num_steps) {\n" "\n" "This is pretty ugly, and isn't it suffering from over-bracketing? My\n" "suggestion would be to break out the invocation of\n" @@ -90,4 +90,4 @@ "Linaro.org \342\224\202 Open source software for ARM SoCs\n" Follow Linaro: Facebook | Twitter | Blog -4d5d84cfb553724ef8ab08902a6a03d7c4fc68e718db132babd3a95ad36f379f +c1d0fd00ac0b57f4ffd3cfc7de9fd8ecd7e7ba38af9e32a3564662e8cae1ab11
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.