Hi Uwe, > -----Original Message----- > From: Uwe Kleine-König <ukleinek@xxxxxxxxxx> > Sent: 05 December 2024 08:28 > Subject: Re: [PATCH v22 3/4] pwm: Add support for RZ/G2L GPT > > Hello Biju, > > > According to me, we should not allow updating periods for second enabled channel. > > Not entirely sure you mean the right thing here. If IO A operates at 5ns and IO B is requested to > set .period = 5 ms, every operation that also changes A is out-of-bounds. So your options are only: > Use the 5ns, or return an error. The latter is hard for consumers because it's unclear what to do > then. I guess, these 2 IO's mostly used in switching circuits. So, can't we return error, if period of IO_A != IO_B? Then the user know the mistake and he will configure with proper values?? For eg: Case 1: IO_A= 5msec, IO_B=5nsec Here IO_B has faster switching(MHz) compared to IO_A (kHZ), So, by allowing IO_A to operate at IO_B frequency, we are doing Some incorrect operation for IO_A. Case 2: IO_A= 5nsec, IO_B=5msec Here we are returning error, that is ok. Cheers, Biju
