Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933796AbbGHVPP (ORCPT ); Wed, 8 Jul 2015 17:15:15 -0400 Received: from vps0.lunn.ch ([178.209.37.122]:36615 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758794AbbGHVPJ (ORCPT ); Wed, 8 Jul 2015 17:15:09 -0400 Date: Wed, 8 Jul 2015 23:08:51 +0200 From: Andrew Lunn To: Vivien Didelot Cc: netdev@vger.kernel.org, "David S. Miller" , Guenter Roeck , linux-kernel@vger.kernel.org, kernel@savoirfairelinux.com Subject: Re: [PATCH 2/2] net: dsa: mv88e6xxx: add SERDES registers to debugfs Message-ID: <20150708210851.GI1357@lunn.ch> References: <1436387779-23407-1-git-send-email-vivien.didelot@savoirfairelinux.com> <1436387779-23407-2-git-send-email-vivien.didelot@savoirfairelinux.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1436387779-23407-2-git-send-email-vivien.didelot@savoirfairelinux.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 855 Lines: 21 On Wed, Jul 08, 2015 at 04:36:19PM -0400, Vivien Didelot wrote: > Add read and write access to a SERDES column in the regs debugfs file > for the Fiber/SERDES registers. Humm. I don't really like this. These SERDES registers are in a complete different address space to the port, globalX registers. I would suggest adding a new file, 'phy', which allows access to all the phy registers, and SERDES at 0x0f. This also makes it simpler to deal with devices which don't have indirect access to the PHYs via Global 2 offset 0x18 and 0x19. For such chips, the debugfs file should not be created. Thanks Andrew -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/