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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3CEACC433EF for ; Fri, 3 Dec 2021 11:14:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1380281AbhLCLR7 (ORCPT ); Fri, 3 Dec 2021 06:17:59 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:56135 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1380313AbhLCLR6 (ORCPT ); Fri, 3 Dec 2021 06:17:58 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1638530074; 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=QkzbCStca4hNYmo0e9TkCL4ZSJm7vJBNxNGPGTXnbPU=; b=CSh7mZhKeknt3c+YTnyUBA/B2TX+dc8cWQQAuACh7IKA9Z9T4h2HmDqSvRIpaydo0PTbqx x65vnDhdkbT1Ci84cDAiFUBxg23P6LHMyVnWaNVBKV19h67K9O3wYDeWTmy/aMqrpUKi6r ydagOaPf7afUUsUnZJdiFq0x4qXLiRw= Received: from mail-ed1-f72.google.com (mail-ed1-f72.google.com [209.85.208.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-408-djAChBWVOY6FaK5mt2t75w-1; Fri, 03 Dec 2021 06:14:31 -0500 X-MC-Unique: djAChBWVOY6FaK5mt2t75w-1 Received: by mail-ed1-f72.google.com with SMTP id m12-20020a056402430c00b003e9f10bbb7dso2241918edc.18 for ; Fri, 03 Dec 2021 03:14:30 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=QkzbCStca4hNYmo0e9TkCL4ZSJm7vJBNxNGPGTXnbPU=; b=7zEQwSeFWju50u6mXBMgMUl4V+llDpbkSjpnVKHM5ih+v+qGgnVJvBHJgD2wcG+gqd 8IonuuZYjRAVOvi3MXBaq1+zEoE+Ovdyfu6kOp/tSuDHaDj8G/ajEOFWkA3pk5eB9rRm 4tM0yUb/Cw8DZHz9QMrR7EiIMqoXP3RcpuNK9q2oAIXAdbh/OnoJ+ULk88RU1MBfugO7 vMuTDv+x/IwDSdoHdOWe4WiR97kXLeLCm7eaCirwWaRtbR82Ow/jpqtcErL38Uk8puGX KCZy4WJTwCHvzKpHB4R5//1sLoMis+M42POeFh151eiUmwT6Eu7ga0g+cky33iOagpCi PVew== X-Gm-Message-State: AOAM532gJACwvoCnw2HXiaCgvSWLyXckQ3FKI3Hz8XFxqm3v3bROh0ou dfxPsp2zYQD/Lmzl7HifzGrchEadzoHKrYkllVbq4VbCsP02sfFKUYyEWBviWO/rKsFF7Di8eQd Dq4xdWXz+Laxk4AKqpyUP X-Received: by 2002:a50:d49d:: with SMTP id s29mr25605744edi.55.1638530069763; Fri, 03 Dec 2021 03:14:29 -0800 (PST) X-Google-Smtp-Source: ABdhPJxhFZCEpKOOhtBu3/bixz8Mpj8tiuA/KCe5gzSlthPY4klqcLCd4BDpFiU7X/6QsKzywNJoVg== X-Received: by 2002:a50:d49d:: with SMTP id s29mr25605713edi.55.1638530069563; Fri, 03 Dec 2021 03:14:29 -0800 (PST) Received: from ?IPV6:2001:1c00:c1e:bf00:1054:9d19:e0f0:8214? (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 c11sm1851940ede.32.2021.12.03.03.14.28 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 03 Dec 2021 03:14:29 -0800 (PST) Message-ID: Date: Fri, 3 Dec 2021 12:14:28 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: Re: [PATCH 1/3] spi: Revert "spi: Remove unused function spi_busnum_to_master()" To: Andy Shevchenko , Stefan Binding Cc: Mark Brown , "Rafael J . Wysocki" , Len Brown , Mark Gross , linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, linux-acpi@vger.kernel.org, platform-driver-x86@vger.kernel.org, patches@opensource.cirrus.com, Lucas Tanure References: <20211202162421.7628-1-sbinding@opensource.cirrus.com> From: Hans de Goede In-Reply-To: Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=hdegoede@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-spi@vger.kernel.org Hi, On 12/3/21 12:06, Andy Shevchenko wrote: > On Thu, Dec 02, 2021 at 04:24:19PM +0000, Stefan Binding wrote: >> From: Lucas Tanure >> >> Revert commit bdc7ca008e1f ("spi: Remove unused function >> spi_busnum_to_master()") >> This function is needed for the spi version of i2c multi >> instantiate driver. > > I understand the intention, but I have no clue from this series (it lacks of > proper cover letter, it lacks of much better and justified commit message in > the patch 3) what is the actual issue. Without these to be provided it's no go > for the series. Please, provide much better description what is the actual > issue you are trying to solve (from patch 3 my guts telling me that this can > be achieved differently without this code being involved). Yes I assume that eventually there will be a follow-up which will actually add some new ACPI HIDs to the new bus-multi-instantiate.c file ? Can we please have (some of) those patches as part of the next version, so that we can see how you will actually use this? Also I'm wondering is this actually about ACPI device's having multiple SpiSerialBusV2 resources in a single _CRS resource list ? Or do you plan to use this for devices with only a single I2cSerialBusV2 or SpiSerialBusV2 resource to e.g. share IRQ lookup code between the 2 cases ? If you plan to use this for devices with only a single I2cSerialBusV2 or SpiSerialBusV2 resource, then I'm going to have to nack this. Each ACPI HID which needs to be handled in this code also needs an entry in the i2c_multi_instantiate_ids[] list in drivers/acpi/scan.c which is code which ends up loaded on every single ACPI system, so we really only want to add HIDs there for the special case of having multiple I2cSerialBusV2 or SpiSerialBusV2 resources in a single ACPI Device / ACPI fwnode. If you are looking to use this as a way to share code for other reasons (which is a good goal to strive for!) please find another way, such as e.g. adding some helper functions to drivers/gpio/gpiolib-acpi.c (note there already are a couple of helpers there which you may use). Regards, Hans