Messages in this thread | | | From | Saravana Kannan <> | Date | Mon, 15 Feb 2021 13:02:25 -0800 | Subject | Re: [PATCH] staging: board: Fix uninitialized spinlock when attaching genpd |
| |
On Mon, Feb 15, 2021 at 11:10 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > > Hi Saravana, > > On Mon, Feb 15, 2021 at 7:37 PM Saravana Kannan <saravanak@google.com> wrote: > > On Mon, Feb 15, 2021 at 7:14 AM Geert Uytterhoeven > > > @@ -148,7 +149,11 @@ static int board_staging_add_dev_domain(struct platform_device *pdev, > > > pd_args.np = np; > > > pd_args.args_count = 0; > > > > > > - return of_genpd_add_device(&pd_args, &pdev->dev); > > > + /* Cfr. device_pm_init_common() */ > > > > What's Cfr? > > "compare to" (from Latin "confer").
Can you please change this to "refer to" or "similar to"? Also, not sure if this comment is even adding anything useful even if you switch the words.
Also, device_pm_init_common() is used in two places outside of drivers/base/ with this change. Maybe better to move it to linux/device.h?
-Saravana
| |