[LEDE-DEV] MT7621 support Jumbo frames

[Gaetano Catalli]

Hi Mingyu Li,

looks fine for me.

On Fri, Apr 28, 2017 at 4:59 AM, Mingyu Li <igvtee at 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 <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



-- 
gaetano catalli