Re: [PATCH v4 2/7] media: s5p-mfc: use generic reserved memory bindings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hello Marek,

On 05/27/2016 02:37 AM, Marek Szyprowski wrote:
> Hello,
> 
> 
> On 2016-05-25 19:36, Rob Herring wrote:
>> On Wed, May 25, 2016 at 11:18:59AM -0400, Javier Martinez Canillas wrote:
>>> Hello Marek,
>>>
>>> On 05/24/2016 09:31 AM, Marek Szyprowski wrote:
>>>> Use generic reserved memory bindings and mark old, custom properties
>>>> as obsoleted.
>>>>
>>>> Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
>>>> ---
>>>>   .../devicetree/bindings/media/s5p-mfc.txt          | 39 +++++++++++++++++-----
>>>>   1 file changed, 31 insertions(+), 8 deletions(-)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt b/Documentation/devicetree/bindings/media/s5p-mfc.txt
>>>> index 2d5787e..92c94f5 100644
>>>> --- a/Documentation/devicetree/bindings/media/s5p-mfc.txt
>>>> +++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt
>>>> @@ -21,15 +21,18 @@ Required properties:
>>>>     - clock-names : from common clock binding: must contain "mfc",
>>>>             corresponding to entry in the clocks property.
>>>>   -  - samsung,mfc-r : Base address of the first memory bank used by MFC
>>>> -            for DMA contiguous memory allocation and its size.
>>>> -
>>>> -  - samsung,mfc-l : Base address of the second memory bank used by MFC
>>>> -            for DMA contiguous memory allocation and its size.
>>>> -
>>>>   Optional properties:
>>>>     - power-domains : power-domain property defined with a phandle
>>>>                  to respective power domain.
>>>> +  - memory-region : from reserved memory binding: phandles to two reserved
>>>> +    memory regions, first is for "left" mfc memory bus interfaces,
>>>> +    second if for the "right" mfc memory bus, used when no SYSMMU
>>>> +    support is available
>>>> +
>>>> +Obsolete properties:
>>>> +  - samsung,mfc-r, samsung,mfc-l : support removed, please use memory-region
>>>> +    property instead
>>>> +
>>>>
>>> I wonder if we should maintain backward compatibility for this driver
>>> since s5p-mfc memory allocation won't work with an old FDT if support
>>> for the old properties are removed.
>> Well, minimally the commit log should indicate that compatibility is
>> being broken.
> 
> Compatibility is only partially broken. I add this to the commit message. Old
> bindings will still work with the new driver when IOMMU is enabled - in such case reserved
> memory regions are ignored so this should not be a big issue. Using IOMMU also increases
> total memory space for the video buffers without wasting it as 'reserved'. Hope that
> once those patches are merged, the IOMMU can be finally enabled in the exynos_defconfig.
>

Yes, a problem is that some Exynos machines (for example the Snow and Peach
Chromebooks) fail to boot when IOMMU is enabled due the bootloader leaving
the FIMD enabled doing DMA operations automatically as you found before.

You proposed to add a "iommu-reserved-mapping" property [0] but that never
landed due not having an agreement on how should be fixed properly IIUC [1].

So that has to be fixed before enabling the Exynos IOMMU support by default.

[0]: http://www.spinics.net/lists/arm-kernel/msg415501.html
[1]: http://www.spinics.net/lists/arm-kernel/msg419747.html 

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux