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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id EF969C6FA82 for ; Tue, 13 Sep 2022 09:21:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:To:From:Reply-To:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=aI5DthhB6OEw3fniDpFKmvMFQ/7ffg9dOJqNGYWbMPI=; b=JoBIMkcLc6OaHh MWvYEY6/VkrW+dK4ekNIKUawtgew1/5KU8y7t5Sz2VLCLBDvWN9AOvAyIBNjJChsKZIh9naHEQqde TriqWU8ujCPFQKhmi5HqVt0XXche0t80/47qkHrZ5V3w28zHpjsL+0yyi1klJhAR51jZc0q1p/u3q AwMkwjAggPfJi2k2S25jCOpdxu2kpAvOpsRA7YNeLBEZV7LIIT9PvFkTIt3ZP1XIFpX3E9eEVfI/N oUaLkn+kw6Hnt0ixSx90Uqd0bkrHs7Lx5teKziyXJvWSOFYiM7rJfkG0M4JddG6cNs9luS4YUpVZn /etOZlpk9wvIdJHKlDfw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oY25A-0053cf-03; Tue, 13 Sep 2022 09:19:52 +0000 Received: from esa.microchip.iphmx.com ([68.232.153.233]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oY254-0053My-Tp for linux-arm-kernel@lists.infradead.org; Tue, 13 Sep 2022 09:19:49 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1663060787; x=1694596787; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=0dFAibKpYm1f7rPnHlhlDVXmQAzDJAhv3nTHnELk21E=; b=N/rUzFn5mjpKyBco1m/dOZ2LQXOeSRxHRnLTRW+p5QOGMf96uZotk09D /+33la6XDrpbeiUD+A0Ejn4xBqTbx4zxkfZjJ6gDAX0gf50/17HVFQ/AO 3peFTd0ZDqzT37lrJMUf9rS7sBOjJuGFWZBsAczp7ktowvQfGnkfxMwUJ xDHHjq3+kXlqWkigTzS0IGt3SryvXdNDu/mfTVrYij8fv2scaL7c8c5KU i1xHZcAg9ucR8A9ztDCzRWkrxZya/NpTI7uj7Qq0H2Azl3I5L7fjlKeyu pqIgZp2dUDj67soudvXpqeaB/wFsZSqZ9H9wY7bp8PGTr+5aiv6sX6S/E A==; X-IronPort-AV: E=Sophos;i="5.93,312,1654585200"; d="scan'208";a="180153374" Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa5.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 13 Sep 2022 02:19:33 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.12; Tue, 13 Sep 2022 02:19:32 -0700 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (10.10.215.89) by email.microchip.com (10.10.87.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.12 via Frontend Transport; Tue, 13 Sep 2022 02:19:32 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XaavlKZXS3YnQnbDeU48MoLl3JT3jVk3TpwiMJ8UWv1YuQufQ1lIRXSR8yTOBM6unPYB5xzbLZG7pUCndRRrfUu4/FXMGMcC4NZuZ6lmQZV28SCu8jdeVuwmBML0w9S6XM6BBRY9w07K4QR+gQxy+SqfBb4h9oRNI5qoFKGWPtiyZ5eioOWHUhtj31xkmERfpm7pkGz9Goaiqa+CBKj5LhiaJp4JQvpxNViqZH9X+tx7dqkiUg2fDXtiUrlY8Z/JXBsUIl8L/KdBkof8TMypczgws8mNy1SxOluBAmKFnuvYmwgTY+9MRx6wVH+p70zNdyOTjgN2sN5VaAIkD3wa2A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=0dFAibKpYm1f7rPnHlhlDVXmQAzDJAhv3nTHnELk21E=; b=jeodYOsaKUaPKQCmEZZIkZooO2ZVs8IGgkgrVpXj2w0Y4RsENpwCko8WjwGUD8SgYnbbnDZ4BadAz2OmJhtZ7eCPP6EdMjadCrUiGzV3VEo2SdY5sNyUO4edYT6REvNIjYMVQUZ89fUkoQkwwnMmHPouGWo0Z3KuVDW4S32Y1kwhK6VzL9b+KcrlbVnM8PG1+TMnmIOJnbJgBLVIJv+Dkuo0qPh7uGcv8dyef2BvV7PvqmlUuwHNZNb8/joTZ3nI/znwr6U4Lc/63sPr4lCXqTfdH9p2QGlidpPiQ6DTFfs+iSwq++Ul3OmtECflmWv+3A17vGE/bsCxGxwbnh0uJw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microchip.com; dmarc=pass action=none header.from=microchip.com; dkim=pass header.d=microchip.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microchiptechnology.onmicrosoft.com; s=selector2-microchiptechnology-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=0dFAibKpYm1f7rPnHlhlDVXmQAzDJAhv3nTHnELk21E=; b=LLjk1oXdqx+d5ZbMg/QF3KGnHby1Ng8msd6Q9hZ5IXtrBXy6QDmBJ49lIcoZuDQCpdwVsMw2Dyp/JoQ+7x2KQ6o1ZME5z1FPpBluKPaCBv3EIP+Hgm+cYRH2bTaSv1nmCC/difDYOg6ufNIZrYb0uyIhoDWur5B5EpRNdNQKBek= Received: from BYAPR11MB2758.namprd11.prod.outlook.com (2603:10b6:a02:c9::11) by PH0PR11MB5579.namprd11.prod.outlook.com (2603:10b6:510:e6::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5612.19; Tue, 13 Sep 2022 09:19:31 +0000 Received: from BYAPR11MB2758.namprd11.prod.outlook.com ([fe80::e8ff:82ac:b76d:c61c]) by BYAPR11MB2758.namprd11.prod.outlook.com ([fe80::e8ff:82ac:b76d:c61c%4]) with mapi id 15.20.5612.022; Tue, 13 Sep 2022 09:19:31 +0000 From: To: , Subject: Re: [PATCH v2 06/13] dt-bindings: serial: atmel,at91-usart: Add SAM9260 compatibles to SAM9x60 Thread-Topic: [PATCH v2 06/13] dt-bindings: serial: atmel,at91-usart: Add SAM9260 compatibles to SAM9x60 Thread-Index: AQHYwfjCQbG755VmlEGUqK2TrhMH7a3VeaSAgAAuKICAAK1+AIAFHfCAgAAyC4CAACijAIABTCEAgAAF4YA= Date: Tue, 13 Sep 2022 09:19:31 +0000 Message-ID: <888b8ab9-f8cb-2e1f-465c-3cb97187fce0@microchip.com> References: <20220906135511.144725-1-sergiu.moga@microchip.com> <20220906135511.144725-7-sergiu.moga@microchip.com> <9aa29d74-b1fc-d00e-dee4-57f277a366ab@linaro.org> <20220909013644.GA3731620-robh@kernel.org> <77d38e3f-6d8c-dbb1-2e66-c768d95b5e35@microchip.com> <5f8ca612-5a89-db3a-42f3-a0613c192a87@linaro.org> In-Reply-To: <5f8ca612-5a89-db3a-42f3-a0613c192a87@linaro.org> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microchip.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: BYAPR11MB2758:EE_|PH0PR11MB5579:EE_ x-ms-office365-filtering-correlation-id: 3078cf37-f018-4843-9ed9-08da956910c2 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: GxNC4igJBrteuMZpi4mGCLHy8qTJFJKyu3g4o8AKQrZEairuO8sLr/9Ro/LRDgr7m2g8O3hbhE3QlsK3Oldu7jkNfcPh/kXT/e4qxvkiT1bVNuFbTHQoB03Bk1WQWMcC61AB/hN/o/ndWOoyAo0CilAqSC+AzDJhV9wJOYFJyBjbdcBAjIDI6TOrGToXTfZ3iI+JgJKxoTOtDiwunY5t7oPHUmcX5mcxBRLd5N3+a7wCCnt+dljSUmdkKo/PzLUPOaO+NZ66XCekzUVv4b5KsgDyFsEoktKih3woOIJu+vnN56FBswPuON7iDz1szymF4rJ9PUpGpRRdyVKmRmJPqZcXzNuQFIcDPxNvUoBDXHCcGZOjLNcejJW1FxMZazvf9VueZRpZYXrFSMAEB4oTHO9lwnXi5dzwALK6oIjz7XfUogU+ESsI1/WRBKrwE42yDacGAsjvqrSmuOVupKoA5OCCJqh9RK75R0jDCofM2cbCNfCeYx92WAac+aLVpp/RFYN7NIuBhXVDT5TXHoQS+/zhqAQoptPjRvxGCP2LkO8AZSdUroi2SI1S3x8/qlKgyHVL1pYv/Qet/jXMR2L7PRTFZ8Cr9zSKNgrbguzQt/llfMlrLBnGJySFVpgaa2C1RI44X7LUJQZu7U5T3fL6CANCmLkz8K90CGxfz74Qti/Mm3VXPsDyaUY2E60frtjeN6nZgnpiy3SbszKtlfxr1z5jvJ7v24r4WUhQhf82nO7QQk+lHW+zXy0eTbnGczuv+ovM5byetLHBRJwg2L7NLF4jNqKBUjX5fSjwZ2lJxds= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BYAPR11MB2758.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(396003)(39860400002)(346002)(136003)(376002)(366004)(451199015)(8676002)(7416002)(5660300002)(2616005)(66946007)(66476007)(41300700001)(6512007)(186003)(26005)(53546011)(316002)(38100700002)(31686004)(2906002)(122000001)(478600001)(4326008)(71200400001)(76116006)(31696002)(36756003)(54906003)(86362001)(64756008)(8936002)(110136005)(83380400001)(6486002)(91956017)(66556008)(66446008)(6506007)(38070700005)(45980500001);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?bTJPdFRrSWtmYjhGU25zTWpDWUlNdTQ0WU1INFhrWDVnTDlzNDZTTVZZeDdk?= =?utf-8?B?aUJzMHh6Ry9jM3RlcGRTSmY3TW9WQStaUUdZajlYTmZHaDJMY1ZzUVJpL2ZQ?= =?utf-8?B?T1FmS3ozYUhOL1dBclRRNzRRSlJRd0t5bVkyUDkxbGxtVVp0QWJSVEJ0UzZj?= =?utf-8?B?NzRQcElwYjRaeUttWlRHT3pITEQ0eGVkd3BwNTFmZXlyM2w1clpyQVdFVm9Q?= =?utf-8?B?T004cU0zd2N2bnRGTWpVS3NTY2FEbW9FQVhKMXFlelVKQ0dwdlVsOC85dWR5?= =?utf-8?B?WDdCREZHSXErcGN6cjBoK3BYMHJ6MFRJdmk1Znc4VVBTTGFJaHFLQUsyQ2RY?= =?utf-8?B?MVJZTjdwZkdNRHlJZzFIL3dwOSt3UW1yTHpuWERKek41T1g1SWk0d2IxYTdL?= =?utf-8?B?anpMaTlBWUFVMEk0WU1TZG5yb2dncFk2TUpPTnF5SCtMWnlOY2FwUm5FSm81?= =?utf-8?B?L1dDN3JOeGlvNnBkVmlOakZQazlCbHZzZEZEd3NEdDhHUy8vUXNIenpxb1FO?= =?utf-8?B?ejdNR0hHVFVCMFkzemdzZDc4K2FUalgxWEpRRURidVRmQ0RYU3V0U1BmV3l6?= =?utf-8?B?U29FMFZSMGpsWFBpbXV5VTNpVDVVdjVQWS84VnBrWmhhQi9IRlMzS2tkekVT?= =?utf-8?B?NmZnK2VlakxvdjU2aDNiSDJDM2ZGOThMSlEybXJjNGxlTlA5RllmT0F5YUdM?= =?utf-8?B?OWovVXcxTjYwbFRqZmN0MzBSem04aCtlL0VOM0xEVTRWOUpWd3dXYmlSc2R4?= =?utf-8?B?cm85K0w0TGJISzQwbW9kRE1BY1VhcUNLcVArWTAvbnVjd1h5K21ReUdjMDl1?= =?utf-8?B?VS9Xd0NOWVVWZ1BZUFFLVnJUdkw2RTZGTWRVZ25SanRoU1NzM0VHdWtTemlz?= =?utf-8?B?aUZJVUZaSGxtVXlyakI3Snlxdy9PK3pKVWN0M3dnbU56TDZKOFpKajVNZFBD?= =?utf-8?B?czBmclB3ci9sQ3VQeUJ3VVJIQndJRDhSSEI5MXdqcmRsSm5jOTJCL0JBdVUy?= =?utf-8?B?cDRieG5vSFJZSnU5Z2NSdXJPdFVjZFBRTDdDVFdBR1FXSTJrd3JwOVlRMklM?= =?utf-8?B?dWtMUXVnNi9mSUYzeTNjMXJ6YTJiM3cvRExtM3pDa3BjTzRqSG8ydmFGS3JE?= =?utf-8?B?WjBqMTYxUTd3MzdjeVkyeVh2NFVkUmRQWExHOUNERThpdGpHZWZMYmVKMjlt?= =?utf-8?B?WjlkbXd4Wm9rSkdpV0tOczNhdUZXSHlGbFJNcTY2Z2N3RlV5WmtzbG1HZlU3?= =?utf-8?B?RHoya0NKOFRJeXh6MHVCSFR6WWQ0V2E3bnBKUnQ4Q2lPYjRiSlY0OWFRUEtz?= =?utf-8?B?UlNob0NabVhkSzY1YVY1UXR1T0pFZjBsc0NyS24yYmpJRmFRS3A1U0N3YVBM?= =?utf-8?B?Y2duRmo5UUJIYVR4VlR0OURJaWVCb2hTN1hZdUNmcEtEU0ZZTFhBS0hIZHB5?= =?utf-8?B?ODI2NFhSWWVzalZCQWkwc20zRGVzcmZCSVF6Z05FdGhlVUlkWEU3Wloxd2lq?= =?utf-8?B?T2NvWnZ4TkNvbmo4S1d1KzJnUEtjSTlab05ieitzS1RUckZFb3pLNXFicU1E?= =?utf-8?B?WStBSDhzbHA1dGY2cTlHVUo0Q0tUdEFmZmdnZ0JuOHQ2TWdWYlZXanQ1YkdU?= =?utf-8?B?Ry9vY2prSXM5TWJDdk1QcStoMVRjZkZ0OTAvL1hIaGR1Y1JJczJkNi9Ca2ZG?= =?utf-8?B?V2xrU3BCckw3WXlEbEFBcy9oei9SU1lDNFloSmJoazdFQ25lVDd2Y3EyNFI3?= =?utf-8?B?SDcvYlNGQndoVTZTSkhOeWNBc3BFQ1NBUVU2Q0dJR1YxWklFN2thRHhTYnBW?= =?utf-8?B?WUwrcDNHTForRVZmQmR6VmpiR3o0V1FINGplSkw3dVg0bUN6YngrQmIvQ1Jl?= =?utf-8?B?TmIrVkc3Q0JnWWk3T2xNZlIrWkZkUXpWYzNzUEJKUFdBRnkwK0p6RkFrYTFm?= =?utf-8?B?ZVJCQlV0ZFhYR0hnVVlUQXhoU0RQeFZxRUdCcGx4dXExREVZNTI4VnpmYSsr?= =?utf-8?B?SGhzVUUyZGtaRXJUT1BXd2ZJbG0zTW1GTU9lQ29XU2d4VlU0aUZGMmhwbFJh?= =?utf-8?B?VUNEYUhFMHFMSmY0dVFjcmpBbGFMZTVma0dHUEJVcHNwNzljcTRnSDZNNHo3?= =?utf-8?Q?RBqLnrZ7kM9TuuyfVE2JNVFX0?= Content-ID: MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB2758.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3078cf37-f018-4843-9ed9-08da956910c2 X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Sep 2022 09:19:31.1485 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: Qzp2AGFC8NVm/pan2pZRU3R1O24CBvtJ8ngbiMcszNTjdxNAdWwCgrCzbi7E5eWtqwU11YyU/jrNsImaNfoyvGyK1Hg3pDh5+c4pfz1o6a8= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5579 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220913_021947_009265_EE384FFC X-CRM114-Status: GOOD ( 22.10 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, alexandre.belloni@bootlin.com, linux-clk@vger.kernel.org, Kavyasree.Kotagiri@microchip.com, Tudor.Ambarus@microchip.com, richard.genoud@gmail.com, gregkh@linuxfoundation.org, radu_nicolae.pirea@upb.ro, lee@kernel.org, linux-spi@vger.kernel.org, linux-kernel@vger.kernel.org, mturquette@baylibre.com, broonie@kernel.org, admin@hifiphile.com, krzysztof.kozlowski+dt@linaro.org, linux-serial@vger.kernel.org, sboyd@kernel.org, jirislaby@kernel.org, Claudiu.Beznea@microchip.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 13.09.2022 11:58, Krzysztof Kozlowski wrote: > On 12/09/2022 15:09, Sergiu.Moga@microchip.com wrote: >> On 12.09.2022 13:44, Krzysztof Kozlowski wrote: >>> On 12/09/2022 09:45, Sergiu.Moga@microchip.com wrote: >>>> On 09.09.2022 04:36, Rob Herring wrote: >>>>> On Thu, Sep 08, 2022 at 03:15:44PM +0000, Sergiu.Moga@microchip.com wrote: >>>>>> On 08.09.2022 15:30, Krzysztof Kozlowski wrote: >>>>>>> On 06/09/2022 15:55, Sergiu Moga wrote: >>>>>>>> Add the AT91SAM9260 serial compatibles to the list of SAM9X60 compatibles >>>>>>>> in order to highlight the incremental characteristics of the SAM9X60 >>>>>>>> serial IP. >>>>>>>> >>>>>>>> Signed-off-by: Sergiu Moga >>>>>>>> --- >>>>>>>> >>>>>>>> >>>>>>>> v1 -> v2: >>>>>>>> - Nothing, this patch was not here before >>>>>>>> >>>>>>>> >>>>>>>> Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml | 2 ++ >>>>>>>> 1 file changed, 2 insertions(+) >>>>>>>> >>>>>>>> diff --git a/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml b/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml >>>>>>>> index b25535b7a4d2..4d80006963c7 100644 >>>>>>>> --- a/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml >>>>>>>> +++ b/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml >>>>>>>> @@ -26,6 +26,8 @@ properties: >>>>>>>> - items: >>>>>>>> - const: microchip,sam9x60-dbgu >>>>>>>> - const: microchip,sam9x60-usart >>>>>>>> + - const: atmel,at91sam9260-dbgu >>>>>>>> + - const: atmel,at91sam9260-usart >>>>>>> >>>>>>> This is weird. You say in commit msg to "highlight the incremental >>>>>>> characteristics" but you basically change here existing compatibles. >>>>>> >>>>>> >>>>>> Does "show that they are incremental IP's" sound better then? >>>>>> >>>>>> >>>>>>> This is not enum, but a list. >>>>>>> >>>>>> >>>>>> >>>>>> What do you mean by this? I know it is a list, I specified so in the >>>>>> commit message. >>>>> >>>>> You are saying that compatible must be exactly the 4 strings above in >>>>> the order listed. You need another entry with another 'items' list. >>>>> >>>>> Rob >>>> >>>> >>>> That is what was intended though: a list of the 4 compatibles in that >>>> exact order. The 4th patch of this series also ensures that all 9x60 >>>> nodes have that exact list of 4 compatibles. >>> >>> The commit msg suggest otherwise - two options, because it is >>> incremental... But this one is not really incremental - you require this >>> one, only one, configuration. It's in general fine, but commit msg >>> should reflect what you are really intend to do here and why you are >>> doing it. >>> >>> >>> Best regards, >>> Krzysztof >> >> >> My apologies, I still do not understand what is wrong with the commit >> message. My intention was to ensure that every 9x60 usart compatible is >> followed by the 9260 compatibles because the 9x60 serial IP is an >> improvement over the 9260 one. Would you prefer it to be just the first >> part of the commit message: `Add the AT91SAM9260 serial compatibles to >> the list of SAM9X60 compatibles`? That way it would really only be what >> the commit does. > > Let me rephrase it: > > What your commit is doing is requiring additional fallback compatibles. > Therefore the commit msg should answer - why do you require additional > fallback compatibles? > The additional fallback compatibles are required because the driver in question only knows about the atmel,at91sam9260-usart compatible. Furthermore, it is also a better representation of the fact that the serial IP of 9x60 is an improvement over the serial IP of 9260 (it contains more hardware features not yet implemented in the driver). > Incremental characteristics sound to me optional. I can increment > sam9x60 with something or I can skip it. But you are not doing it... > sam9x60 was already there and now you require a fallback. > > Best regards, > Krzysztof So, what is your opinion on the following commit message: "Fix sam9x60 compatible list by adding the sam9260 compatibles as fallback, since the atmel_serial driver only knows of the latter's compatible. The atmel_serial driver only has knowledge of the sam9260 compatible because it does not have the sam9x60's serial IP specific features implemented yet and adding an empty compatible without adding support specific to that compatible would be misleading. Thus prefer the fallback mechanism in the detriment of adding an empty compatible in the driver." Thanks, Sergiu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel