From: Dan Carpenter <dan.carpenter@linaro.org>
To: Johannes Stezenbach <js@sig21.net>
Cc: linux-media@vger.kernel.org
Subject: [bug report] [PATCH] dvb: b2c2/flexcop driver refactoring part 2: add modular Flexcop driver
Date: Sat, 4 May 2024 14:24:21 +0300 [thread overview]
Message-ID: <e42e10a1-f414-40c1-a804-36bc67270088@moroto.mountain> (raw)
[ This patch is 19 years old now... :P Sorry! - dan ]
Hello Johannes Stezenbach,
Commit 2add87a95068 ("[PATCH] dvb: b2c2/flexcop driver refactoring
part 2: add modular Flexcop driver") from May 16, 2005 (linux-next),
leads to the following Smatch static checker warning:
drivers/media/usb/b2c2/flexcop-usb.c:199 flexcop_usb_memory_req()
warn: iterator 'i' not incremented
drivers/media/usb/b2c2/flexcop-usb.c
178 static int flexcop_usb_memory_req(struct flexcop_usb *fc_usb,
179 flexcop_usb_request_t req, flexcop_usb_mem_page_t page_start,
180 u32 addr, int extended, u8 *buf, u32 len)
181 {
182 int i, ret = 0;
183 u16 wMax;
184 u32 pagechunk = 0;
185
186 switch (req) {
187 case B2C2_USB_READ_V8_MEM:
188 wMax = USB_MEM_READ_MAX;
189 break;
190 case B2C2_USB_WRITE_V8_MEM:
191 wMax = USB_MEM_WRITE_MAX;
192 break;
193 case B2C2_USB_FLASH_BLOCK:
194 wMax = USB_FLASH_MAX;
195 break;
196 default:
197 return -EINVAL;
198 }
--> 199 for (i = 0; i < len;) {
No i++.
200 pagechunk =
201 wMax < bytes_left_to_read_on_page(addr, len) ?
202 wMax :
203 bytes_left_to_read_on_page(addr, len);
204 deb_info("%x\n",
205 (addr & V8_MEMORY_PAGE_MASK) |
206 (V8_MEMORY_EXTENDED*extended));
207
208 ret = flexcop_usb_v8_memory_req(fc_usb, req,
209 page_start + (addr / V8_MEMORY_PAGE_SIZE),
210 (addr & V8_MEMORY_PAGE_MASK) |
211 (V8_MEMORY_EXTENDED*extended),
212 &buf[i], pagechunk);
^^^^^^^^
I think adding an i++ doesn't make sense. Are we really writing a byte
at a time?
213
214 if (ret < 0)
215 return ret;
216 addr += pagechunk;
217 len -= pagechunk;
218 }
219 return 0;
220 }
regards,
dan carpenter
next reply other threads:[~2024-05-04 11:24 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-04 11:24 Dan Carpenter [this message]
2024-05-04 20:24 ` [bug report] [PATCH] dvb: b2c2/flexcop driver refactoring part 2: add modular Flexcop driver Johannes Stezenbach
2024-05-06 5:56 ` Dan Carpenter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e42e10a1-f414-40c1-a804-36bc67270088@moroto.mountain \
--to=dan.carpenter@linaro.org \
--cc=js@sig21.net \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).