From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752357AbbDBXjb (ORCPT ); Thu, 2 Apr 2015 19:39:31 -0400 Received: from mailout1.samsung.com ([203.254.224.24]:11681 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751295AbbDBXj1 (ORCPT ); Thu, 2 Apr 2015 19:39:27 -0400 X-AuditID: cbfee690-f79ab6d0000046f7-9b-551dd322d386 Message-id: <551DD321.6030707@samsung.com> Date: Fri, 03 Apr 2015 08:39:13 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130106 Thunderbird/17.0.2 MIME-version: 1.0 To: Mark Rutland Cc: "kgene@kernel.org" , Marc Zyngier , "arnd@arndb.de" , "olof@lixom.net" , Catalin Marinas , Will Deacon , "inki.dae@samsung.com" , "chanho61.park@samsung.com" , "sw0312.kim@samsung.com" , "jh80.chung@samsung.com" , "ideal.song@samsung.com" , "a.kesavan@samsung.com" , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-samsung-soc@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v7 1/9] arm64: dts: exynos: Add dts files for 64-bit Exynos5433 SoC References: <1426637856-3730-1-git-send-email-cw00.choi@samsung.com> <1426637856-3730-2-git-send-email-cw00.choi@samsung.com> <20150330160940.GA13731@leverpostej> <5519E2B6.2030905@samsung.com> <20150402173558.GB30669@leverpostej> In-reply-to: <20150402173558.GB30669@leverpostej> Content-type: text/plain; charset=ISO-8859-1 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrLIsWRmVeSWpSXmKPExsWyRsSkRFfpsmyowZHfShaP1yxmsvg76Ri7 xftlPYwWl/drW8w/co7VYtff+4wWk+5PYLG48auN1aL/8Wtmi02Pr7FaXN41h81ixvl9QF13 /rFZLL1+kcni1PXPQLHJL9ksXn48weIg6LFm3hpGj9+/JjF6bFrVyeaxeUm9x5UTTawefVtW MXp83iQXwB7FZZOSmpNZllqkb5fAlfGncydTwUP+iv/L8hoYN/F0MXJySAiYSDQdfc8CYYtJ XLi3nq2LkYtDSGApo8T/J7/YYYouXnrKApGYziixr/0oVNUDRok1HcsZQap4BbQkZqxoBbI5 OFgEVCX+Pw4DCbMBhfe/uMEGYosKhEmsnH6FBaJcUOLH5HtgtoiAukTPri9gNrPAHjaJTW1i ILawQKTE+/3roRa/YZTY/7iTCSTBKWAo0friMitEg47E/tZpbBC2vMTmNW+ZQRokBOZySGy8 +YAZJMEiICDxbfIhFpDjJARkJTYdYIb4TFLi4IobLBMYxWYhuWkWkrGzkIxdwMi8ilE0tSC5 oDgpvchErzgxt7g0L10vOT93EyMw1k//ezZhB+O9A9aHGAU4GJV4eDP2yIQKsSaWFVfmHmI0 BbpiIrOUaHI+MKHklcQbGpsZWZiamBobmVuaKYnzvpb6GSwkkJ5YkpqdmlqQWhRfVJqTWnyI kYmDU6qBsbviUlPO/Pt8k/8aCBcfjpQvOqthdyzr4dSUBbv5VqvrJDO172TufD5B5F7T8/yX Nle8Vt9Rd5kY2WHP88pzQ4/gykOM69zOX5d/XPvD+vqspytOVj6VT+m4/+ttjNnyC3/f/GcI 3nXDXTBvqeFif37HY80L70Smd/pZ9LhySEYauT5jTagTUmIpzkg01GIuKk4EADZQ3pDwAgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrOKsWRmVeSWpSXmKPExsVy+t9jQV2ly7KhBicnSFo8XrOYyeLvpGPs Fu+X9TBaXN6vbTH/yDlWi11/7zNaTLo/gcXixq82Vov+x6+ZLTY9vsZqcXnXHDaLGef3AXXd +cdmsfT6RSaLU9c/A8Umv2SzePnxBIuDoMeaeWsYPX7/msTosWlVJ5vH5iX1HldONLF69G1Z xejxeZNcAHtUA6NNRmpiSmqRQmpecn5KZl66rZJ3cLxzvKmZgaGuoaWFuZJCXmJuqq2Si0+A rltmDtAHSgpliTmlQKGAxOJiJX07TBNCQ9x0LWAaI3R9Q4LgeowM0EDCGsaMP507mQoe8lf8 X5bXwLiJp4uRk0NCwETi4qWnLBC2mMSFe+vZuhi5OIQEpjNK7Gs/CuU8YJRY07GcEaSKV0BL YsaKViCbg4NFQFXi/+MwkDAbUHj/ixtsILaoQJjEyulXWCDKBSV+TL4HZosIqEv07PoCZjML 7GGT2NQmBmILC0RKvN+/ngVi1xtGif2PO5lAEpwChhKtLy6zQjToSOxvncYGYctLbF7zlnkC o8AsJDtmISmbhaRsASPzKkbR1ILkguKk9FwjveLE3OLSvHS95PzcTYzgRPJMegfjqgaLQ4wC HIxKPLwZe2RChVgTy4orcw8xSnAwK4nwrtgtGyrEm5JYWZValB9fVJqTWnyI0RQYAhOZpUST 84FJLq8k3tDYxMzI0sjc0MLI2FxJnFfJvi1ESCA9sSQ1OzW1ILUIpo+Jg1OqgbH2bemCo/q+ MrxfV6z9W7r174o/zo3hllM+53p5vf62U+ZDqY1Lr/6BjQstd7EmnbPeyBp+a295h+mJlFed cS5Fx6wDW8q3VnCZzLx+cH6jTcukWUxBJxwF3k+oF/3jOnXnPcdKKcsFnLa/7S8X8b/LX1/8 Na/R6szd3KSfP5y+OXXZJhSnBymxFGckGmoxFxUnAgDEBvHHOgMAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/03/2015 02:35 AM, Mark Rutland wrote: > On Tue, Mar 31, 2015 at 12:56:38AM +0100, Chanwoo Choi wrote: >> Hi Mark, >> >> On 03/31/2015 01:09 AM, Mark Rutland wrote: >>> Hi, >>> >>> On Wed, Mar 18, 2015 at 12:17:28AM +0000, Chanwoo Choi wrote: >>>> This patch adds new Exynos5433 dtsi to support 64-bit Exynos5433 SoC based on >>>> Octal core CPUs (quad Cortex-A57 and quad Cortex-A53). And Exynos5433 supports >>>> PSCI (Power State Coordination Interface) v0.1. >>>> >>>> This patch includes following dt node to support Exynos5433 SoC: >>>> 1. Octa core for big.LITTLE architecture >>>> - Cortex-A53 LITTLE Quad-core >>>> - Cortex-A57 big Quad-core >>>> - Support PSCI v0.1 >>> >>> Is CPU0 hotplug still broken, or has the FW been fixed? >> >> CPU0 hotplug is still not working. >> >>> >>> I'm very worried about adding a DT that's known broken, especially when >>> we have no idea as to if/when the FW will be fixed judging from prior >>> replies. >> >> As I replied, I can not fix the FW because I don't have any code of FW. > > Surely you are able to contact those who do? > >> I don't have any solution to fix it on Linux kernel level. >> >> So, If you agree, I can add the comment of CPU0 hotplug issue on DT file. > > I disagree. I do not want to add a DT that is known to be broken; > especially when we have no idea how to fix it. It creates long-term > maintenance pain for everyone, and marginal gain for few. A comment does > nothing to aid the end-user. > > So NAK to the PSCI node and PSCI enable method in this dts until we > either have a working firmware, or a reasonable mechanism to handle the > deficiency. There is only CPU0 hotplug issue. Excpet CPU{1-7} are well working. To fix this issue, we must need the help of firmware developer. But, We never get the any help. Also, as I mentioned on previous mail, all Exynos SoCs can not turn off the CPU0. I've never seen Exynos SoC that CP0 hotplug is possible. Thanks, Chanwoo Choi