From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH v2] i2c: Allow ACPI_I2C_OPREGION if I2C is built as a module Date: Wed, 24 Jan 2018 17:09:22 +0100 Message-ID: References: <1514988151-12248-1-git-send-email-eric.auger@redhat.com> <20180124055615.c3qwhwnj7spmcr3w@ninjato> <20180124062725.GK27654@lahna.fi.intel.com> <54ef10bd-6209-6983-1d3a-ddf7d7fea252@redhat.com> <26d6761c-17c9-73d8-42d6-25815b58bc87@redhat.com> <17f80dc5-126e-bf1e-8bcc-95a1e66cd271@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <17f80dc5-126e-bf1e-8bcc-95a1e66cd271@codeaurora.org> Content-Language: en-US Sender: linux-i2c-owner@vger.kernel.org To: Sinan Kaya , Andy Shevchenko Cc: Mika Westerberg , Wolfram Sang , Eric Auger , eric.auger.pro@gmail.com, linux-i2c , Linux Kernel Mailing List , "linux-acpi@vger.kernel.org" List-Id: linux-acpi@vger.kernel.org Hi, On 24-01-18 16:37, Sinan Kaya wrote: > On 1/24/2018 10:23 AM, Hans de Goede wrote: >>> There must be a middle ground somewhere. >> >> One thing which comes to mind is to simply not allow building i2c as a module >> when ACPI is selected, something like this should work I think: >> >>  config I2C >>          tristate "I2C support" >>          select RT_MUTEXES >>          select IRQ_DOMAIN >> +        # force building I2C in on ACPI systems, for opregion availability >> +        depends on y || !ACPI > > This works for me. OK, so feel free to turn it into a proper patch and submit it upstream. Regards, Hans