Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1503692 > unrolled thread

Re: [PATCH 6/7] iio: light: tsl2583: add locking to sysfs *_store() functions

Started byDan Carpenter <dan.carpenter@oracle.com>
First post2016-10-19 16:30 +0200
Last post2016-10-24 11:40 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 6/7] iio: light: tsl2583: add locking to sysfs *_store()  functions Dan Carpenter <dan.carpenter@oracle.com> - 2016-10-19 16:30 +0200
    Re: [PATCH 6/7] iio: light: tsl2583: add locking to sysfs *_store()  functions Jonathan Cameron <jic23@kernel.org> - 2016-10-22 19:40 +0200
      Re: [PATCH 6/7] iio: light: tsl2583: add locking to sysfs *_store()  functions Dan Carpenter <dan.carpenter@oracle.com> - 2016-10-24 11:40 +0200

#1503692 — Re: [PATCH 6/7] iio: light: tsl2583: add locking to sysfs *_store() functions

FromDan Carpenter <dan.carpenter@oracle.com>
Date2016-10-19 16:30 +0200
SubjectRe: [PATCH 6/7] iio: light: tsl2583: add locking to sysfs *_store() functions
Message-ID<su02w-2rf-137@gated-at.bofh.it>
I appologize for laughing, but I am still secretly amused in my heart.

regards,
dan carpenter

[toc] | [next] | [standalone]


#1506568

FromJonathan Cameron <jic23@kernel.org>
Date2016-10-22 19:40 +0200
Message-ID<sv8qZ-6pE-7@gated-at.bofh.it>
In reply to#1503692
On 19/10/16 12:37, Dan Carpenter wrote:
> I appologize for laughing, but I am still secretly amused in my heart.
> 
> regards,
> dan carpenter
> 
Fewer beers or less caffeine for Dan!

Key take away here is keep things simple.  The gotos in my mind
would actually have made sense, but I wouldn't do the refactoring until
you need it. i.e. when you introduce the locking in this patch.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

[toc] | [prev] | [next] | [standalone]


#1507036

FromDan Carpenter <dan.carpenter@oracle.com>
Date2016-10-24 11:40 +0200
Message-ID<svJTz-5GY-15@gated-at.bofh.it>
In reply to#1506568
On Sat, Oct 22, 2016 at 06:29:55PM +0100, Jonathan Cameron wrote:
> On 19/10/16 12:37, Dan Carpenter wrote:
> > I appologize for laughing, but I am still secretly amused in my heart.
> > 
> > regards,
> > dan carpenter
> > 
> Fewer beers or less caffeine for Dan!

;)

> 
> Key take away here is keep things simple.  The gotos in my mind
> would actually have made sense, but I wouldn't do the refactoring until
> you need it. i.e. when you introduce the locking in this patch.

Yeah, I totally agree do it when you need it.  I like gotos if they're
needed.

regards,
dan carpenter

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web