RE: [EXT] Re: Re: [PATCH] spi: imx: add a check for speed_hz before calculating the clock

From: Clark Wang
Date: Tue Apr 27 2021 - 07:25:35 EST



> -----Original Message-----
> From: Mark Brown <broonie@xxxxxxxxxx>
> Sent: Tuesday, April 27, 2021 19:19
> To: Clark Wang <xiaoning.wang@xxxxxxx>
> Cc: shawnguo@xxxxxxxxxx; s.hauer@xxxxxxxxxxxxxx; festevam@xxxxxxxxx;
> kernel@xxxxxxxxxxxxxx; dl-linux-imx <linux-imx@xxxxxxx>; linux-
> spi@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx
> Subject: [EXT] Re: Re: [PATCH] spi: imx: add a check for speed_hz before
> calculating the clock
>
> On Tue, Apr 27, 2021 at 08:33:06AM +0000, Clark Wang wrote:
>
> > However, I notice that you have applied this patch to the next branch?
> > Will you revert this patch?
> > I think you may want to apply this patch I sent before.
> >
> > Author: Clark Wang <xiaoning.wang@xxxxxxx>
> > Date: Mon Dec 14 17:05:04 2020 +0800
> >
> > spi: imx: add 16/32 bits per word support for slave mode
>
> Oh, in this case what happened is that you sent your speed_hz patch as a
> reply to this patch so the speed_hz patch looked like a replacement for it
> which confused both me and the tooling.

I'm sorry to cause you confusion.
1) spi: imx: add 16/32 bits per word support for slave mode and B
2) spi: imx: add a check for speed_hz before calculating the clock
These two patch above are two independent patches.
Now 2) is no longer needed, I just sent 3) to fix the real problem.
3) spi: imx: remove CLK calculation and divider in slave mode

Thank you very much!

Best Regards,
Clark Wang

Attachment: smime.p7s
Description: S/MIME cryptographic signature