From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f174.google.com (mail-pf1-f174.google.com [209.85.210.174]) (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 D34151B2518 for ; Wed, 21 Aug 2024 17:12:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724260353; cv=none; b=rnkav950tegiZuYER9dUHisWGLf1tzJQQ0xFGjkdE9iDhLf3HdnHSBnCkzllQRn4J9Ma0x/thwf1HoYfakNXVlx6QpNLp8+V15L3d16NvNhc4ktjlUj5rhfX37WPUJpM849doJAl29+kTeRfxyW3bKyzRQj3t4AnNjF4PBWpWd4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724260353; c=relaxed/simple; bh=8AMj1gQbdOy6mzRKOvisGTOPkhNd65sI4xgr/VwrPjo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=aQ0M43RkL0cBhnMw8PL9KHP7JxbvMQhLluV7meBhK/6o9sSxT5uLi0evYKTUCFQiUu8fGGo8Q2mQn+lPjGOATtM4UcWbOVv4nTLrGLeQVkiGyu+EWLmAbaC8eZz3s5+ABAEDS2RMrzDoI+pYQIrvu4+ct+nw74n4jHI47oR4348= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=upk5xP46; arc=none smtp.client-ip=209.85.210.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="upk5xP46" Received: by mail-pf1-f174.google.com with SMTP id d2e1a72fcca58-7141285db14so1516003b3a.1 for ; Wed, 21 Aug 2024 10:12:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1724260351; x=1724865151; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=2PGuIX73MOWMnEgrAGHe44EShZU2W6VwRhKHeC2K7JU=; b=upk5xP46OfEFNNtH3SH5yhTKqD57K+7FyHr+IV0pyc6BZcvM+A17QmRF5ESdc8eZOy urpf6pmCGrdNB3DtXMbKpl5nyIbmpCEZ5eeqxV60uXWr8SmH8Bkn5/SdbrwHG8v6ZW5U rP8Rm2Z/arH6YmE6zk0+iKCq26QPe5flVcbPh1dmOB5mjK/VxnhSL88EYbdRCna+DMr2 G2vdCvMj9154bHR21mnFQAnVgSmSs4HawEpsbR8/4HUy8UOt8Yfks4nAhtAmgFhfB8fQ PYNCCBemPFSc5jDp9mxYSIz0NoQ20p35Hw8CqecRUZG4rJc9tiwy+UfE8YFNrbs7V0Il H7kA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1724260351; x=1724865151; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=2PGuIX73MOWMnEgrAGHe44EShZU2W6VwRhKHeC2K7JU=; b=cJRUSTqgFbiqaDO1+1ytagV1b+A4XlpEm46h3eAvIfUIx1wjRAXagmf8XfpuIP9eKU QsNLccDZg7T0DZYCdLmLDtwonYUvvsWBk8HZkJbWQY0ssqAyeyGiN3NGE1suBmhZRETO zJcXzapk0FWcVaYjXz/2tqdKRszs5L61j0rlzkKRK4SGRGnuOTtlwbO6J9/91tSFTydL gWx3RdWA7NbwwyyhearJNcEED2BUmenJBUNAs1YVkQiJXpTRph3SvY2DDrvtSl4Kjtup xbXafD/dwDzyKvlNMN7FeFOHWbcET0mBEoNkFm4p8ZCZEioxH33qFujDRJigShJE4jTw /ELw== X-Forwarded-Encrypted: i=1; AJvYcCXoCnV6ljiDIIxKG7jguyOB6h+GwcfbazaU5vE9St40mS/UAsFgE/oYK6fHufxIOwK4rKIbeG7WVuz7@vger.kernel.org X-Gm-Message-State: AOJu0YyqeId4wqFX6S32HBhEvYKXrpvgmd4d5NZ6n+/nW45KYl14O8Dh v1SsocpmuCg8tEwbQkLR1gmhPO0iMsQwG6ulbl0IDB4gmlFq7SgoCLPHUg9Rpl0= X-Google-Smtp-Source: AGHT+IFahu5nhyR9APNX81eGWYDvWGW2BXzcpC6njxXplKlq16tD6UzEgdsOMomgPVvunvzQIx/1lw== X-Received: by 2002:a05:6a20:c916:b0:1c3:b61c:57cb with SMTP id adf61e73a8af0-1cad818adccmr3000160637.53.1724260350919; Wed, 21 Aug 2024 10:12:30 -0700 (PDT) Received: from p14s ([2604:3d09:148c:c800:167e:272c:2858:d319]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-7142e28d8f9sm528460b3a.70.2024.08.21.10.12.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 21 Aug 2024 10:12:30 -0700 (PDT) Date: Wed, 21 Aug 2024 11:12:27 -0600 From: Mathieu Poirier To: Andrew Davis Cc: Krzysztof Kozlowski , Vignesh Raghavendra , Bjorn Andersson , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Nishanth Menon , Tero Kristo , Philipp Zabel , Hari Nagalla , linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v11 3/9] remoteproc: k3-m4: Add a remoteproc driver for M4F subsystem Message-ID: References: <20240802152109.137243-1-afd@ti.com> <20240802152109.137243-4-afd@ti.com> <98d65c2e-f5a2-4894-b76d-6fa0fb8b6daf@ti.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <98d65c2e-f5a2-4894-b76d-6fa0fb8b6daf@ti.com> On Mon, Aug 19, 2024 at 10:54:11AM -0500, Andrew Davis wrote: > On 8/19/24 10:39 AM, Krzysztof Kozlowski wrote: > > On 19/08/2024 17:32, Mathieu Poirier wrote: > > > > > > > > Please remove. > > > > > Forget this comment since it would cause an error in __rproc_detach(). > > > > > > > > > > > Other than the above I'm good with this driver. That said I can't move forward > > > > > > without a nod from the DT crew. I also noticed a fair amount of code > > > > > > duplication with the k3_r5 and k3_dsp drivers. Dealing with that should not be > > > > > > part of the current work but will need to be done before another k3 driver can > > > > > > be merged. > > > > > > > > > > > > > > > The above still apply though. > > > > > > > > Me or Nishanth will pick up the SoC DT patches via TI SoC tree, once the > > > > driver patches are merged. Feel free to ignore those but queue > > > > dt-bindings (already has DT maintainers ack) and driver patches via > > > > rproc tree. > > > > > > > > > > Can you provide a link where the DT maintainers have acknowledged the bindings? > > > > The reviewed-by tag serves as acknowledgment as well and the binding > > patch has it. Conor gave it on some earlier version of the patchset. I > > did not check if there were any significant changes in the meantime, though. > > > > Was reviewed in v8: > https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20240424190612.17349-2-afd@ti.com/#3302840 > I didn't notice Conor had joined the DT crew and as such was expecting something from either Rob or Krzysztof. I am applying this set. Thanks, Mathieu > If there was any significant changes since I would have dropped the tag. > > Andrew > > > > > Best regards, > > Krzysztof > >