From: Guennadi Liakhovetski <guennadi.liakhovetski@xxxxxxxxxxxxxxx> Doing "if (a >= b) return 1; return 0;" can be trivially simplified to "return a >= b;" Use this for soc_tplg_is_eof(). Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@xxxxxxxxxxxxxxx> --- sound/soc/soc-topology.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/sound/soc/soc-topology.c b/sound/soc/soc-topology.c index 045ef136903d..041e7c680a6b 100644 --- a/sound/soc/soc-topology.c +++ b/sound/soc/soc-topology.c @@ -111,11 +111,7 @@ static int soc_tplg_check_elem_count(struct soc_tplg *tplg, size_t elem_size, static inline int soc_tplg_is_eof(struct soc_tplg *tplg) { - const u8 *end = tplg->hdr_pos; - - if (end >= tplg->fw->data + tplg->fw->size) - return 1; - return 0; + return tplg->hdr_pos >= tplg->fw->data + tplg->fw->size; } static inline unsigned long soc_tplg_get_hdr_offset(struct soc_tplg *tplg) -- 2.17.1 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel