Re: [PATCH v2] input: Add Marine Navigation Keycodes

[Shelby Heffron]

Hi Dmitry,

> > +/* Toggle the focus in or out */
> > +#define KEY_FOCUS_TOGGLE            0x27b
>
> Could it be served by KEY_SELECT?

We have found another keycode that will serve for this. Will remove in
next patch.

> > +
> > +/* Move between highlightable items */
> > +#define KEY_NEXT_HIGHLIGHT          0x27c
> > +#define KEY_PREVIOUS_HIGHLIGHT              0x27d
>
> I wonder if KEY_NEXT and KEY_PREVIOUS would be suitable for your
> application.

Yes, those keycodes should be suitable. Will remove in next patch.

> > +/* Navigate Displays */
> > +#define KEY_NEXT_DISPLAY            0x27f
> > +#define KEY_PREVIOUS_DISPLAY                0x280
>
> I am not familiar with marine navigation devices, could you please
> explain a bit more about displays you are navigating. Are there separate
> physical screens or are there several application screens/views you are
> switching between?

Yes - these keys would navigate between separate physical screens.

> > +#define KEY_BRIGHTNESS_MENU         0x28c
>
> Do you have multiple menus in the application?
We have found another keycode that will serve for this. Will remove in
next patch.


Thanks.
Shelby



________________________________

CONFIDENTIALITY NOTICE: This email and any attachments are for the sole use of the intended recipient(s) and contain information that may be Garmin confidential and/or Garmin legally privileged. If you have received this email in error, please notify the sender by reply email and delete the message. Any disclosure, copying, distribution or use of this communication (including attachments) by someone other than the intended recipient is prohibited. Thank you.