From mboxrd@z Thu Jan 1 00:00:00 1970 From: Minkyu Kang Date: Mon, 24 Feb 2014 19:05:06 +0900 Subject: [U-Boot] [PATCH V2 05/12] board:samsung:common: remove unused max77686 init function In-Reply-To: <000001cf312b$283f1010$78bd3030$%wilczek@samsung.com> References: <1390832143-372-1-git-send-email-p.wilczek@samsung.com> <1392300630-5400-1-git-send-email-p.wilczek@samsung.com> <1392300630-5400-6-git-send-email-p.wilczek@samsung.com> <000301cf2969$ef4b5430$cde1fc90$%wilczek@samsung.com> <530853C6.8010301@samsung.com> <000001cf312b$283f1010$78bd3030$%wilczek@samsung.com> Message-ID: <530B1952.7070707@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 24/02/14 15:39, Piotr Wilczek wrote: > Dear Minkyu Kang, > >> -----Original Message----- >> From: Minkyu Kang [mailto:mk7.kang at samsung.com] >> Sent: Saturday, February 22, 2014 8:38 AM >> To: Rajeshwari Birje; Piotr Wilczek; Rajeshwari S Shinde >> Cc: Jaehoon Chung; u-boot at lists.denx.de; Kyungmin Park >> Subject: Re: [U-Boot] [PATCH V2 05/12] board:samsung:common: remove >> unused max77686 init function >> >> Dear Rajeshwari and Piotr, >> >> On 14/02/14 20:40, Rajeshwari Birje wrote: >>> Hi Piotr, >>> >>> On Fri, Feb 14, 2014 at 3:18 PM, Piotr Wilczek >> wrote: >>>> Hi Rajeshwari, >>>> >>>>> -----Original Message----- >>>>> From: Rajeshwari Birje [mailto:rajeshwari.birje at gmail.com] >>>>> Sent: Friday, February 14, 2014 6:32 AM >>>>> To: Piotr Wilczek >>>>> Cc: u-boot at lists.denx.de; Jaehoon Chung; Kyungmin Park; Rajeshwari >> S >>>>> Shinde >>>>> Subject: Re: [U-Boot] [PATCH V2 05/12] board:samsung:common: remove >>>>> unused max77686 init function >>>>> >>>>> Hi Piotr, >>>>> >>>>> On Thu, Feb 13, 2014 at 7:40 PM, Piotr Wilczek >>>>> >>>>> wrote: >>>>>> This patch removes currently unused max77686_init function. >>>>>> Despite being not used, it's implementation is board specific. >>>>>> >>>>> MAX77686 is required for 5250, but missed it somehow when adding >>>>> 5420 support and making a common config file for both. It is my >>>>> mistake will correct the same You can refer: >>>>> "[U-Boot] [PATCH V5 0/6] SMDK5420: Add S2MPS11 pmic support to >>>>> SMDK5420" by Leela Krishna Amudala It adds a generic way for PMIC >>>>> support. >>>>> http://lists.denx.de/pipermail/u-boot/2014-January/171113.html >>>>> >>>> MAX77686 is also used at Trats2 so max77686_init must be either >>>> generic based on DT or moved to the board file. >>> >>> Generic in the sense you want all registers to be set and there >> values >>> have to come from DT file? >>> Which ever you feel OK is fine with me. >>> >> >> So.. do you agree to apply this patch? >> or need another discussion? >> > I will move max77686_init to smdk5250 board file and prepare v3 of this > patch series. > Do you have any other comments to this series? No. looks good to me. Thanks, Minkyu Kang.