On Wed, May 16, 2012 at 03:41:17PM +0200, Marek Vasut wrote: > Dear Felipe Balbi, > > > On Tue, May 15, 2012 at 10:23:37AM +0200, Marek Vasut wrote: > > > This codes connects the CI13xxx driver with the core found inside of > > > the Freescale i.MX244/i.MX28 CPU. > > > > > > Signed-off-by: Marek Vasut <[email protected]> > > > Cc: Alan Stern <[email protected]> > > > Cc: Detlev Zundel <[email protected]> > > > CC: Dong Aisheng <[email protected]> > > > CC: Fabio Estevam <[email protected]> > > > Cc: Felipe Balbi <[email protected]> > > > Cc: Greg Kroah-Hartman <[email protected]> > > > Cc: Linux ARM kernel <[email protected]> > > > Cc: [email protected] > > > CC: Sascha Hauer <[email protected]> > > > CC: Shawn Guo <[email protected]> > > > Cc: Stefano Babic <[email protected]> > > > CC: Uwe Kleine-König <[email protected]> > > > Cc: Wolfgang Denk <[email protected]> > > > Cc: Wolfram Sang <[email protected]> > > > --- > > > > > > drivers/usb/chipidea/Makefile | 4 + > > > drivers/usb/chipidea/ci13xxx_mxs.c | 201 > > > ++++++++++++++++++++++++++++++++++++ 2 files changed, 205 insertions(+) > > > create mode 100644 drivers/usb/chipidea/ci13xxx_mxs.c > > > > > > diff --git a/drivers/usb/chipidea/Makefile > > > b/drivers/usb/chipidea/Makefile index cc34937..266d57d 100644 > > > --- a/drivers/usb/chipidea/Makefile > > > +++ b/drivers/usb/chipidea/Makefile > > > @@ -12,3 +12,7 @@ endif > > > > > > ifneq ($(CONFIG_ARCH_MSM),) > > > > > > obj-$(CONFIG_USB_CHIPIDEA) += ci13xxx_msm.o > > > > > > endif > > > > > > + > > > +ifneq ($(CONFIG_ARCH_MXS),) > > > + obj-$(CONFIG_USB_CHIPIDEA) += ci13xxx_mxs.o > > > +endif > > > > NAK... let it compile everywhere. > > I just followed the pattern here, honestly I think this is quite OK so far. > Further improvements can be made indeed.
why introduce something we know will change ? Just make sure your code compiles everywhere. What's already there can be changed afterwards. > > > + *data->pdev->dev.dma_mask = DMA_BIT_MASK(32); > > > + data->pdev->dev.coherent_dma_mask = DMA_BIT_MASK(32); > > > > why don't you just point to the parent's ? > > Good idea, thanks :) no problem ;-) -- balbi
signature.asc
Description: Digital signature
