| Message ID | 20200728082330.24948-1-b18007@students.iitmandi.ac.in (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2] Staging : media : atomisp : fixed a brace coding sytle issue | expand |
On Tue, Jul 28, 2020 at 01:53:30PM +0530, Ankit wrote: > From: Ankit Baluni <b18007@students.iitmandi.ac.in> > > Removed braces for a 'if' condition as it contain only single line & > there is no need for braces for such case according to coding style > rules. > > Signed-off-by: Ankit Baluni <b18007@students.iitmandi.ac.in> > > --- > drivers/staging/media/atomisp/pci/atomisp_cmd.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) What changed from v1? Always put that below the --- line as the documentation asks for. thanks, greg k-h
diff --git a/drivers/staging/media/atomisp/pci/atomisp_cmd.c b/drivers/staging/media/atomisp/pci/atomisp_cmd.c index 8ea65bef35d2..28b96b66f4f3 100644 --- a/drivers/staging/media/atomisp/pci/atomisp_cmd.c +++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.c @@ -4981,9 +4981,8 @@ enum mipi_port_id __get_mipi_port(struct atomisp_device *isp, case ATOMISP_CAMERA_PORT_SECONDARY: return MIPI_PORT1_ID; case ATOMISP_CAMERA_PORT_TERTIARY: - if (MIPI_PORT1_ID + 1 != N_MIPI_PORT_ID) { + if (MIPI_PORT1_ID + 1 != N_MIPI_PORT_ID) return MIPI_PORT1_ID + 1; - } /* fall through */ default: dev_err(isp->dev, "unsupported port: %d\n", port);