[PATCH v3] mhi: use irq_flags if controller driver configures it

2021-01-04 Thread Carl Huang
vector, IRQF_NO_SUSPEND will trigger WARNINGS because QCA6390 wants to disable the IRQ during the suspend. To avoid above two issues, QCA6390 driver specifies the irq_flags in case of one MSI vector when mhi_register_controller is called. Signed-off-by: Carl Huang Reviewed-by: Manivannan Sadhasivam

[PATCH v2] mhi: use irq_flags if client driver configures it

2020-12-28 Thread Carl Huang
, IRQF_NO_SUSPEND will trigger WARNINGS because QCA6390 wants to disable the IRQ during the suspend. To avoid above two issues, QCA6390 driver specifies the irq_flags in case of one MSI vector when mhi_register_controller is called. Signed-off-by: Carl Huang --- v2: - document irq_flags added

Re: [PATCH] mhi: use irq_flags if client driver configures it

2020-12-22 Thread Carl Huang
On 2020-12-10 03:48, Jeffrey Hugo wrote: On 12/9/2020 11:34 AM, Hemant Kumar wrote: On 12/7/20 7:55 PM, Carl Huang wrote: If client driver has specified the irq_flags, mhi uses this specified irq_flags. Otherwise, mhi uses default irq_flags. The purpose of this change is to support one MSI

[PATCH] mhi: use irq_flags if client driver configures it

2020-12-07 Thread Carl Huang
, IRQF_NO_SUSPEND will trigger WARNINGS because QCA6390 wants to disable the IRQ during the suspend. To avoid above two issues, QCA6390 driver specifies the irq_flags in case of one MSI vector when mhi_register_controller is called. Signed-off-by: Carl Huang --- drivers/bus/mhi/core/init.c | 9

Re: Regression: QCA6390 fails with "mm/page_alloc: place pages to tail in __free_pages_core()"

2020-11-13 Thread Carl Huang
On 2020-11-13 19:08, Carl Huang wrote: On 2020-11-13 16:17, Pavel Procopiuc wrote: Op 12.11.2020 om 11:48 schreef David Hildenbrand: Trying to understand the code, it looks like there are always two rounds of reqests. The first one always fails ("requesting one big chunk of DMA m

Re: Regression: QCA6390 fails with "mm/page_alloc: place pages to tail in __free_pages_core()"

2020-11-13 Thread Carl Huang
On 2020-11-13 16:17, Pavel Procopiuc wrote: Op 12.11.2020 om 11:48 schreef David Hildenbrand: Trying to understand the code, it looks like there are always two rounds of reqests. The first one always fails ("requesting one big chunk of DMA memory"), the second one (providing multiple chunks of

[PATCH v2] net: qrtr: free flow in __qrtr_node_release

2020-06-30 Thread Carl Huang
mi_helpers] [<9c9ae7de>] qmi_send_request+0x1c/0x20 [qmi_helpers] Signed-off-by: Carl Huang --- net/qrtr/qrtr.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/qrtr/qrtr.c b/net/qrtr/qrtr.c index 2d8d613..0598813 100644 --- a/net/qrtr/qrtr.c +++ b/net/qrt

[PATCH] net: qrtr: free flow in __qrtr_node_release

2020-06-22 Thread Carl Huang
mi_helpers] [<9c9ae7de>] qmi_send_request+0x1c/0x20 [qmi_helpers] Signed-off-by: Carl Huang --- net/qrtr/qrtr.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/qrtr/qrtr.c b/net/qrtr/qrtr.c index 2d8d613..1855857 100644 --- a/net/qrtr/qrtr.c +++ b/net/qrt