From: Frank Rowand <frowand.list@gmail.com>
To: Sricharan R <sricharan@codeaurora.org>,
Rob Herring <robh+dt@kernel.org>,
Robin Murphy <Robin.Murphy@arm.com>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] of: change fixup of dma-ranges size to error
Date: Mon, 10 Apr 2017 04:59:05 -0700 [thread overview]
Message-ID: <58EB7389.7020603@gmail.com> (raw)
In-Reply-To: <19a2a28f-8338-970e-3b5f-05be3362fb9a@codeaurora.org>
On 04/10/17 04:48, Sricharan R wrote:
> Hi Frank,
>
> <snip..>
>
>>>> Can we get back to the basic premise of the proposed patch?
>>>>
>>>> The current code in of_dma_configure() contains a hack that allows the
>>>> dma-ranges property to specify a mask instead of a size. The binding
>>>> in the specification allows a size and does not allow a mask.
>>>>
>>>> The hack was added to account for one or more dts files that did not
>>>> follow the specification. In the mail list discussion of the hack
>>>> you said "Also, we need a WARN here so DTs get fixed."
>>>>
>>>> The hack was first present in Linux 4.1. The only in-tree dts that
>>>> incorrectly contained a mask instead of a size in dma-ranges was
>>>> arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi
>>>>
>>>> That .dtsi was fixed by
>>>> commit c91cb9123cdd ("dtb: amd: Fix DMA ranges in device tree")
>>>> The fix was present in Linux 4.6, May 15, 2016.
>>>>
>>>> I would like to remove the hack. I think that enough time has
>>>> elapsed to allow this change.
>>>
>>> If we have no cases of what I'm concerned about, then removing it is
>>> fine. Is this a dependency for iommu series? Doesn't look like it to
>>> me.
>>
>> This patch is a replacement for patch 03/12 in the iommu series. I
>> think that patch 03/12 of the iommu series could be dropped and my
>> patch could be applied independently of the iommu series.
>>
>> There is likely a conflict between my patch and patch 06/12 of the
>> iommu series because in my patch the first line of the patch chunk
>> of drivers/of/device.c includes a line that is changed in 06/12
>> of the iommu series. If this is the case then the iommu series
>> should take precedence over my patch (and I should subsequently
>> fixup my patch).
>>
>
> Ok, for which i just posted a V11 [1] with patch 03/12 from
> V10 dropped.
>
> [1] https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1373638.html
>
> Regards,
> Sricharan
>
Thanks. I'll revisit this patch after the iommu series gets merged.
-Frank
next prev parent reply other threads:[~2017-04-10 11:59 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-06 6:18 [PATCH] of: change fixup of dma-ranges size to error frowand.list
2017-04-06 14:03 ` Rob Herring
2017-04-06 18:37 ` Frank Rowand
2017-04-06 22:41 ` Rob Herring
2017-04-07 5:18 ` Frank Rowand
2017-04-07 17:09 ` Rob Herring
2017-04-07 23:26 ` Frank Rowand
2017-04-10 11:48 ` Sricharan R
2017-04-10 11:59 ` Frank Rowand [this message]
2017-04-10 13:11 ` Robin Murphy
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=58EB7389.7020603@gmail.com \
--to=frowand.list@gmail.com \
--cc=Robin.Murphy@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=sricharan@codeaurora.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).