From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Date: Sun, 22 Nov 2015 10:09:51 -0700 Subject: [U-Boot] [PATCH 2/2] usb: eth: add Realtek RTL8152B/RTL8153 driver In-Reply-To: References: <1447430581-8805-1-git-send-email-swarren@wwwdotorg.org> <201511191412.45730.marex@denx.de> <201511201838.20549.marex@denx.de> <56501CD6.6050700@wwwdotorg.org> Message-ID: <5651F6DF.6000807@wwwdotorg.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 11/21/2015 09:50 AM, Simon Glass wrote: > Hi, > > On 21 November 2015 at 00:27, Stephen Warren wrote: >> On 11/20/2015 10:38 AM, Marek Vasut wrote: >>> On Friday, November 20, 2015 at 06:35:41 PM, Simon Glass wrote: >>>> Hi, >>> >>> Hi, >>> >>>> On 19 November 2015 at 06:12, Marek Vasut wrote: >>>>> On Thursday, November 19, 2015 at 01:49:16 PM, Anand Moon wrote: >>>>>> Hi Marek, >>>>> >>>>> Hi! >>>>> >>>>> [...] >>>>> >>>>>> Sorry for the mess: Please accept my apology, I will not in the future. >>>>> >>>>> Cool, thanks! :) >>>> >>>> This driver should use driver model, right? >>> >>> Of course. >> >> Hopefully it can support both DM and non-DM? > > Why support non-DM? Isn't that just going to slow us down? I don't think DM is activated everywhere for Ethernet yet is it? I can't see why it'd slow us down. The non-DM code should be quite static, and only have any effect on DM conversion whenever we finally pull the trigger to remove it. It shouldn't be any kind of maintenance issue in the interim.