From: Boris BREZILLON <boris.brezillon@free-electrons.com> To: jjhiblot@traphandler.com, Thierry Reding <thierry.reding@gmail.com>, Nicolas Ferre <nicolas.ferre@atmel.com>, David Airlie <airlied@linux.ie>, Samuel Ortiz <sameo@linux.intel.com>, Lee Jones <lee.jones@linaro.org> Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>, Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pwm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH v2 2/7] pwm: add support for atmel-hlcdc-pwm device Date: Sun, 15 Jun 2014 16:55:27 +0200 [thread overview] Message-ID: <539DB3DF.5050301@free-electrons.com> (raw) In-Reply-To: <539D6328.4020505@traphandler.com> Hello JJ, On 15/06/2014 11:11, Jean-Jacques Hiblot wrote: > > On 06/09/2014 06:04 PM, Boris BREZILLON wrote: >> The HLCDC IP available in some Atmel SoCs (i.e. sam9x5i.e. at91sam9n12, >> at91sam9x5 family or sama5d3 family) provide a PWM device. >> >> This driver add support for this PWM device. >> >> Signed-off-by: Boris BREZILLON <boris.brezillon@free-electrons.com> >> --- >> .../devicetree/bindings/pwm/atmel-hlcdc-pwm.txt | 40 ++++ >> drivers/pwm/Kconfig | 9 + >> drivers/pwm/Makefile | 1 + >> drivers/pwm/pwm-atmel-hlcdc.c | 216 +++++++++++++++++++++ >> 4 files changed, 266 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt >> create mode 100644 drivers/pwm/pwm-atmel-hlcdc.c >> >> diff --git a/Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt b/Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt >> new file mode 100644 >> index 0000000..5e2ba87 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt >> @@ -0,0 +1,40 @@ [...] >> +++ b/drivers/pwm/Kconfig >> @@ -50,6 +50,15 @@ config PWM_ATMEL >> To compile this driver as a module, choose M here: the module >> will be called pwm-atmel. >> >> +config PWM_ATMEL_HLCDC_PWM >> + tristate "Atmel HLCDC PWM support" >> + depends on MFD_ATMEL_HLCDC > I'd personnaly prefer a 'select' instead of 'depends on' here. Or maybe > the MFD driver should enabled y defaut for platforms supporting the hlcdc. I don't have any strong opinion on the "select" vs "depends on" approach. Does anyone else think we should use a 'select' instead of a 'depends on' ? I see at least one benefit, we would be able to see the HLCDC_PWM option even if the HLCDC driver is not enabled. Best Regards, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: boris.brezillon@free-electrons.com (Boris BREZILLON) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 2/7] pwm: add support for atmel-hlcdc-pwm device Date: Sun, 15 Jun 2014 16:55:27 +0200 [thread overview] Message-ID: <539DB3DF.5050301@free-electrons.com> (raw) In-Reply-To: <539D6328.4020505@traphandler.com> Hello JJ, On 15/06/2014 11:11, Jean-Jacques Hiblot wrote: > > On 06/09/2014 06:04 PM, Boris BREZILLON wrote: >> The HLCDC IP available in some Atmel SoCs (i.e. sam9x5i.e. at91sam9n12, >> at91sam9x5 family or sama5d3 family) provide a PWM device. >> >> This driver add support for this PWM device. >> >> Signed-off-by: Boris BREZILLON <boris.brezillon@free-electrons.com> >> --- >> .../devicetree/bindings/pwm/atmel-hlcdc-pwm.txt | 40 ++++ >> drivers/pwm/Kconfig | 9 + >> drivers/pwm/Makefile | 1 + >> drivers/pwm/pwm-atmel-hlcdc.c | 216 +++++++++++++++++++++ >> 4 files changed, 266 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt >> create mode 100644 drivers/pwm/pwm-atmel-hlcdc.c >> >> diff --git a/Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt b/Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt >> new file mode 100644 >> index 0000000..5e2ba87 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/pwm/atmel-hlcdc-pwm.txt >> @@ -0,0 +1,40 @@ [...] >> +++ b/drivers/pwm/Kconfig >> @@ -50,6 +50,15 @@ config PWM_ATMEL >> To compile this driver as a module, choose M here: the module >> will be called pwm-atmel. >> >> +config PWM_ATMEL_HLCDC_PWM >> + tristate "Atmel HLCDC PWM support" >> + depends on MFD_ATMEL_HLCDC > I'd personnaly prefer a 'select' instead of 'depends on' here. Or maybe > the MFD driver should enabled y defaut for platforms supporting the hlcdc. I don't have any strong opinion on the "select" vs "depends on" approach. Does anyone else think we should use a 'select' instead of a 'depends on' ? I see at least one benefit, we would be able to see the HLCDC_PWM option even if the HLCDC driver is not enabled. Best Regards, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
next prev parent reply other threads:[~2014-06-15 14:55 UTC|newest] Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-06-09 16:04 [PATCH v2 0/7] drm: add support for Atmel HLCDC Display Controller Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` [PATCH v2 1/7] mfd: add atmel-hlcdc driver Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-15 8:53 ` Jean-Jacques Hiblot 2014-06-15 8:53 ` Jean-Jacques Hiblot 2014-06-15 14:48 ` Boris BREZILLON 2014-06-15 14:48 ` Boris BREZILLON 2014-06-16 7:46 ` Jean-Jacques Hiblot 2014-06-16 7:46 ` Jean-Jacques Hiblot 2014-06-16 7:46 ` Jean-Jacques Hiblot 2014-06-16 12:50 ` Lee Jones 2014-06-16 12:50 ` Lee Jones 2014-06-16 13:23 ` Boris BREZILLON 2014-06-16 13:23 ` Boris BREZILLON 2014-06-16 13:23 ` Boris BREZILLON 2014-06-16 17:03 ` Lee Jones 2014-06-16 17:03 ` Lee Jones 2014-06-16 17:03 ` Lee Jones 2014-06-16 17:09 ` Boris BREZILLON 2014-06-16 17:09 ` Boris BREZILLON 2014-06-09 16:04 ` [PATCH v2 2/7] pwm: add support for atmel-hlcdc-pwm device Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-13 11:43 ` Alexandre Belloni 2014-06-13 11:43 ` Alexandre Belloni 2014-06-13 12:17 ` Boris BREZILLON 2014-06-13 12:17 ` Boris BREZILLON 2014-06-15 9:11 ` Jean-Jacques Hiblot 2014-06-15 9:11 ` Jean-Jacques Hiblot 2014-06-15 14:55 ` Boris BREZILLON [this message] 2014-06-15 14:55 ` Boris BREZILLON 2014-07-04 13:31 ` Boris BREZILLON 2014-07-04 13:31 ` Boris BREZILLON 2014-06-09 16:04 ` [PATCH v2 3/7] drm: add Atmel HLCDC Display Controller support Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-15 9:32 ` Jean-Jacques Hiblot 2014-06-15 15:00 ` Boris BREZILLON 2014-06-15 15:00 ` Boris BREZILLON 2014-06-17 9:06 ` Boris BREZILLON 2014-07-03 13:34 ` Boris BREZILLON 2014-06-09 16:04 ` [PATCH v2 4/7] ARM: at91/dt: split sama5d3 lcd pin definitions to match RGB mode configs Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-19 7:07 ` Bo Shen 2014-06-19 7:07 ` Bo Shen 2014-06-19 7:07 ` Bo Shen 2014-06-19 7:40 ` Boris BREZILLON 2014-06-19 7:40 ` Boris BREZILLON 2014-06-19 11:42 ` Boris BREZILLON 2014-06-19 11:42 ` Boris BREZILLON 2014-06-09 16:04 ` [PATCH v2 5/7] ARM: at91/dt: define the HLCDC node available on sama5d3 SoCs Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` [PATCH v2 6/7] ARM: at91/dt: add LCD panel description to sama5d3xdm.dtsi Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` [PATCH v2 7/7] ARM: at91/dt: enable the LCD panel on sama5d3xek boards Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-09 16:04 ` Boris BREZILLON 2014-06-19 7:12 ` Bo Shen 2014-06-19 7:12 ` Bo Shen 2014-06-19 7:12 ` Bo Shen 2014-06-19 7:54 ` Boris BREZILLON 2014-06-19 7:54 ` Boris BREZILLON
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=539DB3DF.5050301@free-electrons.com \ --to=boris.brezillon@free-electrons.com \ --cc=airlied@linux.ie \ --cc=alexandre.belloni@free-electrons.com \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=jjhiblot@traphandler.com \ --cc=laurent.pinchart@ideasonboard.com \ --cc=lee.jones@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pwm@vger.kernel.org \ --cc=nicolas.ferre@atmel.com \ --cc=plagnioj@jcrosoft.com \ --cc=sameo@linux.intel.com \ --cc=thierry.reding@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.