* [PATCH] counter: ti-eqep: Enable for TDA4VM/J721E
@ 2021-11-30 19:33 Jason Kridner
2021-11-30 21:16 ` Nishanth Menon
2021-12-04 1:46 ` William Breathitt Gray
0 siblings, 2 replies; 4+ messages in thread
From: Jason Kridner @ 2021-11-30 19:33 UTC (permalink / raw)
To: William Breathitt Gray; +Cc: Jason Kridner, linux-iio, linux-kernel
Counter TI-EQEP was tested on TDA4VM/J721E SoC.
Overlay used for testing EQEP0 and EQEP1:
&cbass_main {
#address-cells = <2>;
#size-cells = <2>;
eqep0: qep@3200000 {
compatible = "ti,am3352-eqep";
reg = <0 0x3200000 0 0x100>;
power-domains = <&k3_pds 94 TI_SCI_PD_EXCLUSIVE>;
clocks = <&k3_clks 94 0>;
clock-names = "eqep0-ficlk";
interrupts = <GIC_SPI 322 IRQ_TYPE_EDGE_RISING>;
interrupt-names = "eqep0";
symlink = "bone/eQEP0"; /* A - P9_42, B - P9_27 */
count_mode = <0>;
swap_input = <0>;
invert_qa = <1>;
invert_qb = <1>;
invert_qi = <0>;
invert_qs = <0>;
};
eqep1: qep@3210000 {
compatible = "ti,am3352-eqep";
reg = <0 0x3210000 0 0x100>;
power-domains = <&k3_pds 95 TI_SCI_PD_EXCLUSIVE>;
clocks = <&k3_clks 95 0>;
clock-names = "eqep1-ficlk";
interrupts = <GIC_SPI 323 IRQ_TYPE_EDGE_RISING>;
interrupt-names = "eqep1";
symlink = "bone/eQEP1"; /* A - P8_35, B - P8_33 */
count_mode = <0>;
swap_input = <0>;
invert_qa = <1>;
invert_qb = <1>;
invert_qi = <0>;
invert_qs = <0>;
};
eqep2: qep@3220000 {
status = "disabled";
compatible = "ti,am3352-eqep";
reg = <0 0x3220000 0 0x100>;
power-domains = <&k3_pds 96 TI_SCI_PD_EXCLUSIVE>;
clocks = <&k3_clks 96 0>;
clock-names = "eqep2-ficlk";
interrupts = <GIC_SPI 324 IRQ_TYPE_EDGE_RISING>;
interrupt-names = "eqep2";
count_mode = <0>;
swap_input = <0>;
invert_qa = <1>;
invert_qb = <1>;
invert_qi = <0>;
invert_qs = <0>;
};
};
Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
---
drivers/counter/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/counter/Kconfig b/drivers/counter/Kconfig
index 2de53ab0dd25..453490059c88 100644
--- a/drivers/counter/Kconfig
+++ b/drivers/counter/Kconfig
@@ -51,7 +51,7 @@ config STM32_LPTIMER_CNT
config TI_EQEP
tristate "TI eQEP counter driver"
- depends on (SOC_AM33XX || COMPILE_TEST)
+ depends on (SOC_AM33XX || ARCH_K3 || COMPILE_TEST)
select REGMAP_MMIO
help
Select this option to enable the Texas Instruments Enhanced Quadrature
--
2.30.2
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] counter: ti-eqep: Enable for TDA4VM/J721E
2021-11-30 19:33 [PATCH] counter: ti-eqep: Enable for TDA4VM/J721E Jason Kridner
@ 2021-11-30 21:16 ` Nishanth Menon
2021-12-04 16:34 ` Jonathan Cameron
2021-12-04 1:46 ` William Breathitt Gray
1 sibling, 1 reply; 4+ messages in thread
From: Nishanth Menon @ 2021-11-30 21:16 UTC (permalink / raw)
To: Jason Kridner; +Cc: William Breathitt Gray, linux-iio, linux-kernel
On 14:33-20211130, Jason Kridner wrote:
> Counter TI-EQEP was tested on TDA4VM/J721E SoC.
>
Thanks for doing this patch,
You could drop the example - This is an example which is probably
better addressed in Documentation/devicetree/bindings/counter/ti-eqep.yaml
or in the actual device tree for j721e itself.
Instead, I would suggest to keep the commit message short:
Enable TI-QEP for TI's K3 architecture as the driver is reusable
or something of that form.
The actual devicetree dts patch should be separate and posted once the
driver support is enabled.
> Overlay used for testing EQEP0 and EQEP1:
>
> &cbass_main {
> #address-cells = <2>;
> #size-cells = <2>;
>
> eqep0: qep@3200000 {
[...]
> };
> };
>
> Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
> ---
> drivers/counter/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/counter/Kconfig b/drivers/counter/Kconfig
> index 2de53ab0dd25..453490059c88 100644
> --- a/drivers/counter/Kconfig
> +++ b/drivers/counter/Kconfig
> @@ -51,7 +51,7 @@ config STM32_LPTIMER_CNT
>
> config TI_EQEP
> tristate "TI eQEP counter driver"
> - depends on (SOC_AM33XX || COMPILE_TEST)
> + depends on (SOC_AM33XX || ARCH_K3 || COMPILE_TEST)
> select REGMAP_MMIO
> help
> Select this option to enable the Texas Instruments Enhanced Quadrature
> --
> 2.30.2
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] counter: ti-eqep: Enable for TDA4VM/J721E
2021-11-30 21:16 ` Nishanth Menon
@ 2021-12-04 16:34 ` Jonathan Cameron
0 siblings, 0 replies; 4+ messages in thread
From: Jonathan Cameron @ 2021-12-04 16:34 UTC (permalink / raw)
To: Nishanth Menon
Cc: Jason Kridner, William Breathitt Gray, linux-iio, linux-kernel
On Tue, 30 Nov 2021 15:16:28 -0600
Nishanth Menon <nm@ti.com> wrote:
> On 14:33-20211130, Jason Kridner wrote:
> > Counter TI-EQEP was tested on TDA4VM/J721E SoC.
> >
> Thanks for doing this patch,
> You could drop the example - This is an example which is probably
> better addressed in Documentation/devicetree/bindings/counter/ti-eqep.yaml
> or in the actual device tree for j721e itself.
>
> Instead, I would suggest to keep the commit message short:
>
> Enable TI-QEP for TI's K3 architecture as the driver is reusable
> or something of that form.
>
> The actual devicetree dts patch should be separate and posted once the
> driver support is enabled.
>
> > Overlay used for testing EQEP0 and EQEP1:
> >
> > &cbass_main {
> > #address-cells = <2>;
> > #size-cells = <2>;
> >
> > eqep0: qep@3200000 {
>
> [...]
>
> > };
> > };
Note this is perfectly reasonable detail to have in a cover letter but I agree
with Nishanth that it is too much for the patch description.
Given we have very little for counter so far this cycle, please +CC
gregkh@linuxfoundation.org on v2. That way we can be lazy and ask him to
queue it up directly if that makes sense at the time.
For background to this, as of last cycle I split the IIO and counter
pull requests for the first time.
Thanks,
Jonathan
> >
> > Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
> > ---
> > drivers/counter/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/counter/Kconfig b/drivers/counter/Kconfig
> > index 2de53ab0dd25..453490059c88 100644
> > --- a/drivers/counter/Kconfig
> > +++ b/drivers/counter/Kconfig
> > @@ -51,7 +51,7 @@ config STM32_LPTIMER_CNT
> >
> > config TI_EQEP
> > tristate "TI eQEP counter driver"
> > - depends on (SOC_AM33XX || COMPILE_TEST)
> > + depends on (SOC_AM33XX || ARCH_K3 || COMPILE_TEST)
> > select REGMAP_MMIO
> > help
> > Select this option to enable the Texas Instruments Enhanced Quadrature
> > --
> > 2.30.2
> >
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] counter: ti-eqep: Enable for TDA4VM/J721E
2021-11-30 19:33 [PATCH] counter: ti-eqep: Enable for TDA4VM/J721E Jason Kridner
2021-11-30 21:16 ` Nishanth Menon
@ 2021-12-04 1:46 ` William Breathitt Gray
1 sibling, 0 replies; 4+ messages in thread
From: William Breathitt Gray @ 2021-12-04 1:46 UTC (permalink / raw)
To: Jason Kridner; +Cc: linux-iio, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 2823 bytes --]
On Tue, Nov 30, 2021 at 02:33:46PM -0500, Jason Kridner wrote:
> Counter TI-EQEP was tested on TDA4VM/J721E SoC.
>
> Overlay used for testing EQEP0 and EQEP1:
>
> &cbass_main {
> #address-cells = <2>;
> #size-cells = <2>;
>
> eqep0: qep@3200000 {
> compatible = "ti,am3352-eqep";
> reg = <0 0x3200000 0 0x100>;
> power-domains = <&k3_pds 94 TI_SCI_PD_EXCLUSIVE>;
> clocks = <&k3_clks 94 0>;
> clock-names = "eqep0-ficlk";
> interrupts = <GIC_SPI 322 IRQ_TYPE_EDGE_RISING>;
> interrupt-names = "eqep0";
> symlink = "bone/eQEP0"; /* A - P9_42, B - P9_27 */
>
> count_mode = <0>;
> swap_input = <0>;
> invert_qa = <1>;
> invert_qb = <1>;
> invert_qi = <0>;
> invert_qs = <0>;
> };
>
> eqep1: qep@3210000 {
> compatible = "ti,am3352-eqep";
> reg = <0 0x3210000 0 0x100>;
> power-domains = <&k3_pds 95 TI_SCI_PD_EXCLUSIVE>;
> clocks = <&k3_clks 95 0>;
> clock-names = "eqep1-ficlk";
> interrupts = <GIC_SPI 323 IRQ_TYPE_EDGE_RISING>;
> interrupt-names = "eqep1";
> symlink = "bone/eQEP1"; /* A - P8_35, B - P8_33 */
>
> count_mode = <0>;
> swap_input = <0>;
> invert_qa = <1>;
> invert_qb = <1>;
> invert_qi = <0>;
> invert_qs = <0>;
> };
>
> eqep2: qep@3220000 {
> status = "disabled";
> compatible = "ti,am3352-eqep";
> reg = <0 0x3220000 0 0x100>;
> power-domains = <&k3_pds 96 TI_SCI_PD_EXCLUSIVE>;
> clocks = <&k3_clks 96 0>;
> clock-names = "eqep2-ficlk";
> interrupts = <GIC_SPI 324 IRQ_TYPE_EDGE_RISING>;
> interrupt-names = "eqep2";
>
> count_mode = <0>;
> swap_input = <0>;
> invert_qa = <1>;
> invert_qb = <1>;
> invert_qi = <0>;
> invert_qs = <0>;
> };
> };
>
> Signed-off-by: Jason Kridner <jkridner@beagleboard.org>
Hello Jason,
The change in this patch is a simple architecture enablement so I don't
have a problem with it given that it's been tested. However, I would
like the commit message simplified as Nishanth recommends before we
merge this patch, so please resubmit it to clean that up. Regardless,
for the code change itself I'll give my ack.
Acked-by: William Breathitt Gray <vilhelm.gray@gmail.com>
> ---
> drivers/counter/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/counter/Kconfig b/drivers/counter/Kconfig
> index 2de53ab0dd25..453490059c88 100644
> --- a/drivers/counter/Kconfig
> +++ b/drivers/counter/Kconfig
> @@ -51,7 +51,7 @@ config STM32_LPTIMER_CNT
>
> config TI_EQEP
> tristate "TI eQEP counter driver"
> - depends on (SOC_AM33XX || COMPILE_TEST)
> + depends on (SOC_AM33XX || ARCH_K3 || COMPILE_TEST)
> select REGMAP_MMIO
> help
> Select this option to enable the Texas Instruments Enhanced Quadrature
> --
> 2.30.2
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-12-04 16:29 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-30 19:33 [PATCH] counter: ti-eqep: Enable for TDA4VM/J721E Jason Kridner
2021-11-30 21:16 ` Nishanth Menon
2021-12-04 16:34 ` Jonathan Cameron
2021-12-04 1:46 ` William Breathitt Gray
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox