From mboxrd@z Thu Jan  1 00:00:00 1970
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: Fri, 28 Feb 2014 15:48:21 +0000
Subject: Re: [PATCH 1/2] iommu: Add driver for Renesas VMSA-compatible IPMMU
Message-Id: <7233184.zvM0XIByjW@avalon>
List-Id: <linux-sh.vger.kernel.org>
References: <1393601099-4413-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <1393601099-4413-2-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <5310AD1C.1050101@codethink.co.uk>
In-Reply-To: <5310AD1C.1050101@codethink.co.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: Ben Dooks <ben.dooks@codethink.co.uk>
Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>, iommu@lists.linux-foundation.org, linux-sh@vger.kernel.org

Hi Ben,

On Friday 28 February 2014 15:37:00 Ben Dooks wrote:
> only minor issue so far:
> > diff --git a/include/linux/platform_data/ipmmu-vmsa.h
> > b/include/linux/platform_data/ipmmu-vmsa.h new file mode 100644
> > index 0000000..8be27e0
> > --- /dev/null
> > +++ b/include/linux/platform_data/ipmmu-vmsa.h
> > @@ -0,0 +1,24 @@
> > +/*
> > + * IPMMU VMSA Platform Data
> > + *
> > + * Copyright (C) 2014 Reneses Electronics Coporation
> 
> typo

That should be easy to fix :-)

> > + * This program is free software; you can redistribute it and/or modify
> > + * it under the terms of the GNU General Public License as published by
> > + * the Free Software Foundation; version 2 of the License.
> > + */
> > +
> > +#ifndef __IPMMU_VMSA_H__
> > +#define __IPMMU_VMSA_H__
> > +
> > +struct ipmmu_vmsa_master {
> > +	const char *name;
> > +	unsigned int utlb;
> > +};
> > +
> > +struct ipmmu_vmsa_platform_data {
> > +	const struct ipmmu_vmsa_master *masters;
> > +	unsigned int num_masters;
> > +};
> > +
> > +#endif /* __IPMMU_VMSA_H__ */
> 
> Also, what about the of-bindings?

That's planned and will come in a second step. I was first trying to get the 
IPMMU working.

-- 
Regards,

Laurent Pinchart