From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 10 Oct 2013 02:42:40 +0200 Subject: [U-Boot] [Patch v1 1/1] usb:smsx95xx LED activity for USB net driver In-Reply-To: <63F59710-432C-42D8-8571-48E7C5993F75@gmail.com> References: <1381203058-4588-1-git-send-email-suriyan.r@gmail.com> <201310100239.24877.marex@denx.de> <63F59710-432C-42D8-8571-48E7C5993F75@gmail.com> Message-ID: <201310100242.40989.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 Dear Joe Hershberger, > On Oct 9, 2013, at 6:39 PM, Marek Vasut wrote: > > Dear Simon Glass, > > > >> +Joe (net maintainer) > >> +Marek (USB) > >> > >> Hi Suriyan, > >> > >> On Mon, Oct 7, 2013 at 9:30 PM, Suriyan Ramasami wrote: > >>> Add LED activity for SMSX95XX USB Ether driver. > >>> > >>> Signed-off-by: ?Suriyan Ramasami" > >>> --- > >>> > >>> drivers/usb/eth/smsc95xx.c | 14 ++++++++++++++ > >>> 1 files changed, 14 insertions(+), 0 deletions(-) > >>> > >>> diff --git a/drivers/usb/eth/smsc95xx.c b/drivers/usb/eth/smsc95xx.c > >>> index 15fd9a9..7bf0a34 100644 > >>> --- a/drivers/usb/eth/smsc95xx.c > >>> +++ b/drivers/usb/eth/smsc95xx.c > >>> @@ -14,6 +14,12 @@ > >>> > >>> /* SMSC LAN95xx based USB 2.0 Ethernet Devices */ > >>> > >>> +/* LED defines */ > >>> +#define LED_GPIO_CFG (0x24) > >>> +#define LED_GPIO_CFG_SPD_LED (0x01000000) > >>> +#define LED_GPIO_CFG_LNK_LED (0x00100000) > >>> +#define LED_GPIO_CFG_FDX_LED (0x00010000) > >>> + > >>> > >>> /* Tx command words */ > >>> #define TX_CMD_A_FIRST_SEG_ 0x00002000 > >>> #define TX_CMD_A_LAST_SEG_ 0x00001000 > >>> > >>> @@ -591,6 +597,14 @@ static int smsc95xx_init(struct eth_device *eth, > >>> bd_t *bd) > >>> > >>> return ret; > >>> > >>> debug("ID_REV = 0x%08x\n", read_buf); > >>> > >>> + /* Configure GPIO pins as LED outputs */ > >>> + write_buf = LED_GPIO_CFG_SPD_LED | LED_GPIO_CFG_LNK_LED | > >>> + LED_GPIO_CFG_FDX_LED; > >>> + ret = smsc95xx_write_reg(dev, LED_GPIO_CFG, write_buf); > >>> + if (ret < 0) > >>> + return ret; > >>> + debug("LED_GPIO_CFG set\n"); > >>> + > >> > >> Seems good. > >> > >> Acked-by: Simon Glass > > > > Applied to usb/next > > > > I kinda wonder if Joe shouldn't pick this one. Is Joe still active ? > > Sure... When I need to be. Heh :) Haven't seen you on IRC for a while (and you surely missed a lot, applied for sjg too). I picked this patch, so no problem here. Best regards, Marek Vasut