From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sinan Kaya Subject: Re: [PATCH V13 08/10] dma: qcom_hidma: read the channel id from HW Date: Wed, 3 Feb 2016 13:32:01 -0500 Message-ID: <56B247A1.20503@codeaurora.org> References: <1454106914-15857-1-git-send-email-okaya@codeaurora.org> <1454106914-15857-9-git-send-email-okaya@codeaurora.org> <20160201153542.GA20396@leverpostej> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160201153542.GA20396@leverpostej> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mark Rutland Cc: dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, marc.zyngier-5wv7dgnIgG8@public.gmane.org, timur-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, cov-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, jcm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, shankerd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, vikrams-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, eric.auger-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, agross-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, arnd-r2nGTMty4D4@public.gmane.org, linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-arm-msm@vger.kernel.org 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? Sinan -- 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 devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965674AbcBCScJ (ORCPT ); Wed, 3 Feb 2016 13:32:09 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:41912 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964967AbcBCScG (ORCPT ); Wed, 3 Feb 2016 13:32:06 -0500 Subject: Re: [PATCH V13 08/10] dma: qcom_hidma: read the channel id from HW To: Mark Rutland References: <1454106914-15857-1-git-send-email-okaya@codeaurora.org> <1454106914-15857-9-git-send-email-okaya@codeaurora.org> <20160201153542.GA20396@leverpostej> Cc: dmaengine@vger.kernel.org, marc.zyngier@arm.com, timur@codeaurora.org, devicetree@vger.kernel.org, cov@codeaurora.org, vinod.koul@intel.com, jcm@redhat.com, shankerd@codeaurora.org, vikrams@codeaurora.org, eric.auger@linaro.org, agross@codeaurora.org, arnd@arndb.de, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org From: Sinan Kaya Message-ID: <56B247A1.20503@codeaurora.org> Date: Wed, 3 Feb 2016 13:32:01 -0500 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <20160201153542.GA20396@leverpostej> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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? Sinan -- 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: okaya@codeaurora.org (Sinan Kaya) Date: Wed, 3 Feb 2016 13:32:01 -0500 Subject: [PATCH V13 08/10] dma: qcom_hidma: read the channel id from HW In-Reply-To: <20160201153542.GA20396@leverpostej> References: <1454106914-15857-1-git-send-email-okaya@codeaurora.org> <1454106914-15857-9-git-send-email-okaya@codeaurora.org> <20160201153542.GA20396@leverpostej> Message-ID: <56B247A1.20503@codeaurora.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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? Sinan -- 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