From: Christoph Hellwig <hch@infradead•org>
To: Tom Murphy <murphyt7@tcd•ie>
Cc: Heiko Stuebner <heiko@sntech•de>,
virtualization@lists•linux-foundation.org,
Matthias Brugger <matthias.bgg@gmail•com>,
Thierry Reding <thierry.reding@gmail•com>,
Will Deacon <will@kernel•org>,
Jean-Philippe Brucker <jean-philippe@linaro•org>,
linux-samsung-soc@vger•kernel.org,
Krzysztof Kozlowski <krzk@kernel•org>,
Jonathan Hunter <jonathanh@nvidia•com>,
linux-rockchip@lists•infradead.org, Kukjin Kim <kgene@kernel•org>,
Andy Gross <agross@kernel•org>,
linux-s390@vger•kernel.org,
Gerald Schaefer <gerald.schaefer@de•ibm.com>,
linux-arm-msm@vger•kernel.org,
linux-mediatek@lists•infradead.org, linux-tegra@vger•kernel.org,
linux-arm-kernel@lists•infradead.org,
David Woodhouse <dwmw2@infradead•org>,
linux-kernel@vger•kernel.org, iommu@lists•linux-foundation.org,
Robin Murphy <robin.murphy@arm•com>
Subject: Re: [PATCH V5 3/5] iommu/dma-iommu: Handle deferred devices
Date: Tue, 20 Aug 2019 02:43:06 -0700 [thread overview]
Message-ID: <20190820094306.GC24154@infradead.org> (raw)
In-Reply-To: <20190815110944.3579-4-murphyt7@tcd.ie>
> +static int handle_deferred_device(struct device *dev,
> + struct iommu_domain *domain)
Nitick: we usually use double tab indents (or indents to after
the opening brace) for multi-line prototyped.
> + if (!is_kdump_kernel())
> + return 0;
> +
> + if (unlikely(ops->is_attach_deferred &&
> + ops->is_attach_deferred(domain, dev)))
> + return iommu_attach_device(domain, dev);
And for multi-line conditionals we also use two-tab indents.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists•infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-08-20 9:43 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-15 11:09 [PATCH V5 0/5] iommu/amd: Convert the AMD iommu driver to the dma-iommu api Tom Murphy
2019-08-15 11:09 ` [PATCH V5 1/5] iommu/amd: Remove unnecessary locking from AMD iommu driver Tom Murphy
2019-08-20 9:41 ` Christoph Hellwig
2019-08-24 7:56 ` Tom Murphy
2019-08-24 22:41 ` Christoph Hellwig
2019-08-15 11:09 ` [PATCH V5 2/5] iommu: Add gfp parameter to iommu_ops::map Tom Murphy
2019-08-19 18:23 ` Robin Murphy
2019-08-20 9:41 ` Christoph Hellwig
2019-08-15 11:09 ` [PATCH V5 3/5] iommu/dma-iommu: Handle deferred devices Tom Murphy
2019-08-19 18:26 ` Robin Murphy
2019-08-20 9:43 ` Christoph Hellwig [this message]
2019-08-15 11:09 ` [PATCH V5 4/5] iommu/dma-iommu: Use the dev->coherent_dma_mask Tom Murphy
2019-08-19 18:39 ` Robin Murphy
2019-08-20 9:43 ` Christoph Hellwig
2019-08-15 11:09 ` [PATCH V5 5/5] iommu/amd: Convert AMD iommu driver to the dma-iommu api Tom Murphy
2019-08-17 3:39 ` [PATCH V5 3/5] iommu/dma-iommu: Handle deferred devices Hillf Danton
2019-08-17 7:19 ` Tom Murphy
2019-09-05 6:18 ` [PATCH V5 0/5] iommu/amd: Convert the AMD iommu driver to the dma-iommu api Christoph Hellwig
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190820094306.GC24154@infradead.org \
--to=hch@infradead$(echo .)org \
--cc=agross@kernel$(echo .)org \
--cc=dwmw2@infradead$(echo .)org \
--cc=gerald.schaefer@de$(echo .)ibm.com \
--cc=heiko@sntech$(echo .)de \
--cc=iommu@lists$(echo .)linux-foundation.org \
--cc=jean-philippe@linaro$(echo .)org \
--cc=jonathanh@nvidia$(echo .)com \
--cc=kgene@kernel$(echo .)org \
--cc=krzk@kernel$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
--cc=linux-arm-msm@vger$(echo .)kernel.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-mediatek@lists$(echo .)infradead.org \
--cc=linux-rockchip@lists$(echo .)infradead.org \
--cc=linux-s390@vger$(echo .)kernel.org \
--cc=linux-samsung-soc@vger$(echo .)kernel.org \
--cc=linux-tegra@vger$(echo .)kernel.org \
--cc=matthias.bgg@gmail$(echo .)com \
--cc=murphyt7@tcd$(echo .)ie \
--cc=robin.murphy@arm$(echo .)com \
--cc=thierry.reding@gmail$(echo .)com \
--cc=virtualization@lists$(echo .)linux-foundation.org \
--cc=will@kernel$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox