Re: [PATCH 3/4] firmware: fold successful fw read early

2015-08-10 Thread Luis R. Rodriguez
On Sun, Aug 9, 2015 at 6:29 AM, Ming Lei wrote: > On Tue, Aug 4, 2015 at 6:00 PM, Luis R. Rodriguez > wrote: >> From: David Howells >> >> We'll be folding in some more checks on fw_read_file_contents(), >> this will make the success case easier to follow. >> >> Signed-off-by: David Howells >>

Re: [PATCH 3/4] firmware: fold successful fw read early

2015-08-10 Thread Luis R. Rodriguez
On Sun, Aug 9, 2015 at 6:29 AM, Ming Lei ming@canonical.com wrote: On Tue, Aug 4, 2015 at 6:00 PM, Luis R. Rodriguez mcg...@do-not-panic.com wrote: From: David Howells dhowe...@redhat.com We'll be folding in some more checks on fw_read_file_contents(), this will make the success case

Re: [PATCH 3/4] firmware: fold successful fw read early

2015-08-09 Thread Ming Lei
On Tue, Aug 4, 2015 at 6:00 PM, Luis R. Rodriguez wrote: > From: David Howells > > We'll be folding in some more checks on fw_read_file_contents(), > this will make the success case easier to follow. > > Signed-off-by: David Howells > Signed-off-by: Luis R. Rodriguez > --- >

Re: [PATCH 3/4] firmware: fold successful fw read early

2015-08-09 Thread Ming Lei
On Tue, Aug 4, 2015 at 6:00 PM, Luis R. Rodriguez mcg...@do-not-panic.com wrote: From: David Howells dhowe...@redhat.com We'll be folding in some more checks on fw_read_file_contents(), this will make the success case easier to follow. Signed-off-by: David Howells dhowe...@redhat.com

[PATCH 3/4] firmware: fold successful fw read early

2015-08-04 Thread Luis R. Rodriguez
From: David Howells We'll be folding in some more checks on fw_read_file_contents(), this will make the success case easier to follow. Signed-off-by: David Howells Signed-off-by: Luis R. Rodriguez --- drivers/base/firmware_class.c | 16 +++- 1 file changed, 7 insertions(+), 9

[PATCH 3/4] firmware: fold successful fw read early

2015-08-04 Thread Luis R. Rodriguez
From: David Howells dhowe...@redhat.com We'll be folding in some more checks on fw_read_file_contents(), this will make the success case easier to follow. Signed-off-by: David Howells dhowe...@redhat.com Signed-off-by: Luis R. Rodriguez mcg...@suse.com --- drivers/base/firmware_class.c | 16