mtd: nand: pxa3xx: use marvell, prefix for custom DT properties
The DT properties for the "enable-arbiter" and "keep-config" config knobs were previously named inconsistently: - The u-boot driver used "nand-enable-arbiter" and "nand-keep-config" names, without Marvell prefixes. - The Linux driver uses "marvell,nand-keep-config" ("enable-arbiter" does not exist anymore in recent kernels, but it also used to be "marvell,nand-enable-arbiter"). - The device trees almost all use "marvell," prefixed names, except for one single instance of "nand-enable-arbiter" without vendor prefix. This commit standardizes on the vendor prefixed version, making the u-boot driver read from DT props "marvell,nand-enable-arbiter" and "marvell,nand-keep-config". The one device tree using the unprefixed version is also changed to use the new naming. This has the side effect of making the previously no-op "marvell," config knobs already present in some DTs actually do something. This was likely the original intention of the DT authors, but note that this commit was not tested on every single impacted board. Signed-off-by: Pierre Bourdon <delroth@gmail.com>
This commit is contained in:
parent
94c30f9c8f
commit
c03f4da187
@ -231,7 +231,7 @@
|
||||
#address-cells = <1>;
|
||||
#size-cells = <0>;
|
||||
clocks = <&CP110_LABEL(syscon0) 1 2>;
|
||||
nand-enable-arbiter;
|
||||
marvell,nand-enable-arbiter;
|
||||
num-cs = <1>;
|
||||
nand-ecc-strength = <8>;
|
||||
nand-ecc-step-size = <512>;
|
||||
|
@ -1862,10 +1862,10 @@ static int pxa3xx_nand_probe_dt(struct udevice *dev, struct pxa3xx_nand_info *in
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
if (dev_read_bool(dev, "nand-enable-arbiter"))
|
||||
if (dev_read_bool(dev, "marvell,nand-enable-arbiter"))
|
||||
pdata->enable_arbiter = 1;
|
||||
|
||||
if (dev_read_bool(dev, "nand-keep-config"))
|
||||
if (dev_read_bool(dev, "marvell,nand-keep-config"))
|
||||
pdata->keep_config = 1;
|
||||
|
||||
/*
|
||||
|
Loading…
Reference in New Issue
Block a user