Hey, again.
On Sun, Dec 02, 2012 at 05:45:15AM -0800, Tejun Heo wrote:
> > +/* Return true if and only if scsi_remove_host() is allowed to finish. */
> > +static bool __scsi_remove_host_done(struct Scsi_Host *shost)
> > +{
> > + lockdep_assert_held(shost->host_lock);
> > +
> > + return list_empty(&shost->__devices);
> > +}
>
> This is a preference thing but I usually find this type of trivial
> wrappers more obfuscating than actually helpful. Dunno what James's
> preference is tho.
Please disregard this one. You're adding more conditions afterwards.
Thanks.
--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
