All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] scsi: hpsa: clean up two indentation issues
@ 2019-01-22 15:18 Colin King
  2019-01-28 16:18   ` Don.Brace
  2019-01-29  5:42   ` Martin K. Petersen
  0 siblings, 2 replies; 6+ messages in thread
From: Colin King @ 2019-01-22 15:18 UTC (permalink / raw)
  To: Don Brace, James E . J . Bottomley, Martin K . Petersen,
	esc.storagedev, linux-scsi
  Cc: kernel-janitors, linux-kernel

From: Colin Ian King <colin.king@canonical.com>

There are two statements that are indented incorrectly. Fix these.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 drivers/scsi/hpsa.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index ff67ef5d5347..5284444fdd10 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -1327,7 +1327,7 @@ static int hpsa_scsi_add_entry(struct ctlr_info *h,
 		dev_warn(&h->pdev->dev, "physical device with no LUN=0,"
 			" suspect firmware bug or unsupported hardware "
 			"configuration.\n");
-			return -1;
+		return -1;
 	}
 
 lun_assigned:
@@ -4110,7 +4110,7 @@ static int hpsa_gather_lun_info(struct ctlr_info *h,
 			"maximum logical LUNs (%d) exceeded.  "
 			"%d LUNs ignored.\n", HPSA_MAX_LUN,
 			*nlogicals - HPSA_MAX_LUN);
