Hi,Ok. WIll update it
On Tue, Sep 25, 2018 at 12:36 PM <dkota@xxxxxxxxxxxxxx> wrote:
>> + unsigned int cur_speed_hz;
>
> unsigned long for Hz? The clk framework uses that type.
cur_speed_hz stores the speed value requested as part of transfer (not
the resultant or rounded off frequency got from clk framework. It is u32
type, i will change cur_speed_hz to u32 type instead of unsigned long.
Code snippet:
mas->cur_speed_hz = xfer->speed_hz;
Change it to "unsigned long" anyway to match the clock framework. In
theory maybe the "xfer" structure will be updated eventually.
Added the changes in V5 to ensure return value doesn't get overwritten.Is it possible for all three conditions above to happen in oneWill check it again by running fullduplex transfer and update.
interrupt? I ask because 'ret' is overwritten and so what may have been
IRQ_HANDLED may become IRQ_NONE which will lead to confusion in the irq
layer. Maybe the handle tx/rx functions can return a bool, that gets
orred together each time so that we know if something handled an
interrupt?