Re: [PATCH 2/2] dma: add Qualcomm Technologies HIDMA channel driver

From: Sinan Kaya
Date: Fri Oct 30 2015 - 21:54:09 EST




On 10/30/2015 6:28 PM, Arnd Bergmann wrote:
I missed that part. If the descriptor count is a hardware feature,
just make that property mandatory. OTOH, if this is an optimization
setting, better drop that property entirely.

I'm going to make this a module parameter instead and get rid of the constant. The reason, I have this default parameter today is that QEMU does not support passing device tree arguments for platform devices.

QEMU allows you to set memory and interrupt resources only for platform devices.

At least now, I can pass the argument via command line before starting QEMU.

I'll put checks that the value needs to come from either DTS/ACPI/command line.

--
Sinan Kaya
Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/