Message ID | 1bc5a16a22c487c478a204ebb7b80a22d2ad9cd0.1585340156.git.geoff@infradead.org (mailing list archive) |
---|---|
State | Accepted |
Commit | 7ee417497a29028502cf952f419ab2635f563d51 |
Headers | show |
Series | [1/9] powerpc/ps3: Remove duplicate error messages | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | Successfully applied on branch powerpc/merge (c6624071c338732402e8c726df6a4074473eaa0e) |
snowpatch_ozlabs/build-ppc64le | success | Build succeeded |
snowpatch_ozlabs/build-ppc64be | success | Build succeeded |
snowpatch_ozlabs/build-ppc64e | success | Build succeeded |
snowpatch_ozlabs/build-pmac32 | success | Build succeeded |
snowpatch_ozlabs/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 11 lines checked |
snowpatch_ozlabs/needsstable | success | Patch has no Fixes tags |
> Remove duplicate memory allocation failure error messages.
A single message can be omitted here.
https://lkml.org/lkml/2017/10/17/870
https://lore.kernel.org/patchwork/patch/842101/
https://lore.kernel.org/linuxppc-dev/e16c8b7d-de3a-6c96-9af4-dd0551cca805@users.sourceforge.net/
Will this detail be reflected in the final commit message?
Regards,
Markus
On Fri, 2020-03-27 at 20:26:23 UTC, Geoff Levand wrote: > From: Markus Elfring <elfring@users.sourceforge.net> > > Remove duplicate memory allocation failure error messages. > > Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> > Signed-off-by: Geoff Levand <geoff@infradead.org> Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/7ee417497a29028502cf952f419ab2635f563d51 cheers
diff --git a/arch/powerpc/platforms/ps3/os-area.c b/arch/powerpc/platforms/ps3/os-area.c index cbddd63caf2d..e8530371aed6 100644 --- a/arch/powerpc/platforms/ps3/os-area.c +++ b/arch/powerpc/platforms/ps3/os-area.c @@ -613,10 +613,8 @@ static int update_flash_db(void) /* Read in header and db from flash. */ header = kmalloc(buf_len, GFP_KERNEL); - if (!header) { - pr_debug("%s: kmalloc failed\n", __func__); + if (!header) return -ENOMEM; - } count = os_area_flash_read(header, buf_len, 0); if (count < 0) {