Re: [PATCH 1/2] iio: cros_ec: do an early exit if not physical_device case

2021-03-07 Thread Alexandru Ardelean
On Sun, Mar 7, 2021 at 1:59 PM Jonathan Cameron wrote: > > On Tue, 2 Mar 2021 11:46:06 +0100 > Enric Balletbo i Serra wrote: > > > Hi all, > > > > On 21/2/21 17:29, Jonathan Cameron wrote: > > > On Mon, 23 Nov 2020 16:40:16 +0200 > > > Alexandru Ardelean wrote: > > > > > >> This whole

Re: [PATCH 1/2] iio: cros_ec: do an early exit if not physical_device case

2021-03-07 Thread Jonathan Cameron
On Tue, 2 Mar 2021 11:46:06 +0100 Enric Balletbo i Serra wrote: > Hi all, > > On 21/2/21 17:29, Jonathan Cameron wrote: > > On Mon, 23 Nov 2020 16:40:16 +0200 > > Alexandru Ardelean wrote: > > > >> This whole code-block was put under one big if() condition/block. > >> This change does an

Re: [PATCH 1/2] iio: cros_ec: do an early exit if not physical_device case

2021-03-02 Thread Enric Balletbo i Serra
Hi all, On 21/2/21 17:29, Jonathan Cameron wrote: > On Mon, 23 Nov 2020 16:40:16 +0200 > Alexandru Ardelean wrote: > >> This whole code-block was put under one big if() condition/block. >> This change does an early return if the 'physical_device' boolean is false, >> thus unindenting the block

Re: [PATCH 1/2] iio: cros_ec: do an early exit if not physical_device case

2021-02-21 Thread Jonathan Cameron
On Mon, 23 Nov 2020 16:40:16 +0200 Alexandru Ardelean wrote: > This whole code-block was put under one big if() condition/block. > This change does an early return if the 'physical_device' boolean is false, > thus unindenting the block by one level. > > No other functional change has been done.

[PATCH 1/2] iio: cros_ec: do an early exit if not physical_device case

2020-11-23 Thread Alexandru Ardelean
This whole code-block was put under one big if() condition/block. This change does an early return if the 'physical_device' boolean is false, thus unindenting the block by one level. No other functional change has been done. Signed-off-by: Alexandru Ardelean ---