-			*nlogicals = HPSA_MAX_LUN;
+		*nlogicals = HPSA_MAX_LUN;
 	}
 	if (*nlogicals + *nphysicals > HPSA_MAX_PHYS_LUN) {
 		dev_warn(&h->pdev->dev,
-- 
2.19.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* RE: [PATCH] scsi: hpsa: clean up two indentation issues
  2019-01-22 15:18 [PATCH] scsi: hpsa: clean up two indentation issues Colin King
  2019-01-28 16:18   ` Don.Brace
@ 2019-01-28 16:18   ` Don.Brace
  1 sibling, 0 replies; 6+ messages in thread
From: Don.Brace @ 2019-01-28 16:18 UTC (permalink / raw)
  To: colin.king, don.brace, jejb, martin.petersen, esc.storagedev, linux-scsi
  Cc: kernel-janitors, linux-kernel


-----Original Message-----
From: linux-scsi-owner@vger.kernel.org [mailto:linux-scsi-owner@vger.kernel.org] On Behalf Of Colin King
Sent: Tuesday, January 22, 2019 9:19 AM
To: Don Brace <don.brace@microsemi.com>; James E . J . Bottomley <jejb@linux.ibm.com>; Martin K . Petersen <martin.petersen@oracle.com>; esc.storagedev@microsemi.com; linux-scsi@vger.kernel.org
Cc: kernel-janitors@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: [PATCH] scsi: hpsa: clean up two indentation issues

From: Colin Ian King <colin.king@canonical.com>

There are two statements that are indented incorrectly. Fix these.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 drivers/scsi/hpsa.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index ff67ef5d5347..5284444fdd10 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -1327,7 +1327,7 @@ static int hpsa_scsi_add_entry(struct ctlr_info *h,
 		dev_warn(&h->pdev->dev, "physical device with no LUN=0,"
 			" suspect firmware bug or unsupported hardware "
 			"configuration.\n");
-			return -1;
+		return -1;
 	}
 
 lun_assigned:
@@ -4110,7 +4110,7 @@ static int hpsa_gather_lun_info(struct ctlr_info *h,
 			"maximum logical LUNs (%d) exceeded.  "
 			"%d LUNs ignored.\n", HPSA_MAX_LUN,
 			*nlogicals - HPSA_MAX_LUN);
-			*nlogicals = HPSA_MAX_LUN;
+		*nlogicals = HPSA_MAX_LUN;
 	}
 	if (*nlogicals + *nphysicals > HPSA_MAX_PHYS_LUN) {
 		dev_warn(&h->pdev->dev,
--
2.19.1

Acked-by: Don Brace <don.brace@microsemi.com>?


^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: [PATCH] scsi: hpsa: clean up two indentation issues
@ 2019-01-28 16:18   ` Don.Brace
  0 siblings, 0 replies; 6+ messages in thread
From: Don.Brace @ 2019-01-28 16:18 UTC (permalink / raw)
  To: colin.king, don.brace, jejb, martin.petersen, esc.storagedev, linux-scsi
  Cc: kernel-janitors, linux-kernel

DQotLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KRnJvbTogbGludXgtc2NzaS1vd25lckB2Z2Vy
Lmtlcm5lbC5vcmcgW21haWx0bzpsaW51eC1zY3NpLW93bmVyQHZnZXIua2VybmVsLm9yZ10gT24g
QmVoYWxmIE9mIENvbGluIEtpbmcNClNlbnQ6IFR1ZXNkYXksIEphbnVhcnkgMjIsIDIwMTkgOTox
OSBBTQ0KVG86IERvbiBCcmFjZSA8ZG9uLmJyYWNlQG1pY3Jvc2VtaS5jb20+OyBKYW1lcyBFIC4g
SiAuIEJvdHRvbWxleSA8amVqYkBsaW51eC5pYm0uY29tPjsgTWFydGluIEsgLiBQZXRlcnNlbiA8
bWFydGluLnBldGVyc2VuQG9yYWNsZS5jb20+OyBlc2Muc3RvcmFnZWRldkBtaWNyb3NlbWkuY29t
OyBsaW51eC1zY3NpQHZnZXIua2VybmVsLm9yZw0KQ2M6IGtlcm5lbC1qYW5pdG9yc0B2Z2VyLmtl
cm5lbC5vcmc7IGxpbnV4LWtlcm5lbEB2Z2VyLmtlcm5lbC5vcmcNClN1YmplY3Q6IFtQQVRDSF0g
c2NzaTogaHBzYTogY2xlYW4gdXAgdHdvIGluZGVudGF0aW9uIGlzc3Vlcw0KDQpGcm9tOiBDb2xp
biBJYW4gS2luZyA8Y29saW4ua2luZ0BjYW5vbmljYWwuY29tPg0KDQpUaGVyZSBhcmUgdHdvIHN0
YXRlbWVudHMgdGhhdCBhcmUgaW5kZW50ZWQgaW5jb3JyZWN0bHkuIEZpeCB0aGVzZS4NCg0KU2ln
bmVkLW9mZi1ieTogQ29saW4gSWFuIEtpbmcgPGNvbGluLmtpbmdAY2Fub25pY2FsLmNvbT4NCi0t
LQ0KIGRyaXZlcnMvc2NzaS9ocHNhLmMgfCA0ICsrLS0NCiAxIGZpbGUgY2hhbmdlZCwgMiBpbnNl
cnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQ0KDQpkaWZmIC0tZ2l0IGEvZHJpdmVycy9zY3NpL2hw
c2EuYyBiL2RyaXZlcnMvc2NzaS9ocHNhLmMgaW5kZXggZmY2N2VmNWQ1MzQ3Li41Mjg0NDQ0ZmRk
MTAgMTAwNjQ0DQotLS0gYS9kcml2ZXJzL3Njc2kvaHBzYS5jDQorKysgYi9kcml2ZXJzL3Njc2kv
aHBzYS5jDQpAQCAtMTMyNyw3ICsxMzI3LDcgQEAgc3RhdGljIGludCBocHNhX3Njc2lfYWRkX2Vu
dHJ5KHN0cnVjdCBjdGxyX2luZm8gKmgsDQogCQlkZXZfd2FybigmaC0+cGRldi0+ZGV2LCAicGh5
c2ljYWwgZGV2aWNlIHdpdGggbm8gTFVOPTAsIg0KIAkJCSIgc3VzcGVjdCBmaXJtd2FyZSBidWcg
b3IgdW5zdXBwb3J0ZWQgaGFyZHdhcmUgIg0KIAkJCSJjb25maWd1cmF0aW9uLlxuIik7DQotCQkJ
cmV0dXJuIC0xOw0KKwkJcmV0dXJuIC0xOw0KIAl9DQogDQogbHVuX2Fzc2lnbmVkOg0KQEAgLTQx
MTAsNyArNDExMCw3IEBAIHN0YXRpYyBpbnQgaHBzYV9nYXRoZXJfbHVuX2luZm8oc3RydWN0IGN0
bHJfaW5mbyAqaCwNCiAJCQkibWF4aW11bSBsb2dpY2FsIExVTnMgKCVkKSBleGNlZWRlZC4gICIN
CiAJCQkiJWQgTFVOcyBpZ25vcmVkLlxuIiwgSFBTQV9NQVhfTFVOLA0KIAkJCSpubG9naWNhbHMg
LSBIUFNBX01BWF9MVU4pOw0KLQkJCSpubG9naWNhbHMgPSBIUFNBX01BWF9MVU47DQorCQkqbmxv
Z2ljYWxzID0gSFBTQV9NQVhfTFVOOw0KIAl9DQogCWlmICgqbmxvZ2ljYWxzICsgKm5waHlzaWNh
bHMgPiBIUFNBX01BWF9QSFlTX0xVTikgew0KIAkJZGV2X3dhcm4oJmgtPnBkZXYtPmRldiwNCi0t
DQoyLjE5LjENCg0KQWNrZWQtYnk6IERvbiBCcmFjZSA8ZG9uLmJyYWNlQG1pY3Jvc2VtaS5jb20+
Pw0KDQo

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: [PATCH] scsi: hpsa: clean up two indentation issues
@ 2019-01-28 16:18   ` Don.Brace
  0 siblings, 0 replies; 6+ messages in thread
From: Don.Brace @ 2019-01-28 16:18 UTC (permalink / raw)
  To: colin.king, don.brace, jejb, martin.petersen, esc.storagedev, linux-scsi
  Cc: kernel-janitors, linux-kernel


-----Original Message-----
From: linux-scsi-owner@vger.kernel.org [mailto:linux-scsi-owner@vger.kernel.org] On Behalf Of Colin King
Sent: Tuesday, January 22, 2019 9:19 AM
To: Don Brace <don.brace@microsemi.com>; James E . J . Bottomley <jejb@linux.ibm.com>; Martin K . Petersen <martin.petersen@oracle.com>; esc.storagedev@microsemi.com; linux-scsi@vger.kernel.org
Cc: kernel-janitors@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: [PATCH] scsi: hpsa: clean up two indentation issues

From: Colin Ian King <colin.king@canonical.com>

There are two statements that are indented incorrectly. Fix these.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 drivers/scsi/hpsa.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index ff67ef5d5347..5284444fdd10 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -1327,7 +1327,7 @@ static int hpsa_scsi_add_entry(struct ctlr_info *h,
 		dev_warn(&h->pdev->dev, "physical device with no LUN=0,"
 			" suspect firmware bug or unsupported hardware "
 			"configuration.\n");
-			return -1;
+		return -1;
 	}
 
 lun_assigned:
@@ -4110,7 +4110,7 @@ static int hpsa_gather_lun_info(struct ctlr_info *h,
 			"maximum logical LUNs (%d) exceeded.  "
 			"%d LUNs ignored.\n", HPSA_MAX_LUN,
 			*nlogicals - HPSA_MAX_LUN);
-			*nlogicals = HPSA_MAX_LUN;
+		*nlogicals = HPSA_MAX_LUN;
 	}
 	if (*nlogicals + *nphysicals > HPSA_MAX_PHYS_LUN) {
 		dev_warn(&h->pdev->dev,
--
2.19.1

Acked-by: Don Brace <don.brace@microsemi.com>?


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] scsi: hpsa: clean up two indentation issues
  2019-01-22 15:18 [PATCH] scsi: hpsa: clean up two indentation issues Colin King
@ 2019-01-29  5:42   ` Martin K. Petersen
  2019-01-29  5:42   ` Martin K. Petersen
  1 sibling, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2019-01-29  5:42 UTC (permalink / raw)
  To: Colin King
  Cc: Don Brace, James E . J . Bottomley, Martin K . Petersen,
	esc.storagedev, linux-scsi, kernel-janitors, linux-kernel


Colin,

> There are two statements that are indented incorrectly. Fix these.

Applied to 5.1/scsi-queue, thanks.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] scsi: hpsa: clean up two indentation issues
@ 2019-01-29  5:42   ` Martin K. Petersen
  0 siblings, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2019-01-29  5:42 UTC (permalink / raw)
  To: Colin King
  Cc: Don Brace, James E . J . Bottomley, Martin K . Petersen,
	esc.storagedev, linux-scsi, kernel-janitors, linux-kernel


Colin,

> There are two statements that are indented incorrectly. Fix these.

Applied to 5.1/scsi-queue, thanks.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-01-29  5:42 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-22 15:18 [PATCH] scsi: hpsa: clean up two indentation issues Colin King
2019-01-28 16:18 ` Don.Brace
2019-01-28 16:18   ` Don.Brace
2019-01-28 16:18   ` Don.Brace
2019-01-29  5:42 ` Martin K. Petersen
2019-01-29  5:42   ` Martin K. Petersen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.