From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932475AbcAMOgp (ORCPT ); Wed, 13 Jan 2016 09:36:45 -0500 Received: from mailout4.w1.samsung.com ([210.118.77.14]:32161 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751052AbcAMOgn (ORCPT ); Wed, 13 Jan 2016 09:36:43 -0500 X-AuditID: cbfec7f5-f79b16d000005389-b5-569660f8b8d5 Subject: Re: [PATCH] [media] media: Kconfig: add dependency of HAS_DMA To: Sakari Ailus , Sudip Mukherjee References: <1451481963-18853-1-git-send-email-sudipm.mukherjee@gmail.com> <20160111125310.GA19742@sudip-pc> <20160112141042.GI576@valkosipuli.retiisi.org.uk> Cc: Mauro Carvalho Chehab , linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, Andrew Morton , hverkuil@xs4all.nl, pawel@osciak.com, kyungmin.park@samsung.com From: Marek Szyprowski Message-id: <569660F7.2000802@samsung.com> Date: Wed, 13 Jan 2016 15:36:39 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-version: 1.0 In-reply-to: <20160112141042.GI576@valkosipuli.retiisi.org.uk> Content-type: text/plain; charset=utf-8; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrMLMWRmVeSWpSXmKPExsVy+t/xK7o/EqaFGUxrNLGYs34Nm8Wpyc+Y LM42vWG3uLxrDptFz4atrBarn1VYTHn7k93izP6VbBYHTk9hduD02DnrLrvH4a8LWTxOzPjN 4vH410s2jy39QLG+LasYPT5vkvM49fUzewBHFJdNSmpOZllqkb5dAlfG+a3uBW+4K6a2rWJu YHzP2cXIySEhYCLx8FQ3G4QtJnHh3nogm4tDSGApo0RH1z0mCOc5o8SE3/9ZQKqEBdwk3q7o YAexRQTCJG6v28UIUbSIUeLm6o9g7cwCDxgl7hyZA1bFJmAo0fW2CyjBwcEroCWx5JouSJhF QFXi0oUusKGiAjESFzuPMIHYvAKCEj8m32MBKecUsJNYddQRJMwsYCbx5eVhVghbXmLzmrfM ExgFZiHpmIWkbBaSsgWMzKsYRVNLkwuKk9JzjfSKE3OLS/PS9ZLzczcxQqLi6w7GpcesDjEK cDAq8fC+tJwWJsSaWFZcmXuIUYKDWUmEtzAcKMSbklhZlVqUH19UmpNafIhRmoNFSZx35q73 IUIC6YklqdmpqQWpRTBZJg5OqQbGOyslnud1TH/9xHfzDsdFnDpX2gyqTFXmJ1aIJMt3pnZV z9xnpXbxdYCWaZLvk7krrbPUWv8wfEh8fv7BVcs7vobTNl/58qCNv3Lp58W2D/dN1Tzcz9zA Xex5m3Xvyh0T88QVbbZ5C/3/HW6X+m5Kvf+FWK2f7xad3ZfOrf0m6e/Whf8nbBYXUGIpzkg0 1GIuKk4EAFvUqR2GAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On 2016-01-12 15:10, Sakari Ailus wrote: > On Mon, Jan 11, 2016 at 06:23:11PM +0530, Sudip Mukherjee wrote: >> On Wed, Dec 30, 2015 at 06:56:03PM +0530, Sudip Mukherjee wrote: >>> The build of m32r allmodconfig fails with the error: >>> drivers/media/v4l2-core/videobuf2-dma-contig.c:484:2: >>> error: implicit declaration of function 'dma_get_cache_alignment' >>> >>> The build of videobuf2-dma-contig.c depends on HAS_DMA and it is >>> correctly mentioned in the Kconfig but the symbol VIDEO_STI_BDISP also >>> selects VIDEOBUF2_DMA_CONTIG, so it is trying to compile >>> videobuf2-dma-contig.c even though HAS_DMA is not defined. >>> >>> Signed-off-by: Sudip Mukherjee >>> --- >> A gentle ping. m32r allmodconfig still fails with next-20160111. Build >> log is at: >> https://travis-ci.org/sudipm-mukherjee/parport/jobs/101536379 > Hi Sudip, > > Even though the issue now manifests itself on m32r, the problem is wider > than that: dma_get_cache_alignment() is only defined if CONFIG_HAS_DMA is > set. > > I wonder if using videobuf2-dma-contig makes any sense if HAS_DMA is > disabled, so perhaps it'd be possible to make it depend on HAS_DMA. VIDEOBUF2_DMA_CONTIG already depends on HAS_DMA, but when driver use select directive for enabling support for VIDEOBUF2_DMA_CONTIG, the dependencies are not checked further. This is known limitation/feature of kconfig system. Best regards -- Marek Szyprowski, PhD Samsung R&D Institute Poland