RE: [PATCHv8 1/4] pwm: Add Freescale FTM PWM driver support

2014-01-05 Thread li.xi...@freescale.com
Hi Arnd, > > Adding Greg here to make sure we have a common understanding in the future. > > ioread*() was indeed introduced for devices that can be both I/O and memory > space, and is a bit slowed than readl() on architectures that don't have > memory mapped I/O space (i.e. x86). > > However,

Re: [PATCHv8 1/4] pwm: Add Freescale FTM PWM driver support

2014-01-03 Thread Arnd Bergmann
On Friday 03 January 2014, li.xi...@freescale.com wrote: > > Subject: Re: [PATCHv8 1/4] pwm: Add Freescale FTM PWM driver support > > > > On Thursday 02 January 2014 17:57:01 Xiubo Li wrote: > > > +static inline u32 ftm_readl(bool big_endian, const void __iomem *addr)

RE: [PATCHv8 1/4] pwm: Add Freescale FTM PWM driver support

2014-01-03 Thread li.xi...@freescale.com
Hi Arnd, > Subject: Re: [PATCHv8 1/4] pwm: Add Freescale FTM PWM driver support > > On Thursday 02 January 2014 17:57:01 Xiubo Li wrote: > > +static inline u32 ftm_readl(bool big_endian, const void __iomem *addr) > > +{ > > + u32 val; > > + &

Re: [PATCHv8 1/4] pwm: Add Freescale FTM PWM driver support

2014-01-02 Thread Arnd Bergmann
On Thursday 02 January 2014 17:57:01 Xiubo Li wrote: > +static inline u32 ftm_readl(bool big_endian, const void __iomem *addr) > +{ > + u32 val; > + > + val = __raw_readl(addr); > + > + if (likely(big_endian)) > + val = be32_to_cpu((__force __be32)val); > + els