From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [RFC v2 16/18] ARM: OMAP2+: AM33XX: Basic suspend resume support Date: Wed, 20 Feb 2013 06:30:00 -0800 Message-ID: <87zjyzqcrb.fsf@linaro.org> References: <1356959231-17335-1-git-send-email-vaibhav.bedia@ti.com> <1356959231-17335-17-git-send-email-vaibhav.bedia@ti.com> <87k3qewcbt.fsf@linaro.org> <87lialzjne.fsf@linaro.org> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from mail-da0-f42.google.com ([209.85.210.42]:61116 "EHLO mail-da0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757742Ab3BTOaE (ORCPT ); Wed, 20 Feb 2013 09:30:04 -0500 Received: by mail-da0-f42.google.com with SMTP id z17so3588397dal.29 for ; Wed, 20 Feb 2013 06:30:04 -0800 (PST) In-Reply-To: (Vaibhav Bedia's message of "Wed, 20 Feb 2013 09:21:41 +0000") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Bedia, Vaibhav" Cc: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "tony@atomide.com" , "Shilimkar, Santosh" , "Cousson, Benoit" , Paul Walmsley "Bedia, Vaibhav" writes: [...] >> IMO, this would be a much cleaner implementation if you just created a >> small driver for the wkup_m3. You're already doing a bunch of >> driver-like stuff for it (requesting base/IRQs, mapping regions, >> firmware, etc.) I think this should separated out into a real driver. >> Then it will be bound to the right omap_device, and normal PM operations >> will work as expected. >> >> Also, doing it this way will be more flexible for those wanting to use >> their own firmware on the M3, or customize the current firmware. > > Hmm... that definitely sounds more flexible. It should also help in the next SoC > AM437x which has a similar PM architecture. Where would you suggest placing > this minimal driver? For now, just leave it in mach-omap2 and we can figure out the right home for it eventually. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@linaro.org (Kevin Hilman) Date: Wed, 20 Feb 2013 06:30:00 -0800 Subject: [RFC v2 16/18] ARM: OMAP2+: AM33XX: Basic suspend resume support In-Reply-To: (Vaibhav Bedia's message of "Wed, 20 Feb 2013 09:21:41 +0000") References: <1356959231-17335-1-git-send-email-vaibhav.bedia@ti.com> <1356959231-17335-17-git-send-email-vaibhav.bedia@ti.com> <87k3qewcbt.fsf@linaro.org> <87lialzjne.fsf@linaro.org> Message-ID: <87zjyzqcrb.fsf@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org "Bedia, Vaibhav" writes: [...] >> IMO, this would be a much cleaner implementation if you just created a >> small driver for the wkup_m3. You're already doing a bunch of >> driver-like stuff for it (requesting base/IRQs, mapping regions, >> firmware, etc.) I think this should separated out into a real driver. >> Then it will be bound to the right omap_device, and normal PM operations >> will work as expected. >> >> Also, doing it this way will be more flexible for those wanting to use >> their own firmware on the M3, or customize the current firmware. > > Hmm... that definitely sounds more flexible. It should also help in the next SoC > AM437x which has a similar PM architecture. Where would you suggest placing > this minimal driver? For now, just leave it in mach-omap2 and we can figure out the right home for it eventually. Kevin