From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CC720C31E40 for ; Mon, 12 Aug 2019 09:21:22 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 997172147A for ; Mon, 12 Aug 2019 09:21:22 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Ogz1Ln69"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b="JxMMK1A9" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 997172147A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=samsung.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:References:In-Reply-To:MIME-Version: Date:Message-ID:From:To:Subject:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=97itcacCx9cjLcWjQ97S1DxJsMRmMBAUbBvyRN9judQ=; b=Ogz1Ln691RYZpM z1yP5wGns8IND7gHqVINfp2etPf/EEoHdIaI8uJVICe4W/aBYtZPBPtVo9tNdm2yuc2MICJttNHqZ 1E4aPGPJ7PW79y0vIvN1fAzGGI30WH+WGxQEpBSa4wZ6JB35azUDedSygeEeLpngoZicLap5JJmKy QLk5eIZVsuawc6rC7UWXnlRXPyAWJwBGFbfS9EM8UlsaanQ2rs21I9dcUJeSIRcyjVBLQsA7kQjK7 cyimLhPWNFJ9cLMMJ4BkxZ9n5wm+N6/U3JpwKM/4Y88i/fmgPpR/MYA1SncyAFK6WfUKQKnMoPFtH Q/fzf6p5aY1qURGaW+Nw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hx6W0-00026K-SX; Mon, 12 Aug 2019 09:21:20 +0000 Received: from mailout1.w1.samsung.com ([210.118.77.11]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hx6Vx-00025C-9g for linux-arm-kernel@lists.infradead.org; Mon, 12 Aug 2019 09:21:19 +0000 Received: from eucas1p1.samsung.com (unknown [182.198.249.206]) by mailout1.w1.samsung.com (KnoxPortal) with ESMTP id 20190812092110euoutp01d3a5d99f5c10f2c4b5dc30869948613a~6Imy1WZFc0508505085euoutp01v for ; Mon, 12 Aug 2019 09:21:10 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout1.w1.samsung.com 20190812092110euoutp01d3a5d99f5c10f2c4b5dc30869948613a~6Imy1WZFc0508505085euoutp01v DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1565601670; bh=XADl3E0ocFQktGQLli+F4PApnzQiN9HeNOsiSPxkCBc=; h=Subject:To:Cc:From:Date:In-Reply-To:References:From; b=JxMMK1A9ww1W6OL8JciMBpBYm6OpY4smWIogbXNv2VZUc413rdU1EpHnt5LOir/M5 LtZlZ3pgZwjqEPEQ2xZvjBkqdSPSdLr4ZZ6OkSMhe80kD7PSf3Mg/Q++Y+trcX0/WD HnOVa8OGMDwafwXJ2D+FMMNeQ4r/uo0WoPmgysGc= Received: from eusmges1new.samsung.com (unknown [203.254.199.242]) by eucas1p1.samsung.com (KnoxPortal) with ESMTP id 20190812092109eucas1p1a8c70054cf60f6f7891e5329afb42db6~6ImyHmub80898908989eucas1p11; Mon, 12 Aug 2019 09:21:09 +0000 (GMT) Received: from eucas1p2.samsung.com ( [182.198.249.207]) by eusmges1new.samsung.com (EUCPMTA) with SMTP id 3A.B2.04469.58F215D5; Mon, 12 Aug 2019 10:21:09 +0100 (BST) Received: from eusmtrp1.samsung.com (unknown [182.198.249.138]) by eucas1p2.samsung.com (KnoxPortal) with ESMTPA id 20190812092108eucas1p2484e38c892c4efd7d84aa26efd6b3b4e~6ImxHzFCL3275032750eucas1p2G; Mon, 12 Aug 2019 09:21:08 +0000 (GMT) Received: from eusmgms2.samsung.com (unknown [182.198.249.180]) by eusmtrp1.samsung.com (KnoxPortal) with ESMTP id 20190812092108eusmtrp19368d65c7cbbab771e52de4cbaa9d227~6Imw4vXoB1828418284eusmtrp1F; Mon, 12 Aug 2019 09:21:08 +0000 (GMT) X-AuditID: cbfec7f2-54fff70000001175-7e-5d512f859754 Received: from eusmtip1.samsung.com ( [203.254.199.221]) by eusmgms2.samsung.com (EUCPMTA) with SMTP id C4.F2.04117.48F215D5; Mon, 12 Aug 2019 10:21:08 +0100 (BST) Received: from [106.120.51.71] (unknown [106.120.51.71]) by eusmtip1.samsung.com (KnoxPortal) with ESMTPA id 20190812092108eusmtip1694326cce266e2d779ea087f823c0236~6ImwV-gVd0771907719eusmtip1g; Mon, 12 Aug 2019 09:21:08 +0000 (GMT) Subject: Re: [PATCH 02/22] ARM: omap1: make omapfb standalone compilable To: Arnd Bergmann From: Bartlomiej Zolnierkiewicz Message-ID: <9b2b4623-f1cb-0e50-691e-e18f8f864c8f@samsung.com> Date: Mon, 12 Aug 2019 11:21:05 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-Brightmail-Tracker: H4sIAAAAAAAAA01Se0hTcRj1t3t3dx1OrtPyy8pqmJWUj0i5pJiFwS0qHPVHFGIrLyrq1E0t LcjIUkzQZiW+C82pWC7fTwxHrvARltoSSTMRjSzTJvmY5HYn+d/5zncO53zwkZi4j+9ERsjj WYVcFiUhhHhj99L7Q/c8pMGeb8v309XTAbRR1S2gBw2/CPpuaQ1BP1pV8+jab8N8+mNrIUEX lGXhdE3JCEF3Dp4OEDIryyrELBpUOKM1PMOZ0eF2guksqhYwYw90PKau7Daj0zfxmIVa5yDr S0K/UDYqIpFVePhfEYY3lxZjsQ3uNwYWM7EU9MQlA1mTQB2B0cEpXgYSkmKqAkF7Qy3BDX8Q LHX3I25YQLAwPklsWApmJywLNYI6wzDGDbMI0lf+IpPKnmIgrb1HYMIO1B54PD1lFmFUHgZv XrXwTQuCOgoP06rMBhHlD3PqVDOPU3thLbXfzG+hLsJYt4bPaezgXd4kbsLWlBRSC4zmShjl CCOTJTwO74Km2UJzGFAzAijLu2PpHQg5RRMCDtvDd129Be+AnpxMnDO8RGBMn7a4mxCoc9Ys bl/Q6gbWa5DrEQegptWDo4/Dz9UaZKKBsgX9rB1XwhZUjbkYR4sg/b6YU7uCplxDbMRmtFRi 2UiSv+m0/E3n5G86J/9/7lOEVyFHNkEZHcYqveTsdXelLFqZIA9zvxYTXYvWn6xnTTffjAwf rnYhikQSG5FmJShYzJclKpOiuxCQmMRBFG9cp0ShsqRkVhETokiIYpVdaDuJSxxFN63GL4up MFk8G8mysaxiY8sjrZ1SUPyFc3a7bZ7beb7O7WjLkeuTNd6+q+ywNK7wls3gwU+ey8KFfSHn ez0Olw4N9RY1nvA2hmrPuqu+bN05x/9tpd/GnNS1+8gCXUojs91KpGc0qx36H83auUpjYfHK vOsMljUQ9Zms8HOjTzm7+rw41haZpXXVdfnlxdX3TfYK0FcJrgyXeblhCqXsH6J5AkNgAwAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrLIsWRmVeSWpSXmKPExsVy+t/xu7ot+oGxBuc3qViseeFg8XfSMXaL K1/fs1k0L17PZjHlz3Imi02Pr7FaXN41h81i9pJ+Fov182+xWey/4uXA5fH71yRGj29fJ7F4 HP66kMXjzrU9bB77565h97jffZzJY/OSeo/jN7YzeXzeJBfAGaVnU5RfWpKqkJFfXGKrFG1o YaRnaGmhZ2RiqWdobB5rZWSqpG9nk5Kak1mWWqRvl6CXsWPxPOaCrXoVF7/1MDcwTlPpYuTk kBAwkZj99hFjFyMXh5DAUkaJvqkX2LsYOYASMhLH15dB1AhL/LnWxQZR85pRYtbml4wgCWEB D4n2PafZQWwRAUWJqS+eMYMUMQvMZZZY9ugyWEJIYDWzxIalwiA2m4CVxMT2VWDNvAJ2Eh+W t7CC2CwCqhL/Ws6BxUUFIiTOvF/BAlEjKHFy5hMwm1MgUKJl9l82EJtZQF3iz7xLzBC2uMSt J/OZIGx5ie1v5zBPYBSahaR9FpKWWUhaZiFpWcDIsopRJLW0ODc9t9hIrzgxt7g0L10vOT93 EyMwarcd+7llB2PXu+BDjAIcjEo8vBE/A2KFWBPLiitzDzFKcDArifCW/AUK8aYkVlalFuXH F5XmpBYfYjQFem4is5Rocj4woeSVxBuaGppbWBqaG5sbm1koifN2CByMERJITyxJzU5NLUgt gulj4uCUamC05Ir9tXx7rcfizUs+qK/g2nX+8N+ko2fTWiod/P6pfPL2Zzv1+9xujo2b2s4e 2tyj8X+JS5di2LFwl+uJ1U3id8wiVXvviEZ3LWb4yD1zVVS0SdmB/x/3fZ3w/uoXv11JbH92 JVw6Z/r/+9mA7yYNBv9XOCpMvW9w9oXacukp8w679wU+TG9yUWIpzkg01GIuKk4EAJ7Ifh7w AgAA X-CMS-MailID: 20190812092108eucas1p2484e38c892c4efd7d84aa26efd6b3b4e X-Msg-Generator: CA X-RootMTR: 20190808212453epcas2p44ff418662ee1acf428c6842ee4488f9f X-EPHeader: CA CMS-TYPE: 201P X-CMS-RootMailID: 20190808212453epcas2p44ff418662ee1acf428c6842ee4488f9f References: <20190808212234.2213262-1-arnd@arndb.de> <20190808212234.2213262-3-arnd@arndb.de> <55c9608d-68c4-17f6-2682-7668d5d7720a@samsung.com> <487da98d-a862-0207-289a-bca8ff18e51a@samsung.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190812_022117_544664_1F3CA8D4 X-CRM114-Status: GOOD ( 29.84 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Aaro Koskinen , Tony Lindgren , Greg Kroah-Hartman , Linus Walleij , Linux Kernel Mailing List , dri-devel , Tomi Valkeinen , linux-omap , Linux ARM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 8/9/19 9:55 PM, Arnd Bergmann wrote: > On Fri, Aug 9, 2019 at 4:36 PM Bartlomiej Zolnierkiewicz > wrote: >> On 8/9/19 1:43 PM, Arnd Bergmann wrote: > >>> >>> That would have been ok as well, but having the addition here was >>> intentional and seems more logical to me as this is where the headers >>> get moved around. >> I see that this is an optimization for making the patch series more >> compact but I think that this addition logically belongs to patch #9 >> (which adds support for COMPILE_TEST) where the new code is required. >> >> Moreover patch description for patch #2 lacks any comment about this >> addition being a preparation for changes in patch #9 so I was quite >> puzzled about its purpose when seeing it first. >> >> Therefore please have mercy on the poor/stupid reviewer and don't do >> such optimizations intentionally (or at least describe them properly >> somewhere).. ;-) > > Ok, I looked at it some more and agree that you are right. I've split it > up further now into patches that make more sense by themselves: > > commit ad71cdc54404ecde2e88678ee6bc7ae7fb8aec97 > Author: Arnd Bergmann > Date: Tue Aug 6 16:08:34 2019 +0200 > > fbdev: omap: avoid using mach/*.h files > > All the headers we actually need are now in include/linux/soc, > so use those versions instead and allow compile-testing on > other architectures. > > Acked-by: Bartlomiej Zolnierkiewicz > Signed-off-by: Arnd Bergmann > > drivers/video/backlight/Kconfig | 4 ++-- > drivers/video/backlight/omap1_bl.c | 4 ++-- > drivers/video/fbdev/omap/Kconfig | 4 ++-- > drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +- > drivers/video/fbdev/omap/lcd_dma.c | 3 ++- > drivers/video/fbdev/omap/lcd_inn1510.c | 2 +- > drivers/video/fbdev/omap/lcd_osk.c | 4 ++-- > drivers/video/fbdev/omap/lcdc.c | 2 ++ > drivers/video/fbdev/omap/omapfb_main.c | 3 +-- > drivers/video/fbdev/omap/sossi.c | 1 + > 10 files changed, 16 insertions(+), 13 deletions(-) > > commit 959e0d68751757e84dd703f60405c7268763dba4 > Author: Arnd Bergmann > Date: Fri Aug 9 21:27:01 2019 +0200 > > fbdev: omap: pass irqs as resource > > To avoid relying on the mach/irqs.h header, stop using > OMAP_LCDC_IRQ and INT_1610_SoSSI_MATCH directly in the driver > code, but instead pass these as resources. > > Signed-off-by: Arnd Bergmann > > arch/arm/mach-omap1/fb.c | 19 ++++++++++++++++++- > drivers/video/fbdev/omap/lcdc.c | 6 +++--- > drivers/video/fbdev/omap/omapfb.h | 2 ++ > drivers/video/fbdev/omap/omapfb_main.c | 16 +++++++++++++++- > drivers/video/fbdev/omap/sossi.c | 2 +- > 5 files changed, 39 insertions(+), 6 deletions(-) > > > commit 6643f7a7da3ca7ce8f2ff094fecab7a0fd706acf > Author: Arnd Bergmann > Date: Fri Aug 9 21:42:31 2019 +0200 > > ARM: omap1: declare a dummy omap_set_dma_priority > > omapfb calls directly into the omap_set_dma_priority() function in > the DMA driver. This prevents compile-testing omapfb on other > architectures. Add an inline function next to the other ones > for non-omap configurations. > > Signed-off-by: Arnd Bergmann > > include/linux/omap-dma.h | 3 +++ > 1 file changed, 3 insertions(+) > > commit 154bfb7ddcecdbca66d9a086776a3108831ef0b9 > Author: Arnd Bergmann > Date: Mon Aug 5 23:15:37 2019 +0200 > > ARM: omap1: move lcd_dma code into omapfb driver > > The omapfb driver is split into platform specific code for omap1, and > driver code that is also specific to omap1. > > Moving both parts into the driver directory simplifies the structure > and avoids the dependency on certain omap machine header files. > > As mach/lcd_dma.h can not be included from include/linux/omap-dma.h > any more now, move the omap_lcd_dma_running() declaration into the > omap-dma header, which matches where it is defined. > > Signed-off-by: Arnd Bergmann > > arch/arm/mach-omap1/Makefile > | 4 ---- > arch/arm/mach-omap1/include/mach/lcdc.h > | 44 -------------------------------------------- > drivers/video/fbdev/Makefile > | 2 +- > drivers/video/fbdev/omap/Makefile > | 5 +++++ > {arch/arm/mach-omap1 => drivers/video/fbdev/omap}/lcd_dma.c > | 4 +++- > {arch/arm/mach-omap1/include/mach => > drivers/video/fbdev/omap}/lcd_dma.h | 2 -- > drivers/video/fbdev/omap/lcdc.c > | 2 +- > drivers/video/fbdev/omap/lcdc.h > | 35 +++++++++++++++++++++++++++++++++++ > drivers/video/fbdev/omap/sossi.c | 1 + > include/linux/omap-dma.h > | 4 ++-- > 10 files changed, 48 insertions(+), 55 deletions(-) > > commit b8ddb98d29a43fecb4387d0d8218935cb1997a28 > Author: Arnd Bergmann > Date: Tue Aug 6 14:59:00 2019 +0200 > > ARM: omap1: innovator: pass lcd control address as pdata > > To avoid using the mach/omap1510.h header file, pass the correct > address as platform data. > > Acked-by: Bartlomiej Zolnierkiewicz > Signed-off-by: Arnd Bergmann > > arch/arm/mach-omap1/board-innovator.c | 3 +++ > drivers/video/fbdev/omap/lcd_inn1510.c | 7 +++++-- > 2 files changed, 8 insertions(+), 2 deletions(-) Thank you for reworking the patch series. > The resulting code is the same as before, I'll post that again along > the rest of the series next week. Should I add your Ack to each > patch already? Yes, please do. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel