Re: [PATCH v2 09/12] intel-ipu3: css hardware setup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Jun 17, 2017 at 01:54:51AM +0300, Andy Shevchenko wrote:
> On Thu, Jun 15, 2017 at 1:19 AM, Yong Zhi <yong.zhi@xxxxxxxxx> wrote:
> 
> Commit message.
> 
> > Signed-off-by: Yong Zhi <yong.zhi@xxxxxxxxx>
> 
> > +static void writes(void *mem, ssize_t len, void __iomem *reg)
> > +{
> > +       while (len >= 4) {
> > +               writel(*(u32 *)mem, reg);
> > +               mem += 4;
> > +               reg += 4;
> > +               len -= 4;
> > +       }
> > +}
> 
> Again, I just looked into patches and first what I see is reinventing the wheel.
> 
> memcpy_toio()

Hi Andy,

That doesn't quite work: the hardware only supports 32-bit access.

So the answer is writesl().

-- 
Sakari Ailus
e-mail: sakari.ailus@xxxxxx	XMPP: sailus@xxxxxxxxxxxxxx



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux