Re: [PATCH 2/2] MIPS: Loongson64: Add dma iocoherency detection support

[Jiaxun Yang]

14.01.2020, 02:52, "Paul Burton" <paulburton89@xxxxxxxxx>:
> Hi Jiaxun,
>
> On Mon, Jan 13, 2020 at 10:07:05PM +0800, Jiaxun Yang wrote:
>> Âdiff --git a/arch/mips/include/asm/mach-loongson64/boot_param.h b/arch/mips/include/asm/mach-loongson64/boot_param.h
>> Âindex 8c286bedff3e..2da2be40ad81 100644
>> Â--- a/arch/mips/include/asm/mach-loongson64/boot_param.h
>> Â+++ b/arch/mips/include/asm/mach-loongson64/boot_param.h
>> Â@@ -115,7 +115,8 @@ struct irq_source_routing_table {
>> ÂÂÂÂÂÂÂÂÂÂu64 pci_io_start_addr;
>> ÂÂÂÂÂÂÂÂÂÂu64 pci_io_end_addr;
>> ÂÂÂÂÂÂÂÂÂÂu64 pci_config_addr;
>> Â- u32 dma_mask_bits;
>> Â+ u16 dma_mask_bits;
>> Â+ u16 dma_noncoherent;
>> ÂÂ} __packed;
>
> This struct is generated by the firmware, right? So does this change
> require that firmware be updated along with the kernel? Or was the
> kernel's definition always incorrect/incomplete?
Hi Paul,

This define added to the firmware by 2013 but somehow never upstream.
That's what Loongson always doing.

Some versions of Loongson-3B1500 failed to maintain DMA coherent.
So they add this parameter to the firmware.
As the maximum dma_mask_bits is 64, the upper 16bit of the old u32 will always
be zero. Which means coherent DMA transformed into the new definition,
that's the expected default behavior. Thus it's safe to do so even if firmware
doesn't understand this parameter.

Thanks

>
> Thanks,
> ÂÂÂÂPaul
--
Jiaxun