From mboxrd@z Thu Jan 1 00:00:00 1970 From: swarren@wwwdotorg.org (Stephen Warren) Date: Tue, 13 Jan 2015 09:19:35 -0700 Subject: [PATCH]dts: add bcm2835-pwm bindings In-Reply-To: <54B53C86.4030903@thomasmore.be> References: <1420471983-29945-1-git-send-email-bart.tanghe@thomasmore.be> <54B1556C.7070401@wwwdotorg.org> <54B53C86.4030903@thomasmore.be> Message-ID: <54B54597.3010508@wwwdotorg.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 01/13/2015 08:40 AM, Bart Tanghe wrote: > On 01/10/2015 05:38 PM, Stephen Warren wrote: >> On 01/05/2015 08:33 AM, Bart Tanghe wrote: >>> To probe the bcm2835-pwm driver properly, this dts bindings has to be added >>> to the bcm2835 dtsi file. >>> In expectation of a bcm2835 clock driver, I've added fixed-clock clock binding. >>> To change the status of the gpio pins into pwm output pins, some additional >>> bindings has to be added but this has not to be done automatically. >>> Is it possible to document this bindings somewhere? >> >> The pinctrl bindings allow you to do this. They're documented in: >> >> Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt >> Documentation/devicetree/bindings/pinctrl/brcm,bcm2835-gpio.txt >> >> pinctrl setup would typically be added to the board file (i.e. >> arch/arm/boot/dts/bcm2835-rpi-b.dts) rather than the SoC file. I think >> on the RPi, the pins that can generate PWM aren't dedicated to PWM by >> the RPi board design (they could be GPIOs and probably other functions >> too) so we probably need to leave it up to individual users to add PWM >> configuration to their DT, if/when they use the PWM feature. >> > > Indeed, the pins aren't dedicated to pwm and can be used for other functions. > Is it usefull to add the pwm pinctrl description to > Documentation/devicetree/bindings/pinctrl/brcm,bcm2835-gpio.txt or is this > useless? As far as I can tell, that file already describes everything required to configure those pins as PWM (or indeed any pins as any function). Can you expand on what you think is missing? From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH]dts: add bcm2835-pwm bindings Date: Tue, 13 Jan 2015 09:19:35 -0700 Message-ID: <54B54597.3010508@wwwdotorg.org> References: <1420471983-29945-1-git-send-email-bart.tanghe@thomasmore.be> <54B1556C.7070401@wwwdotorg.org> <54B53C86.4030903@thomasmore.be> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <54B53C86.4030903-2dq7lIxLGp5qkBXjtECzWg@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Bart Tanghe , lee-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On 01/13/2015 08:40 AM, Bart Tanghe wrote: > On 01/10/2015 05:38 PM, Stephen Warren wrote: >> On 01/05/2015 08:33 AM, Bart Tanghe wrote: >>> To probe the bcm2835-pwm driver properly, this dts bindings has to be added >>> to the bcm2835 dtsi file. >>> In expectation of a bcm2835 clock driver, I've added fixed-clock clock binding. >>> To change the status of the gpio pins into pwm output pins, some additional >>> bindings has to be added but this has not to be done automatically. >>> Is it possible to document this bindings somewhere? >> >> The pinctrl bindings allow you to do this. They're documented in: >> >> Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt >> Documentation/devicetree/bindings/pinctrl/brcm,bcm2835-gpio.txt >> >> pinctrl setup would typically be added to the board file (i.e. >> arch/arm/boot/dts/bcm2835-rpi-b.dts) rather than the SoC file. I think >> on the RPi, the pins that can generate PWM aren't dedicated to PWM by >> the RPi board design (they could be GPIOs and probably other functions >> too) so we probably need to leave it up to individual users to add PWM >> configuration to their DT, if/when they use the PWM feature. >> > > Indeed, the pins aren't dedicated to pwm and can be used for other functions. > Is it usefull to add the pwm pinctrl description to > Documentation/devicetree/bindings/pinctrl/brcm,bcm2835-gpio.txt or is this > useless? As far as I can tell, that file already describes everything required to configure those pins as PWM (or indeed any pins as any function). Can you expand on what you think is missing? -- 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 S1753234AbbAMQSk (ORCPT ); Tue, 13 Jan 2015 11:18:40 -0500 Received: from avon.wwwdotorg.org ([70.85.31.133]:36924 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751318AbbAMQSi (ORCPT ); Tue, 13 Jan 2015 11:18:38 -0500 Message-ID: <54B54597.3010508@wwwdotorg.org> Date: Tue, 13 Jan 2015 09:19:35 -0700 From: Stephen Warren User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: Bart Tanghe , lee@kernel.org, devicetree@vger.kernel.org CC: thierry.reding@gmail.com, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, linux@arm.linux.org.uk, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH]dts: add bcm2835-pwm bindings References: <1420471983-29945-1-git-send-email-bart.tanghe@thomasmore.be> <54B1556C.7070401@wwwdotorg.org> <54B53C86.4030903@thomasmore.be> In-Reply-To: <54B53C86.4030903@thomasmore.be> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/13/2015 08:40 AM, Bart Tanghe wrote: > On 01/10/2015 05:38 PM, Stephen Warren wrote: >> On 01/05/2015 08:33 AM, Bart Tanghe wrote: >>> To probe the bcm2835-pwm driver properly, this dts bindings has to be added >>> to the bcm2835 dtsi file. >>> In expectation of a bcm2835 clock driver, I've added fixed-clock clock binding. >>> To change the status of the gpio pins into pwm output pins, some additional >>> bindings has to be added but this has not to be done automatically. >>> Is it possible to document this bindings somewhere? >> >> The pinctrl bindings allow you to do this. They're documented in: >> >> Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt >> Documentation/devicetree/bindings/pinctrl/brcm,bcm2835-gpio.txt >> >> pinctrl setup would typically be added to the board file (i.e. >> arch/arm/boot/dts/bcm2835-rpi-b.dts) rather than the SoC file. I think >> on the RPi, the pins that can generate PWM aren't dedicated to PWM by >> the RPi board design (they could be GPIOs and probably other functions >> too) so we probably need to leave it up to individual users to add PWM >> configuration to their DT, if/when they use the PWM feature. >> > > Indeed, the pins aren't dedicated to pwm and can be used for other functions. > Is it usefull to add the pwm pinctrl description to > Documentation/devicetree/bindings/pinctrl/brcm,bcm2835-gpio.txt or is this > useless? As far as I can tell, that file already describes everything required to configure those pins as PWM (or indeed any pins as any function). Can you expand on what you think is missing?