[LEDE-DEV] MT7621 support Jumbo frames

[Mingyu Li]

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 <jaapbuurman at 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 <milankocvara at 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  <rosenp at 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 at lists.infradead.org
>>> http://lists.infradead.org/mailman/listinfo/lede-dev
>
> _______________________________________________
> Lede-dev mailing list
> Lede-dev at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/lede-dev