Hi Mingyu Li, looks fine for me.
On Fri, Apr 28, 2017 at 4:59 AM, Mingyu Li <igv...@gmail.com> wrote: > Hi Gaetano Catalli > > i change your mtk_eth_soc.c driver part. because mt7621 don't need to > set FE_GDMA1_FWD_CFG register to enable jumbo frame. and keep > MAX_RX_LENGTH because other chips need it. > > i also made a patch on github > https://github.com/igvtee/source/commit/c4a660d991011e2c0e232758500d56faae93b631 > if you feel ok. i can send pull request to lede project. > > 2017-04-10 15:21 GMT+08:00 Jaap Buurman <jaapbuur...@gmail.com>: >> I am able to test this on my DIR-860l. I could try to compile my own >> build to test this, but I am no C programmer myself. Am I correct in >> the following post about how to set the flag to allow for jumbo frames >> to be set?: >> https://forum.lede-project.org/t/build-for-the-d-link-dir-860l/948/40?u=mushoz >> If so, I will be able to apply the patch myself and test it. I could >> then send a PR if the tests are successful. >> >> Yours sincerely, >> >> Jaap Buurman >> >> On Mon, Apr 10, 2017 at 9:16 AM, Milan Kočvara <milankocv...@gmail.com> >> wrote: >>> Hello all, >>> if someone do a testing patch for jumboFrames, I can lend a helping >>> hand with testing on EdgeRouter X. I think MT7621 know JumboFrames, >>> because of Ubiquiti FMW allow to set MTU 1536B >>> >>> 2017-04-10 2:06 GMT+02:00 <ros...@gmail.com>: >>>> Probably was not in the interest of the driver writers. Based on the >>>> copyrights though, it's mostly LEDE/OpenWRT developers. Try modifying >>>> the code to test if jumbo frames work. >>>> >>>> On Sun, 2017-04-09 at 21:20 +0200, Jaap Buurman wrote: >>>>> Hello all, >>>>> >>>>> I found the message below in a conversation from back in August, 2016 >>>>> in this mailinglist. I did not find a reply to this question. Has >>>>> there ever been one? Or does anyone else happen to know the answer to >>>>> this question? Thank you very much in advance. >>>>> >>>>> Yours sincerely, >>>>> >>>>> Jaap Buurman >>>>> >>>>> August, 2016 message: >>>>> >>>>> >>>>> Hi all, >>>>> >>>>> in the MT7621 ethernet driver code >>>>> (drivers/net/ethernet/mediatek/soc_mt7621.c) the FE_FLAG_JUMBO_FRAME >>>>> flag is not during the device initialization. This prevents the user >>>>> to set an MTU greater than 1500. Is this correct? Does MT7621 support >>>>> jumbo frames? >>>>> >>>>> Thanks. >>>>> >>>> >>>> _______________________________________________ >>>> Lede-dev mailing list >>>> Lede-dev@lists.infradead.org >>>> http://lists.infradead.org/mailman/listinfo/lede-dev >> >> _______________________________________________ >> Lede-dev mailing list >> Lede-dev@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/lede-dev -- gaetano catalli _______________________________________________ Lede-dev mailing list Lede-dev@lists.infradead.org http://lists.infradead.org/mailman/listinfo/lede-dev
