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 E864FC6FD20 for ; Fri, 24 Mar 2023 04:54:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231388AbjCXEyM (ORCPT ); Fri, 24 Mar 2023 00:54:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39324 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229752AbjCXEyL (ORCPT ); Fri, 24 Mar 2023 00:54:11 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 43180EF8C for ; Thu, 23 Mar 2023 21:54:11 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id D3CF26294A for ; Fri, 24 Mar 2023 04:54:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id F1A4EC433D2; Fri, 24 Mar 2023 04:54:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1679633650; bh=2JRudIQh65qZ6tHnsabSRzj18cCSM5gxQ2xSsUScztU=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=G1pKJna+/jOgIGc4r3jD8QlSPnVvOm9k8tkj96noQVOvGLcV+uOddBfsSUWYuz0Rz xzgOgW9zQs/E5UTjaHM/GUVkTcgY2xORPasZ3iSwJ1rMHm2QPEyfh2ol12R98kHVg8 6VeDITodLeutW1kDRBQaaal7eChXprIhmFq1b3+ArWkHms7RqDuHHEgB98fbFdxGCu 4s6ZKuzyqBqeWT0zpnw1z3mltHV9EUaOgqUAxvDuuTCGPITzU2c6oBbNHAurpzTuZc FvyFIBP+ovG0EbICuqI0bg91XZLwBYFMWRoKXdixJykXsy6IE670nP2I5mL8hfwDq+ YIDErTeRkSyNA== Date: Thu, 23 Mar 2023 21:54:08 -0700 From: Jakub Kicinski To: Michal Michalik Cc: netdev@vger.kernel.org, davem@davemloft.net, pabeni@redhat.com, edumazet@google.com Subject: Re: [PATCH net-next v3] tools: ynl: Add missing types to encode/decode Message-ID: <20230323215408.08c76fd5@kernel.org> In-Reply-To: <20230323175129.17122-1-michal.michalik@intel.com> References: <20230323175129.17122-1-michal.michalik@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Thu, 23 Mar 2023 18:51:29 +0100 Michal Michalik wrote: > While testing the tool I noticed we miss the u16 type on payload create. > On the code inspection it turned out we miss also u8 and u64 - add them. > > We also miss the decoding of u16 despite the fact `NlAttr` class > supports it - add it. > > Fixes: e4b48ed460d3 ("tools: ynl: add a completely generic client") No fixes tag, no in-tree spec needs these today, right? Please rebase, too, Jiri sent in just the u8 addition in the meantime.