[PATCH] scsi: Clean up errors in sni_53c710.c

chenguohua@jari.cn posted 1 patch 2 years, 4 months ago
drivers/scsi/sni_53c710.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] scsi: Clean up errors in sni_53c710.c
Posted by chenguohua@jari.cn 2 years, 4 months ago
Fix the following errors reported by checkpatch:

ERROR: space required before the open parenthesis '('

Signed-off-by: GuoHua Cheng <chenguohua@jari.cn>
---
 drivers/scsi/sni_53c710.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/sni_53c710.c b/drivers/scsi/sni_53c710.c
index 678651b9b4dd..e226225d1237 100644
--- a/drivers/scsi/sni_53c710.c
+++ b/drivers/scsi/sni_53c710.c
@@ -86,7 +86,7 @@ static int snirm710_probe(struct platform_device *dev)
 	host->irq = rc = platform_get_irq(dev, 0);
 	if (rc < 0)
 		goto out_put_host;
-	if(request_irq(host->irq, NCR_700_intr, IRQF_SHARED, "snirm710", host)) {
+	if (request_irq(host->irq, NCR_700_intr, IRQF_SHARED, "snirm710", host)) {
 		printk(KERN_ERR "snirm710: request_irq failed!\n");
 		goto out_put_host;
 	}
-- 
2.17.1
Re: [PATCH] scsi: Clean up errors in sni_53c710.c
Posted by James Bottomley 2 years, 4 months ago
On Wed, 2023-09-27 at 16:37 +0800, chenguohua@jari.cn wrote:
> Fix the following errors reported by checkpatch:
> 
> ERROR: space required before the open parenthesis '('

checkpatch is advisory, not mandatory.

While we usually fix its errors and warnings on current patches,
checkpatch changes over time and code it once accepted sometimes no
longer passes.  For that reason we don't accept patches to in-tree
files which now fail checkpatch because there are a huge number of them
and it generates an awful lot of churn which ends up either causing
bugs or interfering with current development.

Regards,

James