* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
@ 2020-07-17 7:06 ` Daniel Vetter
0 siblings, 0 replies; 14+ messages in thread
From: Daniel Vetter @ 2020-07-17 7:06 UTC (permalink / raw)
To: james qian wang (Arm Technology China)
Cc: nd, Greg Kroah-Hartman, Liviu Dudau, Linux Kernel Mailing List,
dri-devel, Qinglang Miao, Mihail Atanassov
On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
<james.qian.wang@arm.com> wrote:
>
> On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
> > From: Liu Shixin <liushixin2@huawei.com>
> >
> > Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
> >
> > Signed-off-by: Liu Shixin <liushixin2@huawei.com>
> > ---
> > drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
> > 1 file changed, 1 insertion(+), 12 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > index 0246b2e94..4a10e6b9e 100644
> > --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
> > return 0;
> > }
> >
> > -static int komeda_register_open(struct inode *inode, struct file *filp)
> > -{
> > - return single_open(filp, komeda_register_show, inode->i_private);
> > -}
> > -
> > -static const struct file_operations komeda_register_fops = {
> > - .owner = THIS_MODULE,
> > - .open = komeda_register_open,
> > - .read_iter = seq_read_iter,
> > - .llseek = seq_lseek,
> > - .release = single_release,
> > -};
> > +DEFINE_SHOW_ATTRIBUTE(komeda_register);
> >
>
> Hi Shixin & Qinglang
>
> Thanks for your patch.
>
> Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
>
> Since your patch is not for drm-misc-next, so seems better
> to leave it to you to merge it. :)
I do think it's for drm-misc-next, what other tree would it be for?
Some people put -next in their patch tag to differentiate from -fixes,
so maintainers know what to do with the patch. It's also not part of a
series, hence I think this is on you to apply it.
Cheers, Daniel
>
> Thanks
> James
>
> > #ifdef CONFIG_DEBUG_FS
> > static void komeda_debugfs_init(struct komeda_dev *mdev)
> > --
> > 2.17.1
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
2020-07-17 7:06 ` Daniel Vetter
@ 2020-07-17 7:59 ` james qian wang (Arm Technology China)
-1 siblings, 0 replies; 14+ messages in thread
From: james qian wang (Arm Technology China) @ 2020-07-17 7:59 UTC (permalink / raw)
To: Daniel Vetter
Cc: Qinglang Miao, Mihail Atanassov, Greg Kroah-Hartman, Liviu Dudau,
Linux Kernel Mailing List, dri-devel, nd
On Fri, Jul 17, 2020 at 09:06:57AM +0200, Daniel Vetter wrote:
> On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
> <james.qian.wang@arm.com> wrote:
> >
> > On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
> > > From: Liu Shixin <liushixin2@huawei.com>
> > >
> > > Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
> > >
> > > Signed-off-by: Liu Shixin <liushixin2@huawei.com>
> > > ---
> > > drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
> > > 1 file changed, 1 insertion(+), 12 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > index 0246b2e94..4a10e6b9e 100644
> > > --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
> > > return 0;
> > > }
> > >
> > > -static int komeda_register_open(struct inode *inode, struct file *filp)
> > > -{
> > > - return single_open(filp, komeda_register_show, inode->i_private);
> > > -}
> > > -
> > > -static const struct file_operations komeda_register_fops = {
> > > - .owner = THIS_MODULE,
> > > - .open = komeda_register_open,
> > > - .read_iter = seq_read_iter,
- .read = seq_read,
+ .read_iter = seq_read_iter,
> > > - .llseek = seq_lseek,
> > > - .release = single_release,
> > > -};
> > > +DEFINE_SHOW_ATTRIBUTE(komeda_register);
> > >
> >
> > Hi Shixin & Qinglang
> >
> > Thanks for your patch.
> >
> > Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
> >
> > Since your patch is not for drm-misc-next, so seems better
> > to leave it to you to merge it. :)
>
> I do think it's for drm-misc-next, what other tree would it be for?
> Some people put -next in their patch tag to differentiate from -fixes,
> so maintainers know what to do with the patch. It's also not part of a
> series, hence I think this is on you to apply it.
>
> Cheers, Daniel
Hi Daniel:
I tried to apply this patch to drm-misc-next, but failed, and found
this patch is actually based on linux-next, and the code base of
linux-next is a little different with our drm-misc-next.
and one of the difference is linux-next has a patch (call it patch-A):
seq_file: switch over direct seq_read method calls to seq_read_iter
https://lkml.org/lkml/2020/7/7/1267
which changed code like below:
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
index 1d767473ba8a06..0246b2e94d8cbd 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
@@ -49,7 +49,7 @@ static int komeda_register_open(struct inode *inode, struct file *filp)
static const struct file_operations komeda_register_fops = {
.owner = THIS_MODULE,
.open = komeda_register_open,
- .read = seq_read,
+ .read_iter = seq_read_iter,
.llseek = seq_lseek,
.release = single_release,
};
And these code will be deleted by this patch, if we merge this patch into
drm-misc-next firstly before the patch-A, that may import a conflict when
we merge our misc into upstreams.
if we want it to be merged into drm-misc, I think we'd better to wait the
upstream (the patch-A) has been synced back to drm-misc.
And what's your opinion ?
Thanks
James
> >
> > Thanks
> > James
> >
> > > #ifdef CONFIG_DEBUG_FS
> > > static void komeda_debugfs_init(struct komeda_dev *mdev)
> > > --
> > > 2.17.1
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
@ 2020-07-17 7:59 ` james qian wang (Arm Technology China)
0 siblings, 0 replies; 14+ messages in thread
From: james qian wang (Arm Technology China) @ 2020-07-17 7:59 UTC (permalink / raw)
To: Daniel Vetter
Cc: nd, Greg Kroah-Hartman, Liviu Dudau, Linux Kernel Mailing List,
dri-devel, Qinglang Miao, Mihail Atanassov
On Fri, Jul 17, 2020 at 09:06:57AM +0200, Daniel Vetter wrote:
> On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
> <james.qian.wang@arm.com> wrote:
> >
> > On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
> > > From: Liu Shixin <liushixin2@huawei.com>
> > >
> > > Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
> > >
> > > Signed-off-by: Liu Shixin <liushixin2@huawei.com>
> > > ---
> > > drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
> > > 1 file changed, 1 insertion(+), 12 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > index 0246b2e94..4a10e6b9e 100644
> > > --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
> > > return 0;
> > > }
> > >
> > > -static int komeda_register_open(struct inode *inode, struct file *filp)
> > > -{
> > > - return single_open(filp, komeda_register_show, inode->i_private);
> > > -}
> > > -
> > > -static const struct file_operations komeda_register_fops = {
> > > - .owner = THIS_MODULE,
> > > - .open = komeda_register_open,
> > > - .read_iter = seq_read_iter,
- .read = seq_read,
+ .read_iter = seq_read_iter,
> > > - .llseek = seq_lseek,
> > > - .release = single_release,
> > > -};
> > > +DEFINE_SHOW_ATTRIBUTE(komeda_register);
> > >
> >
> > Hi Shixin & Qinglang
> >
> > Thanks for your patch.
> >
> > Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
> >
> > Since your patch is not for drm-misc-next, so seems better
> > to leave it to you to merge it. :)
>
> I do think it's for drm-misc-next, what other tree would it be for?
> Some people put -next in their patch tag to differentiate from -fixes,
> so maintainers know what to do with the patch. It's also not part of a
> series, hence I think this is on you to apply it.
>
> Cheers, Daniel
Hi Daniel:
I tried to apply this patch to drm-misc-next, but failed, and found
this patch is actually based on linux-next, and the code base of
linux-next is a little different with our drm-misc-next.
and one of the difference is linux-next has a patch (call it patch-A):
seq_file: switch over direct seq_read method calls to seq_read_iter
https://lkml.org/lkml/2020/7/7/1267
which changed code like below:
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
index 1d767473ba8a06..0246b2e94d8cbd 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
@@ -49,7 +49,7 @@ static int komeda_register_open(struct inode *inode, struct file *filp)
static const struct file_operations komeda_register_fops = {
.owner = THIS_MODULE,
.open = komeda_register_open,
- .read = seq_read,
+ .read_iter = seq_read_iter,
.llseek = seq_lseek,
.release = single_release,
};
And these code will be deleted by this patch, if we merge this patch into
drm-misc-next firstly before the patch-A, that may import a conflict when
we merge our misc into upstreams.
if we want it to be merged into drm-misc, I think we'd better to wait the
upstream (the patch-A) has been synced back to drm-misc.
And what's your opinion ?
Thanks
James
> >
> > Thanks
> > James
> >
> > > #ifdef CONFIG_DEBUG_FS
> > > static void komeda_debugfs_init(struct komeda_dev *mdev)
> > > --
> > > 2.17.1
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
2020-07-17 7:06 ` Daniel Vetter
@ 2020-07-17 8:00 ` miaoqinglang
-1 siblings, 0 replies; 14+ messages in thread
From: miaoqinglang @ 2020-07-17 8:00 UTC (permalink / raw)
To: Daniel Vetter, james qian wang (Arm Technology China)
Cc: Mihail Atanassov, Greg Kroah-Hartman, Liviu Dudau,
Linux Kernel Mailing List, dri-devel, nd
在 2020/7/17 15:06, Daniel Vetter 写道:
> On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
> <james.qian.wang@arm.com> wrote:
>>
>> On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
>>> From: Liu Shixin <liushixin2@huawei.com>
>>>
>>> Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
>>>
>>> Signed-off-by: Liu Shixin <liushixin2@huawei.com>
>>> ---
>>> drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
>>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>> index 0246b2e94..4a10e6b9e 100644
>>> --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>> +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>> @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
>>> return 0;
>>> }
>>>
>>> -static int komeda_register_open(struct inode *inode, struct file *filp)
>>> -{
>>> - return single_open(filp, komeda_register_show, inode->i_private);
>>> -}
>>> -
>>> -static const struct file_operations komeda_register_fops = {
>>> - .owner = THIS_MODULE,
>>> - .open = komeda_register_open,
>>> - .read_iter = seq_read_iter,
>>> - .llseek = seq_lseek,
>>> - .release = single_release,
>>> -};
>>> +DEFINE_SHOW_ATTRIBUTE(komeda_register);
>>>
>>
>> Hi Shixin & Qinglang
>>
>> Thanks for your patch.
>>
>> Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
>>
>> Since your patch is not for drm-misc-next, so seems better
>> to leave it to you to merge it. :)
>
> I do think it's for drm-misc-next, what other tree would it be for?
> Some people put -next in their patch tag to differentiate from -fixes,
> so maintainers know what to do with the patch. It's also not part of a
> series, hence I think this is on you to apply it.
>
Hi James & Daniel,
Sorry I didn't make it clear in commit log, but it do based on linux-next.
I think the reason why James think it's not for drm-misc-next
is conflicts exists when this patch being applied. There's conflicts
because commit <4d4901c6d7> which switched over direct seq_read method
calls to seq_read_iter should applied before this clean-up patch(linkage
listed as below).
https://kernel.googlesource.com/pub/scm/linux/kernel/git/next/linux-next/+/4d4901c6d748efab8aab6e7d2405dadaed0bea50
I can send a new patch based on mainline if needed.
Thanks.
Qinglang
.
>
> Cheers, Daniel
>
>>
>> Thanks
>> James
>>
>>> #ifdef CONFIG_DEBUG_FS
>>> static void komeda_debugfs_init(struct komeda_dev *mdev)
>>> --
>>> 2.17.1
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
@ 2020-07-17 8:00 ` miaoqinglang
0 siblings, 0 replies; 14+ messages in thread
From: miaoqinglang @ 2020-07-17 8:00 UTC (permalink / raw)
To: Daniel Vetter, james qian wang (Arm Technology China)
Cc: Mihail Atanassov, Greg Kroah-Hartman, Liviu Dudau,
Linux Kernel Mailing List, dri-devel, nd
在 2020/7/17 15:06, Daniel Vetter 写道:
> On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
> <james.qian.wang@arm.com> wrote:
>>
>> On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
>>> From: Liu Shixin <liushixin2@huawei.com>
>>>
>>> Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
>>>
>>> Signed-off-by: Liu Shixin <liushixin2@huawei.com>
>>> ---
>>> drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
>>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>> index 0246b2e94..4a10e6b9e 100644
>>> --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>> +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>> @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
>>> return 0;
>>> }
>>>
>>> -static int komeda_register_open(struct inode *inode, struct file *filp)
>>> -{
>>> - return single_open(filp, komeda_register_show, inode->i_private);
>>> -}
>>> -
>>> -static const struct file_operations komeda_register_fops = {
>>> - .owner = THIS_MODULE,
>>> - .open = komeda_register_open,
>>> - .read_iter = seq_read_iter,
>>> - .llseek = seq_lseek,
>>> - .release = single_release,
>>> -};
>>> +DEFINE_SHOW_ATTRIBUTE(komeda_register);
>>>
>>
>> Hi Shixin & Qinglang
>>
>> Thanks for your patch.
>>
>> Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
>>
>> Since your patch is not for drm-misc-next, so seems better
>> to leave it to you to merge it. :)
>
> I do think it's for drm-misc-next, what other tree would it be for?
> Some people put -next in their patch tag to differentiate from -fixes,
> so maintainers know what to do with the patch. It's also not part of a
> series, hence I think this is on you to apply it.
>
Hi James & Daniel,
Sorry I didn't make it clear in commit log, but it do based on linux-next.
I think the reason why James think it's not for drm-misc-next
is conflicts exists when this patch being applied. There's conflicts
because commit <4d4901c6d7> which switched over direct seq_read method
calls to seq_read_iter should applied before this clean-up patch(linkage
listed as below).
https://kernel.googlesource.com/pub/scm/linux/kernel/git/next/linux-next/+/4d4901c6d748efab8aab6e7d2405dadaed0bea50
I can send a new patch based on mainline if needed.
Thanks.
Qinglang
.
>
> Cheers, Daniel
>
>>
>> Thanks
>> James
>>
>>> #ifdef CONFIG_DEBUG_FS
>>> static void komeda_debugfs_init(struct komeda_dev *mdev)
>>> --
>>> 2.17.1
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
2020-07-17 8:00 ` miaoqinglang
@ 2020-07-17 9:10 ` Daniel Vetter
-1 siblings, 0 replies; 14+ messages in thread
From: Daniel Vetter @ 2020-07-17 9:10 UTC (permalink / raw)
To: miaoqinglang
Cc: Daniel Vetter, james qian wang (Arm Technology China),
Mihail Atanassov, Greg Kroah-Hartman, Liviu Dudau,
Linux Kernel Mailing List, dri-devel, nd
On Fri, Jul 17, 2020 at 04:00:36PM +0800, miaoqinglang wrote:
>
>
> 在 2020/7/17 15:06, Daniel Vetter 写道:
> > On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
> > <james.qian.wang@arm.com> wrote:
> > >
> > > On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
> > > > From: Liu Shixin <liushixin2@huawei.com>
> > > >
> > > > Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
> > > >
> > > > Signed-off-by: Liu Shixin <liushixin2@huawei.com>
> > > > ---
> > > > drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
> > > > 1 file changed, 1 insertion(+), 12 deletions(-)
> > > >
> > > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > > index 0246b2e94..4a10e6b9e 100644
> > > > --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > > @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
> > > > return 0;
> > > > }
> > > >
> > > > -static int komeda_register_open(struct inode *inode, struct file *filp)
> > > > -{
> > > > - return single_open(filp, komeda_register_show, inode->i_private);
> > > > -}
> > > > -
> > > > -static const struct file_operations komeda_register_fops = {
> > > > - .owner = THIS_MODULE,
> > > > - .open = komeda_register_open,
> > > > - .read_iter = seq_read_iter,
> > > > - .llseek = seq_lseek,
> > > > - .release = single_release,
> > > > -};
> > > > +DEFINE_SHOW_ATTRIBUTE(komeda_register);
> > > >
> > >
> > > Hi Shixin & Qinglang
> > >
> > > Thanks for your patch.
> > >
> > > Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
> > >
> > > Since your patch is not for drm-misc-next, so seems better
> > > to leave it to you to merge it. :)
> >
> > I do think it's for drm-misc-next, what other tree would it be for?
> > Some people put -next in their patch tag to differentiate from -fixes,
> > so maintainers know what to do with the patch. It's also not part of a
> > series, hence I think this is on you to apply it.
> >
> Hi James & Daniel,
>
> Sorry I didn't make it clear in commit log, but it do based on linux-next.
>
> I think the reason why James think it's not for drm-misc-next
> is conflicts exists when this patch being applied. There's conflicts because
> commit <4d4901c6d7> which switched over direct seq_read method calls to
> seq_read_iter should applied before this clean-up patch(linkage listed as
> below).
>
> https://kernel.googlesource.com/pub/scm/linux/kernel/git/next/linux-next/+/4d4901c6d748efab8aab6e7d2405dadaed0bea50
>
> I can send a new patch based on mainline if needed.
Uh yes this is annoying. We're at feature cutoff so this will likely cause
bad conflicts no matter what if we merge it now, but the clean solution is
to rebase onto drm-misc-next, and then let maintainers sort out the mess
with conflicts. It's a pretty simple change in the above patch, so
shouldn't cause too many troubles.
-Daniel
>
> Thanks.
>
> Qinglang
>
> .
>
> >
> > Cheers, Daniel
> >
> > >
> > > Thanks
> > > James
> > >
> > > > #ifdef CONFIG_DEBUG_FS
> > > > static void komeda_debugfs_init(struct komeda_dev *mdev)
> > > > --
> > > > 2.17.1
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> >
> >
> >
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
@ 2020-07-17 9:10 ` Daniel Vetter
0 siblings, 0 replies; 14+ messages in thread
From: Daniel Vetter @ 2020-07-17 9:10 UTC (permalink / raw)
To: miaoqinglang
Cc: nd, Greg Kroah-Hartman, Liviu Dudau, Linux Kernel Mailing List,
dri-devel, james qian wang (Arm Technology China),
Mihail Atanassov
On Fri, Jul 17, 2020 at 04:00:36PM +0800, miaoqinglang wrote:
>
>
> 在 2020/7/17 15:06, Daniel Vetter 写道:
> > On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
> > <james.qian.wang@arm.com> wrote:
> > >
> > > On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
> > > > From: Liu Shixin <liushixin2@huawei.com>
> > > >
> > > > Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
> > > >
> > > > Signed-off-by: Liu Shixin <liushixin2@huawei.com>
> > > > ---
> > > > drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
> > > > 1 file changed, 1 insertion(+), 12 deletions(-)
> > > >
> > > > diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > > index 0246b2e94..4a10e6b9e 100644
> > > > --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > > +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
> > > > @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
> > > > return 0;
> > > > }
> > > >
> > > > -static int komeda_register_open(struct inode *inode, struct file *filp)
> > > > -{
> > > > - return single_open(filp, komeda_register_show, inode->i_private);
> > > > -}
> > > > -
> > > > -static const struct file_operations komeda_register_fops = {
> > > > - .owner = THIS_MODULE,
> > > > - .open = komeda_register_open,
> > > > - .read_iter = seq_read_iter,
> > > > - .llseek = seq_lseek,
> > > > - .release = single_release,
> > > > -};
> > > > +DEFINE_SHOW_ATTRIBUTE(komeda_register);
> > > >
> > >
> > > Hi Shixin & Qinglang
> > >
> > > Thanks for your patch.
> > >
> > > Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
> > >
> > > Since your patch is not for drm-misc-next, so seems better
> > > to leave it to you to merge it. :)
> >
> > I do think it's for drm-misc-next, what other tree would it be for?
> > Some people put -next in their patch tag to differentiate from -fixes,
> > so maintainers know what to do with the patch. It's also not part of a
> > series, hence I think this is on you to apply it.
> >
> Hi James & Daniel,
>
> Sorry I didn't make it clear in commit log, but it do based on linux-next.
>
> I think the reason why James think it's not for drm-misc-next
> is conflicts exists when this patch being applied. There's conflicts because
> commit <4d4901c6d7> which switched over direct seq_read method calls to
> seq_read_iter should applied before this clean-up patch(linkage listed as
> below).
>
> https://kernel.googlesource.com/pub/scm/linux/kernel/git/next/linux-next/+/4d4901c6d748efab8aab6e7d2405dadaed0bea50
>
> I can send a new patch based on mainline if needed.
Uh yes this is annoying. We're at feature cutoff so this will likely cause
bad conflicts no matter what if we merge it now, but the clean solution is
to rebase onto drm-misc-next, and then let maintainers sort out the mess
with conflicts. It's a pretty simple change in the above patch, so
shouldn't cause too many troubles.
-Daniel
>
> Thanks.
>
> Qinglang
>
> .
>
> >
> > Cheers, Daniel
> >
> > >
> > > Thanks
> > > James
> > >
> > > > #ifdef CONFIG_DEBUG_FS
> > > > static void komeda_debugfs_init(struct komeda_dev *mdev)
> > > > --
> > > > 2.17.1
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> >
> >
> >
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
2020-07-17 9:10 ` Daniel Vetter
@ 2020-09-17 12:36 ` miaoqinglang
-1 siblings, 0 replies; 14+ messages in thread
From: miaoqinglang @ 2020-09-17 12:36 UTC (permalink / raw)
To: james qian wang (Arm Technology China), Daniel Vetter
Cc: Mihail Atanassov, Greg Kroah-Hartman, Liviu Dudau,
Linux Kernel Mailing List, dri-devel, nd
在 2020/7/17 17:10, Daniel Vetter 写道:
> On Fri, Jul 17, 2020 at 04:00:36PM +0800, miaoqinglang wrote:
>>
>>
>> 在 2020/7/17 15:06, Daniel Vetter 写道:
>>> On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
>>> <james.qian.wang@arm.com> wrote:
>>>>
>>>> On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
>>>>> From: Liu Shixin <liushixin2@huawei.com>
>>>>>
>>>>> Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
>>>>>
>>>>> Signed-off-by: Liu Shixin <liushixin2@huawei.com>
>>>>> ---
>>>>> drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
>>>>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>>>>
>>>>> diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>>>> index 0246b2e94..4a10e6b9e 100644
>>>>> --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>>>> +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>>>> @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
>>>>> return 0;
>>>>> }
>>>>>
>>>>> -static int komeda_register_open(struct inode *inode, struct file *filp)
>>>>> -{
>>>>> - return single_open(filp, komeda_register_show, inode->i_private);
>>>>> -}
>>>>> -
>>>>> -static const struct file_operations komeda_register_fops = {
>>>>> - .owner = THIS_MODULE,
>>>>> - .open = komeda_register_open,
>>>>> - .read_iter = seq_read_iter,
>>>>> - .llseek = seq_lseek,
>>>>> - .release = single_release,
>>>>> -};
>>>>> +DEFINE_SHOW_ATTRIBUTE(komeda_register);
>>>>>
>>>>
>>>> Hi Shixin & Qinglang
>>>>
>>>> Thanks for your patch.
>>>>
>>>> Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
>>>>
>>>> Since your patch is not for drm-misc-next, so seems better
>>>> to leave it to you to merge it. :)
>>>
>>> I do think it's for drm-misc-next, what other tree would it be for?
>>> Some people put -next in their patch tag to differentiate from -fixes,
>>> so maintainers know what to do with the patch. It's also not part of a
>>> series, hence I think this is on you to apply it.
>>>
>> Hi James & Daniel,
>>
>> Sorry I didn't make it clear in commit log, but it do based on linux-next.
>>
>> I think the reason why James think it's not for drm-misc-next
>> is conflicts exists when this patch being applied. There's conflicts because
>> commit <4d4901c6d7> which switched over direct seq_read method calls to
>> seq_read_iter should applied before this clean-up patch(linkage listed as
>> below).
>>
>> https://kernel.googlesource.com/pub/scm/linux/kernel/git/next/linux-next/+/4d4901c6d748efab8aab6e7d2405dadaed0bea50
>>
>> I can send a new patch based on mainline if needed.
>
> Uh yes this is annoying. We're at feature cutoff so this will likely cause
> bad conflicts no matter what if we merge it now, but the clean solution is
> to rebase onto drm-misc-next, and then let maintainers sort out the mess
> with conflicts. It's a pretty simple change in the above patch, so
> shouldn't cause too many troubles.
> -Daniel
>
Hi James & Daniel,
I've sent v2 patch based on based on linux-next(20200917), and can be
applied to mainline cleanly now.
Thanks.
>
>>
>> Thanks.
>>
>> Qinglang
>>
>> .
>>
>>>
>>> Cheers, Daniel
>>>
>>>>
>>>> Thanks
>>>> James
>>>>
>>>>> #ifdef CONFIG_DEBUG_FS
>>>>> static void komeda_debugfs_init(struct komeda_dev *mdev)
>>>>> --
>>>>> 2.17.1
>>>> _______________________________________________
>>>> dri-devel mailing list
>>>> dri-devel@lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>>>
>>>
>>>
>>
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH -next] drm/komeda: Convert to DEFINE_SHOW_ATTRIBUTE
@ 2020-09-17 12:36 ` miaoqinglang
0 siblings, 0 replies; 14+ messages in thread
From: miaoqinglang @ 2020-09-17 12:36 UTC (permalink / raw)
To: james qian wang (Arm Technology China), Daniel Vetter
Cc: Mihail Atanassov, Greg Kroah-Hartman, Liviu Dudau,
Linux Kernel Mailing List, dri-devel, nd
在 2020/7/17 17:10, Daniel Vetter 写道:
> On Fri, Jul 17, 2020 at 04:00:36PM +0800, miaoqinglang wrote:
>>
>>
>> 在 2020/7/17 15:06, Daniel Vetter 写道:
>>> On Fri, Jul 17, 2020 at 8:40 AM james qian wang (Arm Technology China)
>>> <james.qian.wang@arm.com> wrote:
>>>>
>>>> On Thu, Jul 16, 2020 at 05:03:33PM +0800, Qinglang Miao wrote:
>>>>> From: Liu Shixin <liushixin2@huawei.com>
>>>>>
>>>>> Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
>>>>>
>>>>> Signed-off-by: Liu Shixin <liushixin2@huawei.com>
>>>>> ---
>>>>> drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 13 +------------
>>>>> 1 file changed, 1 insertion(+), 12 deletions(-)
>>>>>
>>>>> diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>>>> index 0246b2e94..4a10e6b9e 100644
>>>>> --- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>>>> +++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
>>>>> @@ -41,18 +41,7 @@ static int komeda_register_show(struct seq_file *sf, void *x)
>>>>> return 0;
>>>>> }
>>>>>
>>>>> -static int komeda_register_open(struct inode *inode, struct file *filp)
>>>>> -{
>>>>> - return single_open(filp, komeda_register_show, inode->i_private);
>>>>> -}
>>>>> -
>>>>> -static const struct file_operations komeda_register_fops = {
>>>>> - .owner = THIS_MODULE,
>>>>> - .open = komeda_register_open,
>>>>> - .read_iter = seq_read_iter,
>>>>> - .llseek = seq_lseek,
>>>>> - .release = single_release,
>>>>> -};
>>>>> +DEFINE_SHOW_ATTRIBUTE(komeda_register);
>>>>>
>>>>
>>>> Hi Shixin & Qinglang
>>>>
>>>> Thanks for your patch.
>>>>
>>>> Reviewed-by: James Qian Wang <james.qian.wang@arm.com>
>>>>
>>>> Since your patch is not for drm-misc-next, so seems better
>>>> to leave it to you to merge it. :)
>>>
>>> I do think it's for drm-misc-next, what other tree would it be for?
>>> Some people put -next in their patch tag to differentiate from -fixes,
>>> so maintainers know what to do with the patch. It's also not part of a
>>> series, hence I think this is on you to apply it.
>>>
>> Hi James & Daniel,
>>
>> Sorry I didn't make it clear in commit log, but it do based on linux-next.
>>
>> I think the reason why James think it's not for drm-misc-next
>> is conflicts exists when this patch being applied. There's conflicts because
>> commit <4d4901c6d7> which switched over direct seq_read method calls to
>> seq_read_iter should applied before this clean-up patch(linkage listed as
>> below).
>>
>> https://kernel.googlesource.com/pub/scm/linux/kernel/git/next/linux-next/+/4d4901c6d748efab8aab6e7d2405dadaed0bea50
>>
>> I can send a new patch based on mainline if needed.
>
> Uh yes this is annoying. We're at feature cutoff so this will likely cause
> bad conflicts no matter what if we merge it now, but the clean solution is
> to rebase onto drm-misc-next, and then let maintainers sort out the mess
> with conflicts. It's a pretty simple change in the above patch, so
> shouldn't cause too many troubles.
> -Daniel
>
Hi James & Daniel,
I've sent v2 patch based on based on linux-next(20200917), and can be
applied to mainline cleanly now.
Thanks.
>
>>
>> Thanks.
>>
>> Qinglang
>>
>> .
>>
>>>
>>> Cheers, Daniel
>>>
>>>>
>>>> Thanks
>>>> James
>>>>
>>>>> #ifdef CONFIG_DEBUG_FS
>>>>> static void komeda_debugfs_init(struct komeda_dev *mdev)
>>>>> --
>>>>> 2.17.1
>>>> _______________________________________________
>>>> dri-devel mailing list
>>>> dri-devel@lists.freedesktop.org
>>>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
>>>
>>>
>>>
>>
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 14+ messages in thread