Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <b47c51dc-a4c3-ed7b-65ae-3938971f453b@ni.com>
Date:   Fri, 31 Mar 2023 12:59:38 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: [PATCH tty-next 1/2] dt-bindings: serial: ni,ni16650: add
 bindings
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     Gratian Crisan <gratian.crisan@ni.com>,
        Jason Smith <jason.smith@ni.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Jiri Slaby <jirislaby@kernel.org>,
        linux-serial@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20230329154235.615349-1-brenda.streiff@ni.com>
 <20230329154235.615349-2-brenda.streiff@ni.com>
 <9cd03cb0-0924-3431-8fe3-4d5f4a27b1a1@linaro.org>
From:   Brenda Streiff <brenda.streiff@ni.com>
In-Reply-To: <9cd03cb0-0924-3431-8fe3-4d5f4a27b1a1@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cmRxMUFha3pZTCtIR0RsL1BJOG5jWEs5eGQzWlhYS2pVVXNnb0Iwam5RSklr?=
 =?utf-8?B?RGxaR1ZHa25XMlk1ekJ5aGQwSXhCV3owZHdsSDJoMHdHREtEVjB1TnNsZUNH?=
 =?utf-8?B?RnMrZXF3U2RlKzByUVNUZFVhenJhRzlvTHZoRks4d2EyTXFFT1h0YzlwTW9O?=
 =?utf-8?B?MFVwck1MRkZJZ2JrTXJCV1plMHdXT2Z5eVNxVVkrcU90SENNSytQMURUMG95?=
 =?utf-8?B?Z0FDZzdkMzAvNmVVemVYRG50VFJabHJ5blVLNzNLanBkQ0huQzVzZDZodzBM?=
 =?utf-8?B?bWRMYTc3SzNJWC9Ndlo3ZGpqZmI5TzRxRjZnb25LdkFBVlM1cDZKOUtzK0Ru?=
 =?utf-8?B?TmFocnZ6WnErVGhNSWk1S1lQWGw4NHJtUlQ5Z1U4eUdGNlBmNjQyZVRhTnRM?=
 =?utf-8?B?YzVxMVBVWjhMM2hhbjlXbU4wT1JGaU1XNFBFdkJtMGF3UUo2endIV3JoWTNl?=
 =?utf-8?B?Nlk1WHNDZjd4a0dpQXhPSE5tcEZjZ0ZQaXlKcjR6MjQyTDhVem1XRFB4d2d2?=
 =?utf-8?B?WDBEWUFDVWEwTzZzSHJobGxHR1F1ZGk5RnlCVC9tWE8yQ2pKWjViWDIxaHB1?=
 =?utf-8?B?L2xiTG5pUlJGalBZVlVBSmJXZTNoT1VocHhUWWNIV2dDODh0K0pkaFpBWHNF?=
 =?utf-8?B?Y0xvY0pWMGVaallxaWVMd0svQThJc2d0MHVnK1RoU1ZWcHlmWGErYkFRWWlK?=
 =?utf-8?B?cFJOemJRMTlXdFRYTUZkL0x1UnZwNVFVZWZvVUxJcWo2c0dmUFZOak5LUjFU?=
 =?utf-8?B?YmJCdVlXeXJVbk5hdWlPeW5tTkQvMHowb3ZkK2s4emMrVHMyZjdPVWlGNHVQ?=
 =?utf-8?B?WUxER21hSllTV2EvWkcyYnZVSHdXL0s3L2dZQmxuSis1NThHNWN1dWFTQTJ2?=
 =?utf-8?B?VHNJaUtJS01VcUYyY0E0ODlSM2RSVDVQenVtb1ZueFJDK0pza3kvNENFQjFk?=
 =?utf-8?B?YlNWMlJuTGs0WTBudlBCUEZKR2ZFazFCRDE1NVlsWXhzU1o5cndDNWxveitw?=
 =?utf-8?B?b29HdWVtb1VUWXEvUUxXdElnakg2OTg2UUdKS2J6Q2VuZEFMRkNIZ1FZUG5P?=
 =?utf-8?B?YXBuQmh3ZnJwVXJ5U3M5WVlJbE9CZUducnNCTS8xbFFzVXhpWHd6MXVTQ2Z2?=
 =?utf-8?B?ZmJMdThvbFpMWE56NVgwUzV1cDBtNy9VQkFWKzZGSS8rcVluN1B3d0l2UE8x?=
 =?utf-8?B?U1lQV25CTExHeHdPb2RsVElnZDFYVVltbDJHVUpMME5qbVd5dVRWUVg0WlRS?=
 =?utf-8?B?cUVkdzRhOEZpN01pTkp1TDY3ZHRFQzlVNy8yc1lvTER0dHkzWmFuYS9JM013?=
 =?utf-8?B?ZWlPSkdDbWRvYWgwUjVOSVpJZ0hiV2RKWEEydFdsU3g5TFYwV0txUTAvMVMx?=
 =?utf-8?B?OEVNOHNwaEphMXdPNUkxTFB1UUdYTTB0M2d4eU14WG5XQ3J4Um1sd1VmakVw?=
 =?utf-8?B?eGZXTDgraWlGUGQ2bE9GWVJNQlF2WnIyOWExVVY5VW1MS0hkdkFFT0hsazBw?=
 =?utf-8?B?eU4wNUtldHlIUHlnOWZReUVaUG00RE84RjhjcmdqMUNZWUtCL0VIclowNUtJ?=
 =?utf-8?B?SFNtQ0lwRlRScEMrdWhKbHBOcnNvV3dTK1Ezci9lVGhCRkw2UWxNb0pKeWV2?=
 =?utf-8?B?MWIzRWE4NklwWmdkaFZOUjlyTkdrWFpMdTFnNmU0NGp3NmYveEp0em15UUx3?=
 =?utf-8?B?RU5NUWFyMWsvSDhSNmdjT2gzd0ZORWljZUl6Q2xxOU15QXpUZU1uY1pTQ3R2?=
 =?utf-8?B?MWhUK0ZtbDYxTzVJQWk4TGFQc3haTjJpNG52cmNlamV1NnVuZUpJWEtTRGx6?=
 =?utf-8?B?UVdWaDBoU0pMTVFyL2RkeU1jdGQ3NzVENC91YzNsTy94NDNCeG9ZWHhRaUhj?=
 =?utf-8?B?L0tSdU5xUVpWTWpkRStxcEFXS2JFaWV2QjJOUWZCWWErNTAzVkZMV1gzVkhw?=
 =?utf-8?B?dldWWjQzbVgreGxSTGE1dDl5QmxYQ3hRTnkzQm9MY2ZKZ2tBM2VERS9zVHZQ?=
 =?utf-8?B?U09FTDM1VXc0THgyTW9iV0xEOFc2SUcvMXBjRUNmSDFXUThld3dibTE3UkhS?=
 =?utf-8?B?cFZyV3hhUWtVdlhLVVNPU29KcFhrVUJCeitXeEh3SG8rWGd2T1NGOEU1L0c1?=
 =?utf-8?Q?DotBgptFCglL5VU8ChE3Du0Sr?=
