From: Vince Bridgers <vbridgers2013@xxxxxxxxx> Date: Tue, 14 Jan 2014 14:44:35 -0600 > On Tue, Jan 14, 2014 at 1:53 PM, Ben Hutchings > <bhutchings@xxxxxxxxxxxxxx> wrote: >> On Tue, 2014-01-14 at 11:17 -0600, Vince Bridgers wrote: >>> --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c >>> +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c >>> @@ -51,6 +51,10 @@ static int stmmac_probe_config_dt(struct platform_device *pdev, >>> plat->mdio_bus_data = devm_kzalloc(&pdev->dev, >>> sizeof(struct stmmac_mdio_bus_data), >>> GFP_KERNEL); >>> + /* Set the maxmtu to a default of 1500 in case the >>> + * parameter is not present in the device tree >>> + */ >>> + plat->maxmtu = JUMBO_LEN; >> >> The comment disagrees with the definition of JUMBO_LEN. > > ok, I'll address the comment inconsistency. Therefore I'm expecting to see a v4 of this series at some point, thanks. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html