From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BE4BF7DA66 for ; Tue, 2 Dec 2025 00:38:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764635901; cv=none; b=fC/yTL79G0k4aIg00tXuWxlQxoMd5FZRYuFy6dPgDI2fabztIAzt+V/bBSZLFzEcw6meknOTBhIMdrqsVzj4VKe1WqLBMle1RiG/NdN30ah8isPDz4ZVEW6IbRyIqS7pmJEnd1bLe2SgbO0mmDBB/m8CZ1ZT72k0JakIenNy3C8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764635901; c=relaxed/simple; bh=BujrSu+y9SwZZzARvq3aZf5jeZIETdmmjG+R0lJrl3I=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=OdT4Cw6r1SylwtRtCXVaQr3QLFOA7aEp4a+Rxa1iuAuptNJ+hERuBlRZpVBIcSLkqPGdNDo/S+OXLKmqx499OnPqPfiRvsrafcXVDiGuHrEdBYhWHPwZ6bSezDnDqrBQluC1MDMSNoc9AoRVuU/ofpazFlWjh2aAGsF7s5VWr8M= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=networkplumber.org; spf=pass smtp.mailfrom=networkplumber.org; dkim=pass (2048-bit key) header.d=networkplumber-org.20230601.gappssmtp.com header.i=@networkplumber-org.20230601.gappssmtp.com header.b=1X8FCg47; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=networkplumber.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=networkplumber.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=networkplumber-org.20230601.gappssmtp.com header.i=@networkplumber-org.20230601.gappssmtp.com header.b="1X8FCg47" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-47775fb6cb4so35002495e9.0 for ; Mon, 01 Dec 2025 16:38:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1764635898; x=1765240698; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=QHgdrCMSyZrsbol2Ny58yvtqwGpR5hU0tMP2n9Q11Sg=; b=1X8FCg47Qvxdn19DMsVMmYfFFDPdnXzjvRvTskXrqDPBB29uPfF50sF/ybOTBhoN+j fuHH2IIpFkX2+VMXlw55QNWhaL07K3oLEedJ1MHqBzwvr1pNy3niPBgWi0PnQ7e4uyrF BSXwBYEZQC4UQnqIjOFfgZFPXGaFGJpqRq6+TB0YnARqTdbo/NcK/zKlDXGPGY/qO16Z bsDRAtjEl+M1wOu14RAh4vHvyvT6ovY0oMmU2xYhjm3PBJio66az+AcjweAIRdlCsdeE tb9ClMn18zkbHZqfrtk0Lxq6y4iAe7C9hqKYqTzyAaJzrvGFvKdDalofnWozOF5YzEdE SGcw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764635898; x=1765240698; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=QHgdrCMSyZrsbol2Ny58yvtqwGpR5hU0tMP2n9Q11Sg=; b=bpF9Zjal6hB6QK0FgiE1f4OmsjZ8UAYs/NmCTrU10VTfQ6IWqkzkjI20r+11cUmQv2 mWBk4STz2vuBK/L1F9atGowYt1njFJBNIMbFCN0w4GzKUIK8Dp9Vszz4ho3c3wA25t89 S6iqE5/nMvd0guAJDIxOHfKXZa6yM9aUnv1vhH3EUgMoEEVC2npvVaBr535fsihOTqdT +1vH2wnlacA+s2uW5vKsfQSZ3KogLmmfsoo0ipkZ/FGXUpOOZ2EKLrkP7uExRIgfAo/2 Snv7mULxd8Um7WbZHE83boCsXkf7uFIE8DOXzk3oCvmaUd0l8qRR/aFD0lVRhZ1K1Un8 yDtw== X-Forwarded-Encrypted: i=1; AJvYcCVDSsd1n11wxXxA/YxlyTq4ChlFrUMc2lVS3lSwZnsfeZUDv9QDCgs7Fo2BXZ3zW8z8OWZaNh4XQUlIrn8=@vger.kernel.org X-Gm-Message-State: AOJu0Yzmf1Js/nGgSbJBAXqXwMxzkPZpRPWseFrgy/OBOK7sOUgJEkHj vBK8zME0VqYG3O5DeqSyzIZ19cMPBxnSlOZBlcb6J2iFt+j1fdEQGVlciervV8ESWbQ= X-Gm-Gg: ASbGncvITlAwgtBkkHb+XXCOqK+svR/AgEocb80vZxzrV0B1Met2qwpvwdNc9OV80Ai PTxGnESUN71DUHNEjuG3i7v+amypfVv158xH/8qHi9e2OymktWbLoGGSX7+8czMzGvlZVNHnGQ+ zztH7/4V7Aw98sTJmXS5Fr1odoK+oYflFFbtec+Mh4HujaI4Hn0W+CAWeJ30CLePl7GKx3oC2ae 7q+KunvyHcY2rOEucVWXsNgi1BhUiWuNT7FAU4v+0sIxWGxD6LnZUhtGI2m/gu35fwD3TWGhA8g Cu8JUFXLnZstXZALh1p9zSuILeb2qWtacP02g0toi1nr4Kgk2pp/CeWivHORefSrJ65Ule2PDWo Bp60ibrUf0iTdhx32MPUr493839bfbRXQMfzuyRR35H3kazgqTMsVzqzNAv9w9tT4k/oDeN8sNT LVvIxFfzV4q/7KxK7WnfYbepBo8xjiPSkOvpoK/K/zOQksDZCqyjng2Plq1+Nv+7M= X-Google-Smtp-Source: AGHT+IEmxd7MorKQrK7TTaf6Wk64mPP08pnZ/XYkB8I8vFTvRhnlneHMm/ytcv8HZc+8AtFDfsjPCQ== X-Received: by 2002:a05:600c:1caa:b0:477:8b77:155f with SMTP id 5b1f17b1804b1-477c10d4935mr469757915e9.8.1764635897998; Mon, 01 Dec 2025 16:38:17 -0800 (PST) Received: from phoenix.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-47905302ef1sm164565595e9.16.2025.12.01.16.38.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Dec 2025 16:38:17 -0800 (PST) Date: Mon, 1 Dec 2025 16:38:10 -0800 From: Stephen Hemminger To: Vincent Mailhol Cc: netdev@vger.kernel.org, Marc Kleine-Budde , Oliver Hartkopp , David Ahern , Rakuram Eswaran , =?UTF-8?B?U3TDqXBoYW5l?= Grosjean , linux-kernel@vger.kernel.org, linux-can@vger.kernel.org Subject: Re: [PATCH iproute2-next v2 5/7] iplink_can: add initial CAN XL interface Message-ID: <20251201163810.3246dc49@phoenix.local> In-Reply-To: <20251201-canxl-netlink-v2-5-dadfac811872@kernel.org> References: <20251201-canxl-netlink-v2-0-dadfac811872@kernel.org> <20251201-canxl-netlink-v2-5-dadfac811872@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Mon, 01 Dec 2025 23:55:13 +0100 Vincent Mailhol wrote: > > -static void can_print_tdc_opt(struct rtattr *tdc_attr) > +static void can_print_tdc_opt(struct rtattr *tdc_attr, bool is_xl) > { > struct rtattr *tb[IFLA_CAN_TDC_MAX + 1]; > > parse_rtattr_nested(tb, IFLA_CAN_TDC_MAX, tdc_attr); > if (tb[IFLA_CAN_TDC_TDCV] || tb[IFLA_CAN_TDC_TDCO] || > tb[IFLA_CAN_TDC_TDCF]) { > - open_json_object("tdc"); > + const char *tdc = is_xl ? "xtdc" : "tdc"; > + > + open_json_object(tdc); > can_print_nl_indent(); > if (tb[IFLA_CAN_TDC_TDCV]) { > + const char *tdcv_fmt = is_xl ? " xtdcv %u" : " tdcv %u"; Having a format string as variable can break (future) format checking and some compiler flags.