X-OriginatorOrg: ni.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fee9b67f-e040-4904-3168-08db3211b33c
X-MS-Exchange-CrossTenant-AuthSource: SN6PR04MB4973.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Mar 2023 17:59:40.7995
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 87ba1f9a-44cd-43a6-b008-6fdb45a5204e
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ZgQamjsRsdNTkcnElioVlD0pKrwsr5LUqsgPFLM5dknANw+CBue/EGmBZcdCTKtNTx/trgP+NQYPtQvYGAVbRA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR04MB7853
X-Proofpoint-GUID: l2eWjNxlTz5-_bRtNOV3NLPk9Es7Yg6a
X-Proofpoint-ORIG-GUID: l2eWjNxlTz5-_bRtNOV3NLPk9Es7Yg6a
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22
 definitions=2023-03-31_07,2023-03-31_01,2023-02-09_01
X-Proofpoint-Spam-Details: rule=outbound_policy_notspam policy=outbound_policy score=30 suspectscore=0
 mlxscore=0 spamscore=0 priorityscore=1501 impostorscore=0 mlxlogscore=875
 malwarescore=0 phishscore=0 lowpriorityscore=0 clxscore=1011 adultscore=0
 bulkscore=0 classifier=spam adjust=30 reason=mlx scancount=1
 engine=8.12.0-2303200000 definitions=main-2303310144
X-Spam-Status: No, score=-0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
        DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,SPF_HELO_NONE,SPF_NONE
        autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 3/30/23 02:28, Krzysztof Kozlowski wrote
>> +      - enum:
>> +          - ni,ni16550
> 
> As Rob pointed out - you did not test it at all.
> 

I did, with dt-schema 2023.1 and the 'make dt_binding_check' command as
described in Documentation/devicetree/bindings/writing-schema.rst
(with no DT_CHECKER_FLAGS, because I was unaware of it until Rob's post)

Is this a documentation gap, or is the DT_CHECKER_FLAGS option slated to
become the default for 'make dt_binding_check' in the future?