* [PATCH v2] media: platform: Add brask to the match table
@ 2021-12-21 9:53 Zhuohao Lee
2021-12-21 15:39 ` Guenter Roeck
2022-01-11 9:41 ` Hans Verkuil
0 siblings, 2 replies; 3+ messages in thread
From: Zhuohao Lee @ 2021-12-21 9:53 UTC (permalink / raw)
To: hverkuil-cisco, mchehab, bleung, groeck, levinale
Cc: linux-media, linux-kernel, Zhuohao Lee
The Google Brask device uses the same approach as the Google Fizz
which enables the HDMI CEC via the cros-ec-cec driver.
Signed-off-by: Zhuohao Lee <zhuohao@chromium.org>
---
drivers/media/cec/platform/cros-ec/cros-ec-cec.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
index 2d95e16cd248..8c8d8fc5e63e 100644
--- a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
+++ b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
@@ -215,6 +215,8 @@ struct cec_dmi_match {
static const struct cec_dmi_match cec_dmi_match_table[] = {
/* Google Fizz */
{ "Google", "Fizz", "0000:00:02.0", "Port B" },
+ /* Google Brask */
+ { "Google", "Brask", "0000:00:02.0", "Port B" },
};
static struct device *cros_ec_cec_find_hdmi_dev(struct device *dev,
--
2.34.1.307.g9b7440fafd-goog
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] media: platform: Add brask to the match table
2021-12-21 9:53 [PATCH v2] media: platform: Add brask to the match table Zhuohao Lee
@ 2021-12-21 15:39 ` Guenter Roeck
2022-01-11 9:41 ` Hans Verkuil
1 sibling, 0 replies; 3+ messages in thread
From: Guenter Roeck @ 2021-12-21 15:39 UTC (permalink / raw)
To: Zhuohao Lee
Cc: Hans Verkuil, Mauro Carvalho Chehab, Benson Leung, Guenter Roeck,
Alex Levin, linux-media, linux-kernel
On Tue, Dec 21, 2021 at 1:53 AM Zhuohao Lee <zhuohao@chromium.org> wrote:
>
> The Google Brask device uses the same approach as the Google Fizz
> which enables the HDMI CEC via the cros-ec-cec driver.
>
> Signed-off-by: Zhuohao Lee <zhuohao@chromium.org>
Reviewed-by: Guenter Roeck <groeck@chromium.org>
> ---
Side note: Change log goes here. Don't expect maintainers to know what
changed in v2.
> drivers/media/cec/platform/cros-ec/cros-ec-cec.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> index 2d95e16cd248..8c8d8fc5e63e 100644
> --- a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> +++ b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> @@ -215,6 +215,8 @@ struct cec_dmi_match {
> static const struct cec_dmi_match cec_dmi_match_table[] = {
> /* Google Fizz */
> { "Google", "Fizz", "0000:00:02.0", "Port B" },
> + /* Google Brask */
> + { "Google", "Brask", "0000:00:02.0", "Port B" },
> };
>
> static struct device *cros_ec_cec_find_hdmi_dev(struct device *dev,
> --
> 2.34.1.307.g9b7440fafd-goog
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] media: platform: Add brask to the match table
2021-12-21 9:53 [PATCH v2] media: platform: Add brask to the match table Zhuohao Lee
2021-12-21 15:39 ` Guenter Roeck
@ 2022-01-11 9:41 ` Hans Verkuil
1 sibling, 0 replies; 3+ messages in thread
From: Hans Verkuil @ 2022-01-11 9:41 UTC (permalink / raw)
To: Zhuohao Lee, mchehab, bleung, groeck, levinale; +Cc: linux-media, linux-kernel
On 21/12/2021 10:53, Zhuohao Lee wrote:
> The Google Brask device uses the same approach as the Google Fizz
> which enables the HDMI CEC via the cros-ec-cec driver.
For future reference, please add the driver name in the subject as well.
A patch with just "media: platform:" as prefix suggests a patch that
touches on many drivers in media/platform.
I've manually updated the prefix, so no need for you to do anything.
Regards,
Hans
>
> Signed-off-by: Zhuohao Lee <zhuohao@chromium.org>
> ---
> drivers/media/cec/platform/cros-ec/cros-ec-cec.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> index 2d95e16cd248..8c8d8fc5e63e 100644
> --- a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> +++ b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> @@ -215,6 +215,8 @@ struct cec_dmi_match {
> static const struct cec_dmi_match cec_dmi_match_table[] = {
> /* Google Fizz */
> { "Google", "Fizz", "0000:00:02.0", "Port B" },
> + /* Google Brask */
> + { "Google", "Brask", "0000:00:02.0", "Port B" },
> };
>
> static struct device *cros_ec_cec_find_hdmi_dev(struct device *dev,
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-11 9:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-21 9:53 [PATCH v2] media: platform: Add brask to the match table Zhuohao Lee
2021-12-21 15:39 ` Guenter Roeck
2022-01-11 9:41 ` Hans Verkuil
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.