Re: [PATCH v4 11/20] memory: tegra: Use of_device_get_match_data()
From: Thierry Reding <hidden>
Date: 2018-09-24 10:14:01
Also in:
linux-devicetree, linux-iommu, lkml
On Mon, Sep 24, 2018 at 03:41:44AM +0300, Dmitry Osipenko wrote:
quoted hunk ↗ jump to hunk
There is no need to match device with the DT node since it was already matched, use of_device_get_match_data() helper to get the match-data. Signed-off-by: Dmitry Osipenko <digetx@gmail.com> --- drivers/memory/tegra/mc.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-)diff --git a/drivers/memory/tegra/mc.c b/drivers/memory/tegra/mc.c index 5454ffe5b2e0..cdc33f93cf7c 100644 --- a/drivers/memory/tegra/mc.c +++ b/drivers/memory/tegra/mc.c@@ -11,8 +11,7 @@ #include <linux/interrupt.h> #include <linux/kernel.h> #include <linux/module.h> -#include <linux/of.h> -#include <linux/platform_device.h>
It's better not to remove these two because the code still uses functions declared in them. If ever we were going to remove code using linux/of_device.h and then remove the linux/of_device.h include, we'd break the build and have to reintroduce the includes. The same would happen if linux/of_device.h were ever to stop including linux/platform_device.h or linux/of.h. That may sound unlikely, but it has happened in the past with other includes. It can also happen that some restructuring takes place in some headers that is not so obvious and then things can still start falling apart miles away. Thierry
Attachments
- signature.asc [application/pgp-signature] 833 bytes