[PATCH] mtd: rawnand: au1550nd: fix the memory reservation check

[James Hilliard]

request_mem_region() returns a non-NULL pointer on success. The inverted
check rejects a successful reservation without releasing it, and proceeds
to map the NAND registers when the reservation actually failed.

Reject a NULL result so probe proceeds only after claiming the resource.

Fixes: b67a1a02d463 ("MTD: nand: make au1550nd.c a platform_driver")
Cc: stable at vger.kernel.org
Signed-off-by: James Hilliard <james.hilliard1 at gmail.com>
---
 drivers/mtd/nand/raw/au1550nd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/raw/au1550nd.c b/drivers/mtd/nand/raw/au1550nd.c
index f1e99197ab84..6796afe9cd29 100644
--- a/drivers/mtd/nand/raw/au1550nd.c
+++ b/drivers/mtd/nand/raw/au1550nd.c
@@ -276,7 +276,7 @@ static int au1550nd_probe(struct platform_device *pdev)
 		ret = -ENODEV;
 		goto out1;
 	}
-	if (request_mem_region(r->start, resource_size(r), "au1550-nand")) {
+	if (!request_mem_region(r->start, resource_size(r), "au1550-nand")) {
 		dev_err(&pdev->dev, "cannot claim NAND memory area\n");
 		ret = -ENOMEM;
 		goto out1;

---
base-commit: 30daaf3e7897b4a8d9b72e5b0c86dd6e2a4d7d85
change-id: 20261107-submit-au1550nd-resource-claim-v1-71562f83dd01

Best regards,
--  
James Hilliard <james.hilliard1 at gmail.com>