From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH RFC 06/10] ARM: s3c64xx: pm: Add device tree based power domain instantiation Date: Sun, 12 Jan 2014 20:34:10 +0100 Message-ID: <52D2EE32.5060709@gmail.com> References: <1389469372-17199-1-git-send-email-tomasz.figa@gmail.com> <1389469372-17199-7-git-send-email-tomasz.figa@gmail.com> <20140112192910.GW29039@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140112192910.GW29039-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mark Brown Cc: linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Mark Rutland , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Russell King , Pawel Moll , Len Brown , Greg Kroah-Hartman , Tomasz Figa , Ian Campbell , "Rafael J. Wysocki" , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Rob Herring , Bartlomiej Zolnierkiewicz , Kukjin Kim , Pavel Machek , Kumar Gala , Stephen Warren , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On 12.01.2014 20:29, Mark Brown wrote: > On Sat, Jan 11, 2014 at 08:42:48PM +0100, Tomasz Figa wrote: > >> This patch adds support for registering power domains of S3C64xx SoCs >> and binding devices to them using device tree. > >> +#ifdef CONFIG_OF >> +static struct of_device_id s3c64xx_pd_matches[] = { >> + { .compatible = "samsung,s3c6400-clock", }, >> + { .compatible = "samsung,s3c6410-clock", }, >> + { }, >> }; > > Looks like a rebasing issue? Otherwise this looks nice. Hmm, what's the problem here? Maybe I'm just blind, but I fail to see anything wrong (except that the array should be const...). Thanks for reviewing this. Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html