2020-07-24 09:00:27

by Steve Lee

[permalink] [raw]
Subject: [PATCH 3/3] ASoC: max98390: update dsm param bin max size

MAX98390_DSM_PARAM_MAX_SIZE is changed to support extended
register update.

Signed-off-by: Steve Lee <[email protected]>
---
sound/soc/codecs/max98390.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/max98390.h b/sound/soc/codecs/max98390.h
index 5f444e7779b0..dff884f68e3e 100644
--- a/sound/soc/codecs/max98390.h
+++ b/sound/soc/codecs/max98390.h
@@ -650,7 +650,7 @@

/* DSM register offset */
#define MAX98390_DSM_PAYLOAD_OFFSET 16
-#define MAX98390_DSM_PARAM_MAX_SIZE 770
+#define MAX98390_DSM_PARAM_MAX_SIZE 1024
#define MAX98390_DSM_PARAM_MIN_SIZE 670

struct max98390_priv {
--
2.17.1


2020-07-24 10:52:41

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 3/3] ASoC: max98390: update dsm param bin max size

On Fri, Jul 24, 2020 at 05:56:44PM +0900, Steve Lee wrote:
> MAX98390_DSM_PARAM_MAX_SIZE is changed to support extended
> register update.

I'm missing patches 1-2 and have no cover letter - what's the story
with dependencies?


Attachments:
(No filename) (235.00 B)
signature.asc (499.00 B)
Download all attachments

2020-07-27 02:01:06

by Steve Lee

[permalink] [raw]
Subject: RE: [PATCH 3/3] ASoC: max98390: update dsm param bin max size

> -----Original Message-----
> From: Mark Brown <[email protected]>
> Sent: Friday, July 24, 2020 7:51 PM
> To: Steve Lee <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; Ryan Lee
> <[email protected]>; [email protected]
> Subject: Re: [PATCH 3/3] ASoC: max98390: update dsm param bin max size
>
> On Fri, Jul 24, 2020 at 05:56:44PM +0900, Steve Lee wrote:
> > MAX98390_DSM_PARAM_MAX_SIZE is changed to support extended register
> > update.
>
> I'm missing patches 1-2 and have no cover letter - what's the story with
> dependencies?
I will re-send patch ang please ignore this patch.
DSM init param is extended to cover more register so that DSM MAX PARAM SIZE value is changed.
Thanks.!

2020-07-27 13:09:36

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 3/3] ASoC: max98390: update dsm param bin max size

On Mon, Jul 27, 2020 at 02:00:09AM +0000, Steve Lee wrote:

> > I'm missing patches 1-2 and have no cover letter - what's the story with
> > dependencies?

> I will re-send patch ang please ignore this patch.
> DSM init param is extended to cover more register so that DSM MAX PARAM SIZE value is changed.

I applied these on Friday having found the other two patches (you're not
sending things as a thread so they get lost, for example if someone
sends a big series at a similar time like what happened on Friday) so
your new version won't apply, please resend whatever change you want to
make as an incremental patch against current code.

Please do not submit new versions of already applied patches, please
submit incremental updates to the existing code. Modifying existing
commits creates problems for other users building on top of those
commits so it's best practice to only change pubished git commits if
absolutely essential.


Attachments:
(No filename) (958.00 B)
signature.asc (499.00 B)
Download all attachments

2020-07-28 01:18:00

by Steve Lee

[permalink] [raw]
Subject: RE: [PATCH 3/3] ASoC: max98390: update dsm param bin max size

> -----Original Message-----
> From: Mark Brown <[email protected]>
> Sent: Monday, July 27, 2020 10:08 PM
> To: Steve Lee <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; Ryan Lee
> <[email protected]>; [email protected]
> Subject: Re: [PATCH 3/3] ASoC: max98390: update dsm param bin max size
>
> On Mon, Jul 27, 2020 at 02:00:09AM +0000, Steve Lee wrote:
>
> > > I'm missing patches 1-2 and have no cover letter - what's the story
> > > with dependencies?
>
> > I will re-send patch ang please ignore this patch.
> > DSM init param is extended to cover more register so that DSM MAX PARAM
> SIZE value is changed.
>
> I applied these on Friday having found the other two patches (you're not sending
> things as a thread so they get lost, for example if someone sends a big series at
> a similar time like what happened on Friday) so your new version won't apply,
> please resend whatever change you want to make as an incremental patch
> against current code.

>
> Please do not submit new versions of already applied patches, please submit
> incremental updates to the existing code. Modifying existing commits creates
> problems for other users building on top of those commits so it's best practice
> to only change pubished git commits if absolutely essential.

Thanks, I missed patched was already applied. I will send patch upon these patches later.