Re: [PATCH 0/3] Rockchip IOMMU driver and devicetree bindings

From: Heiko Stübner
Date: Thu Oct 23 2014 - 12:20:17 EST


Am Freitag, 24. Oktober 2014, 00:11:06 schrieb Daniel Kurtz:
> On Wed, Oct 22, 2014 at 11:20 PM, Joerg Roedel <joro@xxxxxxxxxx> wrote:
> > On Wed, Oct 01, 2014 at 06:20:40PM +0800, Daniel Kurtz wrote:
> > > Add a driver and devicetree bindings for the IOMMU found in Rockchip
> >
> > RK3288
> >
> > > SoCs.
> > >
> > > Daniel Kurtz (3):
> > > iommu/rockchip: rk3288 iommu driver
> > > dt-bindings: iommu: Add documentation for rockchip iommu
> > > ARM: dts: rk3288: add VOP iommu nodes
> > >
> > > .../devicetree/bindings/iommu/rockchip,iommu.txt | 26 +
> > > arch/arm/boot/dts/rk3288.dtsi | 18 +
> > > drivers/iommu/Kconfig | 11 +
> > > drivers/iommu/Makefile | 1 +
> > > drivers/iommu/rockchip-iommu.c | 924
> >
> > +++++++++++++++++++++
> >
> > > 5 files changed, 980 insertions(+)
> > > create mode 100644
> >
> > Documentation/devicetree/bindings/iommu/rockchip,iommu.txt
> >
> > > create mode 100644 drivers/iommu/rockchip-iommu.c
> >
> > Please make sure you also get Acks for the DT bindings. With that and
> > the issue I mentioned in the other email fixed, I will apply these
> > patches?
>
> Hi Joerg,
>
> Thanks for your reviews.
> Can you help me get these DT bindings Acked?
> AFAICT, the DT maintainers are CC'ed on this patch, I'm not sure what else
> to do.

I think the generally accepted way is, to give the dt maintainers an
appropriate amount of time [0] to complain and assume an implicit ack if no
complaint arrives.

This is especially true, as the dt-binding for the iommu itself does not even
introduce any new properties and simply uses widely established ones.


As a sidenote ... I'd like to take the dts changes in patch 3, once Joerg
takes patches 1 and 2.


Heiko



[0] 3 weeks seems to be the most widely used timespan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/