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 2E83BC04A94 for ; Mon, 31 Jul 2023 06:56:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229824AbjGaG43 (ORCPT ); Mon, 31 Jul 2023 02:56:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39646 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229445AbjGaG42 (ORCPT ); Mon, 31 Jul 2023 02:56:28 -0400 Received: from mail-ed1-x536.google.com (mail-ed1-x536.google.com [IPv6:2a00:1450:4864:20::536]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 63F1A10D for ; Sun, 30 Jul 2023 23:56:26 -0700 (PDT) Received: by mail-ed1-x536.google.com with SMTP id 4fb4d7f45d1cf-52256241c66so8661185a12.1 for ; Sun, 30 Jul 2023 23:56:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1690786585; x=1691391385; 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=B+CfR0bQL2f6+lX0xcgogwy4lwpWoaSnfppJn89aXWk=; b=GUWEp9EcVvFYEpYWiL5hR0u5tRun0bBc6gdFUvEBr2ca3dLBFSo6wwuoZiXMZyYcGD toPpNRTJmFbjOdjqrqdN9KUbye/dlF3PdTH+hVVWddJ3NQm+3YemGfRml6CO3sQZz0AJ k40y+yRttCxEEBZfU6nPOUUuRJb1y3VSgGEDu3AEyi/ON0hOWEvwOpeBRdXfZP1dY5hN iIfhwvprSpoM5NNiKDagcfVcEz/9BE6EQQqPaByg1RwYmcjw+R3zzMI0VuCQhPL+y3w1 di6R+PU9hprzInMd9hTptP6RBthie/K7BbO0a2lbPIAlHveUdJqQN2yfVh77ZhR7hJi7 hqcw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690786585; x=1691391385; 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=B+CfR0bQL2f6+lX0xcgogwy4lwpWoaSnfppJn89aXWk=; b=Fjc97RKTLXqpgH0l/NbnvxHgmhC50Rm2pyF5pa3N3IQONS+uVl2tQRKSXMhor0mYKT Rb7eVgntdq/h4xS2SxpChsZk6N28Kro9u3v5jorEZunUcX76JhEQd7NBGSeVqAEbiV8s FOAgyqz4Z1oRecoTkrOY+NoJSVsnPT3Ks6/bIA+uBXa4t/0a6S7W3KAxgWjmdjhZ9pE3 lpPqFlrDFQmxBMeQHyzS2m3qJeYOnmfJnUFO/UF6u3Uf95+UkMYkg1Y+DN1OxBWWs/4L R0ayAd+Iqf+fJIFo5P1i6aWn/gq2XPXAAx+RYkkKi4l0Ty/OZ8X8YDIv/H/gn+BbmP+Y x2uA== X-Gm-Message-State: ABy/qLZVnsMnZo8xOHdRcLjPh4xKPFXIkAQBtnkyOGyJjY+XbERg0zvB UlD0hMV2lUAf1wm/+d9x3azq8Q== X-Google-Smtp-Source: APBJJlGCJK+vLRByTvuO6krg5VsOL4GRQzkfn6Akp0lQsgEeTOlxYlC1pqxpSpG6aT11G5ROzBnjWg== X-Received: by 2002:aa7:c84b:0:b0:51d:95ac:22ed with SMTP id g11-20020aa7c84b000000b0051d95ac22edmr9574982edt.1.1690786584828; Sun, 30 Jul 2023 23:56:24 -0700 (PDT) Received: from [192.168.1.20] ([178.197.222.183]) by smtp.gmail.com with ESMTPSA id u24-20020a056402065800b0051bfc85afaasm5011296edx.86.2023.07.30.23.56.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 30 Jul 2023 23:56:24 -0700 (PDT) Message-ID: Date: Mon, 31 Jul 2023 08:56:22 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.1 Subject: Re: [PATCH 07/17] MIPS: loongson32: Convert UART platform device to DT Content-Language: en-US To: Keguang Zhang Cc: linux-mips@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Thomas Bogendoerfer , Rob Herring , Krzysztof Kozlowski , Conor Dooley References: <20230729134318.1694467-1-keguang.zhang@gmail.com> <20230729134318.1694467-8-keguang.zhang@gmail.com> <2ee2336c-d54f-6fb7-0142-8d0e08cce25b@linaro.org> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 31/07/2023 05:32, Keguang Zhang wrote: > On Mon, Jul 31, 2023 at 11:04 AM Keguang Zhang wrote: >> >> On Sun, Jul 30, 2023 at 4:26 PM Krzysztof Kozlowski >> wrote: >>> >>> On 29/07/2023 15:43, Keguang Zhang wrote: >>>> Add UART device nodes for Loongson-1 boards, >>>> and drop the legacy platform devices and data accordingly. >>>> >>>> Signed-off-by: Keguang Zhang >>>> --- >>>> arch/mips/boot/dts/loongson/loongson1.dtsi | 54 +++++++++++++++++++ >>> >>> Same problem - DTS is always separate. It seems you made this mistake >>> everywhere, so entire patchset needs to be fixed. Keep all DTS - your >>> base board and extending it - at the end of the patchset and squash it. >>> There is little point to add new DTS in steps (e.g. first add incomplete >>> broken DTS and then immediately fix it... no, instead just add correct >>> and complete DTS). >>> >> Sorry. I thought it would be easier to review for split patches. >> Thanks for the explanation. >> Will send v2 with one complete DTS. > > Hello Thomas, > May I ask your opinion about the way to delete the obsolete platform devices? > Should I delete them in one patch? Or in separated patches? subsystem patches are split per subsystem. arch-code can be either together or also split per type of driver. Removal of code is incremental. Just add extending it, but your patch was not extending DTS, but adding new one. Best regards, Krzysztof