2 Apr
2016
2 Apr
'16
1:31 a.m.
On Fri, 2016-04-01 at 18:25 -0500, Scott Wood wrote:
On Fri, 2016-04-01 at 16:59 +0530, Mugunthan V N wrote:
+static int nand_child_pre_probe(struct udevice *dev) +{
- nand_info_t *nand = dev_get_uclass_priv(dev);
- void *priv = dev_get_priv(dev);
- /*
* Store nand device priv pointer in nand_info so that* it can be used by nand command*/- nand->priv = priv;
Wouldn't it make more sense to have a pointer to the device in the NAND struct, and let the driver manage both privs as it chooses?
This makes even less sense after seeing patch 5/9, which assumes dev priv is nand_info_t, and stores its own data in nand->priv. Won't this overwrite that?
-Scott