Re: [Outreachy kernel] [PATCH 2/7] staging: vt6655: Use incrementation in `idx`

From: Karolina Drobnik
Date: Mon Nov 01 2021 - 06:08:41 EST


Hi Praveen,

On Fri, 2021-10-29 at 21:11 +0530, Praveen Kumar wrote:
> However, as mentioned in previous patch, we can also make *i* as
> unsigned short and that should IMO remove the requirement of cast.

I agree, it should remove the cast but I think I'll leave `idx` as it
is.

> But again, this works as well. I'm fine with either. Thanks and sorry
> for the confusion.

No worries, thanks for looking at my changes. It's good to double check
everything :)


Thanks,
Karolina