On Wed, Apr 29, 2020 at 08:28:25PM -0400, vincent.cheng...@renesas.com wrote:
> @@ -871,6 +880,69 @@ static int idtcm_set_pll_mode(struct idtcm_channel 
> *channel,
>  
>  /* PTP Hardware Clock interface */
>  
> +/**
> + * @brief Maximum absolute value for write phase offset in picoseconds
> + *
> + * Destination signed register is 32-bit register in resolution of 50ps
> + *
> + * 0x7fffffff * 50 =  2147483647 * 50 = 107374182350
> + */
> +static int _idtcm_adjphase(struct idtcm_channel *channel, s32 deltaNs)
> +{
> +     struct idtcm *idtcm = channel->idtcm;
> +
> +     int err;
> +     u8 i;
> +     u8 buf[4] = {0};
> +     s32 phaseIn50Picoseconds;
> +     s64 phaseOffsetInPs;

Kernel coding style uses lower_case_underscores rather than lowerCamelCase.

> +
> +     if (channel->pll_mode != PLL_MODE_WRITE_PHASE) {
> +
> +             kthread_cancel_delayed_work_sync(
> +                     &channel->write_phase_ready_work);
> +
> +             err = idtcm_set_pll_mode(channel, PLL_MODE_WRITE_PHASE);
> +
> +             if (err)
> +                     return err;
> +
> +             channel->write_phase_ready = 0;
> +
> +             kthread_queue_delayed_work(channel->kworker,
> +                                        &channel->write_phase_ready_work,
> +                                        msecs_to_jiffies(WR_PHASE_SETUP_MS));

Each PHC driver automatically has a kworker provided by the class
layer.  In order to use it, set ptp_clock_info.do_aux_work to your
callback function and then call ptp_schedule_worker() when needed.

See drivers/net/ethernet/ti/cpts.c for example.

Thanks,
Richard

Reply via email to