linux-kernel - [PATCH] sbus: bbc_i2c.c: fixed various coding style issues

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <20230315062518.256368-1-michaelharriscode@gmail.com>
Date:   Tue, 14 Mar 2023 23:25:18 -0700
From:   Michael Harris <michaelharriscode@...il.com>
To:     davem@...emloft.net
Cc:     sparclinux@...r.kernel.org, linux-kernel@...r.kernel.org,
        Michael Harris <michaelharriscode@...il.com>
Subject: [PATCH] sbus: bbc_i2c.c: fixed various coding style issues

Fixed various issues given from checkpatch.pl

Signed-off-by: Michael Harris <michaelharriscode@...il.com>
---
 drivers/sbus/char/bbc_i2c.c | 29 +++++++++++++++--------------
 1 file changed, 15 insertions(+), 14 deletions(-)

diff --git a/drivers/sbus/char/bbc_i2c.c b/drivers/sbus/char/bbc_i2c.c
index 537e55cd038d..c0df890f1aec 100644
--- a/drivers/sbus/char/bbc_i2c.c
+++ b/drivers/sbus/char/bbc_i2c.c
@@ -16,7 +16,7 @@
 #include <linux/of.h>
 #include <linux/of_device.h>
 #include <asm/bbc.h>
-#include <asm/io.h>
+#include <linux/io.h>
 
 #include "bbc_i2c.h"
 
@@ -32,7 +32,7 @@
 
 #define I2C_PCF_START    (I2C_PCF_PIN | I2C_PCF_ESO | I2C_PCF_ENI | I2C_PCF_STA | I2C_PCF_ACK)
 #define I2C_PCF_STOP     (I2C_PCF_PIN | I2C_PCF_ESO | I2C_PCF_STO | I2C_PCF_ACK)
-#define I2C_PCF_REPSTART (              I2C_PCF_ESO | I2C_PCF_STA | I2C_PCF_ACK)
+#define I2C_PCF_REPSTART (I2C_PCF_ESO | I2C_PCF_STA | I2C_PCF_ACK)
 #define I2C_PCF_IDLE     (I2C_PCF_PIN | I2C_PCF_ESO               | I2C_PCF_ACK)
 
 #define I2C_PCF_INI 0x40   /* 1 if not initialized */
@@ -63,8 +63,8 @@ static void set_device_claimage(struct bbc_i2c_bus *bp, struct platform_device *
 	}
 }
 
-#define claim_device(BP,ECHILD)		set_device_claimage(BP,ECHILD,1)
-#define release_device(BP,ECHILD)	set_device_claimage(BP,ECHILD,0)
+#define claim_device(BP, ECHILD)		set_device_claimage(BP, ECHILD, 1)
+#define release_device(BP, ECHILD)	set_device_claimage(BP, ECHILD, 0)
 
 struct platform_device *bbc_i2c_getdev(struct bbc_i2c_bus *bp, int index)
 {
@@ -72,8 +72,10 @@ struct platform_device *bbc_i2c_getdev(struct bbc_i2c_bus *bp, int index)
 	int curidx = 0, i;
 
 	for (i = 0; i < NUM_CHILDREN; i++) {
-		if (!(op = bp->devs[i].device))
+		if (!bp->devs[i].device) {
+			op = bp->devs[i].device;
 			break;
+		}
 		if (curidx == index)
 			goto out;
 		op = NULL;
@@ -85,6 +87,7 @@ struct platform_device *bbc_i2c_getdev(struct bbc_i2c_bus *bp, int index)
 		return op;
 	return NULL;
 }
+EXPORT_SYMBOL(bbc_i2c_getdev);
 
 struct bbc_i2c_client *bbc_i2c_attach(struct bbc_i2c_bus *bp, struct platform_device *op)
 {
@@ -110,6 +113,7 @@ struct bbc_i2c_client *bbc_i2c_attach(struct bbc_i2c_bus *bp, struct platform_de
 
 	return client;
 }
+EXPORT_SYMBOL(bbc_i2c_attach);
 
 void bbc_i2c_detach(struct bbc_i2c_client *client)
 {
@@ -119,6 +123,7 @@ void bbc_i2c_detach(struct bbc_i2c_client *client)
 	release_device(bp, op);
 	kfree(client);
 }
+EXPORT_SYMBOL(bbc_i2c_detach);
 
 static int wait_for_pin(struct bbc_i2c_bus *bp, u8 *status)
 {
@@ -177,6 +182,7 @@ int bbc_i2c_writeb(struct bbc_i2c_client *client, unsigned char val, int off)
 	writeb(I2C_PCF_STOP, bp->i2c_control_regs + 0x0);
 	return ret;
 }
+EXPORT_SYMBOL(bbc_i2c_writeb);
 
 int bbc_i2c_readb(struct bbc_i2c_client *client, unsigned char *byte, int off)
 {
@@ -226,6 +232,7 @@ int bbc_i2c_readb(struct bbc_i2c_client *client, unsigned char *byte, int off)
 
 	return ret;
 }
+EXPORT_SYMBOL(bbc_i2c_readb);
 
 int bbc_i2c_write_buf(struct bbc_i2c_client *client,
 		      char *buf, int len, int off)
@@ -242,6 +249,7 @@ int bbc_i2c_write_buf(struct bbc_i2c_client *client,
 	}
 	return ret;
 }
+EXPORT_SYMBOL(bbc_i2c_write_buf);
 
 int bbc_i2c_read_buf(struct bbc_i2c_client *client,
 		     char *buf, int len, int off)
@@ -259,13 +267,6 @@ int bbc_i2c_read_buf(struct bbc_i2c_client *client,
 
 	return ret;
 }
-
-EXPORT_SYMBOL(bbc_i2c_getdev);
-EXPORT_SYMBOL(bbc_i2c_attach);
-EXPORT_SYMBOL(bbc_i2c_detach);
-EXPORT_SYMBOL(bbc_i2c_writeb);
-EXPORT_SYMBOL(bbc_i2c_readb);
-EXPORT_SYMBOL(bbc_i2c_write_buf);
 EXPORT_SYMBOL(bbc_i2c_read_buf);
 
 static irqreturn_t bbc_i2c_interrupt(int irq, void *dev_id)
@@ -291,7 +292,7 @@ static void reset_one_i2c(struct bbc_i2c_bus *bp)
 	writeb(I2C_PCF_IDLE, bp->i2c_control_regs + 0x0);
 }
 
-static struct bbc_i2c_bus * attach_one_i2c(struct platform_device *op, int index)
+static struct bbc_i2c_bus *attach_one_i2c(struct platform_device *op, int index)
 {
 	struct bbc_i2c_bus *bp;
 	struct device_node *dp;
@@ -341,7 +342,7 @@ static struct bbc_i2c_bus * attach_one_i2c(struct platform_device *op, int index
 	writeb(I2C_PCF_PIN | I2C_PCF_ES1, bp->i2c_control_regs + 0x0);
 	bp->clock = readb(bp->i2c_control_regs + 0x01);
 
-	printk(KERN_INFO "i2c-%d: Regs at %p, %d devices, own %02x, clock %02x.\n",
+	pr_info("i2c-%d: Regs at %p, %d devices, own %02x, clock %02x.\n",
 	       bp->index, bp->i2c_control_regs, entry, bp->own, bp->clock);
 
 	reset_one_i2c(bp);
-- 
2.34.1