From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Mon, 9 Feb 2015 20:18:52 +0100 Subject: [U-Boot] [PATCH 2/2] net: phy: Add ability to program the ksz9031 skew values from the uboot env In-Reply-To: <1423493073-9462-3-git-send-email-vbridger@opensource.altera.com> References: <1423493073-9462-1-git-send-email-vbridger@opensource.altera.com> <1423493073-9462-3-git-send-email-vbridger@opensource.altera.com> Message-ID: <201502092018.52128.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Monday, February 09, 2015 at 03:44:33 PM, Vince Bridgers wrote: > This patch adds the ability for the Micrel ksz9031 PHY driver to detect > skew settings from the UBoot environment and program those values at PHY > initialization time. This is a first step towards setting these values from > the UBoot environment or the Micrel PHY devicetree once supported. The skew > values may be negative or positive, therefore depends on UBoot environment > support to read positive or negative values in units of picoseconds. > > Signed-off-by: Vince Bridgers Hi! We already do this kind of a programming in board/altera/socfpga/socfpga.c in board_phy_config(), don't we ? Also, see [1], once I apply this, the DT support (not DM) for SoCFPGA will become mandatory. Won't it make more sense to pull these values from the DT instead of poluting the board environment with those please ? Thanks! [1] http://git.denx.de/?p=u-boot/u-boot- socfpga.git;a=shortlog;h=refs/heads/topic/vip-arriav Best regards, Marek Vasut