From mboxrd@z Thu Jan 1 00:00:00 1970 From: walter harms Date: Mon, 20 Nov 2017 20:15:27 +0000 Subject: Re: [PATCH] USB: serial: iuu_phoenix: remove redundant assignment of DIV to itself Message-Id: <5A1337DF.2060404@bfs.de> List-Id: References: <20171120174015.7191-1-colin.king@canonical.com> In-Reply-To: <20171120174015.7191-1-colin.king@canonical.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Colin King Cc: Johan Hovold , Greg Kroah-Hartman , linux-usb@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Am 20.11.2017 18:40, schrieb Colin King: > From: Colin Ian King > > The assignment of DIV to itself is redundant and can be removed. > > Signed-off-by: Colin Ian King > --- > drivers/usb/serial/iuu_phoenix.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/usb/serial/iuu_phoenix.c b/drivers/usb/serial/iuu_phoenix.c > index 397a8012ffa3..62c91e360baf 100644 > --- a/drivers/usb/serial/iuu_phoenix.c > +++ b/drivers/usb/serial/iuu_phoenix.c > @@ -472,7 +472,6 @@ static int iuu_clk(struct usb_serial_port *port, int dwFrq) > } > } > P2 = ((P - PO) / 2) - 4; > - DIV = DIV; > PUMP = 0x04; > PBmsb = (P2 >> 8 & 0x03); > PBlsb = P2 & 0xFF; These all all-upper-case stuff makes me a bit nervous. Normally this is reserved for #define ( I assume that the programmer refers to the original documentation) a point to change ? btw: i noticed int frq = (int)dwFrq; since dwFrq is already an in, the cast is useless. just ym 2 cents, re, wh