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 28766C0015E for ; Sat, 5 Aug 2023 21:10:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229483AbjHEVKs (ORCPT ); Sat, 5 Aug 2023 17:10:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49206 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229460AbjHEVKr (ORCPT ); Sat, 5 Aug 2023 17:10:47 -0400 Received: from mail-ej1-x635.google.com (mail-ej1-x635.google.com [IPv6:2a00:1450:4864:20::635]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 02F8EE7 for ; Sat, 5 Aug 2023 14:10:46 -0700 (PDT) Received: by mail-ej1-x635.google.com with SMTP id a640c23a62f3a-99c0cb7285fso425232266b.0 for ; Sat, 05 Aug 2023 14:10:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691269844; x=1691874644; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Te/iuHusfR0sKZcOV1o10/BbIF4/9EQr1cV9bQ+zMAA=; b=A4jiRySvUj9KFHw3mJ8Ljc0XzHgBEIg6Qsdnu2ivSFoLgjJqEPvhmbCXBO82qbCjuh XgqvhPQEWiub8ol4Te+2bhuLt+gawSFmo3aU07dbt5/XfJxBECNX9N8XUcn/QWRQI3tv dSYI0+2fooby+xpsjNdh0AaV2WxbwLotenEIcFTLQtwYqZ7J+fX3uD3tZiHpBV+Tdq06 8nI2JMXrlY18yxJQmZXoFtTDr6BzvBQTKKGiZaNnl/5cq0nJ/I73jtrBagvW40R2090U oLw67U/yiqaWCt7wTzk9izFQ3O1SCiG6rkQaTHiM6KQCq00y47wkYLwbruiwDpVCz39+ QIEA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691269844; x=1691874644; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Te/iuHusfR0sKZcOV1o10/BbIF4/9EQr1cV9bQ+zMAA=; b=A4qjk3XrUR7fWal355qqliT3DKe+JufpIlPEf2qoaWeeSQZaK3UwEjyaCr69ksYAH6 owsg+G3LzyzYsRVjXONVh4zoTWhqzSIfQgYLiBqI0zwWfJlgjOj/NNrw6oVkqXt6397C FX8YhUtb3HASm9noex6V1nvEhVxta+FY1JuVXTs8ke5PlHYev6sS1iFTL0EYne8SDEED sZ7WaWN+RiGc150fmItE/1uvZUO2bMYn+YB0QIgFtBgrDJ5VSd29GqC3YfV23caOzHyy EjBML7kCFCeAPqx4tCZLJNrvyKzRtwrMR4Crk/lbQbVidrisLU+lC0uWnBHrpDg8JBfj WC0w== X-Gm-Message-State: AOJu0Yxc7MoqY+JqMAqlOph7QYr6TkOO7r57diqSuX/F7hzdHN4lofa5 d2ZvFMNpTW3MehPXJEFpx6oyeEQxBQOVpam8lbo= X-Google-Smtp-Source: AGHT+IFqJSazSoqvAvxX9mRgx7+oZ2LN3CzLXMX1HNes/QFXBiiDV7pmQ8/iMUTNvjP0bgMKXb7DlQ== X-Received: by 2002:a17:906:3001:b0:992:33ba:2eb4 with SMTP id 1-20020a170906300100b0099233ba2eb4mr4383582ejz.71.1691269844379; Sat, 05 Aug 2023 14:10:44 -0700 (PDT) Received: from [192.168.1.20] ([178.197.222.245]) by smtp.gmail.com with ESMTPSA id a8-20020a170906244800b00992b2c55c67sm3147806ejb.156.2023.08.05.14.10.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 05 Aug 2023 14:10:43 -0700 (PDT) Message-ID: Date: Sat, 5 Aug 2023 23:10:42 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH v2 2/2] dt-bindings: media: Add OV5642 Content-Language: en-US To: Conor Dooley , Fabio Estevam Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20230801234047.136099-1-festevam@denx.de> <20230801234047.136099-2-festevam@denx.de> <20230802-outreach-national-92083a670b58@spud> From: Krzysztof Kozlowski In-Reply-To: <20230802-outreach-national-92083a670b58@spud> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 02/08/2023 17:25, Conor Dooley wrote: > Hey, > > On Tue, Aug 01, 2023 at 08:40:47PM -0300, Fabio Estevam wrote: >> Add DT bindings for OmniVision OV5642 Image Sensor. >> >> Signed-off-by: Fabio Estevam > > Thanks for doing this. It looks good to me, although I got a complaint > from git while applying it locally: > Applying: dt-bindings: trivial-devices: Remove the OV5642 entry > Applying: dt-bindings: media: Add OV5642 > /stuff/linux/.git/worktrees/linux-dt/rebase-apply/patch:119: trailing whitespace. > > warning: 1 line adds whitespace errors. > > I think you can probably squash both patches and add a Yes, they should be squashed, otherwise it is not really bisectable. Compatible disappears after first patch... and the first patch does not have any sense on its own. Even if this is not trivial device, removing it without documentation is not the way. Best regards, Krzysztof