Hi Markus, On Fri, Jul 5, 2019 at 9:43 AM Markus Elfring <Markus.Elfring@xxxxxx> wrote: > > From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> > Date: Fri, 5 Jul 2019 18:33:58 +0200 > > Avoid an extra function call by using a ternary operator instead of > a conditional statement for a setting selection. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> > --- > arch/xtensa/mm/init.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) Thanks for the patch. It doesn't change generated code since PHYS_OFFSET is a build-time constant, but it's a nice cleanup. Applied to my xtensa tree. -- Thanks. -- Max
