kernel: use accepted version of bcm47xxpart fix commit
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
This commit is contained in:
parent
070693477a
commit
404508001e
3 changed files with 18 additions and 3 deletions
|
@ -1,4 +1,6 @@
|
||||||
|
From 237ea0d4762cc14d0fc80e80d61f0f08e1050c7f Mon Sep 17 00:00:00 2001
|
||||||
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
|
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
|
||||||
|
Date: Thu, 12 Apr 2018 07:24:52 +0200
|
||||||
Subject: [PATCH] mtd: bcm47xxpart: improve handling TRX partition size
|
Subject: [PATCH] mtd: bcm47xxpart: improve handling TRX partition size
|
||||||
MIME-Version: 1.0
|
MIME-Version: 1.0
|
||||||
Content-Type: text/plain; charset=UTF-8
|
Content-Type: text/plain; charset=UTF-8
|
||||||
|
@ -6,7 +8,7 @@ Content-Transfer-Encoding: 8bit
|
||||||
|
|
||||||
When bcm47xxpart finds a TRX partition (container) it's supposed to jump
|
When bcm47xxpart finds a TRX partition (container) it's supposed to jump
|
||||||
to the end of it and keep looking for more partitions. TRX and its
|
to the end of it and keep looking for more partitions. TRX and its
|
||||||
subpartitions are handled be a separated parser.
|
subpartitions are handled by a separate parser.
|
||||||
|
|
||||||
The problem with old code was relying on the length specified in a TRX
|
The problem with old code was relying on the length specified in a TRX
|
||||||
header. That isn't reliable as TRX is commonly modified to have checksum
|
header. That isn't reliable as TRX is commonly modified to have checksum
|
||||||
|
@ -22,7 +24,10 @@ This makes code more optimal & reliable thanks to skipping data that
|
||||||
shouldn't be parsed.
|
shouldn't be parsed.
|
||||||
|
|
||||||
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
|
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
|
||||||
|
Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
|
||||||
---
|
---
|
||||||
|
drivers/mtd/bcm47xxpart.c | 22 ++++++++++++++++++----
|
||||||
|
1 file changed, 18 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
--- a/drivers/mtd/bcm47xxpart.c
|
--- a/drivers/mtd/bcm47xxpart.c
|
||||||
+++ b/drivers/mtd/bcm47xxpart.c
|
+++ b/drivers/mtd/bcm47xxpart.c
|
|
@ -1,4 +1,6 @@
|
||||||
|
From 237ea0d4762cc14d0fc80e80d61f0f08e1050c7f Mon Sep 17 00:00:00 2001
|
||||||
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
|
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
|
||||||
|
Date: Thu, 12 Apr 2018 07:24:52 +0200
|
||||||
Subject: [PATCH] mtd: bcm47xxpart: improve handling TRX partition size
|
Subject: [PATCH] mtd: bcm47xxpart: improve handling TRX partition size
|
||||||
MIME-Version: 1.0
|
MIME-Version: 1.0
|
||||||
Content-Type: text/plain; charset=UTF-8
|
Content-Type: text/plain; charset=UTF-8
|
||||||
|
@ -6,7 +8,7 @@ Content-Transfer-Encoding: 8bit
|
||||||
|
|
||||||
When bcm47xxpart finds a TRX partition (container) it's supposed to jump
|
When bcm47xxpart finds a TRX partition (container) it's supposed to jump
|
||||||
to the end of it and keep looking for more partitions. TRX and its
|
to the end of it and keep looking for more partitions. TRX and its
|
||||||
subpartitions are handled be a separated parser.
|
subpartitions are handled by a separate parser.
|
||||||
|
|
||||||
The problem with old code was relying on the length specified in a TRX
|
The problem with old code was relying on the length specified in a TRX
|
||||||
header. That isn't reliable as TRX is commonly modified to have checksum
|
header. That isn't reliable as TRX is commonly modified to have checksum
|
||||||
|
@ -22,7 +24,10 @@ This makes code more optimal & reliable thanks to skipping data that
|
||||||
shouldn't be parsed.
|
shouldn't be parsed.
|
||||||
|
|
||||||
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
|
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
|
||||||
|
Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
|
||||||
---
|
---
|
||||||
|
drivers/mtd/bcm47xxpart.c | 22 ++++++++++++++++++----
|
||||||
|
1 file changed, 18 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
--- a/drivers/mtd/bcm47xxpart.c
|
--- a/drivers/mtd/bcm47xxpart.c
|
||||||
+++ b/drivers/mtd/bcm47xxpart.c
|
+++ b/drivers/mtd/bcm47xxpart.c
|
|
@ -1,4 +1,6 @@
|
||||||
|
From 237ea0d4762cc14d0fc80e80d61f0f08e1050c7f Mon Sep 17 00:00:00 2001
|
||||||
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
|
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
|
||||||
|
Date: Thu, 12 Apr 2018 07:24:52 +0200
|
||||||
Subject: [PATCH] mtd: bcm47xxpart: improve handling TRX partition size
|
Subject: [PATCH] mtd: bcm47xxpart: improve handling TRX partition size
|
||||||
MIME-Version: 1.0
|
MIME-Version: 1.0
|
||||||
Content-Type: text/plain; charset=UTF-8
|
Content-Type: text/plain; charset=UTF-8
|
||||||
|
@ -6,7 +8,7 @@ Content-Transfer-Encoding: 8bit
|
||||||
|
|
||||||
When bcm47xxpart finds a TRX partition (container) it's supposed to jump
|
When bcm47xxpart finds a TRX partition (container) it's supposed to jump
|
||||||
to the end of it and keep looking for more partitions. TRX and its
|
to the end of it and keep looking for more partitions. TRX and its
|
||||||
subpartitions are handled be a separated parser.
|
subpartitions are handled by a separate parser.
|
||||||
|
|
||||||
The problem with old code was relying on the length specified in a TRX
|
The problem with old code was relying on the length specified in a TRX
|
||||||
header. That isn't reliable as TRX is commonly modified to have checksum
|
header. That isn't reliable as TRX is commonly modified to have checksum
|
||||||
|
@ -22,7 +24,10 @@ This makes code more optimal & reliable thanks to skipping data that
|
||||||
shouldn't be parsed.
|
shouldn't be parsed.
|
||||||
|
|
||||||
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
|
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
|
||||||
|
Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
|
||||||
---
|
---
|
||||||
|
drivers/mtd/bcm47xxpart.c | 22 ++++++++++++++++++----
|
||||||
|
1 file changed, 18 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
--- a/drivers/mtd/bcm47xxpart.c
|
--- a/drivers/mtd/bcm47xxpart.c
|
||||||
+++ b/drivers/mtd/bcm47xxpart.c
|
+++ b/drivers/mtd/bcm47xxpart.c
|
Loading…
Reference in a new issue