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=-12.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, 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 64CA6C4743F for ; Tue, 8 Jun 2021 09:00:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4942661153 for ; Tue, 8 Jun 2021 09:00:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230450AbhFHJC2 (ORCPT ); Tue, 8 Jun 2021 05:02:28 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:32386 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230293AbhFHJCX (ORCPT ); Tue, 8 Jun 2021 05:02:23 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1623142828; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=q8pQTS5vz7W3pBaGE0d38TVTbjri0nLjhaoLXCAvVt4=; b=HcVt12BMAiXCn35SHF3s8IDfBi88MB7DNNuQQMVJTDJE2d6qvfqTQWfPi2Twgfy321Nnr7 e6e0Z6U066d+QOXMm/7zTQO1pL5MrvpA1klcaA7r/Z8g8mrcMfPfe9bTyRCbELmi+AiHn5 f+17Iu/gCGAcNeydCnzusoWj2tIOKro= Received: from mail-ed1-f69.google.com (mail-ed1-f69.google.com [209.85.208.69]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-464-nvvWfvfzMqCWgKWJQ0H1rA-1; Tue, 08 Jun 2021 05:00:27 -0400 X-MC-Unique: nvvWfvfzMqCWgKWJQ0H1rA-1 Received: by mail-ed1-f69.google.com with SMTP id j13-20020aa7de8d0000b029038fc8e57037so10537349edv.0 for ; Tue, 08 Jun 2021 02:00:26 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=q8pQTS5vz7W3pBaGE0d38TVTbjri0nLjhaoLXCAvVt4=; b=BTrKlRQbl9jpipmfgdmYnmD4fMqH6CFl2eqyW+jzuNxiG/1966G9MtE+WUln8rQS3i aCedXaM4/uWwSvYrBIoLRxL2/G2wtAh0CjrFmp/L4toEwg6hbi0RGroFI4tWonpnnM42 Brf6nx2wiS81YeiHQIDsYo/cRNSI6zKt3YuImHxi75Z/TiY2ZaPYnUQIddy9Tv6C2uBg M1L+8HxI9tcC9dP39fdde7iaBgMRrT3gqtz7szeJ7QP36n4bRP8MEPKD7Jec30tX4ksz XqKg7tSQ1cu1VJXuDcH/jU3u/RCaWiMU+XGSVfuBMUew/iWtvLDCf5BMuY8oXoTH+NWV ihsA== X-Gm-Message-State: AOAM533YFThEPFYfp4++RdY/hzzN4aKHgdP2ah8uO0sCncktI96v04VD SLslDurAR100IcM8t3dntOpuai88oBGSlK7NsabWxKJrWClSxKjH08ZiSNjnVvB150irQm73taI GVGvhSrapt35eaKq2XRHW X-Received: by 2002:a17:906:a04e:: with SMTP id bg14mr5775590ejb.366.1623142825966; Tue, 08 Jun 2021 02:00:25 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw0+0lBjVs9CJK1gPDtNITf0ZqRD1SRcnD3pEOFRUdvDJbycYeSORdCpGk0gKsOlfgba/chOA== X-Received: by 2002:a17:906:a04e:: with SMTP id bg14mr5775558ejb.366.1623142825703; Tue, 08 Jun 2021 02:00:25 -0700 (PDT) Received: from x1.localdomain (2001-1c00-0c1e-bf00-1054-9d19-e0f0-8214.cable.dynamic.v6.ziggo.nl. [2001:1c00:c1e:bf00:1054:9d19:e0f0:8214]) by smtp.gmail.com with ESMTPSA id di16sm8488697edb.62.2021.06.08.02.00.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 08 Jun 2021 02:00:25 -0700 (PDT) Subject: Re: [PATCH v5 0/6] Introduce intel_skl_int3472 module To: Daniel Scally , "Rafael J . Wysocki" , Andy Shevchenko , Lee Jones , Maximilian Luz , linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-gpio@vger.kernel.org, linux-i2c@vger.kernel.org, platform-driver-x86@vger.kernel.org, devel@acpica.org Cc: Len Brown , Mika Westerberg , Linus Walleij , Bartosz Golaszewski , Wolfram Sang , Mark Gross , Robert Moore , Erik Kaneda , laurent.pinchart@ideasonboard.com, kieran.bingham@ideasonboard.com References: <20210603224007.120560-1-djrscally@gmail.com> From: Hans de Goede Message-ID: <4400512a-b788-7074-d3c6-0ec228b43d7e@redhat.com> Date: Tue, 8 Jun 2021 11:00:24 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 MIME-Version: 1.0 In-Reply-To: <20210603224007.120560-1-djrscally@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org Hi, On 6/4/21 12:40 AM, Daniel Scally wrote: > Hello all > > Bit longer than hoped but here's v5. > > v4: > https://lore.kernel.org/lkml/20210520140928.3252671-1-djrscally@gmail.com/ > > v3 > https://lore.kernel.org/lkml/20210222130735.1313443-1-djrscally@gmail.com/ > > v2 > https://lore.kernel.org/platform-driver-x86/20210118003428.568892-1-djrscally@gmail.com/ > > v1 > https://lore.kernel.org/linux-media/20201130133129.1024662-1-djrscally@gmail.com/T/#m91934e12e3d033da2e768e952ea3b4a125ee3e67 > > The only changes are the dropped patches, renamed functions in 2/6 and most of > Andy's suggestions on 5/6 - I didn't hit them all yet but didn't want to delay > this any more. > > Series level changelog: > > - Dropped all but the essential patches to simplify merge plan - thanks > Hans. Thank you. Andy has already sends me a pull-req for the gpiolib-acpi changes and I expect Rafael to send me a pull-req (from an immutable branch) for the ACPI bits soon-ish. When I merge those both into pdx86 I should be able to pick-up 5/6. Given the long time this has been in the making I'm tempted to do that (pick up 5/6 as is) and then we can do further cleanups like looking into using the existing fixed-regulator code later. My mean reason to do this would be to get this code into the hands of users starting with the 5.14 kernel. So question, how usable is this (from the kernel pov, I know userspace needs work too) once I merge 5/6. Is the kernel-side support for the cameras on some Surface devices then complete or are more patches necessary in other subsystems ? Note another advantage of just merging 5/6 as is and doing the fixed-regulator bits on top, is that having those in a separate commit makes it easier to see if these indeed result in a nice cleanup (vs sticking with the current code). Regards, Hans > > Daniel Scally (6): > ACPI: scan: Extend acpi_walk_dep_device_list() > ACPI: scan: Add function to fetch dependent of acpi device > gpiolib: acpi: Export acpi_get_gpiod() > gpiolib: acpi: Add acpi_gpio_get_io_resource() > platform/x86: Add intel_skl_int3472 driver > mfd: tps68470: Remove tps68470 MFD driver > > MAINTAINERS | 5 + > drivers/acpi/ec.c | 2 +- > drivers/acpi/pmic/Kconfig | 2 +- > drivers/acpi/pmic/intel_pmic_chtdc_ti.c | 2 +- > drivers/acpi/scan.c | 104 ++++- > drivers/gpio/Kconfig | 2 +- > drivers/gpio/gpiolib-acpi.c | 61 ++- > drivers/i2c/i2c-core-acpi.c | 8 +- > drivers/mfd/Kconfig | 18 - > drivers/mfd/Makefile | 1 - > drivers/mfd/tps68470.c | 97 ---- > drivers/platform/surface/aggregator/core.c | 6 +- > drivers/platform/surface/surface3_power.c | 22 +- > .../platform/surface/surface_acpi_notify.c | 7 +- > drivers/platform/x86/Kconfig | 2 + > drivers/platform/x86/Makefile | 1 + > drivers/platform/x86/intel-int3472/Kconfig | 30 ++ > drivers/platform/x86/intel-int3472/Makefile | 5 + > .../intel_skl_int3472_clk_and_regulator.c | 196 ++++++++ > .../intel-int3472/intel_skl_int3472_common.c | 106 +++++ > .../intel-int3472/intel_skl_int3472_common.h | 118 +++++ > .../intel_skl_int3472_discrete.c | 417 ++++++++++++++++++ > .../intel_skl_int3472_tps68470.c | 137 ++++++ > include/acpi/acpi_bus.h | 8 + > include/linux/acpi.h | 11 +- > include/linux/gpio/consumer.h | 2 + > 26 files changed, 1205 insertions(+), 165 deletions(-) > delete mode 100644 drivers/mfd/tps68470.c > create mode 100644 drivers/platform/x86/intel-int3472/Kconfig > create mode 100644 drivers/platform/x86/intel-int3472/Makefile > create mode 100644 drivers/platform/x86/intel-int3472/intel_skl_int3472_clk_and_regulator.c > create mode 100644 drivers/platform/x86/intel-int3472/intel_skl_int3472_common.c > create mode 100644 drivers/platform/x86/intel-int3472/intel_skl_int3472_common.h > create mode 100644 drivers/platform/x86/intel-int3472/intel_skl_int3472_discrete.c > create mode 100644 drivers/platform/x86/intel-int3472/intel_skl_int3472_tps68470.c >