On Kha, 2015-10-08 at 22:05 +0200, Corentin LABBE wrote:
> > This patch adds Altera PCIe MSI driver. This soft IP supports configurable
> > number of vectors, which is a dts parameter.
> >
> > Signed-off-by: Ley Foon Tan <lf...@altera.com>
> > Reviewed-by: Marc Zyngier <marc.zyng...@arm.com>
> > +
> > +static inline void msi_writel(struct altera_msi *msi, u32 value, u32 reg)
> > +{
> > + writel_relaxed(value, msi->csr_base + reg);
> > +}
> > +
> > +static inline u32 msi_readl(struct altera_msi *msi, u32 reg)
> > +{
> > + return readl_relaxed(msi->csr_base + reg);
> > +}
> > +
>
> You could set value and reg parameter as const
Noted.
>
> > +
> > +static int altera_irq_domain_alloc(struct irq_domain *domain, unsigned int
> > virq,
> > + unsigned int nr_irqs, void *args)
> > +{
> > + struct altera_msi *msi = domain->host_data;
> > + unsigned long bit;
> > + u32 mask;
> > +
> > + WARN_ON(nr_irqs != 1);
> > + mutex_lock(&msi->lock);
> > +
> > + bit = find_first_zero_bit(msi->used, msi->num_of_vectors);
> > + if (bit >= msi->num_of_vectors)
> > + return -ENOSPC;
> > +
> > + set_bit(bit, msi->used);
> > +
> > + mutex_unlock(&msi->lock);
> > +
> > + irq_domain_set_info(domain, virq, bit, &altera_msi_bottom_irq_chip,
> > + domain->host_data, handle_simple_irq,
> > + NULL, NULL);
> > +
> > + mask = msi_readl(msi, MSI_INTMASK);
> > + mask |= 1 << bit;
> > + msi_writel(msi, mask, MSI_INTMASK);
> > +
> > + return 0;
> > +}
>
> You do not unlock the mutex when returning -ENOSPC
> And again some parameter could be set as const
Good catch, will fix that.
Do you mean add the const for altera_irq_domain_alloc()?
It is defined by the struct irq_domain_ops.
Regards
Ley Foon
________________________________
Confidentiality Notice.
This message may contain information that is confidential or otherwise
protected from disclosure. If you are not the intended recipient, you are
hereby notified that any use, disclosure, dissemination, distribution, or
copying of this message, or any attachments, is strictly prohibited. If you
have received this message in error, please advise the sender by reply e-mail,
and delete the message and any attachments. Thank you.
