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 AF93AC7619A for ; Wed, 22 Mar 2023 10:35:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230128AbjCVKfc (ORCPT ); Wed, 22 Mar 2023 06:35:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48090 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230272AbjCVKfa (ORCPT ); Wed, 22 Mar 2023 06:35:30 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 07C5E521C4 for ; Wed, 22 Mar 2023 03:34:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1679481282; 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: in-reply-to:in-reply-to:references:references; bh=sAw1kb+1MttUiaf7ZI4njfkrYYVFnkuOfRgsWc6OSNg=; b=U6ADbOB4labQQJP4oCd8f2cIwF/qRaceJ4ZvcuhHtpEgp5WllQoiF+St++w+JgVfcsbvCE 02TNrKQBMf5SzTLRtdcmzhaV44UNtzPX4NCZ7LjiyDs0msxAf8E9xFInSeqwYFKohZ25ni wKB8HQhsZlMQa/Bxqd7U/0J1dG/d/3I= Received: from mail-wm1-f70.google.com (mail-wm1-f70.google.com [209.85.128.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-595-8_YykLbENgWPUTLmpp3cwg-1; Wed, 22 Mar 2023 06:34:41 -0400 X-MC-Unique: 8_YykLbENgWPUTLmpp3cwg-1 Received: by mail-wm1-f70.google.com with SMTP id m10-20020a05600c4f4a00b003ed74161838so7634638wmq.6 for ; Wed, 22 Mar 2023 03:34:40 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1679481280; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=sAw1kb+1MttUiaf7ZI4njfkrYYVFnkuOfRgsWc6OSNg=; b=MOti7JFQITRGkvy8HttIwI1xV/qPs03owpOj5j769pQkUC2J+zzse8JqS5sJGpI8G8 ihGQYzUmiUErP+u+lwWBPK72PTnNhsAwN4YPv/Xfw5bmz/DLjw+rvUfpPopLhZibfGSJ Ckm7IppCuEf22y0DXrzp9ArHBilIPZhd+af0lFvroDhK/r2sebHO4/w4XhU5LmnnndeR D9fAoYIV6UcQVy2GuJBnutNeTs9N+3K4CClqhGBFThDqoOrRSezRUS1SeW/Q9xI0tdeK jkrPDnn/Z0SVHDDpySTNiQuRFO3ZaoczRvvuxwGwU0GLa8NFeLjjZsl8fcQLmsKR+eBL 6PJw== X-Gm-Message-State: AO0yUKWcz7RGaGOhXgALz7pgkvBxH54kMvHdjy/g5BTwoazPo/0vorcj BsVgz2p38O/S0YbOvW5str3774ZXte0FQW5pMdXpKoJlrx4MinvBxbisxR3VIam5yfzz2nSXotO bdyQJOtYFgiqgFuE5Vqgp X-Received: by 2002:a05:600c:ad3:b0:3eb:2b88:9adc with SMTP id c19-20020a05600c0ad300b003eb2b889adcmr4782442wmr.25.1679481280058; Wed, 22 Mar 2023 03:34:40 -0700 (PDT) X-Google-Smtp-Source: AK7set9OJEunbX5gMc6MC+xeRbrZtBcXx8fDHwdAWSwBMT23hK8lYvS2ivGhxvkhzDa1tAAGkQFT7w== X-Received: by 2002:a05:600c:ad3:b0:3eb:2b88:9adc with SMTP id c19-20020a05600c0ad300b003eb2b889adcmr4782415wmr.25.1679481279722; Wed, 22 Mar 2023 03:34:39 -0700 (PDT) Received: from localhost (205.pool92-176-231.dynamic.orange.es. [92.176.231.205]) by smtp.gmail.com with ESMTPSA id x33-20020a05600c18a100b003ede3f5c81fsm10182838wmp.41.2023.03.22.03.34.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 22 Mar 2023 03:34:39 -0700 (PDT) From: Javier Martinez Canillas To: Andy Shevchenko , Matti Vaittinen Cc: Matti Vaittinen , Noralf =?utf-8?Q?Tr=C3=B8nnes?= , Masahiro Yamada , Randy Dunlap , Shreeya Patel , Krzysztof Kozlowski , Jonathan Cameron , devicetree@vger.kernel.org, Zhigang Shi , Maxime Ripard , Heikki Krogerus , Lars-Peter Clausen , Paul Gazzillo , Maxime Ripard , =?utf-8?Q?Ma=C3=ADra?= Canal , Rob Herring , Dmitry Osipenko , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, kunit-dev@googlegroups.com, Stephen Boyd , Emma Anholt , Liam Beguin , Greg Kroah-Hartman , Maarten Lankhorst , Thomas Zimmermann , Daniel Vetter , David Gow , "Rafael J. Wysocki" , Brendan Higgins , David Airlie , linux-kselftest@vger.kernel.org Subject: Re: [PATCH v5 0/8] Support ROHM BU27034 ALS sensor In-Reply-To: References: Date: Wed, 22 Mar 2023 11:34:38 +0100 Message-ID: <87edphnkg1.fsf@minerva.mail-host-address-is-not-set> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org Andy Shevchenko writes: Hello Andy, > On Wed, Mar 22, 2023 at 11:05:23AM +0200, Matti Vaittinen wrote: > >> Revision history: >> v4 => v5: Mostly fixes to review comments from Andy and Jonathan. >> - more accurate change-log in individual patches > >> - copy code from DRM test helper instead of moving it to simplify >> merging > > 1) Why do you think this is a problem? > 2) How would we avoid spreading more copies of the same code in the future? > > > 1) Merge conflicts is not a bad thing. It shows that people tested their code > in isolation and stabilized it before submitting to the upper maintainer. > > https://yarchive.net/comp/linux/git_merges_from_upstream.html > > 2) Spreading the same code when we _know_ this, should be very well justified. > Merge conflict is an administrative point, and not a technical obstacle to > avoid. > I believe this was suggested by Maxime and the rationale is that by just copying the helpers for now, that would make it easier to land instead of requiring coordination between different subystems. Otherwise the IIO tree will need to provide an inmutable branch for the DRM tree to merge and so on. I agree with Maxime that a little bit of duplication (that can be cleaned up by each subsystem at their own pace) is the path of least resistance. -- Best regards, Javier Martinez Canillas Core Platforms Red Hat