On 03/10/17 23:35, Dmitry Osipenko wrote:
> Currently fuse driver requests DMA channel from an arbitrary DMA device,
> it is not a problem since there is only one DMA provider for Tegra20 yet,
> but it will become troublesome once another provider will be added.
>
> Signed-off-by: Dmitry Osipenko <digetx@xxxxxxxxx>
> ---
> drivers/soc/tegra/fuse/fuse-tegra20.c | 12 +++++++++++-
> 1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/soc/tegra/fuse/fuse-tegra20.c b/drivers/soc/tegra/fuse/fuse-tegra20.c
> index a33f48c06771..e9e127a24c70 100644
> --- a/drivers/soc/tegra/fuse/fuse-tegra20.c
> +++ b/drivers/soc/tegra/fuse/fuse-tegra20.c
> @@ -37,6 +37,11 @@
> #define FUSE_UID_LOW 0x08
> #define FUSE_UID_HIGH 0x0c
>
> +static const struct of_device_id apbdma_of_match[] = {
> + { .compatible = "nvidia,tegra20-apbdma" },
> + { },
> +};
> +
> static u32 tegra20_fuse_read_early(struct tegra_fuse *fuse, unsigned int offset)
> {
> return readl_relaxed(fuse->base + FUSE_BEGIN + offset);
> @@ -96,6 +101,11 @@ static u32 tegra20_fuse_read(struct tegra_fuse *fuse, unsigned int offset)
> return value;
> }
>
> +static bool dma_filter(struct dma_chan *chan, void *filter_param)
> +{
> + return of_match_device(apbdma_of_match, chan->device->dev) != NULL;
Is it simpler just to use of_device_is_compatible() here?
Cheers
Jon
--
nvpublic
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
