Re: [PATCH net-next 5/7] r8169: add a new chip for RTL8111G

From: David Miller
Date: Tue Apr 02 2013 - 19:26:09 EST


From: Francois Romieu <romieu@xxxxxxxxxxxxx>
Date: Wed, 3 Apr 2013 01:20:08 +0200

> hayeswang <hayeswang@xxxxxxxxxxx> :
>> Francois Romieu [mailto:romieu@xxxxxxxxxxxxx]
> [...]
>> > There is close to zero added value for this stuff in the kernel.
>> > You may as well move it completely into the firmware.
>>
>> Do you mean all of the phy settings ? I have checked these settings with
>> our hw engineers. These are not firmware.
>
> Undocumented configuration data which is subject to change over time ?
>
> No one outside of Realtek can make any sense of this opaque pile of data.
> There is no point in me or anybody else rubber stamping it for inclusion.

Right, so either document all of these indirect registers being programmed
in the PHY or move it to firmware.
--
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/