Hi William Breathitt Gray, Thanks for the feedback. > Subject: Re: [PATCH RFC 0/8] Add RZ/G2L MTU3a MFD and Counter driver > > On Mon, Sep 26, 2022 at 02:21:06PM +0100, Biju Das wrote: > > The RZ/G2L multi-function timer pulse unit 3 (MTU3a) is embedded in > > the Renesas RZ/G2L family SoC's. It consists of eight 16-bit timer > > channels and one 32-bit timer channel. It supports the following > > functions > > - Counter > > - Timer > > - PWM > > > > This patch series aim to add MFD and counter driver for MTU3a. > > Subsequent patch seies will add TImer and PWM driver support also > > enhancements to counter driver. > > Hello Biju, > > I see this device consists of several channels, but only one Count is > defined in the counter patch ("Channel 1 Count"). Do all channels > support counting, or is it limited to just one channel? It is like this MTU1 channel :- 1 16-bit phase counter MTU2-Channel :- 1 16-bit phase counter MTU1 + MTU2 channel combined:- 1 32-bit phase counter Other channels are not supporting phase counting. Each counter device will have 1 channel. Currently it supports 16-bit phase counting. Please see my test program. Am I missing something here? My test program:- echo 1 > /sys/bus/counter/devices/counter0/count0/enable echo 50 > /sys/bus/counter/devices/counter0/count0/ceiling devmem2 0x10001391 b 0x00 # Enable phase clock selection A for MTU2. echo 1 > /sys/bus/counter/devices/counter1/count0/enable echo 50 > /sys/bus/counter/devices/counter1/count0/ceiling for i in {1..5}; do cat /sys/bus/counter/devices/counter0/count0/count ; cat /sys/bus/counter/devices/counter0/count0/direction; cat /sys/bus/counter/devices/counter1/count0/count; cat /sys/bus/counter/devices/counter1/count0/direction; done Cheers, Biju