| Message ID | 20150319173947.GA12061@amitoj-Inspiron-3542 (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Amitoj Kaur Chawla, On 2015.03.19 19:39, Amitoj Kaur Chawla wrote: > The edits have been made to remove C99 Comments and properly indent > the if-else statements in the file while taking care of the braces according to > Linux coding style. > > Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com> > --- > drivers/staging/sm750fb/ddk750_chip.c | 25 +++++++++++-------------- > 1 file changed, 11 insertions(+), 14 deletions(-) > > diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/staging/sm750fb/ddk750_chip.c > index 33add64..33fa456 100644 > --- a/drivers/staging/sm750fb/ddk750_chip.c > +++ b/drivers/staging/sm750fb/ddk750_chip.c > @@ -17,25 +17,22 @@ logical_chip_type_t getChipType(void) > char physicalRev; > logical_chip_type_t chip; > > - physicalID = devId750;//either 0x718 or 0x750 > + physicalID = devId750;/* either 0x718 or 0x750 */ Add a space here after ;? > physicalRev = revId750; > > - if (physicalID == 0x718) > - { > - chip = SM718; > - } > - else if (physicalID == 0x750) > - { > - chip = SM750; > + if (physicalID == 0x718) { > + chip = SM718; > + } > + else if (physicalID == 0x750) { > + chip = SM750; > /* SM750 and SM750LE are different in their revision ID only. */ > - if (physicalRev == SM750LE_REVISION_ID){ > + if (physicalRev == SM750LE_REVISION_ID) { > chip = SM750LE; > } > - } > - else > - { > - chip = SM_UNKNOWN; > - } > + } > + else { > + chip = SM_UNKNOWN; > + } > > return chip; > } >
On Thu, 2015-03-19 at 23:09 +0530, Amitoj Kaur Chawla wrote: > The edits have been made to remove C99 Comments and properly indent > the if-else statements in the file while taking care of the braces according to > Linux coding style. Please scan your proposed patches with checkpatch and fix possible style defects before sending them. -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/staging/sm750fb/ddk750_chip.c b/drivers/staging/sm750fb/ddk750_chip.c index 33add64..33fa456 100644 --- a/drivers/staging/sm750fb/ddk750_chip.c +++ b/drivers/staging/sm750fb/ddk750_chip.c @@ -17,25 +17,22 @@ logical_chip_type_t getChipType(void) char physicalRev; logical_chip_type_t chip; - physicalID = devId750;//either 0x718 or 0x750 + physicalID = devId750;/* either 0x718 or 0x750 */ physicalRev = revId750; - if (physicalID == 0x718) - { - chip = SM718; - } - else if (physicalID == 0x750) - { - chip = SM750; + if (physicalID == 0x718) { + chip = SM718; + } + else if (physicalID == 0x750) { + chip = SM750; /* SM750 and SM750LE are different in their revision ID only. */ - if (physicalRev == SM750LE_REVISION_ID){ + if (physicalRev == SM750LE_REVISION_ID) { chip = SM750LE; } - } - else - { - chip = SM_UNKNOWN; - } + } + else { + chip = SM_UNKNOWN; + } return chip; }
The edits have been made to remove C99 Comments and properly indent the if-else statements in the file while taking care of the braces according to Linux coding style. Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com> --- drivers/staging/sm750fb/ddk750_chip.c | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-)