Received: by 2002:a25:f815:0:0:0:0:0 with SMTP id u21csp2178983ybd; Mon, 24 Jun 2019 02:03:59 -0700 (PDT) X-Google-Smtp-Source: APXvYqyGC514MAU3gDuJSGlpfkB15Vekm0bdZKuld9LPBDt61+Dxpi3jgSRMUJLvUATFriPL480S X-Received: by 2002:a17:90a:9903:: with SMTP id b3mr23407173pjp.80.1561367039413; Mon, 24 Jun 2019 02:03:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1561367039; cv=none; d=google.com; s=arc-20160816; b=LStswNj7suHEME7+7fggGfLcyMCUdddRy1QPT1SDeNsvxrba1l6FGct82Fvkms5HYf FaFe3VZ7BCDWWDqtVcK9gLDhjw5s5+C0RA4EYYsEnYuW7wjINNkT3Yp+oj8SpUyHnHE7 jDY+r31Y1wLRS4rTqdbq0W1ZWlNGtYhRMSJvSjiYA/uixyzQnIYfzN2KO5QjJLxTftXG 3U7xY4iMrZnqvMlHQE2JqzpRezRo4wNf+Qi4YPdjEIO3qlLTjITuC8A7nTfV0YF6QSm8 jiihiZ2M1/X5LfAl5gmsZR09bimH/DMGHC5hRrNM8K0p7777cCs2g5wTQOE2WuEZOQlR xv2w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:organization:from:references:cc:to:subject; bh=WJiPj0SHdLIxNXszneYe5Y/oG3Xlfs6w0bGRwLU9USA=; b=YVeflTd7fkFt8K/yHM/Vt5ph5xCUGhk2/KIF3Y5EuCESENdxaglGdfgsoAsy/YP57H ACrza2bVwWFnZnyZE5cXpj5sLloa2OTDojicOCocAzrNITT4m5tYzAAUGyBhB3gg7gd7 CIbS8+mvHsVTrsRKbmPV7bap75hjOYLYb3jlv8GB+iLmFnkykPAWhWgMlDQ3XFlWYjOZ PKRaEWo+FDAl0gkQH6oXqUIPxDtkxdM63CuXpr6zHdB+vzSUpxgJTU5CNwgBTVx44zIm 873zzn1VhzH2cCJaNn+BnNWiASUhtKrqg+iWHXLC13VExOX1u9vYKk/NFq/iEpsmlYeY EX5w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v5si8348808pgs.285.2019.06.24.02.03.42; Mon, 24 Jun 2019 02:03:59 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728043AbfFXI7U (ORCPT + 99 others); Mon, 24 Jun 2019 04:59:20 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:55623 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726557AbfFXI7U (ORCPT ); Mon, 24 Jun 2019 04:59:20 -0400 Received: from [192.168.1.110] ([77.4.138.202]) by mrelayeu.kundenserver.de (mreue011 [212.227.15.167]) with ESMTPSA (Nemesis) id 1MJVY8-1hzD7W12sq-00JruV; Mon, 24 Jun 2019 10:59:18 +0200 Subject: Re: [PATCH 1/3] include: linux: i2c: more helpers for declaring i2c drivers To: Wolfram Sang Cc: "Enrico Weigelt, metux IT consult" , linux-kernel@vger.kernel.org, linus.walleij@linaro.org, bgolaszewski@baylibre.com, linux-gpio@vger.kernel.org, linux-i2c@vger.kernel.org References: <1560796779-17117-1-git-send-email-info@metux.net> <20190621211744.GC950@kunai> <20190624084427.GA1014@kunai> From: "Enrico Weigelt, metux IT consult" Organization: metux IT consult Message-ID: <205d0ef7-d487-006b-d104-88958f40e197@metux.net> Date: Mon, 24 Jun 2019 10:59:17 +0200 User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 MIME-Version: 1.0 In-Reply-To: <20190624084427.GA1014@kunai> Content-Type: text/plain; charset=windows-1252 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K1:kVx291ZekfkMnc6WJzsoAVGadykZkXcOImRJ4KvFPHdNlmO8X/w wDVWLmsrflIK8awd/K9cSRQ18j/vONP/bvXB6ZqUdqjmZFLoJdqpR8zsfGxyUYPo7aw5OTq JTWaPVTxkWAt9RDtJ8x+xcczZqZUYacyI6ljlgrQfbl6e+VfRyuU605WdSdh2P1L53dnNHb oBmbyA1UGpa/KoX9vsv5w== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:swNYtjFfWCA=:Jx2Lqwjcro1pBpKYHO0/pU +A1W/g1q42uZpjEit3HRp0zwiLLJHbmGa6ZR56uXgJ90ynEVvLvx0SFGfGQEIfpt7msP05TRu hR/qwuHAVkYCmAL3G+ZOc1DK5zK88e2bUgZRsLDIQgk1Lxj+RYm73fx5BRtTE/zl+IkIDYPdx taT4jr067Tw76Ht39NXb1Are5dq69lwUM79TqytyRbBY/2SBjQM172VjXJ28UG2AezM/qmVGY 1okO761KsA9gewuK58pamhxz3zfrTGC1CUc6dPCKt7i49JXHWSBi5D0qI9fpgjlOYcl4xvOc1 /c/MwWmVgGuTUjQM5z7r+QIGbmoN2Kuwv4jobmMwyWlcL/NtyGfSErZ51YluK8tF5fyoxw7uq creyd+1iDAJYEGPk6gANoiqV8iubpjtbkG6WD/kF55tZxIXeN5G7Zl9RXCNZQYUvctTce9jVp PYQLXfeGasLg39y/FEj5oIS+OTZn4zgFqvYxrRQw5P4bngTRYqdRVCbleM7BN7WTJrwBCNCb3 6hs/p34Ores8bYLoboOzauGD/XktVcuNQJ1ZuOQSjbkTq5bD1DZ02+VNCM0HePYvOwDeJmLWu 29Z7fd77nF8MOzTFPxKbfSg8L6suqRslFE0Z5cOLoZT7TG3nb3h0C+znK0yylVkiucSH/aJ++ v+pIMbLe6iIhaIp2osaPIs1HzpTFQN/CSMCLYEQ7AgHtXG3WmtIDGYhqxC3PoEff2yehi8/Lx YBmCLnZ0D49Mk6dGlEqOt7iBMaY9KvJwvD6X36Q7qIw9xLjD0cKiSfEP16g= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 24.06.19 10:44, Wolfram Sang wrote: > The correct solution is to fix the boot dependency in the affected I2C > client drivers. That definately needs HW and thorough testing. > > It may also need something better than the current deferred probe. Big > topic. So, then the current approach of using subsys_initcall() can't be changed easily, right now. But planned for the future (or at least not introducing new caes). But: how does that conflict w/ just moving the existing redundant pieces into a helper macro ? The logic stays the same - just using a shorter notation. (assuming my patch isn't buggy ;-)). I can add a remark in the function documentation that this shall only by used in rare cases, and maybe something like "that's just legacy - introducing new caller is most certainly wrong" ;-) --mtx -- Enrico Weigelt, metux IT consult Free software and Linux embedded engineering info@metux.net -- +49-151-27565287