All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Liang He" <windhl@126.com>
To: "kernel test robot" <lkp@intel.com>
Cc: palmer@dabbelt.com, paul.walmsley@sifive.com,
	aou@eecs.berkeley.edu, kbuild-all@lists.01.org,
	linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re:Re: [PATCH] drivers: soc: sifive: Add missing of_node_put() in sifive_l2_cache.c
Date: Thu, 16 Jun 2022 12:54:50 +0800 (CST)	[thread overview]
Message-ID: <3e6469ba.38b2.1816add33c5.Coremail.windhl@126.com> (raw)
In-Reply-To: <202206160559.fVKJx0ST-lkp@intel.com>





At 2022-06-16 05:58:05, "kernel test robot" <lkp@intel.com> wrote:
>Hi Liang,
>
>Thank you for the patch! Yet something to improve:
>
>[auto build test ERROR on linus/master]
>[also build test ERROR on v5.19-rc2 next-20220615]
>[If your patch is applied to the wrong git tree, kindly drop us a note.
>And when submitting patch, we suggest to use '--base' as documented in
>https://git-scm.com/docs/git-format-patch]
>

When I use --base, there are too many prerequests-patch-id caused by my local commits. 
I don't know if it will cause other troubles. So I resent a new patch still without this '--base'.
Is it Ok?

Sorry, I forget to say in new patch that  is based on v5.19-rc2 mainline git repo.

>url:    https://github.com/intel-lab-lkp/linux/commits/Liang-He/drivers-soc-sifive-Add-missing-of_node_put-in-sifive_l2_cache-c/20220615-202528
>base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 018ab4fabddd94f1c96f3b59e180691b9e88d5d8
>config: riscv-allyesconfig (https://download.01.org/0day-ci/archive/20220616/202206160559.fVKJx0ST-lkp@intel.com/config)
>compiler: riscv64-linux-gcc (GCC) 11.3.0
>reproduce (this is a W=1 build):
>        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>        chmod +x ~/bin/make.cross
>        # https://github.com/intel-lab-lkp/linux/commit/49c4086768b5aff410a4a19ca740f8ae8e211844
>        git remote add linux-review https://github.com/intel-lab-lkp/linux
>        git fetch --no-tags linux-review Liang-He/drivers-soc-sifive-Add-missing-of_node_put-in-sifive_l2_cache-c/20220615-202528
>        git checkout 49c4086768b5aff410a4a19ca740f8ae8e211844
>        # save the config file
>        mkdir build_dir && cp config build_dir/.config
>        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/soc/
>
>If you fix the issue, kindly add following tag where applicable
>Reported-by: kernel test robot <lkp@intel.com>
>

Thanks for this report, now I make a new patch and add "Reported-by: kernel test robot <lkp@intel.com>"


>All errors (new ones prefixed by >>):
>
>   drivers/soc/sifive/sifive_l2_cache.c: In function 'sifive_l2_init':
>>> drivers/soc/sifive/sifive_l2_cache.c:224:17: error: expected ';' before 'goto'
>     224 |                 goto out_put;
>         |                 ^~~~
>
>

Thanks for all your effort to improve the patch.

WARNING: multiple messages have this Message-ID (diff)
From: "Liang He" <windhl@126.com>
To: "kernel test robot" <lkp@intel.com>
Cc: palmer@dabbelt.com, paul.walmsley@sifive.com,
	aou@eecs.berkeley.edu,  kbuild-all@lists.01.org,
	linux-riscv@lists.infradead.org,  linux-kernel@vger.kernel.org
Subject: Re:Re: [PATCH] drivers: soc: sifive: Add missing of_node_put() in sifive_l2_cache.c
Date: Thu, 16 Jun 2022 12:54:50 +0800 (CST)	[thread overview]
Message-ID: <3e6469ba.38b2.1816add33c5.Coremail.windhl@126.com> (raw)
In-Reply-To: <202206160559.fVKJx0ST-lkp@intel.com>





At 2022-06-16 05:58:05, "kernel test robot" <lkp@intel.com> wrote:
>Hi Liang,
>
>Thank you for the patch! Yet something to improve:
>
>[auto build test ERROR on linus/master]
>[also build test ERROR on v5.19-rc2 next-20220615]
>[If your patch is applied to the wrong git tree, kindly drop us a note.
>And when submitting patch, we suggest to use '--base' as documented in
>https://git-scm.com/docs/git-format-patch]
>

When I use --base, there are too many prerequests-patch-id caused by my local commits. 
I don't know if it will cause other troubles. So I resent a new patch still without this '--base'.
Is it Ok?

Sorry, I forget to say in new patch that  is based on v5.19-rc2 mainline git repo.

>url:    https://github.com/intel-lab-lkp/linux/commits/Liang-He/drivers-soc-sifive-Add-missing-of_node_put-in-sifive_l2_cache-c/20220615-202528
>base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 018ab4fabddd94f1c96f3b59e180691b9e88d5d8
>config: riscv-allyesconfig (https://download.01.org/0day-ci/archive/20220616/202206160559.fVKJx0ST-lkp@intel.com/config)
>compiler: riscv64-linux-gcc (GCC) 11.3.0
>reproduce (this is a W=1 build):
>        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>        chmod +x ~/bin/make.cross
>        # https://github.com/intel-lab-lkp/linux/commit/49c4086768b5aff410a4a19ca740f8ae8e211844
>        git remote add linux-review https://github.com/intel-lab-lkp/linux
>        git fetch --no-tags linux-review Liang-He/drivers-soc-sifive-Add-missing-of_node_put-in-sifive_l2_cache-c/20220615-202528
>        git checkout 49c4086768b5aff410a4a19ca740f8ae8e211844
>        # save the config file
>        mkdir build_dir && cp config build_dir/.config
>        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/soc/
>
>If you fix the issue, kindly add following tag where applicable
>Reported-by: kernel test robot <lkp@intel.com>
>

Thanks for this report, now I make a new patch and add "Reported-by: kernel test robot <lkp@intel.com>"


>All errors (new ones prefixed by >>):
>
>   drivers/soc/sifive/sifive_l2_cache.c: In function 'sifive_l2_init':
>>> drivers/soc/sifive/sifive_l2_cache.c:224:17: error: expected ';' before 'goto'
>     224 |                 goto out_put;
>         |                 ^~~~
>
>

Thanks for all your effort to improve the patch.
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

WARNING: multiple messages have this Message-ID (diff)
From: Liang He <windhl@126.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH] drivers: soc: sifive: Add missing of_node_put() in sifive_l2_cache.c
Date: Thu, 16 Jun 2022 12:54:50 +0800	[thread overview]
Message-ID: <3e6469ba.38b2.1816add33c5.Coremail.windhl@126.com> (raw)
In-Reply-To: <202206160559.fVKJx0ST-lkp@intel.com>

[-- Attachment #1: Type: text/plain, Size: 2497 bytes --]





At 2022-06-16 05:58:05, "kernel test robot" <lkp@intel.com> wrote:
>Hi Liang,
>
>Thank you for the patch! Yet something to improve:
>
>[auto build test ERROR on linus/master]
>[also build test ERROR on v5.19-rc2 next-20220615]
>[If your patch is applied to the wrong git tree, kindly drop us a note.
>And when submitting patch, we suggest to use '--base' as documented in
>https://git-scm.com/docs/git-format-patch]
>

When I use --base, there are too many prerequests-patch-id caused by my local commits. 
I don't know if it will cause other troubles. So I resent a new patch still without this '--base'.
Is it Ok?

Sorry, I forget to say in new patch that  is based on v5.19-rc2 mainline git repo.

>url:    https://github.com/intel-lab-lkp/linux/commits/Liang-He/drivers-soc-sifive-Add-missing-of_node_put-in-sifive_l2_cache-c/20220615-202528
>base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 018ab4fabddd94f1c96f3b59e180691b9e88d5d8
>config: riscv-allyesconfig (https://download.01.org/0day-ci/archive/20220616/202206160559.fVKJx0ST-lkp(a)intel.com/config)
>compiler: riscv64-linux-gcc (GCC) 11.3.0
>reproduce (this is a W=1 build):
>        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>        chmod +x ~/bin/make.cross
>        # https://github.com/intel-lab-lkp/linux/commit/49c4086768b5aff410a4a19ca740f8ae8e211844
>        git remote add linux-review https://github.com/intel-lab-lkp/linux
>        git fetch --no-tags linux-review Liang-He/drivers-soc-sifive-Add-missing-of_node_put-in-sifive_l2_cache-c/20220615-202528
>        git checkout 49c4086768b5aff410a4a19ca740f8ae8e211844
>        # save the config file
>        mkdir build_dir && cp config build_dir/.config
>        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/soc/
>
>If you fix the issue, kindly add following tag where applicable
>Reported-by: kernel test robot <lkp@intel.com>
>

Thanks for this report, now I make a new patch and add "Reported-by: kernel test robot <lkp@intel.com>"


>All errors (new ones prefixed by >>):
>
>   drivers/soc/sifive/sifive_l2_cache.c: In function 'sifive_l2_init':
>>> drivers/soc/sifive/sifive_l2_cache.c:224:17: error: expected ';' before 'goto'
>     224 |                 goto out_put;
>         |                 ^~~~
>
>

Thanks for all your effort to improve the patch.

  reply	other threads:[~2022-06-16  4:55 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-15 12:23 [PATCH] drivers: soc: sifive: Add missing of_node_put() in sifive_l2_cache.c Liang He
2022-06-15 12:23 ` Liang He
2022-06-15 21:58 ` kernel test robot
2022-06-15 21:58   ` kernel test robot
2022-06-16  4:54   ` Liang He [this message]
2022-06-16  4:54     ` Liang He
2022-06-16  4:54     ` Liang He
2022-06-16  5:12 ` Christophe JAILLET
2022-06-16  5:12   ` Christophe JAILLET
2022-06-16  5:33   ` Liang He
2022-06-16  5:33     ` Liang He
2022-06-16 18:45     ` Christophe JAILLET
2022-06-16 18:45       ` Christophe JAILLET

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3e6469ba.38b2.1816add33c5.Coremail.windhl@126.com \
    --to=windhl@126.com \
    --cc=aou@eecs.berkeley.edu \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=lkp@intel.com \
    --cc=palmer@dabbelt.com \
    --cc=paul.walmsley@sifive.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.