From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: RE: [PATCH v5 6/6] spi: s3c64xx: add device tree support Date: Sun, 15 Jul 2012 19:39:50 +0900 Message-ID: <127d01cd6276$295b3a80$7c11af80$%kim@samsung.com> References: <1342021265-11212-1-git-send-email-thomas.abraham@linaro.org> <1342021265-11212-7-git-send-email-thomas.abraham@linaro.org> <4FFF51EE.4080609@samsung.com> <117c01cd60ea$90f76ac0$b2e64040$%kim@samsung.com> <118901cd60fd$66c26260$34472720$%kim@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: spi-devel-general@lists.sourceforge.net, devicetree-discuss@lists.ozlabs.org, broonie@opensource.wolfsonmicro.com, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, rob.herring@calxeda.com, grant.likely@secretlab.ca, jaswinder.singh@linaro.org To: 'Thomas Abraham' Return-path: In-reply-to: Content-language: ko Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-spi.vger.kernel.org Thomas Abraham wrote: > > On 13 July 2012 19:12, Kukjin Kim wrote: > > Thomas Abraham wrote: > >> > >> On 13 July 2012 16:58, Kukjin Kim wrote: > >> > Thomas Abraham wrote: > >> >> > >> >> On 13 July 2012 04:08, Kukjin Kim wrote: > >> >> > Thomas, the DMACH_DT_PROP is available only on pl330 now. So seems > >> occur > >> >> > following build error with s3c6400_defconfig. > >> >> > > >> >> > drivers/spi/spi-s3c64xx.c: In function 's3c64xx_spi_get_dmares': > >> >> > drivers/spi/spi-s3c64xx.c:1098: error: 'DMACH_DT_PROP' undeclared > >> (first > >> >> use > >> >> > in this function) > >> >> > drivers/spi/spi-s3c64xx.c:1098: error: (Each undeclared identifier > is > >> >> > reported only once > >> >> > drivers/spi/spi-s3c64xx.c:1098: error: for each function it > appears > >> in.) > >> >> > make[3]: *** [drivers/spi/spi-s3c64xx.o] Error 1 > >> >> > make[2]: *** [drivers/spi] Error 2 > >> >> > > >> >> > >> >> Dear Mr. Kim, > >> >> > >> >> Thanks for considering these patches to be merged via your tree. For > >> >> the build break you mentioned above, there was a patch submitted > some > >> >> time back to handle this. The link to that patch is: > >> >> http://lists.infradead.org/pipermail/linux-arm-kernel/2012- > >> >> June/102627.html. > >> >> Could you please apply this patch also to your tree. > >> >> > >> > Sure, it looks good to me, will apply that then this series will be > >> rebased > >> > on top of that. > >> > >> There are six more patches that enable spi device tree support for > >> Exynos4 and Exynos5 platforms. Could you consider to apply those > >> patches as well. The link to those patches is: > >> http://www.mail-archive.com/linux-samsung- > >> soc@vger.kernel.org/msg10851.html. > >> > > Yeah, I remember, but I think some patches is needed to re-work as per > > comments...? > > I checked again for any pending comments on v2 version. There are no > pending comments for the v2 version of this patch series. Thanks. OK, I checked v2 series you mentioned. Looks OK to me and applied into next/dt-samsung because it is related in supporting dt. Note, I applied all of them into same dt branch, the fist one 'fix the incorrect hierarchy of spi controller bus clock for exynos4' can be sent to upstream via fixes now though. And I changed the clock name for exynos4_clk_mdout_spix to 'mdout_spi' for consensus when I applied. If any problems, please let me know. Thanks. Best regards, Kgene. -- Kukjin Kim , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd.