Received: by 2002:a05:7412:98c1:b0:fa:551:50a7 with SMTP id kc1csp35671rdb; Fri, 5 Jan 2024 01:39:22 -0800 (PST) X-Google-Smtp-Source: AGHT+IEb8EXiXGZPpsfWcaewglwpXnHKWF8CGDh5JhIZInOBb7ZmsfMKESYoTSffVnbO/uC/7wUT X-Received: by 2002:a50:d59b:0:b0:554:1100:99e9 with SMTP id v27-20020a50d59b000000b00554110099e9mr1898653edi.9.1704447562693; Fri, 05 Jan 2024 01:39:22 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704447562; cv=none; d=google.com; s=arc-20160816; b=XSZTvWKCDFB1bK6NDmDxezYjBUssKTcHim5lWtdoP33axaLkCsnEADscoGRby0PkBJ iOmCYoze0bg3ljrV9IJhpOZ0YUgVFdooO+hVZatwiph6w/Kl0c4Il3n8gIYugAnBg4uh sZ3cad3yuU/UiPb5pt8EMt/SUDbDHRhwkdzO4rOGeT6br3IB/PtgXunnpaquSS1Tfs+6 RcY/cwiYegoXHoSfavf8NQ/nLSls8LhoUsvaIg6Tj24xWTrIyM8O9otS9Ak5A+kEwA07 yDDSVrhUObRoMkEeMdtjM9tJ0TNcW84MsYxPJAhJrZnr+Ex1/0IrwOa1FVaqfYWMKrUZ nx6A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:list-unsubscribe:list-subscribe :list-id:precedence; bh=ADivkWK4lWs4Zr5dQX009Mb26e7oIT3IDUBbQqeyIfg=; fh=eV4LK3XrLxRggMj73ev+uyHr88Td9deaUFjx6bXA4Bo=; b=PfHLFeRviD3qnSURtMoXwtYNd5jbA/UqWvNfqeCawvQoeJ6gjNWO7RSnnqBQA6CG7e XuKsybgbh4fxcx3GB6BCFyDPitU7EJkhFfrxtKtAamEiRpEA0K1yXACq1c9Th5trSLJY wvzSr78vZxxInHT8u/xTp5qoj17bH6dL7nQDsqpViK75nhEQBmkXDZae0uJIOVJFr2no 4qXkFJ1Vtw7TwYhdntNv76JykoTL61hUYZJ0igPmLPXPBJG19LyXm/+w4R88yCyP2Aqq pMZuM2AzeVJAcIX7yjC5LSm1GragX63SCxnP2dgK0EpfWoksgl0yGajSp/NZenZWimIs JwBQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-17669-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-17669-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id dj17-20020a05640231b100b005542033f1ddsi520666edb.152.2024.01.05.01.39.22 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Jan 2024 01:39:22 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-17669-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-17669-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-17669-linux.lists.archive=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 6C3361F22D11 for ; Fri, 5 Jan 2024 09:39:22 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 4D393249F3; Fri, 5 Jan 2024 09:39:12 +0000 (UTC) X-Original-To: linux-kernel@vger.kernel.org Received: from mail-yw1-f170.google.com (mail-yw1-f170.google.com [209.85.128.170]) (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 25C01241E7; Fri, 5 Jan 2024 09:39:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=linux-m68k.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-yw1-f170.google.com with SMTP id 00721157ae682-5e74b4d5445so12278947b3.1; Fri, 05 Jan 2024 01:39:09 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1704447549; x=1705052349; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ADivkWK4lWs4Zr5dQX009Mb26e7oIT3IDUBbQqeyIfg=; b=nOqYqhs43LlfFJuJRjiotdqHn8KAICw422qKN6oSoKWgKhONbh+foVK/RFLJ4oEGgq MkTtp8ZSHwsV6xghu7u9xCAXaJ1Gc2RaQxlk/wingY27DbHdepsB/+e0WPxYxs9CDHco hvJmO5N1TXWrn3VtGCpsQZMSfZ5lO4UmnWt1mPRHHVLXM/JpFGee5iuXKH+76g0JfYEp dDTKsH6BpnqQTTfK3wxgD75SGMCv51eY718WbqzyqliX1Yk3St49L5mp54QQIYSemERd Mjpvl5vytf3OlvXLyCFr+CN1au8+k//MYFLdEpagwTgX8TLhCpZKyfBWSnksliCyT5rY ycNw== X-Gm-Message-State: AOJu0YzRxQ/+pVcqCU3/uv3gwONcaRIwT763vbeb0ImGwi6VglTkbw4M voE5o52YAVrkl4Eqo5PBxH/QIIUH6cT0DA== X-Received: by 2002:a81:8701:0:b0:5f0:b992:d5f8 with SMTP id x1-20020a818701000000b005f0b992d5f8mr1946479ywf.64.1704447548942; Fri, 05 Jan 2024 01:39:08 -0800 (PST) Received: from mail-yw1-f174.google.com (mail-yw1-f174.google.com. [209.85.128.174]) by smtp.gmail.com with ESMTPSA id s62-20020a818241000000b005f48f4e0d4bsm509457ywf.64.2024.01.05.01.39.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 05 Jan 2024 01:39:08 -0800 (PST) Received: by mail-yw1-f174.google.com with SMTP id 00721157ae682-5e734d6cbe4so12338917b3.3; Fri, 05 Jan 2024 01:39:08 -0800 (PST) X-Received: by 2002:a0d:f4c1:0:b0:5e8:f2e7:c48a with SMTP id d184-20020a0df4c1000000b005e8f2e7c48amr1763943ywf.98.1704447548262; Fri, 05 Jan 2024 01:39:08 -0800 (PST) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240105082339.1468817-1-claudiu.beznea.uj@bp.renesas.com> <20240105082339.1468817-4-claudiu.beznea.uj@bp.renesas.com> In-Reply-To: <20240105082339.1468817-4-claudiu.beznea.uj@bp.renesas.com> From: Geert Uytterhoeven Date: Fri, 5 Jan 2024 10:38:56 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH net-next v3 03/19] net: ravb: Make reset controller support mandatory To: Claudiu Cc: s.shtylyov@omp.ru, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, richardcochran@gmail.com, p.zabel@pengutronix.de, yoshihiro.shimoda.uh@renesas.com, wsa+renesas@sang-engineering.com, netdev@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, geert+renesas@glider.be, Claudiu Beznea Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi Claudiu, On Fri, Jan 5, 2024 at 9:24=E2=80=AFAM Claudiu w= rote: > From: Claudiu Beznea > > On the RZ/G3S SoC the reset controller is mandatory for the IP to work. > The device tree binding documentation for the ravb driver specifies that > the resets are mandatory. Based on this, make the resets mandatory also i= n > driver for all ravb devices. > > Reviewed-by: Sergey Shtylyov > Reviewed-by: Geert Uytterhoeven > Signed-off-by: Claudiu Beznea > --- a/drivers/net/ethernet/renesas/ravb_main.c > +++ b/drivers/net/ethernet/renesas/ravb_main.c > @@ -2645,7 +2645,7 @@ static int ravb_probe(struct platform_device *pdev) > return -EINVAL; > } > > - rstc =3D devm_reset_control_get_optional_exclusive(&pdev->dev, NU= LL); > + rstc =3D devm_reset_control_get_exclusive(&pdev->dev, NULL); > if (IS_ERR(rstc)) > return dev_err_probe(&pdev->dev, PTR_ERR(rstc), > "failed to get cpg reset\n"); Upon second look, you also have to make config RAVB select RESET_CONTROLLER. Currently, you can build an R-Car Gen[234] kernel with RESET_CONTROLLER disabled, causing devm_reset_control_get_exclusive() to fail unconditionally. Gr{oetje,eeting}s, Geert --=20 Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k= .org In personal conversations with technical people, I call myself a hacker. Bu= t when I'm talking to journalists I just say "programmer" or something like t= hat. -- Linus Torvalds