[PATCH V14 3/4] ptp: Added a clock driver for the IXP46x.
    Richard Cochran 
    richardcochran at gmail.com
       
    Mon Apr 18 18:06:07 EST 2011
    
    
  
On Mon, Apr 18, 2011 at 08:56:03AM +0200, Arnd Bergmann wrote:
> On Monday 18 April 2011, Richard Cochran wrote:
> > +
> > +       lo = __raw_readl(®s->channel[ch].src_uuid_lo);
> > +       hi = __raw_readl(®s->channel[ch].src_uuid_hi);
> > +
> 
> I guess you should use readl(), not __raw_readl() here. The __raw_* functions
> are not meant for device drivers.
Krzysztof had a different opinion about this.
    https://lkml.org/lkml/2011/1/8/67
Anyway, it is his driver, and I just followed what he does elsewhere
in the driver. It make sense to me to keep the driver consistent.
Maybe we should make the change throughout?
Thanks,
Richard
    
    
More information about the devicetree-discuss
mailing list