Commit e373991e authored by Uwe Kleine-König's avatar Uwe Kleine-König Committed by Uwe Kleine-König
Browse files

pwm: rzg2l-gpt: Accept requests for too high period length 



The period setting is shared for each pair of PWM channels. So if the
twin channel is in use, the period must not be changed. According to the
usual practise to pick the next smaller possible period, accept a
request for a period that is bigger than the unchangable value.

Signed-off-by: default avatarUwe Kleine-König <u.kleine-koenig@baylibre.com>
Tested-by: default avatarBiju Das <biju.das.jz@bp.renesas.com>
Link: https://lore.kernel.org/r/20250423095715.2952692-2-u.kleine-koenig@baylibre.com


Signed-off-by: default avatarUwe Kleine-König <ukleinek@kernel.org>
parent b6b5683e

drivers/pwm/pwm-rzg2l-gpt.c

+8 −4
Original line number Diff line number Diff line
@@ -270,15 +270,19 @@ static int rzg2l_gpt_config(struct pwm_chip *chip, struct pwm_device *pwm, 
	 * prescale and period can NOT be modified when there are multiple IOs

	 * in use with different settings.

	 */

	if (rzg2l_gpt->channel_request_count[ch] > 1 && period_ticks != rzg2l_gpt->period_ticks[ch])

	if (rzg2l_gpt->channel_request_count[ch] > 1) {

		if (period_ticks < rzg2l_gpt->period_ticks[ch])

			return -EBUSY;

		else

			period_ticks = rzg2l_gpt->period_ticks[ch];

	}



	prescale = rzg2l_gpt_calculate_prescale(rzg2l_gpt, period_ticks);

	pv = rzg2l_gpt_calculate_pv_or_dc(period_ticks, prescale);



	duty_ticks = mul_u64_u64_div_u64(state->duty_cycle, rzg2l_gpt->rate_khz, USEC_PER_SEC);

	if (duty_ticks > RZG2L_MAX_TICKS)

		duty_ticks = RZG2L_MAX_TICKS;

	if (duty_ticks > period_ticks)

		duty_ticks = period_ticks;

	dc = rzg2l_gpt_calculate_pv_or_dc(duty_ticks, prescale);



	/*