linux-kernel - RE: [PATCH 4/9] dmaengine: pl330: add new items for pl330 private data

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 10 Dec 2015 06:38:09 +0000
From:	"Wang, Annie" <Annie.Wang@....com>
To:	Vinod Koul <vinod.koul@...el.com>
CC:	Mika Westerberg <mika.westerberg@...ux.intel.com>,
	Joerg Roedel <joro@...tes.org>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-serial@...r.kernel.org" <linux-serial@...r.kernel.org>,
	"dmaengine@...r.kernel.org" <dmaengine@...r.kernel.org>,
	"iommu@...ts.linux-foundation.org" <iommu@...ts.linux-foundation.org>,
	Borislav Petkov <bp@...en8.de>,
	"Huang, Ray" <Ray.Huang@....com>,
	"Wan, Vincent" <Vincent.Wan@....com>, "Xue, Ken" <Ken.Xue@....com>,
	"Li, Tony" <Tony.Li@....com>
Subject: RE: [PATCH 4/9] dmaengine: pl330: add new items for pl330 private
 data

>-----Original Message-----
>From: Vinod Koul [mailto:vinod.koul@...el.com]
>Sent: Thursday, December 10, 2015 12:09 PM
>To: Wang, Annie
>Cc: Mika Westerberg; Joerg Roedel; Greg Kroah-Hartman; Rafael J. Wysocki;
>linux-acpi@...r.kernel.org; linux-kernel@...r.kernel.org; linux-
>serial@...r.kernel.org; dmaengine@...r.kernel.org; iommu@...ts.linux-
>foundation.org; Borislav Petkov; Huang, Ray; Wan, Vincent; Xue, Ken; Li, Tony
>Subject: Re: [PATCH 4/9] dmaengine: pl330: add new items for pl330 private data
>
>On Fri, Dec 04, 2015 at 11:24:21AM +0800, Wang Hongcheng wrote:
>> has_no_cap_mask means this device has no preset cap mask.
>> mcbuf_sz means bytes to allocate for MC buffer.
>
>MC ?
Size of MicroCode buffers for each channel. I will update the comment.

>> flags is for irq sharing, default is non-shared, in AMD Carrizo, pl330
>> shares IRQ with its corresponding UART device.
>>
>> Signed-off-by: Wang Hongcheng <annie.wang@....com>
>> ---
>>  drivers/acpi/acpi_apd.c    | 13 ++++++++++++-
>>  drivers/dma/pl330.c        | 19 +++++++++++++------
>>  include/linux/amba/pl330.h |  3 +++
>>  3 files changed, 28 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c index
>> eb3316a..7a582f5 100644
>> --- a/drivers/acpi/acpi_apd.c
>> +++ b/drivers/acpi/acpi_apd.c
>> @@ -21,6 +21,7 @@
>>  #include <linux/amba/bus.h>
>>  #include <linux/kernel.h>
>>  #include <linux/sizes.h>
>> +#include <linux/amba/pl330.h>
>>  #include <linux/interrupt.h>
>>
>>  #include "internal.h"
>> @@ -35,6 +36,16 @@ struct apd_private_data;
>>  #define ACPI_APD_SYSFS	BIT(0)
>>  #define ACPI_APD_PM	BIT(1)
>>
>> +static u8 peri_id[2] = { 0, 1 };
>> +
>> +static struct dma_pl330_platdata amd_pl330 = {
>> +	.nr_valid_peri = 2,
>> +	.peri_id = peri_id,
>> +	.has_no_cap_mask = true,
>> +	.mcbuf_sz = 0,
>> +	.flags = IRQF_SHARED,
>> +};
>
>Why not DT or ACPI for this?
>
>--
>~Vinod

We choose to use private data, as pl330 already has  struct dma_pl330_platdata. 
Physically DMA share ACPI device with UART, however, BIOS believes DMA and UART is one device.
We can't  get irq share info from ACPI. And we don't use DT. 

Regards,
Annie




--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/