Re: [PATCH V13 08/10] dma: qcom_hidma: read the channel id from HW

From: Mark Rutland
Date: Wed Feb 03 2016 - 13:37:27 EST


On Wed, Feb 03, 2016 at 01:32:01PM -0500, Sinan Kaya wrote:
> Hi Mark,
>
> On 2/1/2016 10:35 AM, Mark Rutland wrote:
> >> Removing the flexibility to choose the event channel as there is no real
> >> > use case right now. We have been using the values in ACPI that match the HW
> >> > defaults. OS is reading the event-channel from the HW register now.
> > Fold this in to the patches adding the binding and the driver.
> >
> > There is no reason to add this then remove it within the same series.
> >
> > Mark.
> >
>
> Before I post the next rev, I'd like to see if I missed something in the series when
> it comes to your comments.
>
> Can you please take a look and let me know?

Sorry, but it's going to be far easier to review with useless code
removed, so I will wait until the next posting where this patch is
folded in.

Thanks,
Mark.