Messages in this thread | | | Date | Sat, 2 Sep 2017 13:37:24 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH 05/11] mux: Add Intel Cherrytrail USB mux driver |
| |
On Sat, Sep 02, 2017 at 01:19:05PM +0300, Andy Shevchenko wrote: > > +static void intel_cht_usb_mux_vbus_work(struct work_struct *work) > > +{ > > + struct intel_cht_usb_mux *mux = > > + container_of(work, struct intel_cht_usb_mux, vbus_work); > > + bool vbus_present = false; > > + int i; > > unsigned int i; ? >
int i is fine. Static checkers which complain about stuff like that should be uninstalled.
> > + > > + for (i = 0; i < ARRAY_SIZE(vbus_cable_ids); i++) {
regards, dan carpenter
| |