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