| Message ID | 20240613045902.12029-2-davthompson@nvidia.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Style fixes for EDAC/bluefield driver | expand |
On Thu, Jun 13, 2024 at 12:59:01AM -0400, David Thompson wrote: > The routine bluefield_edac_mc_probe() has an extra empty line, > so it must be removed. This extra line is flagged when running > "scripts/checkpatch --strict": > CHECK: Blank lines aren't necessary before a close brace '}' > #324: FILE: drivers/edac/bluefield_edac.c:324: > + > +} > > Signed-off-by: David Thompson <davthompson@nvidia.com> > Reviewed-by: Shravan Kumar Ramani <shravankr@nvidia.com> > --- > v2 > - Updated commit message per review comments > --- > drivers/edac/bluefield_edac.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/edac/bluefield_edac.c b/drivers/edac/bluefield_edac.c > index 5b3164560648..1f6f39a7dbf3 100644 > --- a/drivers/edac/bluefield_edac.c > +++ b/drivers/edac/bluefield_edac.c > @@ -320,7 +320,6 @@ static int bluefield_edac_mc_probe(struct platform_device *pdev) > edac_mc_free(mci); > > return ret; > - > } > > static void bluefield_edac_mc_remove(struct platform_device *pdev) > -- Did you get my reply to this and did you read it? https://lore.kernel.org/r/20240612200056.GGZmn-eO15ePolbAPo@fat_crate.local
diff --git a/drivers/edac/bluefield_edac.c b/drivers/edac/bluefield_edac.c index 5b3164560648..1f6f39a7dbf3 100644 --- a/drivers/edac/bluefield_edac.c +++ b/drivers/edac/bluefield_edac.c @@ -320,7 +320,6 @@ static int bluefield_edac_mc_probe(struct platform_device *pdev) edac_mc_free(mci); return ret; - } static void bluefield_edac_mc_remove(struct platform_device *pdev)