Re: [RFC v2 0/7] pch_can/c_can: fix races and add PCH support to c_can

[Alexander Stein <alexander.stein@systec-electronic.com>]

Hello Wolfgang,

On Thursday 06 December 2012 15:31:20, Wolfgang Grandegger wrote:
> > diff --git a/drivers/net/can/c_can/c_can_pci.c b/drivers/net/can/c_can/c_can_pci.c
> > index 2516ea9..0ac4d43 100644
> > --- a/drivers/net/can/c_can/c_can_pci.c
> > +++ b/drivers/net/can/c_can/c_can_pci.c
> > @@ -74,13 +74,37 @@ static void c_can_pci_write_reg_aligned_to_32bit(struct c_can_priv *priv,
> >  static u16 c_can_pci_read_reg_32bit(struct c_can_priv *priv,
> >                                     enum reg index)
> >  {
> > -       return (u16)ioread32(priv->base + 2 * priv->regs[index]);
> > +       unsigned long flags;
> > +       u16 reg;
> > +
> > +       spin_lock_irqsave(&priv->testlock, flags);
> > +       reg = (u16)ioread32(priv->base + 2 * priv->regs[index]);
> > +       spin_unlock_irqrestore(&priv->testlock, flags);
> > +
> > +       return reg;
> >  }
> >  
> >  static void c_can_pci_write_reg_32bit(struct c_can_priv *priv,
> >                                       enum reg index, u16 val)
> >  {
> > -       iowrite32((u32)val, priv->base + 2 * priv->regs[index]);
> > +       u16 reg;
> > +       unsigned long flags;
> > +       int retries;
> > +
> > +       retries = 0;
> > +
> > +       spin_lock_irqsave(&priv->testlock, flags);
> > +
> > +       do
> > +       {
> > +               iowrite32((u32)val, priv->base + 2 * priv->regs[index]);
> 
> I think it's enough to write the message once.

I can reply to this one immediatly. Not it is not sufficent to write once. This was my 1st implementation (iowrite32 just above the do) and noticed in case the write doesn't succeed I won't succeed until all ioread32 have passed. IMO I must write again if it failed before.

Alexander