mirror of
https://gitee.com/mirrors_PX4/PX4-Autopilot.git
synced 2026-10-24 12:13:35 +08:00
Dequote Kconfig strings that may be used as components of a path
git-svn-id: https://nuttx.svn.sourceforge.net/svnroot/nuttx/trunk@5057 7fd9a85b-ad96-42d3-883c-3090e2eb8679
This commit is contained in:
@@ -2101,23 +2101,10 @@ static inline int lpc17_phyinit(struct lpc17_driver_s *priv)
|
||||
/* Disable auto-configuration. Set the fixed speed/duplex mode.
|
||||
* (probably more than little redundant).
|
||||
*
|
||||
* REVISIT: Revisit the following CONFIG_PHY_CEMENT_DISABLE work-around:
|
||||
*
|
||||
* "... I found this using a LPC1768 dev board with a DP83848I PHY. I'm
|
||||
* using CONFIG_PHY_DP83848C for that ... I found that a static
|
||||
* configuration for the PHY gave errors. I didn't investigate why.
|
||||
*
|
||||
* "My problem however was that autonegotiation seemingly failed -
|
||||
* however on debugging I saw the calls for autonegotation were actually
|
||||
* successful. I tracked the code down to [the following logic] and
|
||||
* I saw that after the negotation completes, those negotiated
|
||||
* parameters are then used to set a fixed speed and duplex, "just
|
||||
* in case". It was that setting of a static configuration which
|
||||
* failed for me."
|
||||
*
|
||||
* "I'm not sure if that's required for other situations or not, so I
|
||||
* added a #define to optionally elide the call to cement the
|
||||
* configuration. My PHY appears to be happy with this for the moment."
|
||||
* REVISIT: Revisit the following CONFIG_PHY_CEMENT_DISABLE work-around.
|
||||
* It is should not needed if CONFIG_PHY_AUTONEG is defined and is known
|
||||
* cause a problem for at least one PHY (DP83848I PHY). It might be
|
||||
* safe just to remove this elided coded for all PHYs.
|
||||
*/
|
||||
|
||||
#ifndef CONFIG_PHY_CEMENT_DISABLE
|
||||
|
||||
Reference in New Issue
Block a user