From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Fri, 20 Nov 2015 23:32:11 +0100 Subject: [U-Boot] [PATCH v2] usb: add support for generic EHCI devices In-Reply-To: <1448054913.2964.19.camel@synopsys.com> References: <1447860381-2645-1-git-send-email-abrodkin@synopsys.com> <201511202148.05092.marex@denx.de> <1448054913.2964.19.camel@synopsys.com> Message-ID: <201511202332.12008.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 Friday, November 20, 2015 at 10:28:34 PM, Alexey Brodkin wrote: > Hi Marek, Hi! > On Fri, 2015-11-20 at 21:48 +0100, Marek Vasut wrote: > > On Wednesday, November 18, 2015 at 04:26:21 PM, Alexey Brodkin wrote: > > > This driver is meant to be used with any EHCI-compatible host > > > controller in case if there's no need for platform-specific > > > glue such as setup of controller or PHY's power mode via > > > GPIOs etc. > > > > > > Signed-off-by: Alexey Brodkin > > > Reviewed-by: Simon Glass > > > Reviewed-by: Marek Vasut > > > Cc: Stephen Warren > > > --- > > > > > > Changes compared to v1: > > > * Updated commit message with removal of Synopsys board mention > > > * Cleaned-up ehci_usb_remove() > > > > [...] > > > > > +static int ehci_usb_probe(struct udevice *dev) > > > +{ > > > + struct ehci_hccr *hccr = (struct ehci_hccr *)dev_get_addr(dev); > > > + struct ehci_hcor *hcor; > > > + > > > + hcor = (struct ehci_hcor *)((uint32_t)hccr + > > > > This should be uintptr_t for the sake of 64bit systems, no ? > > Hm, that's a good point! > Indeed I was only thinking about 32-bit systems that I work with. You work for hardware design company, no ? I'd expect you guys thing about 256byte long cachelines and other such stuff :-) > So please do that change. > > What's interesting most of other USB drivers do use "uint32_t" so > there's a room for improvement it seems :) Patches are welcome ;-) Besides, please use u32 instead of uint32_t. > -Alexey Best regards, Marek Vasut