Page MenuHomeFreeBSD
Differential D33130

mmc: Fix HS200/HS400 capability check
ClosedPublic
Actions

Authored by kd on Nov 26 2021, 2:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 6, 3:37 AM
Unknown Object (File)
Sat, Mar 1, 2:15 PM
Unknown Object (File)
Thu, Feb 27, 10:39 AM
Unknown Object (File)
Thu, Feb 27, 7:36 AM
Unknown Object (File)
Sat, Feb 22, 11:39 AM
Unknown Object (File)
Feb 15 2025, 11:25 PM
Unknown Object (File)
Jan 30 2025, 7:45 AM
Unknown Object (File)
Jan 30 2025, 3:41 AM
View All 78 Files
Subscribers
imp

Details

Reviewers
mw
wma
mmel
manu
Commits
rG7aa6014fb16f: mmc: Fix HS200/HS400 capability check
rG8661e085fb95: mmc: Fix HS200/HS400 capability check
Summary

HS200 and HS400 speeds can be enabled either with 1.2, or 1.8V signaling voltage.
Because of that we have four cabability flags: MMC_CAP_MMC_HS200_120, MMC_CAP_MMC_HS200_180, MMC_CAP_MMC_HS400_120, MMC_CAP_MMC_HS400_180.
MMC logic only enables HS200/HS400 mode if both flags are set for the corresponding speed.
Fix that by being more permissive in host timing cap check.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
sys/
dev/
mmc/
6 lines

Diff 99157

View Options

sys/dev/mmc/mmc.c

Show All 25 Lines
* *
* $FreeBSD$ * $FreeBSD$
*/ */
#ifndef _LINUXKPI_LINUX_DEBUGFS_H_ #ifndef _LINUXKPI_LINUX_DEBUGFS_H_
#define _LINUXKPI_LINUX_DEBUGFS_H_ #define _LINUXKPI_LINUX_DEBUGFS_H_
#include <linux/fs.h> #include <linux/fs.h>
#include <linux/module.h>
#include <linux/seq_file.h> #include <linux/seq_file.h>
#include <linux/types.h> #include <linux/types.h>
void debugfs_remove(struct dentry *dentry); MALLOC_DECLARE(M_DFSINT);
struct debugfs_reg32 {
char *name;
unsigned long offset;
};
struct debugfs_regset32 {
const struct debugfs_reg32 *regs;
int nregs;
};
struct dentry *debugfs_create_file(const char *name, umode_t mode, struct dentry *debugfs_create_file(const char *name, umode_t mode,
struct dentry *parent, void *data, struct dentry *parent, void *data,
const struct file_operations *fops); const struct file_operations *fops);
struct dentry *debugfs_create_file_unsafe(const char *name, umode_t mode,
struct dentry *parent, void *data,
const struct file_operations *fops);
struct dentry *debugfs_create_mode_unsafe(const char *name, umode_t mode,
struct dentry *parent, void *data,
const struct file_operations *fops,
const struct file_operations *fops_ro,
const struct file_operations *fops_wo);
struct dentry *debugfs_create_dir(const char *name, struct dentry *parent); struct dentry *debugfs_create_dir(const char *name, struct dentry *parent);
struct dentry *debugfs_create_symlink(const char *name, struct dentry *parent, struct dentry *debugfs_create_symlink(const char *name, struct dentry *parent,
const char *dest); const char *dest);
void debugfs_remove(struct dentry *dentry);
void debugfs_remove_recursive(struct dentry *dentry); void debugfs_remove_recursive(struct dentry *dentry);
#endif #define DEFINE_DEBUGFS_ATTRIBUTE(__fops, __get, __set, __fmt) \
DEFINE_SIMPLE_ATTRIBUTE(__fops, __get, __set, __fmt)
void debugfs_create_bool(const char *name, umode_t mode, struct dentry *parent,
bool *value);
void debugfs_create_ulong(const char *name, umode_t mode, struct dentry *parent,
unsigned long *value);
#endif /* _LINUXKPI_LINUX_DEBUGFS_H_ */
hselaskyUnsubmitted
Done
Inline Actions

BSD style: All return values should be wrapped by ()'s.

hselasky: BSD style: All return values should be wrapped by ()'s.
jfreeAuthorUnsubmitted
Done
Inline Actions

Got it. I will change that! :)

jfree: Got it. I will change that! :)