From b4d2c67bddd1a53fe91a9c331ee50ffa3e9bc0cd Mon Sep 17 00:00:00 2001 From: Jeenu Viswambharan Date: Tue, 21 Feb 2017 16:01:31 +0000 Subject: [PATCH] Remove redundant assert Static checks flag an assert added in commit 1f786b0 that compares unsigned value to 0, which will never fail. Change-Id: I4b02031c2cfbd9a25255d12156919dda7d4805a0 Signed-off-by: Jeenu Viswambharan --- drivers/io/io_memmap.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/io/io_memmap.c b/drivers/io/io_memmap.c index 5104fb19a..a97df6b29 100644 --- a/drivers/io/io_memmap.c +++ b/drivers/io/io_memmap.c @@ -125,8 +125,6 @@ static int memmap_block_open(io_dev_info_t *dev_info, const uintptr_t spec, int result = -ENOMEM; const io_block_spec_t *block_spec = (io_block_spec_t *)spec; - assert(block_spec->length >= 0); - /* Since we need to track open state for seek() we only allow one open * spec at a time. When we have dynamic memory we can malloc and set * entity->info.