From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guru Das Srinagesh Subject: Re: [RESEND v5 2/2] pwm: core: Convert period and duty cycle to u64 Date: Thu, 13 Feb 2020 11:39:26 -0800 Message-ID: <20200213193926.GA20183@codeaurora.org> References: <20200213101802.owpluhixtpor3qi3@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from alexa-out-sd-02.qualcomm.com ([199.106.114.39]:13065 "EHLO alexa-out-sd-02.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726282AbgBMTj0 (ORCPT ); Thu, 13 Feb 2020 14:39:26 -0500 Content-Disposition: inline In-Reply-To: <20200213101802.owpluhixtpor3qi3@pengutronix.de> Sender: linux-pwm-owner@vger.kernel.org List-Id: linux-pwm@vger.kernel.org To: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= Cc: linux-pwm@vger.kernel.org, Thierry Reding , Subbaraman Narayanamurthy , linux-kernel@vger.kernel.org On Thu, Feb 13, 2020 at 11:18:02AM +0100, Uwe Kleine-König wrote: > Hello, > > On Wed, Feb 12, 2020 at 10:54:08AM -0800, Guru Das Srinagesh wrote: > > @@ -305,8 +305,8 @@ struct pwm_chip { > > * @duty_cycle: duty cycle of the PWM signal (in nanoseconds) > > */ > > struct pwm_capture { > > - unsigned int period; > > - unsigned int duty_cycle; > > + u64 period; > > + u64 duty_cycle; > > }; > > Is this last hunk a separate change? > > Otherwise looks fine. Hi Uwe, No, this is very much a part of the change and not a separate one. If things look good to you, could you please add an Acked-by or Reviewed-by tag to the patches so that they get accepted by Thierry? Thank you. Guru Das.