On Fri, Aug 23, 2013 at 02:40:47AM -0700, Marc C wrote: > +static inline int ata_fpdma_dsm_supported(struct ata_device *dev) bool? > +{ > + if ((dev->flags & ATA_DFLAG_NCQ_SEND_RECV) && > + (dev->ncq_send_recv_cmds[ATA_LOG_NCQ_SEND_RECV_DSM_OFFSET] & > + ATA_LOG_NCQ_SEND_RECV_DSM_TRIM)) { > + return 1; > + } else { > + return 0; > + } And again, you can just use a return statement. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html