mirror of https://gitee.com/openkylin/linux.git
tty: nozomi: remove unneeded break
A break is not needed if it is preceded by a return Add explicit fallthrough Signed-off-by: Tom Rix <trix@redhat.com> Link: https://lore.kernel.org/r/20201019175732.3289-1-trix@redhat.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
974e454d6f
commit
1a460c3607
|
@ -414,11 +414,9 @@ static void read_mem32(u32 *buf, const void __iomem *mem_addr_start,
|
||||||
buf16 = (u16 *) buf;
|
buf16 = (u16 *) buf;
|
||||||
*buf16 = __le16_to_cpu(readw(ptr));
|
*buf16 = __le16_to_cpu(readw(ptr));
|
||||||
goto out;
|
goto out;
|
||||||
break;
|
|
||||||
case 4: /* 4 bytes */
|
case 4: /* 4 bytes */
|
||||||
*(buf) = __le32_to_cpu(readl(ptr));
|
*(buf) = __le32_to_cpu(readl(ptr));
|
||||||
goto out;
|
goto out;
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
while (i < size_bytes) {
|
while (i < size_bytes) {
|
||||||
|
@ -460,15 +458,14 @@ static u32 write_mem32(void __iomem *mem_addr_start, const u32 *buf,
|
||||||
buf16 = (const u16 *)buf;
|
buf16 = (const u16 *)buf;
|
||||||
writew(__cpu_to_le16(*buf16), ptr);
|
writew(__cpu_to_le16(*buf16), ptr);
|
||||||
return 2;
|
return 2;
|
||||||
break;
|
|
||||||
case 1: /*
|
case 1: /*
|
||||||
* also needs to write 4 bytes in this case
|
* also needs to write 4 bytes in this case
|
||||||
* so falling through..
|
* so falling through..
|
||||||
*/
|
*/
|
||||||
|
fallthrough;
|
||||||
case 4: /* 4 bytes */
|
case 4: /* 4 bytes */
|
||||||
writel(__cpu_to_le32(*buf), ptr);
|
writel(__cpu_to_le32(*buf), ptr);
|
||||||
return 4;
|
return 4;
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
while (i < size_bytes) {
|
while (i < size_bytes) {
|
||||||
|
|
Loading…
Reference in New Issue