Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Date:   Fri, 22 Sep 2023 18:23:46 +0900
From:   Jeongtae Park <jtp.park@samsung.com>
To:     Dave Jiang <dave.jiang@intel.com>
Cc:     Dan Williams <dan.j.williams@intel.com>,
        Alison Schofield <alison.schofield@intel.com>,
        Vishal Verma <vishal.l.verma@intel.com>,
        Ben Widawsky <bwidawsk@kernel.org>,
        Jonathan Cameron <jonathan.cameron@huawei.com>,
        Davidlohr Bueso <dave@stgolabs.net>,
        Fan Ni <fan.ni@samsung.com>, linux-cxl@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Kyungsan Kim <ks0204.kim@samsung.com>,
        Wonjae Lee <wj28.lee@samsung.com>,
        Hojin Nam <hj96.nam@samsung.com>,
        Junhyeok Im <junhyeok.im@samsung.com>,
        Jehoon Park <jehoon.park@samsung.com>,
        Jeongtae Park <jeongtae.park@gmail.com>
Subject: Re: [PATCH 6/7] cxl/memdev: Fix whitespace error/warnings
Message-ID: <ZQ1dIr//9aYNktzI@jtpark-7920>
MIME-Version: 1.0
In-Reply-To: <e14ac1d0-ef8e-74ab-0616-f3203580b371@intel.com>
Content-Type: multipart/mixed;
        boundary="----UO.gBI9h3KzXx7fAUzAwhoREi3kPxaIpZHHzoQQrAoo4vDHp=_2351a_"
CMS-TYPE: 102P
DLP-Filter: Pass
References: <20230921025110.3717583-1-jtp.park@samsung.com>
        <CGME20230921024832epcas2p24325cdfdb6fbbcf489ea3d442258e86b@epcas2p2.samsung.com>
        <20230921025110.3717583-7-jtp.park@samsung.com>
        <e14ac1d0-ef8e-74ab-0616-f3203580b371@intel.com>
Precedence: bulk

------UO.gBI9h3KzXx7fAUzAwhoREi3kPxaIpZHHzoQQrAoo4vDHp=_2351a_
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline

On Thu, Sep 21, 2023 at 03:57:25PM -0700, Dave Jiang wrote:
> 
> 
> On 9/20/23 19:51, Jeongtae Park wrote:
> > ERROR: code indent should use tabs where possible
> > WARNING: please, no spaces at the start of a line
> > 
> > Signed-off-by: Jeongtae Park <jtp.park@samsung.com>
> 
> two different issues?
> 

Yes, they are. I will seperate and resend them.

> > ---
> >  drivers/cxl/core/memdev.c | 12 ++++++------
> >  1 file changed, 6 insertions(+), 6 deletions(-)
> > 
> > diff --git a/drivers/cxl/core/memdev.c b/drivers/cxl/core/memdev.c
> > index f99e7ec3cc40..ce67df163452 100644
> > --- a/drivers/cxl/core/memdev.c
> > +++ b/drivers/cxl/core/memdev.c
> > @@ -935,11 +935,11 @@ static void cxl_fw_cancel(struct fw_upload *fwl)
> >  }
> >  
> >  static const struct fw_upload_ops cxl_memdev_fw_ops = {
> > -        .prepare = cxl_fw_prepare,
> > -        .write = cxl_fw_write,
> > -        .poll_complete = cxl_fw_poll_complete,
> > -        .cancel = cxl_fw_cancel,
> > -        .cleanup = cxl_fw_cleanup,
> > +	.prepare = cxl_fw_prepare,
> > +	.write = cxl_fw_write,
> > +	.poll_complete = cxl_fw_poll_complete,
> > +	.cancel = cxl_fw_cancel,
> > +	.cleanup = cxl_fw_cleanup,
> >  };
> >  
> >  static void devm_cxl_remove_fw_upload(void *fwl)
> > @@ -1010,7 +1010,7 @@ static int cxl_memdev_security_init(struct cxl_memdev *cxlmd)
> >  	}
> >  
> >  	return devm_add_action_or_reset(cxlds->dev, put_sanitize, mds);
> > - }
> > +}
> >  
> >  struct cxl_memdev *devm_cxl_add_memdev(struct cxl_dev_state *cxlds)
> >  {

------UO.gBI9h3KzXx7fAUzAwhoREi3kPxaIpZHHzoQQrAoo4vDHp=_2351a_
Content-Type: text/plain; charset="utf-8"


------UO.gBI9h3KzXx7fAUzAwhoREi3kPxaIpZHHzoQQrAoo4vDHp=_2351a_--