From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fgw23-7.mail.saunalahti.fi (fgw23-7.mail.saunalahti.fi [62.142.5.84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0AC1618858A for ; Sat, 28 Dec 2024 23:41:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=62.142.5.84 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1735429266; cv=none; b=VlQBtZfn645K01Q6DMvp+pogKSqqN5BugdmWE7p2lS9yPPaGz/ZCH0LrsRlat4uhCTRsCAQDzmH/7QbbQBCGeXEwr+0hnYRnwL7YCpkjdmr04hx1UZQCp9W19wF5bu7LVUDF8Aywf//5GKpPjTeC0yH9rqHFBMqlpTdYWxmQbc8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1735429266; c=relaxed/simple; bh=GxT6OU5DMMOzXUdLwmBKpReesSqumSB7KsrSv8mTWmE=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=pQ3KNd2eH4huVF1vuFXBf9MsnsSNr38kmWhgYdD0NjSKqgP+Tg+0/gTtxRdOLhJtQx+3k0Zln7b/c1DbUXquaXl4NJtbJXpZYX8nFwAu6hvFXwuyPQGQzHy5b0ViIgYkXxXd+3uEaCRJKfl5ew8WNj6Qwp9c+ykCsosbL7dZRBk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com; spf=fail smtp.mailfrom=gmail.com; arc=none smtp.client-ip=62.142.5.84 Authentication-Results: smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=gmail.com Received: from localhost (88-113-25-143.elisa-laajakaista.fi [88.113.25.143]) by fgw21.mail.saunalahti.fi (Halon) with ESMTP id 2e11aa48-c575-11ef-8899-005056bdd08f; Sun, 29 Dec 2024 01:40:54 +0200 (EET) From: Andy Shevchenko Date: Sun, 29 Dec 2024 01:40:54 +0200 To: Andi Shyti Cc: Prabhakar , Chris Brandt , Geert Uytterhoeven , Wolfram Sang , Philipp Zabel , linux-renesas-soc@vger.kernel.org, linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, Biju Das , Fabrizio Castro , Lad Prabhakar Subject: Re: [PATCH v3 0/8] i2c: riic: driver cleanup and improvements Message-ID: References: <20241227115154.56154-1-prabhakar.mahadev-lad.rj@bp.renesas.com> 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-Disposition: inline In-Reply-To: Fri, Dec 27, 2024 at 11:10:22PM +0100, Andi Shyti kirjoitti: ... > > i2c: riic: Use dev_err_probe in probe and riic_init_hw functions > > i2c: riic: Use local `dev` pointer in `dev_err_probe()` > > i2c: riic: Use BIT macro consistently > > i2c: riic: Use GENMASK() macro for bitmask definitions > > i2c: riic: Make use of devres helper to request deasserted reset line > > i2c: riic: Mark riic_irqs array as const > > i2c: riic: Use predefined macro and simplify clock tick calculation > > i2c: riic: Add `riic_bus_barrier()` to check bus availability > > merged to i2c/i2c-host. There are some comments, up to you how to proceed, they seem not to be any critical. -- With Best Regards, Andy Shevchenko