From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932528AbdKOP2A (ORCPT ); Wed, 15 Nov 2017 10:28:00 -0500 Received: from mail-ot0-f196.google.com ([74.125.82.196]:54530 "EHLO mail-ot0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932411AbdKOP1s (ORCPT ); Wed, 15 Nov 2017 10:27:48 -0500 X-Google-Smtp-Source: AGs4zMa7otxcbbIY1BYBq3T4P6e0igvpW6DWoLWzm8/sj0l59wrPEFDsc7p0aF2EtT/WpHll4+ZyrA== Date: Wed, 15 Nov 2017 09:27:46 -0600 From: Rob Herring To: Eugen Hristev Cc: nicolas.ferre@microchip.com, alexandre.belloni@free-electrons.com, linux-iio@vger.kernel.org, lars@metafoo.de, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, ludovic.desroches@microchip.com, jic23@kernel.org Subject: Re: [PATCH v3 1/4] dt-bindings: iio: at91-sama5d2_adc: add optional dma property Message-ID: <20171115152746.w4giwt7kdqm4i3yl@rob-hp-laptop> References: <1510750608-8697-1-git-send-email-eugen.hristev@microchip.com> <1510750608-8697-2-git-send-email-eugen.hristev@microchip.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1510750608-8697-2-git-send-email-eugen.hristev@microchip.com> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 15, 2017 at 02:56:45PM +0200, Eugen Hristev wrote: > Added property for DMA configuration of the device. > > Signed-off-by: Eugen Hristev > --- > Changes in v3: > None, but we discussed on the ML about whether we should have "dma-names" > present in the binding even if it's only one. > The helpers in the kernel to retrieve the channel info rely on the > presence of this property, so I am resending the patch based on this. > If another solution is better, please advise and I can try it and > resend the patch. Really the kernel APIs should accept a NULL name and return the DMA channel when there is only one. This is how the clk_get API works for example. > Documentation/devicetree/bindings/iio/adc/at91-sama5d2_adc.txt | 7 +++++++ > 1 file changed, 7 insertions(+) In any case, not really a big deal. Acked-by: Rob Herring From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v3 1/4] dt-bindings: iio: at91-sama5d2_adc: add optional dma property Date: Wed, 15 Nov 2017 09:27:46 -0600 Message-ID: <20171115152746.w4giwt7kdqm4i3yl@rob-hp-laptop> References: <1510750608-8697-1-git-send-email-eugen.hristev@microchip.com> <1510750608-8697-2-git-send-email-eugen.hristev@microchip.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1510750608-8697-2-git-send-email-eugen.hristev-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org> Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Eugen Hristev Cc: nicolas.ferre-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org, alexandre.belloni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org, linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, ludovic.desroches-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org, jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org List-Id: devicetree@vger.kernel.org On Wed, Nov 15, 2017 at 02:56:45PM +0200, Eugen Hristev wrote: > Added property for DMA configuration of the device. > > Signed-off-by: Eugen Hristev > --- > Changes in v3: > None, but we discussed on the ML about whether we should have "dma-names" > present in the binding even if it's only one. > The helpers in the kernel to retrieve the channel info rely on the > presence of this property, so I am resending the patch based on this. > If another solution is better, please advise and I can try it and > resend the patch. Really the kernel APIs should accept a NULL name and return the DMA channel when there is only one. This is how the clk_get API works for example. > Documentation/devicetree/bindings/iio/adc/at91-sama5d2_adc.txt | 7 +++++++ > 1 file changed, 7 insertions(+) In any case, not really a big deal. Acked-by: Rob Herring From mboxrd@z Thu Jan 1 00:00:00 1970 From: robh@kernel.org (Rob Herring) Date: Wed, 15 Nov 2017 09:27:46 -0600 Subject: [PATCH v3 1/4] dt-bindings: iio: at91-sama5d2_adc: add optional dma property In-Reply-To: <1510750608-8697-2-git-send-email-eugen.hristev@microchip.com> References: <1510750608-8697-1-git-send-email-eugen.hristev@microchip.com> <1510750608-8697-2-git-send-email-eugen.hristev@microchip.com> Message-ID: <20171115152746.w4giwt7kdqm4i3yl@rob-hp-laptop> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Nov 15, 2017 at 02:56:45PM +0200, Eugen Hristev wrote: > Added property for DMA configuration of the device. > > Signed-off-by: Eugen Hristev > --- > Changes in v3: > None, but we discussed on the ML about whether we should have "dma-names" > present in the binding even if it's only one. > The helpers in the kernel to retrieve the channel info rely on the > presence of this property, so I am resending the patch based on this. > If another solution is better, please advise and I can try it and > resend the patch. Really the kernel APIs should accept a NULL name and return the DMA channel when there is only one. This is how the clk_get API works for example. > Documentation/devicetree/bindings/iio/adc/at91-sama5d2_adc.txt | 7 +++++++ > 1 file changed, 7 insertions(+) In any case, not really a big deal. Acked-by: Rob Herring