Re: [PATCH 02/10] net: stmmac: Honor DT parameter to force DMAstore and forward mode

From: David Miller
Date: Fri Dec 06 2013 - 16:26:15 EST


From: Chen-Yu Tsai <wens@xxxxxxxx>
Date: Sat, 7 Dec 2013 01:29:35 +0800

> @@ -47,6 +47,7 @@ static int stmmac_probe_config_dt(struct platform_device *pdev,
> plat->bus_id = 0;
>
> of_property_read_u32(np, "snps,phy-addr", &plat->phy_addr);
> + plat->force_sf_dma_mode = of_property_read_bool(np, "snps,force_sf_dma_mode");

Will this do the right thing for when the property is not present?
Right now the force_sf_dma_mode value is always false.

In fact won't it override the explicit settings done elsewhere in the
driver?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/