Commit 8a4606c6 authored by Aleksa Vučković's avatar Aleksa Vučković Committed by Greg Kroah-Hartman

drivers: dio: brace should be on the previous line

This patch fixes the checkpatch.pl errors and warnings:

< ERROR: open brace '{' following struct go on the same line
< +struct dioname
< +{
<
< ERROR: that open brace { should be on the previous line
< +static struct dioname names[] =
< +{
<
< ERROR: that open brace { should be on the previous line
< +        for (scode = 0; scode < DIO_SCMAX; ++scode)
< +        {
<
< WARNING: braces {} are not necessary for any arm of this statement
< +        if (scode >= DIOII_SCBASE) {
< [...]
< +        } else if (scode > DIO_SCMAX || scode < 0)
< [...]
< +        else if (DIO_SCINHOLE(scode))
< [...]
Signed-off-by: default avatarAleksa Vučković <aleksav013@gmail.com>
Link: https://lore.kernel.org/r/20220127135054.27281-2-aleksav013@gmail.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 10f4629f
...@@ -57,8 +57,7 @@ struct dio_bus dio_bus = { ...@@ -57,8 +57,7 @@ struct dio_bus dio_bus = {
* I think we do this by copying them into newly kmalloc()ed memory and * I think we do this by copying them into newly kmalloc()ed memory and
* marking the names[] array as .initdata ? * marking the names[] array as .initdata ?
*/ */
struct dioname struct dioname {
{
int id; int id;
const char *name; const char *name;
}; };
...@@ -67,8 +66,7 @@ struct dioname ...@@ -67,8 +66,7 @@ struct dioname
#define DIONAME(x) { DIO_ID_##x, DIO_DESC_##x } #define DIONAME(x) { DIO_ID_##x, DIO_DESC_##x }
#define DIOFBNAME(x) { DIO_ENCODE_ID( DIO_ID_FBUFFER, DIO_ID2_##x), DIO_DESC2_##x } #define DIOFBNAME(x) { DIO_ENCODE_ID( DIO_ID_FBUFFER, DIO_ID2_##x), DIO_DESC2_##x }
static struct dioname names[] = static struct dioname names[] = {
{
DIONAME(DCA0), DIONAME(DCA0REM), DIONAME(DCA1), DIONAME(DCA1REM), DIONAME(DCA0), DIONAME(DCA0REM), DIONAME(DCA1), DIONAME(DCA1REM),
DIONAME(DCM), DIONAME(DCMREM), DIONAME(DCM), DIONAME(DCMREM),
DIONAME(LAN), DIONAME(LAN),
...@@ -190,8 +188,7 @@ static int __init dio_init(void) ...@@ -190,8 +188,7 @@ static int __init dio_init(void)
request_resource(&iomem_resource, &dio_bus.resources[i]); request_resource(&iomem_resource, &dio_bus.resources[i]);
/* Register all devices */ /* Register all devices */
for (scode = 0; scode < DIO_SCMAX; ++scode) for (scode = 0; scode < DIO_SCMAX; ++scode) {
{
u_char prid, secid = 0; /* primary, secondary ID bytes */ u_char prid, secid = 0; /* primary, secondary ID bytes */
u_char *va; u_char *va;
unsigned long pa; unsigned long pa;
...@@ -267,9 +264,9 @@ subsys_initcall(dio_init); ...@@ -267,9 +264,9 @@ subsys_initcall(dio_init);
*/ */
unsigned long dio_scodetophysaddr(int scode) unsigned long dio_scodetophysaddr(int scode)
{ {
if (scode >= DIOII_SCBASE) { if (scode >= DIOII_SCBASE)
return (DIOII_BASE + (scode - 132) * DIOII_DEVSIZE); return (DIOII_BASE + (scode - 132) * DIOII_DEVSIZE);
} else if (scode > DIO_SCMAX || scode < 0) else if (scode > DIO_SCMAX || scode < 0)
return 0; return 0;
else if (DIO_SCINHOLE(scode)) else if (DIO_SCINHOLE(scode))
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment