Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ucm: add support for sof-glkda7219max #26

Open
wants to merge 1 commit into
base: sof_ucm1
Choose a base branch
from

Conversation

aiChaoSONG
Copy link

@aiChaoSONG aiChaoSONG commented Jan 16, 2020

add support for sof-glkda7219max, this ucm is for GLK chromebook with da7219 codec

@aiChaoSONG aiChaoSONG changed the title ucm: add support sof sof-glkda7219max ucm: add support for sof-glkda7219max Jan 16, 2020
@aiChaoSONG
Copy link
Author

@plbossart Can we close #7 by this PR?

Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs a bit of adaptation, and we'd need to have a ucm v2 support

ucm/sof-glkda7219max/HiFi.conf Outdated Show resolved Hide resolved
CapturePCM "hw:sofglkda7219max,1"
CaptureChannels "2"
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no jack detection information?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@plbossart As I see from amixer contents, there is no headset jack detection info. only three HDMI jack detection info, and they are added.
'Headphone Jack Switch' is not the one we want.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's in the machine driver

	/*
	 * Headset buttons map to the google Reference headset.
	 * These can be configured by userspace.
	 */
	ret = snd_soc_card_jack_new(rtd->card, "Headset Jack",
			SND_JACK_HEADSET | SND_JACK_BTN_0 | SND_JACK_BTN_1 |
			SND_JACK_BTN_2 | SND_JACK_BTN_3 | SND_JACK_LINEOUT,
			&broxton_headset, NULL, 0);

Copy link
Author

@aiChaoSONG aiChaoSONG Jan 19, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@plbossart Yes, "Headset Jack" is in the driver, but I am sure it's not in the amixer contents (attached). seems we got an issue here.
I add some info as chrome did:

JackName "sofglkda7219max Headset Jack"
JackControl "Headset Jack"
JackType "gpio"

but jack detection not work at all, there is always a headset playback/ headset capture item in sound settings even without a headset inserted. we have the same situation on helios
I am kind of confused.
glk_amixer.log

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@juimonen can you help here?

Value {
PlaybackPCM "hw:sofglkda7219max,1"
PlaybackChannels "2"
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no JackControl?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@plbossart for HDMI we got JackControl "HDMI/DP,pcm=7 Jack", but for headset, seems no such info in amixer contents, so headphone JackControl, and jack detection related is not present in this ucm.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well jack detection should be added here for headphones/headset. It's supported in the machine driver so there's no reason to remove it.

ucm/sof-glkda7219max/sof-glkda7219max.conf Show resolved Hide resolved
ucm/sof-glkda7219max/sof-glkda7219max.conf Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants