Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751353AbdISJZJ (ORCPT ); Tue, 19 Sep 2017 05:25:09 -0400 Received: from mx07-00178001.pphosted.com ([62.209.51.94]:29654 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750982AbdISJZH (ORCPT ); Tue, 19 Sep 2017 05:25:07 -0400 From: Gabriel FERNANDEZ To: "mturquette@baylibre.com" , "sboyd@codeaurora.org" , Alexandre TORGUE , "linux-clk@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , David Binderma Subject: Re: linux-4.14-rc1/drivers/clk/clk-stm32h7.c: 2 * possible typo ? Thread-Topic: linux-4.14-rc1/drivers/clk/clk-stm32h7.c: 2 * possible typo ? Thread-Index: AQHTMSfo2BeTO7hs90yo0g46AnUoVaK7zbEAgAAAqIA= Date: Tue, 19 Sep 2017 09:24:29 +0000 Message-ID: <0abb5cce-f98f-b3c1-4d53-7ae4bd0e44dd@st.com> References: <12ce6ec0-d351-4ffe-a5a0-cdc09f6aee39@st.com> <73cadb19-a75e-4938-265f-890cc5a4d3b4@st.com> In-Reply-To: <73cadb19-a75e-4938-265f-890cc5a4d3b4@st.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.46] Content-Type: text/plain; charset="utf-8" Content-ID: MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-09-19_04:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id v8J9PDw3004864 Content-Length: 540 Lines: 33 I missed David ... On 09/19/2017 11:22 AM, Gabriel FERNANDEZ wrote: > Hi David, > > I confirm it's typo issue. I 'm ready to send a fix for that. > > Many Thank's > > Gabriel > >> Hello there, >> >> 1. >> >> linux-4.14-rc1/drivers/clk/clk-stm32h7.c:387]: (style) Same expression >> on both sides of '&&'. >> >> Source code is >> >>     if (gcfg->mux && gcfg->mux) { >> >> 2. >> >> [linux-4.14-rc1/drivers/clk/clk-stm32h7.c:413]: (style) Same expression >> on both sides of '&&'. >> >> Duplicate. >> >> Regards >> >> David Binderman