From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758640AbbCDOrd (ORCPT ); Wed, 4 Mar 2015 09:47:33 -0500 Received: from mailout4.w1.samsung.com ([210.118.77.14]:43509 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755514AbbCDOrb (ORCPT ); Wed, 4 Mar 2015 09:47:31 -0500 X-AuditID: cbfec7f5-b7fc86d0000066b7-8a-54f71a6b7555 From: Kamil Debski To: "'Axel Lin'" , "'Kishon Vijay Abraham I'" Cc: "'Kukjin Kim'" , linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org References: <1425344923.4315.3.camel@phoenix> In-reply-to: <1425344923.4315.3.camel@phoenix> Subject: RE: [PATCH] phy: samsung-usb2: Remove NULL terminating entry from phys array Date: Wed, 04 Mar 2015 15:47:28 +0100 Message-id: <07b201d0568a$234c4160$69e4c420$%debski@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 Content-transfer-encoding: 7bit X-Mailer: Microsoft Office Outlook 12.0 Thread-index: AdBVTp36JSght5YRRsGSM0NecxGt3ABOwj9A Content-language: pl X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrDLMWRmVeSWpSXmKPExsVy+t/xq7rZUt9DDK7OsrE4Mucrs0X/49fM Fhee9rBZXN41h81ixvl9TA6sHm0/yzw2repk8zh+YzuTx+dNcgEsUVw2Kak5mWWpRfp2CVwZ Dc/nMxasFKh41enRwLiNt4uRg0NCwERi7/uELkZOIFNM4sK99WxdjFwcQgJLGSVW9yxgAkkI CTQwSfSsUwCpZxPQlFh1zwMkLCLgJ9G+YSkjiM0skCax+Og9RohyXYljX1rYQWxOAT2Jgw1L 2EBsYYFwiVmTu1hBxrAIqEos2BkNEuYVcJCYd3kuM4QtKPFj8j0WiJHqEpPmLWKGsOUlNq95 ywxxsbrEo7+6EBcYSTxZPgmqRETibsNz1gmMQrOQTJqFZNIsJJNmIWlZwMiyilE0tTS5oDgp PddIrzgxt7g0L10vOT93EyMkBr7uYFx6zOoQowAHoxIPb0HStxAh1sSy4srcQ4wSHMxKIrzP 2b6HCPGmJFZWpRblxxeV5qQWH2Jk4uCUamA8Gx2yw9O2NFb9zp6J6zSzGX43fX+juexCUYnh tqrrayvs7EMvxZRoP7KdkeH4t6o0Jeqo7N9zxs32KxYwRs+U/LRE85JS0yxRfgkjP06PrVqc k7Pmfbtwfe41zdb3zIaF2VplHipst6Q32i0XOH4uUy8nJM6n8kXye92Tee593//tXKp6WESJ pTgj0VCLuag4EQArIbtLXwIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Axel Lin [mailto:axel.lin@ingics.com] Sent: Tuesday, March 03, 2015 2:09 AM > > Current code uses num_phys settings to tell the number of entries in > phys. > Thus remove the NULL terminating entry from phys array which is not > necessary. The patch looks good. > Signed-off-by: Axel Lin Acked-by: Kamil Debski Best wishes, -- Kamil Debski Samsung R&D Institute Poland > --- > drivers/phy/phy-exynos4210-usb2.c | 1 - drivers/phy/phy-exynos4x12- > usb2.c | 1 - drivers/phy/phy-exynos5250-usb2.c | 1 - > 3 files changed, 3 deletions(-) > > diff --git a/drivers/phy/phy-exynos4210-usb2.c b/drivers/phy/phy- > exynos4210-usb2.c > index 236a52a..f30bbb0 100644 > --- a/drivers/phy/phy-exynos4210-usb2.c > +++ b/drivers/phy/phy-exynos4210-usb2.c > @@ -250,7 +250,6 @@ static const struct samsung_usb2_common_phy > exynos4210_phys[] = { > .power_on = exynos4210_power_on, > .power_off = exynos4210_power_off, > }, > - {}, > }; > > const struct samsung_usb2_phy_config exynos4210_usb2_phy_config = { > diff --git a/drivers/phy/phy-exynos4x12-usb2.c b/drivers/phy/phy- > exynos4x12-usb2.c > index 0b9de88..765da90 100644 > --- a/drivers/phy/phy-exynos4x12-usb2.c > +++ b/drivers/phy/phy-exynos4x12-usb2.c > @@ -361,7 +361,6 @@ static const struct samsung_usb2_common_phy > exynos4x12_phys[] = { > .power_on = exynos4x12_power_on, > .power_off = exynos4x12_power_off, > }, > - {}, > }; > > const struct samsung_usb2_phy_config exynos3250_usb2_phy_config = { > diff --git a/drivers/phy/phy-exynos5250-usb2.c b/drivers/phy/phy- > exynos5250-usb2.c > index 1c139aa..2ed1735 100644 > --- a/drivers/phy/phy-exynos5250-usb2.c > +++ b/drivers/phy/phy-exynos5250-usb2.c > @@ -391,7 +391,6 @@ static const struct samsung_usb2_common_phy > exynos5250_phys[] = { > .power_on = exynos5250_power_on, > .power_off = exynos5250_power_off, > }, > - {}, > }; > > const struct samsung_usb2_phy_config exynos5250_usb2_phy_config = { > -- > 1.9.1 >